Re: [PATCH scsi-misc-2.6 08/13] scsi: move request preps in other places into prep_fn()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2005-03-31 at 18:08 +0900, Tejun Heo wrote:
> 	Move request preparations scattered in scsi_request_fn() and
> 	scsi_dispatch_cmd() into scsi_prep_fn().
> 
> 	* CDB_SIZE check in scsi_dispatch_cmd()
> 	* SCSI-2 LUN preparation in scsi_dispatch_cmd()
> 	* scsi_init_cmd_errh() in scsi_request_fn()
> 
> 	No invalid request reaches scsi_request_fn() anymore.

This one, I like, there's just one small problem:

You can't move scsi_init_cmd_errh() out of the request function path:
It's where we set up the sense buffer handling, so it has to be done
every time the command is prepared for execution (the prep function is
only called once)---think what happens if we turn a command around for
retry based on a sense indication.

So redo the patch and I'll put it in.

James


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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux