Re: [Xen-devel] FIXME question

2016-06-27 Thread Wu, Feng
> -Original Message- > From: Andrew Cooper [mailto:andrew.coop...@citrix.com] > Sent: Monday, June 27, 2016 5:51 PM > To: Jan Beulich ; Xu, Quan > Cc: Tian, Kevin ; Wu, Feng ; xen- > de...@lists.xen.org > Subject: Re: [Xen-devel] FIXME question > > On 27/06/

Re: [Xen-devel] FIXME question

2016-06-27 Thread Andrew Cooper
On 27/06/16 10:32, Jan Beulich wrote: On 27.06.16 at 11:23, wrote: >> Hi, >> >> When I read IOMMU code, >> In xen/drivers/passthrough/vtd/intremap.c : pi_update_irte().. >> There are a FIXME -- >> '' >> * FIXME: For performance reasons we should store the 'iommu' pointer in >> * 'struct msi_d

Re: [Xen-devel] FIXME question

2016-06-27 Thread Jan Beulich
>>> On 27.06.16 at 11:23, wrote: > Hi, > > When I read IOMMU code, > In xen/drivers/passthrough/vtd/intremap.c : pi_update_irte().. > There are a FIXME -- > '' > * FIXME: For performance reasons we should store the 'iommu' pointer in > * 'struct msi_desc' in some other place, so we don't need to

Re: [Xen-devel] FIXME question

2016-06-27 Thread Wu, Feng
> -Original Message- > From: Xu, Quan > Sent: Monday, June 27, 2016 5:24 PM > To: xen-devel@lists.xen.org > Cc: Jan Beulich ; Tian, Kevin ; Wu, > Feng > Subject: FIXME question > > Hi, > > When I read IOMMU code, > In xen/drivers/passthrough/vtd/intremap.c : pi_update_irte().. > There