On 09.09.2022 03:25, Daniel P. Smith wrote: > --- a/xen/xsm/flask/hooks.c > +++ b/xen/xsm/flask/hooks.c > @@ -38,6 +38,12 @@ > #include <conditional.h> > #include "private.h" > > +#ifdef CONFIG_X86 > +#include <asm/pv/shim.h> > +#else > +#define pv_shim false > +#endif
Hmm, yet another instance of such. We have already several (and dissimilar) instances scattered around, so I guess we really want to centralize this (e.g. by moving things from x86'es pv/shim.h to a non-asm header). Not something you need to do right here, but something for 4.18. Jan