On Fri, 2014-12-19 at 01:55 +0800, Fu Wei wrote:
> Signed-off-by: Fu Wei <fu.wei@linaro.org>
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

I've tested the code at
git://git.linaro.org/people/fu.wei/grub.git#multiboot_xen_support_upstream_v4.0
which I believe is the same as this posting on an AMD Seattle system
with a 40_custom containing:

        menuentry 'Baremetal' {
                insmod gzio
                insmod part_msdos
                insmod ext2
                set root='hd0,msdos2'
                search --no-floppy --fs-uuid --set=root d4178d5c-96a8-46ee-a304-0d87baa545cd
                linux /boot/vmlinuz console=ttyAMA0,115200n8 earlycon=pl011,0xe1010000 root=/dev/sda2 rootwait
        }
        menuentry 'Xen' {
                insmod gzio
                insmod part_msdos
                insmod ext2
                set root='hd0,msdos2'
                search --no-floppy --fs-uuid --set=root d4178d5c-96a8-46ee-a304-0d87baa545cd
                multiboot /boot/xen no-bootscrub console=dtuart conswitch=x dtuart=/smb/serial@e1010000 noreboot sync_console dom0_mem=256M dom0_max_vcpus=1
                module /boot/vmlinuz-xen console=hvc0 earlycon=pl011,0xe1010000 root=/dev/sda2 rootwait
        }

(my system is too confused for 10_linux or 20_linux_xen right now, but I
believe this is representative of what they would emit)

The result was that I could boot both Baremetal and Xen via grub. So:

Tested-by: Ian Campbell <ian.campbell@citrix.com>

I've not looked at the code (although I did review several older
iterations), would it be useful to the grub maintainers for me to do so?

Ian.