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

From: Maarten Lankhorst
Date: Tue Dec 17 2024 - 15:17:57 EST


Hey,

Den 2024-12-17 kl. 19:23, skrev Tejun Heo:
Hello,

On Tue, Dec 17, 2024 at 06:37:22PM +0100, Maarten Lankhorst wrote:
Den 2024-12-17 kl. 18:11, skrev Tejun Heo:
On Tue, Dec 17, 2024 at 03:28:50PM +0100, Maarten Lankhorst wrote:
Now that all patches look good, what is needed to merge the series? Without
patch 6/7 as it is a hack for testing.

There were some questions raised about device naming. One thing we want to
get right from the beginning is the basic interface.

Thanks.

I believe it was solved. The conclusion appears to be that we go with how we
defined it in this series. drm/$pciid/$regionname. With the only regions
defined now being VRAM. Main memory will be a followup, but requires some
discussions on hwo to be prevent double accounting, and what to do with the
limited amount of mappable memory.

Provided Johannes is okay with the series, how do you want to route the
series? If you want to route it through drm, that's fine by me and please
feel free to add:

Acked-by: Tejun Heo <tj@xxxxxxxxxx>

Thanks.


Thank you!

I've discussed this with the DRM maintainers. What was suggested is to create a topic branch, merge it to drm-misc whichwhere it will be picked up into drm.git during the next pull request. At the same time the topic branch can be also be merged into the cgroup tree if needed.

The drm-misc tree already handles dma-buf and fbdev core, think DMEM could fit in there too.

Cheers,
~Maarten