James Bottomley wrote: > On Fri, 2006-04-28 at 09:28 -0500, Brian King wrote: >> Any comments on the patch below? > > The mechanistic comment is that the max_cmd_len parameter should be per > target not per device. I don't have a problem with that. It could go in the target as well. > However, since this is SATA only, and SATA is supposed to be moving out > of the SCSI subsystem, I'm a bit loath to add things we'll have to > attempt to extract later. What would be the consequence of simply > lowering the host max_cmd_len by the result returned from IDENTIFY? That is what is done today, which works fine if you only have one device per host, but when you have multiple devices per host, there is no *good* value to put in the host max_cmd_len, hence the patch. This could also be completely contained in libata as my previous post suggests, if Jeff is OK with that. Brian -- Brian King eServer Storage I/O IBM Linux Technology Center - : send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
