On Fri, 20 Apr 2012 18:23:26 -0700, Ben Widawsky <b...@bwidawsk.net> wrote: > The only time irq_get should fail is during unload or suspend. Both of > these points should try to quiesce the GPU before disabling interrupts > and so the atomic polling should never occur. > > This was recommended by Chris Wilson as a way of reducing added > complexity to the polled wait which I introduced in an RFC patch. > > 09:57 < ickle_> it's only there as a fudge for waiting after irqs > after uninstalled during s&r, we aren't actually meant to hit it > 09:57 < ickle_> so maybe we should just kill the code there and fix the > breakage > > Cc: Chris Wilson <ch...@chris-wilson.co.uk> > Signed-off-by: Ben Widawsky <benjamin.widaw...@intel.com> > ---
> + if (WARN_ON(!ring->irq_get(ring))) > + return -EBUSY; I think this is now a -ENODEV; :) -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
