Re: [Xen-devel] RE: [PATCH] [RFC] VT-d: always clean up dpci timers.

[Tim Deegan <Tim.Deegan@xxxxxxxxxx>]

At 15:47 +0100 on 25 Jul (1311608879), Keir Fraser wrote:
> On 25/07/2011 15:21, "Tim Deegan" <Tim.Deegan@xxxxxxxxxx> wrote:
> 
> >> Could we make need_iommu(d) sticky? Being able to clear it doesn't seem an
> >> important case (such a domain is probably being torn down anyway) and
> >> clearly it can lead to fragility. The fact that presumably we'd end up 
> >> doing
> >> unnecessary IOMMU PT work for the remaining lifetime of the domain doesn't
> >> seem a major downside to me.
> > 
> > If you prefer it that way.  TBH I think I prefer the other way though:
> > things that gate on need_iommu() should be cleaned up by
> > iommu->teardown().
> 
> Is your original patch still proposed as a valid alternative to this one?

Yes; I prefer the original. 

Tim.

-- 
Tim Deegan <Tim.Deegan@xxxxxxxxxx>
Principal Software Engineer, Xen Platform Team
Citrix Systems UK Ltd.  (Company #02937203, SL9 0BG)

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel