On Wed, Nov 09, 2011 at 12:17:06PM -0800, Eric Anholt wrote: > On Wed, 09 Nov 2011 19:15:06 +0000, Chris Wilson <chris at chris-wilson.co.uk> wrote: > > On Wed, 09 Nov 2011 10:57:00 -0800, Eric Anholt <eric at anholt.net> wrote: > > > On Wed, 9 Nov 2011 17:32:26 +0000, Chris Wilson <chris at chris-wilson.co.uk> wrote: > > > > We are advised that in order to workaround the TLB invalidation being > > > > performed asynchronously with the command streamer that we need to use > > > > a post-sync write along with the invalidations in the MI_FLUSH_DW so > > > > that all operations are complete before executing the next command in > > > > the ringbuffer. > > > > > > > > References: https://bugs.freedesktop.org/show_bug.cgi?id=37990 > > > > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk> > > > > Cc: Jesse Barnes <jbarnes at virtuousgeek.org> > > > > > > I find the "References:" to a bug that this is not confirmed to fix to > > > be very misleading. > > > > Because it references a bug I think is due to TLB invalidations as the > > glitches look very similar to earlier TLB invalidation. > > That sort of context included with the bz reference would make me feel > more comfortable. What's the status of this patch? If you resend can you clarify the bugzilla reference and also include a Bspec/PRM/whatever citation for this workaround? Thanks, Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48
