On Thu, Apr 23, 2009 at 01:29:31AM -0400, Martin K. Petersen wrote:
> Extract physical block size and lowest aligned LBA from READ
> CAPACITY(16) response and adjust queue parameters.
>
> Report physical block size and alignment when applicable.
>
> Signed-off-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx>
>
.. snip ..
> - if (sdkp->first_scan || old_capacity != sdkp->capacity)
> + if (sdkp->first_scan || old_capacity != sdkp->capacity) {
> sd_printk(KERN_NOTICE, sdkp,
> - "%llu %d-byte hardware sectors: (%s/%s)\n",
> + "%llu %d-byte logical blocks: (%s/%s)\n",
> (unsigned long long)sdkp->capacity,
> sector_size, cap_str_10, cap_str_2);
> +
> + if (sdkp->hw_sector_size != sector_size)
> + sd_printk(KERN_NOTICE, sdkp,
> + "%u-byte physical blocks\n",
> + sdkp->hw_sector_size);
With the changes in the read_capacity_16 (where you set the
hw_sector_size) this won't be printed (at least on first probe).
Should this be printed irregardless of that?
Another question - should the Documentation/*.txt have an entry about what a
'logical block', and 'physical block' is vs. 'hardware sector' ?
--
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
