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

RE: [PATCH 3/6] iommu: remove iommu_lookup_page() and the lookup_page() method...

> -----Original Message-----
> From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Sent: 24 July 2020 19:39
> To: Paul Durrant <paul@xxxxxxx>; xen-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Paul Durrant <pdurrant@xxxxxxxxxx>; Kevin Tian <kevin.tian@xxxxxxxxx>; 
> Jan Beulich
> <jbeulich@xxxxxxxx>
> Subject: Re: [PATCH 3/6] iommu: remove iommu_lookup_page() and the 
> lookup_page() method...
> On 24/07/2020 17:46, Paul Durrant wrote:
> > From: Paul Durrant <pdurrant@xxxxxxxxxx>
> >
> > ... from iommu_ops.
> >
> > This patch is essentially a reversion of dd93d54f "vtd: add lookup_page 
> > method
> > to iommu_ops". The code was intended to be used by a patch that has long-
> > since been abandoned. Therefore it is dead code and can be removed.
> And by this, you mean the work that you only partial unstreamed, with
> the remainder of the feature still very much in use by XenServer?

I thought we basically decided to bin the original PV IOMMU idea though? 

> Please don't go breaking in-use things, simply because we're fixing
> Xen's IOMMU mess once large XSA at a time...
> As far as I can tell, this patch doesn't interact with any others in the
> series.

I can leave it, but I still don't think anything other than current XenServer 
will ever use it... so it really ought to just be in the downstream patch queue 


> ~Andrew



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