On April 25, 2016 6:07 PM, Jan Beulich <jbeul...@suse.com> wrote:
> >>> On 18.04.16 at 16:00, <quan...@intel.com> wrote:
> > --- a/xen/drivers/passthrough/vtd/iommu.c
> > +++ b/xen/drivers/passthrough/vtd/iommu.c
> > @@ -617,11 +617,12 @@ static void intel_iommu_iotlb_flush_all(struct
> > domain
> > *d)
> >  }
> >
> >  /* clear one page's page table */
> > -static void dma_pte_clear_one(struct domain *domain, u64 addr)
> > +static int dma_pte_clear_one(struct domain *domain, u64 addr)
> 
> This, at the latest in this series, is where __must_check should come into 
> play.
> 

Agreed.
Quan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to