On Tue, Jun 25, 2013 at 05:10:42PM +0100, Chris Wilson wrote: > On Tue, Jun 25, 2013 at 06:47:02PM +0300, Mika Kuoppala wrote: > > From: Chris Wilson <chris at chris-wilson.co.uk> > > > > As our contexts are more general than the logical contexts supported by > > the hardware, for instance they allow per context hangcheck tracking, it > > is beneficial to group tasks across rings belonging to the same context. > > Context switching is already a no-op for unsupported rings, we just > > suffered from a little too overzealous parameter checking on entry - now > > we just check that the context is valid. > > We have to drop this temporarily as the validation was being provided by > i915_gem_context_get_hang_stats(), which is now notably absent. > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre I haven't been actively reading the list recently, but I have this same patch in the PPGTT series. I'll make sure I get it upstream in my series if you guys are okay with dropping it here. -- Ben Widawsky, Intel Open Source Technology Center