[Intel-gfx] [PATCH 0/8] drm/i915/context
chris at chris-wilson.co.uk
Wed Feb 2 15:29:37 PST 2011
I don't think we should be pinning contexts for their entire lifetime.
The page should only be pinned whilst it is being referenced by the
hardware, and it looks like you already parts of the infrastructure to
be able to track this. You will also need to track seqno and retire
Being picky, there are still renames ;-) But it is much easier to see
what you are introducing in each step, thanks!
I'm not convinced by the locking strategy, there seems to be quite a few
holes left, and is something else that is much easier to make it correct
from the beginning and incrementally adjust.
And then there are lots of little changes to try and make the code
tidier... But those are probably best left for another day.
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx