>>> On 18.05.16 at 10:08, <quan...@intel.com> wrote: > --- a/xen/common/memory.c > +++ b/xen/common/memory.c > @@ -633,9 +633,9 @@ static long > memory_exchange(XEN_GUEST_HANDLE_PARAM(xen_memory_exchange_t) arg) > return rc; > } > > -static int xenmem_add_to_physmap(struct domain *d, > - struct xen_add_to_physmap *xatp, > - unsigned int start) > +static int __must_check xenmem_add_to_physmap(struct domain *d, > + struct xen_add_to_physmap > *xatp, > + unsigned int start) > {
As before - either you do this adding of annotations completely, or you stop at the IOMMU / MM boundary. And note that this addition was not covered by the R-b tag I had offered during the v4 review. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel