On Wed, Apr 02, 2014 at 12:42:11PM +0100, Damien Lespiau wrote: > On Thu, Mar 27, 2014 at 06:00:10PM +0000, oscar.mateo@xxxxxxxxx wrote: > > @@ -543,6 +545,9 @@ static int init_ring_common_lrc(struct intel_engine *ring) > > ringbuf->space = ringbuf->size; > > ringbuf->last_retired_head = -1; > > > > + I915_WRITE_IMR(ring, ~(ring->irq_enable_mask | ring->irq_keep_mask)); > > + I915_WRITE(RING_HWSTAM(ring->mmio_base), ~(ring->irq_enable_mask | ring->irq_keep_mask)); > > + > > return 0; > > } > > > > Two little things: > > - I don't see any place where we look at the interrupt reporting in > the HWS page, so we could just initialize HWSTAM to 0xffffffff It's an old w/a to make interrupt signalling a little bit more coherent. No idea whether we still need it since we don't really have a good testcase for interrupts ... I guess we could give it a shot with a patch and a big commit message citing all the history. -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