On Tue, Jul 08, 2014 at 10:39:26AM +0100, Chris Wilson wrote: > On Thu, Jul 03, 2014 at 04:27:58PM +0100, [email protected] wrote: > > From: Oscar Mateo <[email protected]> > > > > This is preparatory work for Execlists: we plan to use it later to > > allocate our own context objects (since Logical Ring Contexts do > > not have the same kind of backing objects). > > > > No functional changes. > > > > Reviewed-by: Jesse Barnes <[email protected]> > > Signed-off-by: Oscar Mateo <[email protected]> > > The series looks fine. My gripe is that the split between context and > ring is messy (I think that a context is now a view of the entire > hardware state and so our bookkeeping should reflect that) and leads to > some subtle bugs where the bookkeeping on the ring is not transferred > correctly to the context. However, this series is a good starting point > to fix the more esoteric problems. > > Reviewed-by: Chris Wilson <[email protected]>
Thanks for patches & review, all merged to dinq. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
