Alan wrote: > > A few questions are raised, libata SATA hasn't been resizing disks in the > > past, should we only disable HPA on PATA only, a la drivers/ide? In any > > drivers/ide disable HPA on SATA ports it manages too. > > > event, supporting HPA lets people who had partitions which extend beyond > > the protected area boot with libata PATA. Some policy decision needs to be > > made too... Right now, I'm just disabling it unilaterally. > > Last time we had a Red Hat bitching session about it the main thing the > installer guys wanted was a way to find out what size the disk thought it > was before we twiddled with it, so that they can make an intelligent > attempt to handle partitioning in the case where for example a disk is > unpartitioned. A partitioned disk isn't so much of a problem as the > partitioning itself tells you the a lot about the state of disk and > intended HPA. > > What do the Ubuntu installer/partitioning folk think ? Suspiciously similar to the Red Hat installer folks. :-) Our partitioner (and Debian's) asks libparted to create a partition table in memory (ped_disk_new_fresh) and then works entirely based on what libparted reports about that partition table before actually committing it. This obviously only works if you can find out the size of the disk before writing anything to it. libparted uses ioctl(BLKGETSIZE[64]) to get the size of the disk at the moment. Making that report the non-HPA size would be enough to make libparted create new partition tables properly, but it would defeat the purpose of Kyle's patch in that you probably couldn't use libparted to work with existing partition tables extending beyond the HPA. Adding say ioctl(BLKGETNONHPASIZE) or something and teaching libparted to create new partition tables excluding the HPA but still be able to read existing partition tables extending into the HPA seems like it'd be the best approach, and sounds pretty straightforward. I'd be happy to do the libparted side of that. -- Colin Watson [cjwatson@xxxxxxxxxx] - 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
