>>>>> "Konrad" == Konrad Rzeszutek <konrad@xxxxxxxxxxxxxxx> writes: Konrad> The read_capacity_16 sets the sdkp->hw_sector_size to Konrad> sector_size: > @@ -1402,6 +1413,7 @@ static int read_capacity_10(struct scsi_ ^^^^^^^^^^^^^^^^ That's the fallback setting for devices that only respond to READ CAPACITY(10). Konrad> That function also returns that value, which sd_read_capacity Konrad> assigns to the sector_size variable. So when the check for Konrad> sdkp->hw_sector_size inequality is reached, it is skiped as Konrad> sdkp->hw_sector_size == sector_size. No? For devices that support READ CAPACITY(16) (and are thus capable of reporting physical block size) the two values may differ and the message is printed. Konrad> Is that doing a read_capacity_16 or _10? 16. >> The protocols have been referring to logical blocks for ages. Konrad> Do they refer to physical blocks as well? Yep. -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: 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