grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 04/18] xen: prepare common code for Xen PVH support


From: Daniel Kiper
Subject: Re: [PATCH v2 04/18] xen: prepare common code for Xen PVH support
Date: Thu, 18 Oct 2018 16:59:55 +0200
User-agent: Mutt/1.5.21 (2010-09-15)

On Tue, Oct 09, 2018 at 01:03:03PM +0200, Juergen Gross wrote:
> Some common code needs to be special cased for Xen PVH mode. This hits
> mostly Xen PV mode specific areas.
>
> Signed-off-by: Juergen Gross <address@hidden>
> ---
>  grub-core/kern/i386/tsc.c         | 2 +-
>  include/grub/i386/pc/int.h        | 3 +++
>  include/grub/i386/tsc.h           | 2 +-
>  include/grub/i386/xen/hypercall.h | 5 ++++-
>  include/grub/kernel.h             | 4 +++-
>  5 files changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/grub-core/kern/i386/tsc.c b/grub-core/kern/i386/tsc.c
> index f266eb131..43fee3a13 100644
> --- a/grub-core/kern/i386/tsc.c
> +++ b/grub-core/kern/i386/tsc.c
> @@ -65,7 +65,7 @@ grub_tsc_init (void)
>
>    tsc_boot_time = grub_get_tsc ();
>
> -#ifdef GRUB_MACHINE_XEN
> +#if defined (GRUB_MACHINE_XEN) || defined (GRUB_MACHINE_XENPVH)

s/GRUB_MACHINE_XENPVH/GRUB_MACHINE_XEN_PVH/

>    (void) (grub_tsc_calibrate_from_xen () || calibrate_tsc_hardcode());
>  #elif defined (GRUB_MACHINE_EFI)
>    (void) (grub_tsc_calibrate_from_pmtimer () || grub_tsc_calibrate_from_pit 
> () || grub_tsc_calibrate_from_efi() || calibrate_tsc_hardcode());
> diff --git a/include/grub/i386/pc/int.h b/include/grub/i386/pc/int.h
> index 16a53e4fe..46fb1e397 100644
> --- a/include/grub/i386/pc/int.h
> +++ b/include/grub/i386/pc/int.h
> @@ -51,9 +51,12 @@ struct grub_bios_int_registers
>  #define  GRUB_CPU_INT_FLAGS_DEFAULT   0
>  #endif
>
> +#ifndef GRUB_MACHINE_XENPVH
>  void EXPORT_FUNC (grub_bios_interrupt) (grub_uint8_t intno,
>                                       struct grub_bios_int_registers *regs)
>       __attribute__ ((regparm(3)));
> +#endif

Is it an issue with this declaration? I think that you should take care
about grub-core/kern/i386/int.S. So, relevant Makefile should be
updated instead of this declaration.

Daniel



reply via email to

[Prev in Thread] Current Thread [Next in Thread]