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

[Juergen Gross <jgross@xxxxxxxx>]

On 25/01/18 15:45, Hans van Kranenburg wrote:
> 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?

No. Your change looks sensible to me. Do you mind me adding that patch
to my series? If no, could you send it to me with your Signed-off-by: ?


Juergen

> 
> -$ 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