On Thu, 2017-10-05 at 11:54 -0700, Michel Thierry wrote: > On 10/5/2017 11:28 AM, Chris Wilson wrote: > > Quoting Michel Thierry (2017-10-05 19:19:27) > > > WaDisableCtxRestoreArbitration was only applied for bdw and chv, but > > > this changed after the code got moved to gen8_emit_bb_start (and, at > > > least in my tree, there is no gen9_emit_bb_start). > > > > But I need to make sure we enable arbitration at some point. :) > > It may not need the w/a name, but I was making very sure we didn't start > > a BB without ARB, or context-switch into the middle of a batch without > > ARB. That was my thinking, that there was no harm in overkill if we set > > it more often than strictly necessary. > > Ok, thanks for the detailed explanation and yes, probably no harm in > setting MI_ARB_ON. Please add a comment if this is the case. W/As implementing required logic for platforms not covered by the W/A is confusing when they are being reviewed or double-checked. Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
