Re: [Xen-devel] [PATCH v5 6/7] xen/guest: prepare hypervisor ops to use alternative calls

[Roger Pau Monné <roger.pau@xxxxxxxxxx>]

On Mon, Mar 02, 2020 at 11:43:14AM +0100, Jan Beulich wrote:
> On 28.02.2020 17:52, Roger Pau Monné wrote:
> > On Fri, Feb 28, 2020 at 05:29:32PM +0100, Jan Beulich wrote:
> >> On 19.02.2020 18:43, Roger Pau Monne wrote:
> >>> --- a/xen/arch/x86/guest/hyperv/hyperv.c
> >>> +++ b/xen/arch/x86/guest/hyperv/hyperv.c
> >>> @@ -199,7 +199,7 @@ static void __init e820_fixup(struct e820map *e820)
> >>>          panic("Unable to reserve Hyper-V hypercall range\n");
> >>>  }
> >>>  
> >>> -static const struct hypervisor_ops ops = {
> >>> +static const struct hypervisor_ops __initdata ops = {
> >>
> >> This needs to be __initconstrel in order to avoid triggering
> >> (possibly only in the future) section mismatch warnings with
> >> at least some gcc versions. With this and the other instance
> >> adjusted
> > 
> > I can do that when posting a new version, unless you want to pick this
> > earlier and adjust on commit.
> 
> Is this to mean that this 2nd to last patch in the series is
> fully independent of the earlier five (also contextually)?

Right, patches 5 to 7 should be completely independent, as they only
modify code related to Xen running as a guest.

> Then of course I'd be fine to make the adjustments and commit.
> Please confirm if so.

Yes please, do the adjustments on commit if you don't mind.

Thanks, Roger.

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