[Intel-gfx] [PATCH 33/39] drm/i915/perf: add a parameter to control the size of OA buffer

[Lucas De Marchi]

On Fri, Aug 23, 2019 at 1:13 AM Lionel Landwerlin
<lionel.g.landwerlin at intel.com> wrote:
>
> On 22/08/2019 12:43, Joonas Lahtinen wrote:
> > Quoting Lucas De Marchi (2019-08-16 11:04:57)
> >> From: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> >>
> >> The way our hardware is designed doesn't seem to let us use the
> >> MI_RECORD_PERF_COUNT command without setting up a circular buffer.
> >>
> >> In the case where the user didn't request OA reports to be available
> >> through the i915 perf stream, we can set the OA buffer to the minimum
> >> size to avoid consuming memory which won't be used by the driver.
> >>
> >> Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> >> Cc: Matthew Auld <matthew.auld at intel.com>
> >> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> >> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
> >> ---
> >>   drivers/gpu/drm/i915/i915_perf.c | 98 +++++++++++++++++++++-----------
> >>   drivers/gpu/drm/i915/i915_reg.h  |  2 +
> >>   include/uapi/drm/i915_drm.h      |  7 +++
> > Any patch touching i915_drm.h should have a link to the corresponding
> > userspace series.
> >
> > Regards, Joonas
> >
> Might be worth dropping this patch from the series then.
>
> I don't think we have anything public or any coming userspace release
> making use of this.

This is dropped from v3.

Lucas De Marchi

>
>
> Cheers,
>
>
> -Lionel
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx