> -----Original Message----- > From: Thomas Gleixner [mailto:[email protected]] > Sent: Thursday, May 21, 2015 6:51 PM > To: Wu, Feng > Cc: [email protected]; [email protected]; [email protected]; > [email protected]; [email protected] > Subject: Re: [v6 6/8] iommu, x86: Setup Posted-Interrupts capability for Intel > iommu > > On Thu, 21 May 2015, Feng Wu wrote: > > @@ -647,6 +647,20 @@ static int __init intel_enable_irq_remapping(void) > > + /* > > + * Set Posted-Interrupts capability. > > + */ > > + if (!disable_irq_post) { > > + intel_irq_remap_ops.capability |= 1 << IRQ_POSTING_CAP; > > + > > + for_each_iommu(iommu, drhd) > > + if (!cap_pi_support(iommu->cap)) { > > + intel_irq_remap_ops.capability &= > > + ~(1 << IRQ_POSTING_CAP); > > + break; > > + } > > + } > > + > > > @@ -874,6 +894,20 @@ static int reenable_irq_remapping(int eim) > > + /* > > + * Set Posted-Interrupts capability. > > + */ > > + if (!disable_irq_post) { > > + intel_irq_remap_ops.capability |= 1 << IRQ_POSTING_CAP; > > + > > + for_each_iommu(iommu, drhd) > > + if (!cap_pi_support(iommu->cap)) { > > + intel_irq_remap_ops.capability &= > > + ~(1 << IRQ_POSTING_CAP); > > + break; > > + } > > + } > > + > > Verbatim copy and paste. Ever heard about the concept of helper functions?
Good idea! Thanks, Feng > > Thanks, > > tglx _______________________________________________ iommu mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
