On Fri, Aug 23, 2019 at 1:13 AM Lionel Landwerlin <lionel.g.landwer...@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.landwer...@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.lahti...@linux.intel.com> > >> Cc: Matthew Auld <matthew.a...@intel.com> > >> Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> > >> Signed-off-by: Lucas De Marchi <lucas.demar...@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@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
