Re: [PATCH-for-5.1] accel/xen: Fix xen_enabled() behavior on target-agnostic objects

[Peter Maydell <peter.maydell@xxxxxxxxxx>]

On Mon, 3 Aug 2020 at 17:35, Anthony PERARD <anthony.perard@xxxxxxxxxx> wrote:
>
> On Tue, Jul 28, 2020 at 12:09:25PM +0200, Philippe Mathieu-Daudé wrote:
> > CONFIG_XEN is generated by configure and stored in "config-target.h",
> > which is (obviously) only include for target-specific objects.
> > This is a problem for target-agnostic objects as CONFIG_XEN is never
> > defined and xen_enabled() is always inlined as 'false'.
> >
> > Fix by following the KVM schema, defining CONFIG_XEN_IS_POSSIBLE
> > when we don't know to force the call of the non-inlined function,
> > returning the xen_allowed boolean.
> >
> > Fixes: da278d58a092 ("accel: Move Xen accelerator code under accel/xen/")
> > Reported-by: Paul Durrant <pdurrant@xxxxxxxxxx>
> > Suggested-by: Peter Maydell <peter.maydell@xxxxxxxxxx>
> > Signed-off-by: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>
>
> Reviewed-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

Note that rc3 is tomorrow so if you want this in 5.1 it would
be a good idea to send a pullreq with it today...

thanks
-- PMM