On Thu, Nov 20, 2014 at 08:05:27AM +0000, Chris Wilson wrote: > On Thu, Nov 20, 2014 at 12:33:05AM +0100, Daniel Vetter wrote: > > Since it's not for the rings but engine, and its specifically for > > render state and workarounds. > > Nope. It is for initialising the context. > > setup_workarounds() would be for constructing the table to be applied > once. The current function is still snafu by adding to the workarounds > every time we reinitialize the default context. Well we also have a ring->init_context which does the other part. I agree it's a snafu, but I've figured this one here will get us a tiny step closer. Well, I'll drop it. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
