[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2 of 2] PV-GRUB: add support for btrfs

On Sat, Mar 10, 2012 at 06:47:00AM -0800, Teck Choon Giam wrote:
> I tried to apply this in my xen-4.1-testing changeset
> 23264:3f158dd40b31 tree but failed.  See comment below.  Thanks.
> On Fri, Mar 9, 2012 at 11:03 AM, Matt Wilson <msw@xxxxxxxxxx> wrote:
> > This patch adds btrfs support to the GRUB tree used to build PV-GRUB.
> > The original patch is from Gentoo:
> > https://bugs.gentoo.org/show_bug.cgi?id=283637
> >
> > Signed-off-by: Matt Wilson <msw@xxxxxxxxxx>
> >
> > +             if (! devwrite (*saved_sector - part_start, 1, stage2_buffer))
> In stock grub-0.97 it is as below:
> if (! devwrite (saved_sector - part_start, 1, stage2_buffer))
> So my question is how you get it to apply in your tree without other
> patch to change the above from saved_sector to *saved_sector?
> I have tried to grep saved_sector string in xen tree which I believe
> all other patches in grub.patches directory doesn't touch that line or
> am I totally wrong about this?

You're right, a hunk of this patch fails to apply. I missed the reject
file being created, and the main functionality appears to work. I will
rework the btrfs patch when I have some time to apply cleanly.  That
would also probably be a good time change the GRUB patching step to
abort loudly if a patch doesn't apply cleanly.

Thanks for catching this!


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.