[Intel-xe] [PATCH 0/7] drm/xe: fdinfo memory stats

[Upadhyay, Tejas]

> -----Original Message-----
> From: Ghimiray, Himal Prasad <himal.prasad.ghimiray at intel.com>
> Sent: Thursday, September 7, 2023 2:52 PM
> To: Upadhyay, Tejas <tejas.upadhyay at intel.com>; intel-
> xe at lists.freedesktop.org
> Cc: Iddamsetty, Aravind <aravind.iddamsetty at intel.com>
> Subject: Re: [PATCH 0/7] drm/xe: fdinfo memory stats
> 
> 
> On 31-08-2023 14:35, Tejas Upadhyay wrote:
> > This series enables enables fdinfo memory stats for xe.
> >
> > I added tracking of most classes of bo (user bo, page tables, ring,
> > lrc) which contribute to client's memory usage. This follows similar
> > lines to what Rob's msm code and tvrtko's i915 code is doing with some
> > xe way of showing and tracking/fecthing objects which adds to client
> > memory stats.
> >
> > Example fdinfo format:
> >
> > #cat /proc/4620/fdinfo/3
> > pos:    0
> > flags:  0100002
> > mnt_id: 23
> > ino:    1116
> > drm-driver:     xe
> > drm-client-id:  13
> > drm-pdev:       0000:4d:00.0
> > drm-total-system:       0
> > drm-shared-system:      0
> > drm-active-system:      0
> > drm-resident-system:    0
> > drm-purgeable-system:   0
> > drm-total-gtt:  4 KiB
> > drm-shared-gtt: 0
> > drm-active-gtt: 0
> > drm-resident-gtt:       4 KiB
> > drm-purgeable-gtt:      0
> > drm-total-vram0:        20 KiB
> > drm-shared-vram0:       0
> > drm-active-vram0:       0
> > drm-resident-vram0:     20 KiB
> > drm-purgeable-vram0:    0
> > drm-total-vram1:        20 KiB
> > drm-shared-vram1:       0
> > drm-active-vram1:       0
> > drm-resident-vram1:     20 KiB
> > drm-purgeable-vram1:    0
> 
> drm-purgeable-vram0, drm-purgeable-vram1 ?
> Are they correct entries ? Seems issue in PATCH7.

Yes I will disable purgeable flag for vrams while exporting stats. As discussed with Aravind, purgeable is only applicable on BO in PL_SYSTEM because by default they are considered purgeable.

Thanks,
Tejas 
> 
> BR
> Himal
> 
> >
> > Signed-off-by: Tejas Upadhyay <tejas.upadhyay at intel.com>
> >
> > Tejas Upadhyay (7):
> >    drm/xe: Add drm-client infrastructure
> >    drm/xe: Interface xe drm client with fdinfo interface
> >    drm/xe: Add tracking support for bos per client
> >    drm/xe: Record each drm client with its VM
> >    drm/xe: Track page table memory usage for client
> >    drm/xe: Account ring buffer and context state storage
> >    drm/xe: Implement fdinfo memory stats printing
> >
> >   drivers/gpu/drm/xe/Makefile          |   1 +
> >   drivers/gpu/drm/xe/xe_bo.c           |   7 +
> >   drivers/gpu/drm/xe/xe_bo.h           |  11 ++
> >   drivers/gpu/drm/xe/xe_bo_types.h     |  10 ++
> >   drivers/gpu/drm/xe/xe_device.c       |  19 ++-
> >   drivers/gpu/drm/xe/xe_device_types.h |   6 +
> >   drivers/gpu/drm/xe/xe_drm_client.c   | 228
> +++++++++++++++++++++++++++
> >   drivers/gpu/drm/xe/xe_drm_client.h   |  71 +++++++++
> >   drivers/gpu/drm/xe/xe_lrc.c          |   7 +-
> >   drivers/gpu/drm/xe/xe_pt.c           |   4 +-
> >   drivers/gpu/drm/xe/xe_vm.c           |  10 +-
> >   drivers/gpu/drm/xe/xe_vm_types.h     |   2 +
> >   12 files changed, 372 insertions(+), 4 deletions(-)
> >   create mode 100644 drivers/gpu/drm/xe/xe_drm_client.c
> >   create mode 100644 drivers/gpu/drm/xe/xe_drm_client.h
> >