[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 09/11] IOMMU: propagate IOMMU Device-TLB flush error up to IOMMU suspending

On April 25, 2016 7:53 PM, Jan Beulich <JBeulich@xxxxxxxx> wrote:
> >>> On 18.04.16 at 16:00, <quan.xu@xxxxxxxxx> wrote:
> > --- a/xen/drivers/passthrough/vtd/iommu.c
> > +++ b/xen/drivers/passthrough/vtd/iommu.c

> > -static void iommu_flush_all(void)
> > +static int iommu_flush_all(void)
> __must_check

The iommu_flush_all() is also called in intel_iommu_hwdom_init()  and 
As we were on the same page, we can ignore the error code propagation for these 
two call trees.

I wonder whether we really need this '__must_check' or not, furthermore, 
print-out message
Is pointless (at least, to me).

Jan, could I ignore this '__must_check '?


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.