Re: [PATCH v2 02/18] loader/linux: support passing rsdp address via boot params

[Daniel Kiper]

On Thu, Oct 18, 2018 at 04:53:01PM +0200, Juergen Gross wrote:
> On 18/10/2018 16:48, Daniel Kiper wrote:
> > On Thu, Oct 18, 2018 at 04:36:28PM +0200, Juergen Gross wrote:
> >> On 18/10/2018 16:30, Daniel Kiper wrote:
> >>> On Thu, Oct 18, 2018 at 04:18:26PM +0200, Juergen Gross wrote:
> >>>> On 18/10/2018 16:13, Daniel Kiper wrote:
> >>>>> On Tue, Oct 09, 2018 at 01:03:01PM +0200, Juergen Gross wrote:
> >>>>>> Xen PVH guests will have the RSDP at an arbitrary address. Support that
> >>>>>> by passing the RSDP address via the boot parameters to Linux.
> >>>>>>
> >>>>>> The new protocol version 2.14 requires to set version to 0x8000 ored
> >>>>>> with the actually use protocol version (the minimum of the kernel
> >>>>>> supplied protocol version and the grub2 supported protocol version)
> >>>>>> if 2.14 or higher are in effect.
> >>>>>>
> >>>>>> Signed-off-by: Juergen Gross <address@hidden>
> >>>>>> ---
> >>>>>> V2: add oring 0x8000 to version field
> >>>>>> ---
> >>>>>>  grub-core/loader/i386/linux.c | 9 +++++++++
> >>>>>>  include/grub/i386/linux.h     | 5 ++++-
> >>>>>>  2 files changed, 13 insertions(+), 1 deletion(-)
> >>>>>>
> >>>>>> diff --git a/grub-core/loader/i386/linux.c 
> >>>>>> b/grub-core/loader/i386/linux.c
> >>>>>> index 4eab55a2d..f96309476 100644
> >>>>>> --- a/grub-core/loader/i386/linux.c
> >>>>>> +++ b/grub-core/loader/i386/linux.c
> >>>>>> @@ -35,6 +35,7 @@
> >>>>>>  #include <grub/i18n.h>
> >>>>>>  #include <grub/lib/cmdline.h>
> >>>>>>  #include <grub/linux.h>
> >>>>>> +#include <grub/machine/kernel.h>
> >>>>>
> >>>>> Probably this change belongs to another patch.
> >>>>
> >>>> I don't think so.
> >>>
> >>> You do not add anything to this header here and out of the blue you
> >>> include it in this file. So, why it is needed here?
> >>
> >> Ah, now I see your problem.
> >>
> >> machine/kernel.h will be the header which eventually defines
> >> GRUB_KERNEL_USE_RSDP_ADDR.
> >
> > So, please move this to the proper patch.
>
> Okay, if you like that better.

Yes, I will. Thanks!

Daniel