<p dir="ltr"><br>
On Mar 23, 2015 7:13 PM, "Sowmini Varadhan" <<a href="mailto:sowmini.varadhan@oracle.com">sowmini.varadhan@oracle.com</a>> wrote:<br>
><br>
> On (03/24/15 09:21), Benjamin Herrenschmidt wrote:<br>
> ><br>
> > So we have two choices here that I can see:<br>
> ><br>
> >  - Keep that old platform use the old/simpler allocator<br>
><br>
> Problem with that approach is that the base "struct iommu" structure<br>
> for sparc gets a split personality: the older one is used with<br>
> the older allocator, and other ugly things ensue.  (alternatively,<br>
> you end up duplicating a version of the code with the flush_all<br>
> inlined).<br>
><br>
> >  - Try to regain the bulk of that benefit with the new one<br>
> ><br>
> > Sowmini, I see various options for the second choice. We could stick to<br>
> > 1 pool, and basically do as before, ie, if we fail on the first pass of<br>
> > alloc, it means we wrap around and do a flush, I don't think that will<br>
> > cause a significant degradation from today, do you ? We might have an<br>
> > occasional additional flush but I would expect it to be in the noise.<br>
><br>
> Isn't this essentially what I have in patch v5 here:<br>
> <a href="http://www.spinics.net/lists/sparclinux/msg13534.html">http://www.spinics.net/lists/sparclinux/msg13534.html</a><br>
><br>
> (the ops->reset is the flushall indirection, can be renamed if the<br>
> latter is preferred)<br>
><br>
> > Dave, what's your feeling there ? Does anybody around still have some<br>
> > HW that we can test with ?<br>
><br>
> I actually tested this on a V440 and a ultra45 (had a heck of a<br>
> time finding these, since the owners keep them turned off because<br>
> they are too noisy and consume too much power :-).</p>
<p dir="ltr">So we need tests then more than hw?.... I have an ultra1, ultra10 and t2000 I can test on if needed. And I'd appreciate my caches not being flushed excessively on these boxes :-) too. I'll see if I can't get Gentoo on the u10 tonight.</p>
<p dir="ltr">Also... It would be more "green" to make the code run faster on these boxes than otherwise! </p>