Account for the total size of buffer object requested to amdgpu by
buffer type on a per cgroup basis.

x prefix in the control file name x.bo_requested.amd.stat signify
experimental.

Change-Id: Ifb680c4bcf3652879a7a659510e25680c2465cf6
Signed-off-by: Kenny Ho <kenny...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.c | 56 +++++++++++++++++++++
 drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.h |  3 ++
 drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c     | 13 +++++
 include/uapi/drm/amdgpu_drm.h               | 24 ++++++---
 4 files changed, 90 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.c
index 853b77532428..e3d98ed01b79 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.c
@@ -7,6 +7,57 @@
 #include "amdgpu_ring.h"
 #include "amdgpu_drmcgrp.h"
 
+void amdgpu_drmcgrp_count_bo_req(struct task_struct *task, struct drm_device 
*dev,
+               u32 domain, unsigned long size)
+{
+       struct drmcgrp *drmcgrp = get_drmcgrp(task);
+       struct drmcgrp_device_resource *ddr;
+       struct drmcgrp *p;
+       struct amd_drmcgrp_dev_resource *a_ddr;
+        int i;
+
+       if (drmcgrp == NULL)
+               return;
+
+       ddr = drmcgrp->dev_resources[dev->primary->index];
+
+       mutex_lock(&ddr->ddev->mutex);
+       for (p = drmcgrp; p != NULL; p = parent_drmcgrp(drmcgrp)) {
+               a_ddr = ddr_amdddr(p->dev_resources[dev->primary->index]);
+
+               for (i = 0; i < __MAX_AMDGPU_MEM_DOMAIN; i++)
+                       if ( (1 << i) & domain)
+                               a_ddr->bo_req_count[i] += size;
+       }
+       mutex_unlock(&ddr->ddev->mutex);
+}
+
+int amd_drmcgrp_bo_req_stat_read(struct seq_file *sf, void *v)
+{
+       struct drmcgrp *drmcgrp = css_drmcgrp(seq_css(sf));
+       struct drmcgrp_device_resource *ddr = NULL;
+       struct amd_drmcgrp_dev_resource *a_ddr = NULL;
+       int i, j;
+
+       seq_puts(sf, "---\n");
+       for (i = 0; i < MAX_DRM_DEV; i++) {
+               ddr = drmcgrp->dev_resources[i];
+
+               if (ddr == NULL || ddr->ddev->vid != amd_drmcgrp_vendor_id)
+                       continue;
+
+               a_ddr = ddr_amdddr(ddr);
+
+               seq_printf(sf, "card%d:\n", i);
+               for (j = 0; j < __MAX_AMDGPU_MEM_DOMAIN; j++)
+                       seq_printf(sf, "  %s: %llu\n", 
amdgpu_mem_domain_names[j], a_ddr->bo_req_count[j]);
+       }
+
+       return 0;
+}
+
+
+
 void amdgpu_drmcgrp_count_cs(struct task_struct *task, struct drm_device *dev,
                enum amdgpu_ring_type r_type)
 {
@@ -55,6 +106,11 @@ int amd_drmcgrp_cmd_submit_accounting_read(struct seq_file 
*sf, void *v)
 
 
 struct cftype files[] = {
+       {
+               .name = "x.bo_requested.amd.stat",
+               .seq_show = amd_drmcgrp_bo_req_stat_read,
+               .flags = CFTYPE_NOT_ON_ROOT,
+       },
        {
                .name = "x.cmd_submitted.amd.stat",
                .seq_show = amd_drmcgrp_cmd_submit_accounting_read,
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.h 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.h
index f894a9a1059f..8b9d61e47dde 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drmcgrp.h
@@ -11,10 +11,13 @@
 struct amd_drmcgrp_dev_resource {
        struct drmcgrp_device_resource ddr;
        u64 cs_count[__MAX_AMDGPU_RING_TYPE];
+       u64 bo_req_count[__MAX_AMDGPU_MEM_DOMAIN];
 };
 
 void amdgpu_drmcgrp_count_cs(struct task_struct *task, struct drm_device *dev,
                enum amdgpu_ring_type r_type);
+void amdgpu_drmcgrp_count_bo_req(struct task_struct *task, struct drm_device 
*dev,
+               u32 domain, unsigned long size);
 
 static inline struct amd_drmcgrp_dev_resource *ddr_amdddr(struct 
drmcgrp_device_resource *ddr)
 {
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
index 7b3d1ebda9df..339e1d3edad8 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
@@ -31,6 +31,17 @@
 #include <drm/amdgpu_drm.h>
 #include "amdgpu.h"
 #include "amdgpu_display.h"
+#include "amdgpu_drmcgrp.h"
+
+char const *amdgpu_mem_domain_names[] = {
+       [AMDGPU_MEM_DOMAIN_CPU]         = "cpu",
+       [AMDGPU_MEM_DOMAIN_GTT]         = "gtt",
+       [AMDGPU_MEM_DOMAIN_VRAM]        = "vram",
+       [AMDGPU_MEM_DOMAIN_GDS]         = "gds",
+       [AMDGPU_MEM_DOMAIN_GWS]         = "gws",
+       [AMDGPU_MEM_DOMAIN_OA]          = "oa",
+       [__MAX_AMDGPU_MEM_DOMAIN]       = "_max"
+};
 
 void amdgpu_gem_object_free(struct drm_gem_object *gobj)
 {
@@ -52,6 +63,8 @@ int amdgpu_gem_object_create(struct amdgpu_device *adev, 
unsigned long size,
        struct amdgpu_bo_param bp;
        int r;
 
+       amdgpu_drmcgrp_count_bo_req(current, adev->ddev, initial_domain, size);
+
        memset(&bp, 0, sizeof(bp));
        *obj = NULL;
        /* At least align on page size */
diff --git a/include/uapi/drm/amdgpu_drm.h b/include/uapi/drm/amdgpu_drm.h
index 370e9a5536ef..531726443104 100644
--- a/include/uapi/drm/amdgpu_drm.h
+++ b/include/uapi/drm/amdgpu_drm.h
@@ -72,6 +72,18 @@ extern "C" {
 #define DRM_IOCTL_AMDGPU_FENCE_TO_HANDLE DRM_IOWR(DRM_COMMAND_BASE + 
DRM_AMDGPU_FENCE_TO_HANDLE, union drm_amdgpu_fence_to_handle)
 #define DRM_IOCTL_AMDGPU_SCHED         DRM_IOW(DRM_COMMAND_BASE + 
DRM_AMDGPU_SCHED, union drm_amdgpu_sched)
 
+enum amdgpu_mem_domain {
+       AMDGPU_MEM_DOMAIN_CPU,
+       AMDGPU_MEM_DOMAIN_GTT,
+       AMDGPU_MEM_DOMAIN_VRAM,
+       AMDGPU_MEM_DOMAIN_GDS,
+       AMDGPU_MEM_DOMAIN_GWS,
+       AMDGPU_MEM_DOMAIN_OA,
+       __MAX_AMDGPU_MEM_DOMAIN
+};
+
+extern char const *amdgpu_mem_domain_names[];
+
 /**
  * DOC: memory domains
  *
@@ -95,12 +107,12 @@ extern "C" {
  * %AMDGPU_GEM_DOMAIN_OA       Ordered append, used by 3D or Compute engines
  * for appending data.
  */
-#define AMDGPU_GEM_DOMAIN_CPU          0x1
-#define AMDGPU_GEM_DOMAIN_GTT          0x2
-#define AMDGPU_GEM_DOMAIN_VRAM         0x4
-#define AMDGPU_GEM_DOMAIN_GDS          0x8
-#define AMDGPU_GEM_DOMAIN_GWS          0x10
-#define AMDGPU_GEM_DOMAIN_OA           0x20
+#define AMDGPU_GEM_DOMAIN_CPU          (1 << AMDGPU_MEM_DOMAIN_CPU)
+#define AMDGPU_GEM_DOMAIN_GTT          (1 << AMDGPU_MEM_DOMAIN_GTT)
+#define AMDGPU_GEM_DOMAIN_VRAM         (1 << AMDGPU_MEM_DOMAIN_VRAM)
+#define AMDGPU_GEM_DOMAIN_GDS          (1 << AMDGPU_MEM_DOMAIN_GDS)
+#define AMDGPU_GEM_DOMAIN_GWS          (1 << AMDGPU_MEM_DOMAIN_GWS)
+#define AMDGPU_GEM_DOMAIN_OA           (1 << AMDGPU_MEM_DOMAIN_OA)
 #define AMDGPU_GEM_DOMAIN_MASK         (AMDGPU_GEM_DOMAIN_CPU | \
                                         AMDGPU_GEM_DOMAIN_GTT | \
                                         AMDGPU_GEM_DOMAIN_VRAM | \
-- 
2.19.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to