Messages in this thread | | | From | "Wu, Feng" <> | Subject | RE: [v6 6/8] iommu, x86: Setup Posted-Interrupts capability for Intel iommu | Date | Fri, 22 May 2015 01:03:17 +0000 |
| |
> -----Original Message----- > From: Thomas Gleixner [mailto:tglx@linutronix.de] > Sent: Thursday, May 21, 2015 6:51 PM > To: Wu, Feng > Cc: joro@8bytes.org; dwmw2@infradead.org; jiang.liu@linux.intel.com; > iommu@lists.linux-foundation.org; linux-kernel@vger.kernel.org > 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
| |