Quoting Ramalingam C (2019-12-20 15:11:57)
> From: Lukasz Fiedorowicz <lukasz.fiedorowicz@xxxxxxxxx>
>
> Debugfs i915_gem_object is extended to enable the IGTs to
> detect the LMEM's availability and the total size of LMEM.
>
> v2: READ_ONCE is used [Chris]
>
> Signed-off-by: Lukasz Fiedorowicz <lukasz.fiedorowicz@xxxxxxxxx>
> Signed-off-by: Matthew Auld <matthew.auld@xxxxxxxxx>
> Signed-off-by: Stuart Summers <stuart.summers@xxxxxxxxx>
> Signed-off-by: Ramalingam C <ramalingam.c@xxxxxxxxx>
> Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> ---
> drivers/gpu/drm/i915/i915_debugfs.c | 6 +++++-
> drivers/gpu/drm/i915/intel_memory_region.c | 5 ++++-
> drivers/gpu/drm/i915/intel_memory_region.h | 2 ++
> 3 files changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index d28468eaed57..7e8e5112da25 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -373,7 +373,11 @@ static int i915_gem_object_info(struct seq_file *m, void *data)
> atomic_read(&i915->mm.free_count),
> i915->mm.shrink_memory);
>
> - seq_putc(m, '\n');
> + if (HAS_LMEM(i915)) {
> + seq_printf(m, "LMEM total: %llu bytes, available %llu bytes\n",
> + READ_ONCE(i915->mm.regions[INTEL_REGION_LMEM]->total),
> + READ_ONCE(i915->mm.regions[INTEL_REGION_LMEM]->avail));
It has to be %pa &total, &avail. That removes the READ_ONE() :(
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
