On Thu, Mar 13, 2025 at 09:38:58AM -0700, Pierrick Bouvier wrote: > Those functions are used by system/physmem.c, and are called only if > xen is enabled (which happens only if CONFIG_XEN is not set). You mean, 's/is not set/is set/'? > > So we can crash in case those are called. > > Acked-by: Richard Henderson <richard.henderson@xxxxxxxxxx> > Signed-off-by: Pierrick Bouvier <pierrick.bouvier@xxxxxxxxxx> > --- > diff --git a/hw/xen/xen_stubs.c b/hw/xen/xen_stubs.c > new file mode 100644 > index 00000000000..19cee84bbb4 > --- /dev/null > +++ b/hw/xen/xen_stubs.c > + > +void xen_invalidate_map_cache(void) > +{ Is this stub actually necessary? xen_invalidate_map_cache() doesn't seems to be used outside of xen's code. In anycase: Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx> Thanks, -- Anthony Perard | Vates XCP-ng Developer XCP-ng & Xen Orchestra - Vates solutions web: https://vates.tech