linux-kernel - Re: [PATCH v2 0/7] kernel/cgroups: Add "dmem" memory accounting cgroup.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <29a71119-04de-4c76-a98a-d0fcb906390f@gmx.de>
Date: Sun, 8 Dec 2024 13:15:34 +0100
From: Friedrich Vock <friedrich.vock@....de>
To: Maarten Lankhorst <dev@...khorst.se>, linux-kernel@...r.kernel.org,
 intel-xe@...ts.freedesktop.org, dri-devel@...ts.freedesktop.org,
 Tejun Heo <tj@...nel.org>, Zefan Li <lizefan.x@...edance.com>,
 Johannes Weiner <hannes@...xchg.org>,
 Andrew Morton <akpm@...ux-foundation.org>, Maxime Ripard <mripard@...nel.org>
Cc: cgroups@...r.kernel.org, linux-mm@...ck.org,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>
Subject: Re: [PATCH v2 0/7] kernel/cgroups: Add "dmem" memory accounting
 cgroup.

Hi,

On 04.12.24 14:44, Maarten Lankhorst wrote:
> New update. Instead of calling it the 'dev' cgroup, it's now the 'dmem' cgroup.

Thanks! I think this version looks pretty good.

>
> Because it only deals with memory regions, the UAPI has been updated to use dmem.min/low/max/current, and to make the API cleaner, the names are changed too.
>
> dmem.current could contain a line like:
> "drm/0000:03:00.0/vram0 1073741824"
>
> But I think using "drm/card0/vram0" instead of PCIID would perhaps be good too. I'm open to changing it to that based on feedback.

Agree, allowing userspace to reference DRM devices via "cardN" syntax
sounds good. What about other subsystems potentially using dmem cgroups?
I'm not familiar with the media subsystem, but I imagine we might be
dealing with things like USB devices there? Is something like a
"deviceN" possible there as well, or would device IDs look completely
different?

Regards,
Friedrich

>
> I've created an IGT test for min and max, and found the changes
> from Friedrich Vock sent as feedback were needed.
> I've integrated those into the first patch.
>
> Maarten Lankhorst (5):
>    kernel/cgroup: Add "dmem" memory accounting cgroup
>    drm/ttm: Handle cgroup based eviction in TTM
>    drm/xe: Implement cgroup for vram
>    drm/amdgpu: Add cgroups implementation
>    drm/xe: Hack to test with mapped pages instead of vram.
>
> Maxime Ripard (2):
>    drm/drv: Add drmm managed registration helper for dmem cgroups.
>    drm/gem: Add cgroup memory accounting for VRAM helper.
>
>   Documentation/admin-guide/cgroup-v2.rst       |  58 +-
>   Documentation/core-api/cgroup.rst             |   9 +
>   Documentation/core-api/index.rst              |   1 +
>   Documentation/gpu/drm-compute.rst             |  54 ++
>   drivers/gpu/drm/amd/amdgpu/amdgpu_vram_mgr.c  |   4 +
>   drivers/gpu/drm/drm_drv.c                     |  32 +
>   drivers/gpu/drm/drm_gem_vram_helper.c         |  15 +-
>   drivers/gpu/drm/ttm/tests/ttm_bo_test.c       |  18 +-
>   .../gpu/drm/ttm/tests/ttm_bo_validate_test.c  |   4 +-
>   drivers/gpu/drm/ttm/tests/ttm_resource_test.c |   2 +-
>   drivers/gpu/drm/ttm/ttm_bo.c                  |  54 +-
>   drivers/gpu/drm/ttm/ttm_resource.c            |  23 +-
>   drivers/gpu/drm/xe/xe_ttm_sys_mgr.c           |   5 +
>   drivers/gpu/drm/xe/xe_ttm_vram_mgr.c          |   8 +
>   include/drm/drm_drv.h                         |   5 +
>   include/drm/ttm/ttm_resource.h                |  12 +-
>   include/linux/cgroup_dmem.h                   |  67 ++
>   include/linux/cgroup_subsys.h                 |   4 +
>   include/linux/page_counter.h                  |   2 +-
>   init/Kconfig                                  |  10 +
>   kernel/cgroup/Makefile                        |   1 +
>   kernel/cgroup/dmem.c                          | 861 ++++++++++++++++++
>   mm/page_counter.c                             |   4 +-
>   23 files changed, 1219 insertions(+), 34 deletions(-)
>   create mode 100644 Documentation/core-api/cgroup.rst
>   create mode 100644 Documentation/gpu/drm-compute.rst
>   create mode 100644 include/linux/cgroup_dmem.h
>   create mode 100644 kernel/cgroup/dmem.c
>