[illumos-Developer] [REVIEW] 1303 Grub should support large sector disks

Garrett D'Amore garrett at damore.org
Fri Jul 29 17:47:09 BST 2011


On 07/29/2011 08:47 AM, Hans Rosenfeld wrote:
> Hi,
>
> our Grub currently fails to boot large sector disks (with ashift != 9).
>
> Here is a webrev to fix this: http://cr.illumos.org/view/6qc99xkh/
>
> I took the liberty to also do some minor cleanups of the code I had to
> touch. I tested it on my laptop, which has (at this time) two rpools,
> one with ashift=12 on a 4k-disk, and another with ashift=9.
>
> The patched grub is able to boot both pools, regardless of which disk it
> was loaded from.
>
>
> Hans
>

This looks really very good.  Nice job Hans!  Do you want to start an 
RTI for this?  (Would be good to get another reviewer as well.)

     - Garrett



More information about the Developer mailing list