On Wed, 16 Jan 2019 at 15:36, Lionel Landwerlin <[email protected]> wrote: > > No issues have been raised about this yet, but this should be reset > every time we enable the stream, otherwise we might have a stale value > from the previous round of enable/disable. > > Signed-off-by: Lionel Landwerlin <[email protected]> > --- > drivers/gpu/drm/i915/i915_perf.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/i915/i915_perf.c > b/drivers/gpu/drm/i915/i915_perf.c > index 575701e2aabc..4c5d2ee4d6e3 100644 > --- a/drivers/gpu/drm/i915/i915_perf.c > +++ b/drivers/gpu/drm/i915/i915_perf.c > @@ -1876,6 +1876,8 @@ static void i915_oa_stream_enable(struct > i915_perf_stream *stream) > { > struct drm_i915_private *dev_priv = stream->dev_priv; > > + dev_priv->perf.oa.pollin = false; > + > dev_priv->perf.oa.ops.oa_enable(stream);
Don't we fumigate oa.polling way down in ops.oa_enable() ? _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
