> Ibragimov, the problem with your patch is that it means that users who > had previously created a 1k block file system on an advanced format > disk with 4k physical sectors but 512 byte logical sectors would no > longer be able to mount that file system. Yes, it was stupid for them > to do so, and they are deserving of plenty of mocking. But > technically, it is something that should work, even if it's not > advisable from a performance point of view. I haven't thought about such case. It renders the patch useless. > Maybe we can create a blacklist of broken drives that report a logical > blocksize of 512 but which really have a logical block size of 4k? That would be great, but then one must recalculate LBA and sector count on every read and write request to hardware. I tried first to implement a kind of blacklist and simply changed ATA ID (to report 4k logical), but realized it was a bad idea. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html