>>>>> "Jens" == Jens Axboe <jaxboe@xxxxxxxxxxxx> writes: Jens> So it's just the hint, not the actual hardware sector size. The Jens> naming is pretty bad on that, physical and logic... This is the official T10/T13 terminology. Logical block size is the unit used when talking to the device via ACS/SBC. Physical block size is the unit used by the device internally. There's a pretty vague description of what that means in SBC. But it was obviously conceived mainly to handle disk drives with 512-byte logical and 4096-byte physical (hardware) sectors. The notion of using the physical block size as an indicator of erase block size is something this unknown vendor came up with. And 1MB is quite the ways out of range for what we expect in this field. But my point is that devices with physical block sizes bigger than the system page size are already out there. So we need to do the right thing here. And since this is not something affecting runtime behavior I'm in favor of deferring the decision what to do about it to userland. Jens> So that does look better, but in that case I don't think that sd Jens> should dump a warning. That's fine by me. Jens> Does mkfs do the right thing? Depends on which mkfs it is. Mike has tested things and can chip in here... -- Martin K. Petersen Oracle Linux Engineering -- 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
