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

[Hans van Kranenburg <hans@xxxxxxxxxxx>]

On 01/25/2018 03:53 PM, Juergen Gross wrote:
> 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: ?

Sure, just did, with a bit reworded commit message.

K

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


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