[Intel-gfx] [PATCH 08/22] drm/i915/perf: rate limit spurious oa report notice

[Chris Wilson]

On Thu, May 11, 2017 at 04:43:31PM +0100, Lionel Landwerlin wrote:
> From: Robert Bragg <robert at sixbynine.org>
> 
> This change is pre-emptively aiming to avoid a potential cause of kernel
> logging noise in case some condition were to result in us seeing invalid
> OA reports.
> 
> The workaround for the OA unit's tail pointer race condition is what
> avoids the primary known cause of invalid reports being seen and with
> that in place we aren't expecting to see this notice but it can't be
> entirely ruled out.
> 
> Just in case some condition does lead to the notice then it's likely
> that it will be triggered repeatedly while attempting to append a
> sequence of reports and depending on the configured OA sampling
> frequency that might be a large number of repeat notices.
> 
> v2: (Chris) avoid inconsistent warning on throttle with
>     printk_ratelimit()
> v3: (Matt) init and summarise with stream init/close not driver init/fini
> 
> Signed-off-by: Robert Bragg <robert at sixbynine.org>
> Reviewed-by: Matthew Auld <matthew.auld at intel.com>

I've pushed up to this point as this looks to be the end of the bug
fixes and the start of the feature work. However Lionel, then I noticed
that you haven't been adding your s-o-b for the patches you've handled,
it's just a step to ensure the provenance of every patch entering the
kernel through us.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre