Re: [Xen-users] pvgrub2(-like?) booting methods for PVHv2 guests

[Hans van Kranenburg <hans@xxxxxxxxxxx>]

By the way, Juergen,

On 01/25/2018 07:01 AM, Juergen Gross wrote:
> On 24/01/18 17:37, Hans van Kranenburg wrote:
>> On 01/23/2018 06:41 AM, Juergen Gross wrote:
>>> On 23/01/18 05:46, Andy Smith wrote:
>>>> Hi,
>>>>
>>>> What are the current options for booting PVHv2 guests in Xen 4.10.0?
>>>>
>>>> As far as I can see there is only direct kernel booting, i.e.
>>>> supplying "kernel = /path/to/vmlinux".
>>>>
>>>> In PV land there is pvgrub which is very convenient for being able
>>>> to use kernel and initramfs, but I am not seeing support for that in
>>>> upstream grub2. I see that Juergen Gross post a patch series that
>>>> claims to add this feature:
>>>>
>>>>     <https://lists.gnu.org/archive/html/grub-devel/2017-11/msg00054.html>
>>>>
>>>> …but this doesn't seem to have been applied yet. Is it an ongoing
>>>> idea or has that method been abandoned?
>>>
>>> Still waiting for review.
>>
>> Since I wanted to try to test this, I scraped the patches off the
>> grub-devel list and did the following:
>>
>> 1. clone https://git.savannah.gnu.org/git/grub.git
>> 2. apply patches on top of master, plus an extra one with a puzzle piece
>> to make it actually compile (see attachment)
>>
>> [...]
I needed to add this one also to make it compile. Didn't you run into this?

-$ git show
commit a3589cc5b5675f302ac430a6fdc0eed4e24ab2ba
Author: Hans van Kranenburg <hans@xxxxxxxxxxx>
Date:   Tue Jan 23 20:03:42 2018 +0100

    grub-module-verifier: Ignore all_video for xenpvh

    This solves the build failing with "Error: no symbol table and no
    .moddeps section"

    Also see:
    - 6371e9c10433578bb236a8284ddb9ce9e201eb59
    - https://savannah.gnu.org/bugs/?49012

diff --git a/util/grub-module-verifier.c b/util/grub-module-verifier.c
index 9179285a5..8c713e015 100644
--- a/util/grub-module-verifier.c
+++ b/util/grub-module-verifier.c
@@ -128,6 +128,7 @@ struct platform_whitelist {

 static struct platform_whitelist whitelists[] = {
   {"i386", "xen", (const char *[]) {"all_video", 0}},
+  {"i386", "xenpvh", (const char *[]) {"all_video", 0}},
   {"x86_64", "xen", (const char *[]) {"all_video", 0}},
   {"sparc64", "ieee1275", (const char *[]) {"all_video", 0}},


Hans

_______________________________________________
Xen-users mailing list
Xen-users@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-users