public inbox for cgroups@vger.kernel.org
 help / color / mirror / Atom feed
From: Tejun Heo <tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Matt Roper <matthew.d.roper-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 1/2] Documentation/cgroup-v1: fix outdated programming details
Date: Tue, 2 Jan 2018 07:05:02 -0800	[thread overview]
Message-ID: <20180102150502.GC3668920@devbig577.frc2.facebook.com> (raw)
In-Reply-To: <20171229203449.GS5820-b/RNqDZ/lqH1fpGqjiHozbKMmGWinSIL2HeeBUIffwg@public.gmane.org>

Hello, Matt.

On Fri, Dec 29, 2017 at 12:34:49PM -0800, Matt Roper wrote:
> If a system is already using a cgroups-v2 hierarchy to manage other
> system resources via the standard controllers, I think it would be ideal
> if we could leverage that existing process organization to supply i915
> with desired driver-specific policy and resource assignments.  Since
> cgroup controllers don't seem to support this at the moment, is there an
> alternate mechanism I should be looking at instead?  Or is this a type
> of use case that we may want to evolve cgroups to support in a different
> manner?

cgroup membership of a task and the hierarchical relationships of
cgroups can be determined trivially.  Unless the resource in question
needs to and can strictly follow the resource rules for cgroup
controllers, which can become really involving and invasive, the
better and easier approach is using cgroup membership as an extra
information from the subsystem, which is how the network and bpf
handle cgroup membership too.

Thanks.

-- 
tejun

  parent reply	other threads:[~2018-01-02 15:05 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-29 20:01 [PATCH 1/2] Documentation/cgroup-v1: fix outdated programming details Matt Roper
2017-12-29 20:02 ` [PATCH 2/2] cgroup: Update documentation reference Matt Roper
2018-01-02 15:00   ` Tejun Heo
     [not found] ` <20171229200200.18873-1-matthew.d.roper-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2017-12-29 20:34   ` [PATCH 1/2] Documentation/cgroup-v1: fix outdated programming details Matt Roper
     [not found]     ` <20171229203449.GS5820-b/RNqDZ/lqH1fpGqjiHozbKMmGWinSIL2HeeBUIffwg@public.gmane.org>
2018-01-02 15:05       ` Tejun Heo [this message]
     [not found]         ` <20180102150502.GC3668920-4dN5La/x3IkLX0oZNxdnEQ2O0Ztt9esIQQ4Iyu8u01E@public.gmane.org>
2018-01-03  0:12           ` Matt Roper
     [not found]             ` <20180103001202.GX5820-b/RNqDZ/lqH1fpGqjiHozbKMmGWinSIL2HeeBUIffwg@public.gmane.org>
2018-01-03  0:37               ` Tejun Heo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180102150502.GC3668920@devbig577.frc2.facebook.com \
    --to=tj-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
    --cc=cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=matthew.d.roper-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox