[PATCH RFC v4 02/16] cgroup: Introduce cgroup for drm subsystem

[Tejun Heo]

On Fri, Nov 29, 2019 at 01:00:36AM -0500, Kenny Ho wrote:
> On Tue, Oct 1, 2019 at 10:31 AM Michal Koutný <mkoutny at suse.com> wrote:
> > On Thu, Aug 29, 2019 at 02:05:19AM -0400, Kenny Ho <Kenny.Ho at amd.com> wrote:
> > > +struct cgroup_subsys drm_cgrp_subsys = {
> > > +     .css_alloc      = drmcg_css_alloc,
> > > +     .css_free       = drmcg_css_free,
> > > +     .early_init     = false,
> > > +     .legacy_cftypes = files,
> > Do you really want to expose the DRM controller on v1 hierarchies (where
> > threads of one process can be in different cgroups, or children cgroups
> > compete with their parents)?
> 
> (Sorry for the delay, I have been distracted by something else.)
> Yes, I am hoping to make the functionality as widely available as
> possible since the ecosystem is still transitioning to v2.  Do you see
> inherent problem with this approach?

Integrating with memcg could be more challenging on cgroup1.  That's
one of the reasons why e.g. cgroup-aware pagecache writeback is only
on cgroup2.

-- 
tejun