Re: [PATCH 00/18] i915 HW Context Support

[Daniel Vetter <daniel@ffwll.ch>]

On Mon, Mar 19, 2012 at 11:14:32AM +0100, Daniel Vetter wrote:
> On Sun, Mar 18, 2012 at 01:39:40PM -0700, Ben Widawsky wrote:
> > The patches have changed quite a bit since the RFC, and therefore I
> > didn't feel comfortable trying to do v2 information. I didn't feel
> > comfortable taking the few r-bs that I had from the RFC except for the
> > one patch that I applied wholesale.
> > 
> > Summary:
> > - Completely redid the patch splitting.
> 
> I've only done a quick and cursory reading, but I like the new splitting
> _much_ more. The storyline behind these patches is now much clearer. I'll
> try to do a more in-depth review later this week.

Ok, I've gone through it and noticed a few things - mostly stuff that are
imo more complicated than necessary and that could be cut out.

Safe for the tlb flush wa I haven't cross-checked anything with Bspec, but
I don't expect any surprises there. I also haven't checked how good the
test coverage is (safe for suggesting that one test for execbuf failure
handling). But again, that's something which can be easily fixed.

The last thing I'm wondering is: How ready is mesa for this? I'd like to
merge this only when the mesa patches are ready to put it to good use.
Otherwise we run the decent risk of shipping broken code, which could end
up in a decent pain for userspace (worst case we have to add a new flag to
claim 'fixed context support').

Cheers, Daniel
-- 
Daniel Vetter
Mail: daniel@ffwll.ch
Mobile: +41 (0)79 365 57 48