> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Thursday, February 22, 2018 5:50 PM > > >>> On 22.02.18 at 10:44, <roger....@citrix.com> wrote: > > On Wed, Oct 11, 2017 at 05:27:05AM -0600, Jan Beulich wrote: > >> >>> On 11.10.17 at 05:03, <haozhong.zh...@intel.com> wrote: > >> > --- a/xen/drivers/passthrough/vtd/iommu.c > >> > +++ b/xen/drivers/passthrough/vtd/iommu.c > >> > @@ -1105,7 +1105,13 @@ static void dma_msi_set_affinity(struct > irq_desc > >> > *desc, const cpumask_t *mask) > >> > > >> > spin_lock_irqsave(&iommu->register_lock, flags); > >> > dmar_writel(iommu->reg, DMAR_FEDATA_REG, msg.data); > >> > - dmar_writeq(iommu->reg, DMAR_FEADDR_REG, msg.address); > >> > + dmar_writel(iommu->reg, DMAR_FEADDR_REG, msg.address_lo); > >> > + /* > >> > + * When x2APIC is not enabled, DMAR_FEUADDR_REG is reserved > and > >> > + * it's not necessary to update it. > >> > + */ > >> > + if (x2apic_enabled) > >> > >> I'm pretty sure it was pointed out before that the style here is > >> wrong (missing spaces). That's easy to fix while committing, but > >> anyway. > > > > It seems like this was not applied. Can someone fix the style and > > apply it? > > I would have fixed the style issue, but I have no VT-d maintainer > ack on record. It is the patch submitter to go hunt for acks. >
I didn't realize no ack from me. Anyway, here it is: Acked-by: Kevin Tian <kevin.t...@intel.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel