lkml.org 
[lkml]   [2005]   [May]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH scsi-misc-2.6 04/04] scsi: remove unnecessary scsi_wait_req_end_io()
From
Date
On Sun, 2005-05-15 at 10:15 +0900, Tejun Heo wrote:
> I've made two new versions of the same patch. The first one just
> BUG() such cases, and the second one makes scsi_prep_fn() tell
> scsi_request_fn() to kill requests instead of doing itself w/
> BLKPREP_KILL. In both cases, I made req->flags error case a BUG().
> If you don't like it, feel free to drop that part.
>
> Oh... one more thing. I forgot to mention the scsi_kill_requests()
> path. As it's a temporary fix, I just left it as it is (terminate
> commands w/ end_that_*). I guess this patch should be pushed after
> removal of that kludge. But with or without this patch, that path
> will leak resources.

I suppose it's not surprising that I don't like either.

You remove the code that handles the BLKPREP_KILL case and then contort
the request functions to try not to do it. We have to handle this case,
it's not optional, so just leave the code that does it in.

James


-
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-05-16 18:11    [W:0.053 / U:0.528 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site