Hello, On Wed, Feb 09, 2011 at 02:45:48PM -0500, Phillip Susi wrote: > Then why did you have libata default to NOT unlocking it, which seems to > be the correct thing to do? When Ubuntu switched to libata they patched > it to default to unlocking the HPA because the old ide driver did. This > seems wrong and causes breakage. Tejun's patch seems to be a good > compromise between the two choices: unlock if you think it will help, > otherwise, don't, since it might hurt. I don't think that patch supports your argument to begin with but even if it did don't get too hung up on the fact that I (or anyone else) did something contrary in the past. I've done a lot of stupid and horrible things and changed mind more times than anyone should care about. I thought it would be a good and safe compromise for distros which don't want to unlock by default but look at where we are. We just ended up with failures which are more obscure. Your proposed addition will only push things further that direction. It might not necessarily be a bad thing. Maybe the level of obscurity then goes beyond certain level and we wouldn't have to care about that. But much better solution seems to be exporting the bios size to others and letting the ones which understand storage configuration better deal with it. At the block or libata layer, we simply don't have enough information to make those decisions in a reliable manner. Thanks. -- tejun -- 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
