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 contexts...
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 -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
