Re: [PATCH 4/4] aic7xxx: Enable 16-bit CDBs

[Hannes Reinecke <hare@suse.de>]

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@suse.de			      +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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html