Yoshinori K. Okuji wrote:
>> Incidentally, could you perhaps offer your comments on the attached patch?
> ChangeLog, please.

Any preferred length, formatting, ?..

"Shuffle BLKGETSIZE and HD_GETGEO so that a correct sector count can
be retrieved for device-mapper devices in the stage2 simulator."

A more verbose changelog could include this too:
"GRUB would previously die with "24 : Attempt to access block outside
partition" in the partition walking code when retrieving an extended
partition table outside of the default CHS bounds (2.441 MB)."

I'd appreciate it if the code could get some tests and reviews, as I'm
nowhere near home in the world of GRUB.

Specifically, I'd like the person who originally wrote device.c to
comment on the ordering of HDIO_GETGEO and BLKGETSIZE.  I'm guessing
it's just random, but I could be mistaken!


_______________________________________________
Bug-grub mailing list
Bug-grub@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-grub

Reply via email to