lkml.org 
[lkml]   [2001]   [Apr]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] Fix scsi_unblock_requests()
Date
From
> In its current implementation, scsi_unblock_requests() simply
> clears host_self_blocked in the SCSI host struct. This means
> that either a transaction must complete or a new transaction

Suppose the queue is unblocked from inside the functions called to process
the request. In that situation the old code is correct and your code might
introduce other problems

> unblocks. scsi_queue_next_request() verifies all other state
> to ensure queuing new transactions is safe prior to proceeding.

Including recursion ?

The patch seems right apart from checking these details out further.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:24    [W:0.062 / U:0.620 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site