Re: [PATCH 0/6 V4] fdinfo shared stats

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 15, 2024 at 9:18 AM Christian König
<ckoenig.leichtzumerken@xxxxxxxxx> wrote:
>
> Am 12.02.24 um 22:04 schrieb Alex Deucher:
> > We had a request to add shared buffer stats to fdinfo for amdgpu and
> > while implementing that, Christian mentioned that just looking at
> > the GEM handle count doesn't take into account buffers shared with other
> > subsystems like V4L or RDMA.  Those subsystems don't use GEM, so it
> > doesn't really matter from a GPU top perspective, but it's more
> > correct if you actually want to see shared buffers.
> >
> > After further discussions, add a helper and update all fdinfo
> > implementations to use that helper for consistency.
> >
> > v4: switch drm_gem_object_is_shared_for_memory_stats() to an inline function
>
> I'm still not sure if looking at the actual handle count is the right
> approach, but it's certainly better than before.

Well, it's consistent across drivers.

>
> So Reviewed-by: Christian König <christian.koenig@xxxxxxx> for the
> entire series.
>
> Should I take this through drm-misc-next?

Yes, please.

Thanks,

Alex

>
> Regards,
> Christian.
>
> >
> > Alex Deucher (6):
> >    Documentation/gpu: Update documentation on drm-shared-*
> >    drm: add drm_gem_object_is_shared_for_memory_stats() helper
> >    drm: update drm_show_memory_stats() for dma-bufs
> >    drm/amdgpu: add shared fdinfo stats
> >    drm/i915: Update shared stats to use the new gem helper
> >    drm/xe: Update shared stats to use the new gem helper
> >
> >   Documentation/gpu/drm-usage-stats.rst      |  2 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_fdinfo.c |  4 ++++
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 11 +++++++++++
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_object.h |  6 ++++++
> >   drivers/gpu/drm/drm_file.c                 |  2 +-
> >   drivers/gpu/drm/i915/i915_drm_client.c     |  2 +-
> >   drivers/gpu/drm/xe/xe_drm_client.c         |  2 +-
> >   include/drm/drm_gem.h                      | 13 +++++++++++++
> >   8 files changed, 38 insertions(+), 4 deletions(-)
> >
>




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux