> -----Original Message----- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 17 October 2017 13:53 > To: Paul Durrant <paul.durr...@citrix.com> > Cc: Andrew Cooper <andrew.coop...@citrix.com>; George Dunlap > <george.dun...@citrix.com>; Ian Jackson <ian.jack...@citrix.com>; Wei Liu > <wei.l...@citrix.com>; Stefano Stabellini <sstabell...@kernel.org>; xen- > de...@lists.xenproject.org; KonradRzeszutek Wilk > <konrad.w...@oracle.com>; Daniel de Graaf <dgde...@tycho.nsa.gov>; Tim > (Xen.org) <t...@xen.org> > Subject: RE: [PATCH v11 05/11] x86/mm: add HYPERVISOR_memory_op to > acquire guest resources > > >>> On 17.10.17 at 14:28, <paul.durr...@citrix.com> wrote: > >> -----Original Message----- > >> > >> > --- a/xen/include/xsm/dummy.h > >> > +++ b/xen/include/xsm/dummy.h > >> > @@ -724,3 +724,9 @@ static XSM_INLINE int xsm_xen_version > >> (XSM_DEFAULT_ARG uint32_t op) > >> > return xsm_default_action(XSM_PRIV, current->domain, NULL); > >> > } > >> > } > >> > + > >> > +static XSM_INLINE int > xsm_domain_resource_map(XSM_DEFAULT_ARG > >> struct domain *d) > >> > +{ > >> > + XSM_ASSERT_ACTION(XSM_DM_PRIV); > >> > + return xsm_default_action(action, current->domain, d); > >> > +} > >> > >> Perhaps better place this near something similar/related (also for > >> some of the other additions further down)? > > > > Looking at this again it seems that various related things, e.g. > > domain_memory_map, are x86 only so adding at the end seems like the > best > > thing to do. > > Well, okay then (unless Daniel, whom it looks like you forgot to Cc, > has a better suggestion). >
Yes, I realised that I forgot to cc him in since the code was added. He's on the v12 list. Paul > Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel