Re: [PATCH scsi-misc-2.6 04/04] scsi: remove unnecessaryscsi_wait_req_end_io()

From: James Bottomley
Date: Mon May 16 2005 - 11:09:31 EST


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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/