Matthew Wilcox wrote: > On Sun, Oct 21, 2007 at 11:19:22AM -0500, James Bottomley wrote: >> But, I suppose the main point is: is this really necessary? The >> default, when unspecified, is 16 bytes and the mid-layer can't deliver >> anything larger. I would anticipate the large CDB infrastructure will >> have some separate enabling, so why not simply do nothing until that >> gets merged? > > You're sadly mistaken ... the default is 12 byte cdbs: > > shost->max_cmd_len = 12; > That's what I thought, too. Hence the patch. But if we set the default to 16 byte cdbs you can of course ignore the patch. Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@xxxxxxx +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Markus Rex, HRB 16746 (AG Nürnberg) - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html