[Intel-gfx] [PATCH] fix interrupt request miss problem in bsd ring for g4x
Chris Wilson
chris at chris-wilson.co.uk
Wed Apr 27 09:39:14 CEST 2011
On Wed, 27 Apr 2011 14:08:57 +0800, "Feng, Boqun" <boqun.feng at intel.com> wrote:
> I am very sorry for my careless about whitespace.
>
> But my patch will not affect gen6+ paths, for gen6+, it use gen6_bsd_ring
> , bsd_ring is only used by g4x and ironlake.
Reviewer error, sorry. Saw the gen6_* in the diff header as the function
affected and believed it.
> Besides, since bsd_ring_get_irq/bsd_ring_put_irq/ring_get_irq/ring_put_irq
> are only used by bsd_ring, can we use a patch to merge them into two function?
Yes, once upon a time they differed, now they are the same so please do
merge them and give them a more useful name: g4x_ring_* so that there is a
constant reminder that g4x also has a BSD ring and that the functions are
not expected to be used with earlier chipsets.
Daniel has done similar things for gen6 once we decided to drop the
pre-production workarounds.
Obviously that is a separate patch to the bug fix. Thanks,
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list