Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1464779 > unrolled thread
| Started by | Andy Lutomirski <luto@amacapital.net> |
|---|---|
| First post | 2016-08-17 22:20 +0200 |
| Last post | 2016-08-21 07:40 +0200 |
| Articles | 5 — 4 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [Documentation] State of CPU controller in cgroup v2 Andy Lutomirski <luto@amacapital.net> - 2016-08-17 22:20 +0200
Re: [Documentation] State of CPU controller in cgroup v2 Tejun Heo <tj@kernel.org> - 2016-08-20 18:00 +0200
Re: [Documentation] State of CPU controller in cgroup v2 Andy Lutomirski <luto@amacapital.net> - 2016-08-20 20:50 +0200
Re: [Documentation] State of CPU controller in cgroup v2 Mike Galbraith <umgwanakikbuti@gmail.com> - 2016-08-22 12:20 +0200
Re: [Documentation] State of CPU controller in cgroup v2 James Bottomley <James.Bottomley@HansenPartnership.com> - 2016-08-21 07:40 +0200
| From | Andy Lutomirski <luto@amacapital.net> |
|---|---|
| Date | 2016-08-17 22:20 +0200 |
| Subject | Re: [Documentation] State of CPU controller in cgroup v2 |
| Message-ID | <s7ftE-5TG-1@gated-at.bofh.it> |
On Aug 5, 2016 7:07 PM, "Tejun Heo" <tj@kernel.org> wrote: > > Hello, > > There have been several discussions around CPU controller support. > Unfortunately, no consensus was reached and cgroup v2 is sorely > lacking CPU controller support. This document includes summary of the > situation and arguments along with an interim solution for parties who > want to use the out-of-tree patches for CPU controller cgroup v2 > support. I'll post the two patches as replies for reference. > > Thanks. > > > CPU Controller on Control Group v2 > > August, 2016 Tejun Heo <tj@kernel.org> > > > While most controllers have support for cgroup v2 now, the CPU > controller support is not upstream yet due to objections from the > scheduler maintainers on the basic designs of cgroup v2. This > document explains the current situation as well as an interim > solution, and details the disagreements and arguments. The latest > version of this document can be found at the following URL. > > https://git.kernel.org/cgit/linux/kernel/git/tj/cgroup.git/tree/Documentation/cgroup-v2-cpu.txt?h=cgroup-v2-cpu > > > CONTENTS > > 1. Current Situation and Interim Solution > 2. Disagreements and Arguments > 2-1. Contentious Restrictions > 2-1-1. Process Granularity > 2-1-2. No Internal Process Constraint > 2-2. Impact on CPU Controller > 2-2-1. Impact of Process Granularity > 2-2-2. Impact of No Internal Process Constraint > 2-3. Arguments for cgroup v2 > 3. Way Forward > 4. References > > > 1. Current Situation and Interim Solution > > All objections from the scheduler maintainers apply to cgroup v2 core > design, and there are no known objections to the specifics of the CPU > controller cgroup v2 interface. The only blocked part is changes to > expose the CPU controller interface on cgroup v2, which comprises the > following two patches: > > [1] sched: Misc preps for cgroup unified hierarchy interface > [2] sched: Implement interface for cgroup unified hierarchy > > The necessary changes are superficial and implement the interface > files on cgroup v2. The combined diffstat is as follows. > > kernel/sched/core.c | 149 +++++++++++++++++++++++++++++++++++++++++++++++-- > kernel/sched/cpuacct.c | 57 ++++++++++++------ > kernel/sched/cpuacct.h | 5 + > 3 files changed, 189 insertions(+), 22 deletions(-) > > The patches are easy to apply and forward-port. The following git > branch will always carry the two patches on top of the latest release > of the upstream kernel. > > git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git/cgroup-v2-cpu > > There also are versioned branches going back to v4.4. > > git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git/cgroup-v2-cpu-$KERNEL_VER > > While it's difficult to tell whether the CPU controller support will > be merged, there are crucial resource control features in cgroup v2 > that are only possible due to the design choices that are being > objected to, and every effort will be made to ease enabling the CPU > controller cgroup v2 support out-of-tree for parties which choose to. > > > 2. Disagreements and Arguments > > There have been several lengthy discussion threads [3][4] on LKML > around the structural constraints of cgroup v2. The two that affect > the CPU controller are process granularity and no internal process > constraint. Both arise primarily from the need for common resource > domain definition across different resources. > > The common resource domain is a powerful concept in cgroup v2 that > allows controllers to make basic assumptions about the structural > organization of processes and controllers inside the cgroup hierarchy, > and thus solve problems spanning multiple types of resources. The > prime example for this is page cache writeback: dirty page cache is > regulated through throttling buffered writers based on memory > availability, and initiating batched write outs to the disk based on > IO capacity. Tracking and controlling writeback inside a cgroup thus > requires the direct cooperation of the memory and the IO controller. > > This easily extends to other areas, such as CPU cycles consumed while > performing memory reclaim or IO encryption. > > > 2-1. Contentious Restrictions > > For controllers of different resources to work together, they must > agree on a common organization. This uniform model across controllers > imposes two contentious restrictions on the CPU controller: process > granularity and the no-internal-process constraint. > > > 2-1-1. Process Granularity > > For memory, because an address space is shared between all threads > of a process, the terminal consumer is a process, not a thread. > Separating the threads of a single process into different memory > control domains doesn't make semantical sense. cgroup v2 ensures > that all controller can agree on the same organization by requiring > that threads of the same process belong to the same cgroup. I haven't followed all of the history here, but it seems to me that this argument is less accurate than it appears. Linux, for better or for worse, has somewhat orthogonal concepts of thread groups (processes), mms, and file tables. An mm has VMAs in it, and VMAs can reference things (files, etc) that hold resources. (Two mms can share resources by mapping the same thing or using fork().) File tables hold files, and files can use resources. Both of these are, at best, moderately good approximations of what actually holds resources. Meanwhile, threads (tasks) do syscalls, take page faults, *allocate* resources, etc. So I think it's not really true to say that the "terminal consumer" of anything is a process, not a thread. While it's certainly easier to think about assigning processes to cgroups, and I certainly agree that, in the common case, it's the right thing to do, I don't see why requiring it is a good idea. Can we turn this around: what actually goes wrong if cgroup v2 were to allow assigning individual threads if a user specifically requests it? > > There are other reasons to enforce process granularity. One > important one is isolating system-level management operations from > in-process application operations. The cgroup interface, being a > virtual filesystem, is very unfit for multiple independent > operations taking place at the same time as most operations have to > be multi-step and there is no way to synchronize multiple accessors. > See also [5] Documentation/cgroup-v2.txt, "R-2. Thread Granularity" I don't buy this argument at all. System-level code is likely to assign single process *trees*, which are a different beast entirely. I.e. you fork, move the child into a cgroup, and that child and its children stay in that cgroup. I don't see how the thread/process distinction matters. On the contrary: with cgroup namespaces, one could easily create a cgroup namespace, shove a process in it, and let that process delegate its threads to child cgroups however it likes. (Well, children of the namespace root.) > > > 2-1-2. No Internal Process Constraint > > cgroup v2 does not allow processes to belong to any cgroup which has > child cgroups when resource controllers are enabled on it (the > notable exception being the root cgroup itself). Can you elaborate on this exception? How do you get any of the supposed benefits of not having processes and cgroups exist as siblings when you make an exception for the root? Similarly, if you make an exception for the root, what do you do about cgroup namespaces where the apparent root isn't the global root? --Andy
[toc] | [next] | [standalone]
| From | Tejun Heo <tj@kernel.org> |
|---|---|
| Date | 2016-08-20 18:00 +0200 |
| Message-ID | <s8gQF-4Uh-3@gated-at.bofh.it> |
| In reply to | #1464779 |
Hello, Andy. On Wed, Aug 17, 2016 at 01:18:24PM -0700, Andy Lutomirski wrote: > > 2-1-1. Process Granularity > > > > For memory, because an address space is shared between all threads > > of a process, the terminal consumer is a process, not a thread. > > Separating the threads of a single process into different memory > > control domains doesn't make semantical sense. cgroup v2 ensures > > that all controller can agree on the same organization by requiring > > that threads of the same process belong to the same cgroup. > > I haven't followed all of the history here, but it seems to me that > this argument is less accurate than it appears. Linux, for better or > for worse, has somewhat orthogonal concepts of thread groups > (processes), mms, and file tables. An mm has VMAs in it, and VMAs can > reference things (files, etc) that hold resources. (Two mms can share > resources by mapping the same thing or using fork().) File tables > hold files, and files can use resources. Both of these are, at best, > moderately good approximations of what actually holds resources. > Meanwhile, threads (tasks) do syscalls, take page faults, *allocate* > resources, etc. > > So I think it's not really true to say that the "terminal consumer" of > anything is a process, not a thread. The terminal consumer is actually the mm context. A task may be the allocating entity but not always for itself. This becomes clear whenever an entity is allocating memory on behalf of someone else - get_user_pages(), khugepaged, swapoff and so on (and likely userfaultfd too). When a task is trying to add a page to a VMA, the task might not have any relationship with the VMA other than that it's operating on it for someone else. The page has to be charged to whoever is responsible for the VMA and the only ownership which can be established is the containing mm_struct. While a mm_struct technically may not map to a process, it is a very close approxmiation which is hardly ever broken in practice. > While it's certainly easier to think about assigning processes to > cgroups, and I certainly agree that, in the common case, it's the > right thing to do, I don't see why requiring it is a good idea. Can > we turn this around: what actually goes wrong if cgroup v2 were to > allow assigning individual threads if a user specifically requests it? Consider the scenario where you have somebody faulting on behalf of a foreign VMA, but the thread who created and is actively using that VMA is in a different cgroup than the process leader. Who are we going to charge? All possible answers seem erratic. Please note that I agree that thread granularity can be useful for some resources; however, my points are 1. it should be scoped so that the resource distribution tree as a whole can be shared across different resources, and, 2. cgroup filesystem interface isn't a good interface for the purpose. I'll continue the second point below. > > there are other reasons to enforce process granularity. One > > important one is isolating system-level management operations from > > in-process application operations. The cgroup interface, being a > > virtual filesystem, is very unfit for multiple independent > > operations taking place at the same time as most operations have to > > be multi-step and there is no way to synchronize multiple accessors. > > See also [5] Documentation/cgroup-v2.txt, "R-2. Thread Granularity" > > I don't buy this argument at all. System-level code is likely to > assign single process *trees*, which are a different beast entirely. > I.e. you fork, move the child into a cgroup, and that child and its > children stay in that cgroup. I don't see how the thread/process > distinction matters. Good point on the multi-process issue, this is something which nagged me a bit while working on rgroup, although I have to point out that the issue here is one of not going far enough rather than the approach being wrong. There are limitations to scoping it to individual processes but that doesn't negate the underlying problem or the usefulness of in-process control. For system-level and process-level operations to not step on each other's toes, they need to agree on the granularity boundary - system-level should be able to treat an application hierarchy as a single unit. A possible solution is allowing rgroup hirearchies to span across process boundaries and implementing cgroup migration operations which treat such hierarchies as a single unit. I'm not yet sure whether the boundary should be at program groups or rgroups. > On the contrary: with cgroup namespaces, one could easily create a > cgroup namespace, shove a process in it, and let that process delegate > its threads to child cgroups however it likes. (Well, children of the > namespace root.) cgroup namespace solves just one piece of the whole problem and not in a very robust way. It's okay for containers but not so for individual applications. * Using namespace is neither trivial or dependable. It requires explicit mount setups, and, more importantly, an application can't rely on a specific namespace setup being there, unlike a setpriority() extension. This affects application designs in the first place and severely hampers the accessibility and thus usefulness of in-application resource control. * While it makes the names consistent from inside, it doesn't solve the atomicity issues when system and application operate on the subtree concurrently. Imagine system level operation trying to relocate the namespace. While the symbolic names can be made to stay the same before and after. That's about it. During migration, depending on how migration is implemented, some may see path linking back to the old or new location. Even the open files for the filesystem knobs wouldn't work after such migration. * It's just a bad interface if one has to use setpriority(2) to set a thread priority but resort to opening a file, parse path, open another file, write a number string which uses a completely different value range to it for thread groups. > > 2-1-2. No Internal Process Constraint > > > > cgroup v2 does not allow processes to belong to any cgroup which has > > child cgroups when resource controllers are enabled on it (the > > notable exception being the root cgroup itself). > > Can you elaborate on this exception? How do you get any of the > supposed benefits of not having processes and cgroups exist as > siblings when you make an exception for the root? Similarly, if you > make an exception for the root, what do you do about cgroup namespaces > where the apparent root isn't the global root? Having a special case doesn't necessarily get in the way of benefiting from a set of general rules. The root cgroup is inherently special as it has to be the catch-all scope for entities and resource consumptions which can't be tied to any specific consumer - irq handling, packet rx, journal writes, memory reclaim from global memory pressure and so on. None of sub-cgroups have to worry about them. These base-system operations are special regardless of cgroup and we already have sometimes crude ways to affect their behaviors where necessary through sysctl knobs, priorities on specific kernel threads and so on. cgroup doesn't change the situation all that much. What gets left in the root cgroup usually are the base-system operations which are outside the scope of cgroup resource control in the first place and cgroup resource graph can treat the root as an opaque anchor point. There can be other ways to deal with the issue; however, treating root cgroup this way has the big advantage of minimizing the gap between configurations without and with cgroups both in terms of mental model and implementation. Hopefully, the case of a namespace root is clear now. If it's gonna have a sub-hierarchy, it itself can't contain processes but the system root just contains base-system entities and resources which a namespace root doesn't have to worry about. Ignoring base-system stuff, a namespace root is topologically in the same position as the system root in the cgroup resource graph. Thanks. -- tejun
[toc] | [prev] | [next] | [standalone]
| From | Andy Lutomirski <luto@amacapital.net> |
|---|---|
| Date | 2016-08-20 20:50 +0200 |
| Message-ID | <s8jvb-6Bq-1@gated-at.bofh.it> |
| In reply to | #1466824 |
On Sat, Aug 20, 2016 at 8:56 AM, Tejun Heo <tj@kernel.org> wrote: > Hello, Andy. > > On Wed, Aug 17, 2016 at 01:18:24PM -0700, Andy Lutomirski wrote: >> > 2-1-1. Process Granularity >> > >> > For memory, because an address space is shared between all threads >> > of a process, the terminal consumer is a process, not a thread. >> > Separating the threads of a single process into different memory >> > control domains doesn't make semantical sense. cgroup v2 ensures >> > that all controller can agree on the same organization by requiring >> > that threads of the same process belong to the same cgroup. >> >> I haven't followed all of the history here, but it seems to me that >> this argument is less accurate than it appears. Linux, for better or >> for worse, has somewhat orthogonal concepts of thread groups >> (processes), mms, and file tables. An mm has VMAs in it, and VMAs can >> reference things (files, etc) that hold resources. (Two mms can share >> resources by mapping the same thing or using fork().) File tables >> hold files, and files can use resources. Both of these are, at best, >> moderately good approximations of what actually holds resources. >> Meanwhile, threads (tasks) do syscalls, take page faults, *allocate* >> resources, etc. >> >> So I think it's not really true to say that the "terminal consumer" of >> anything is a process, not a thread. > > The terminal consumer is actually the mm context. A task may be the > allocating entity but not always for itself. > > This becomes clear whenever an entity is allocating memory on behalf > of someone else - get_user_pages(), khugepaged, swapoff and so on (and > likely userfaultfd too). When a task is trying to add a page to a > VMA, the task might not have any relationship with the VMA other than > that it's operating on it for someone else. The page has to be > charged to whoever is responsible for the VMA and the only ownership > which can be established is the containing mm_struct. This surprises me a bit. If I do access_process_vm(), then I would have expected the charge to go the caller, not the mm being accessed. What happens if a program calls read(2), though? A page may be inserted into page cache on behalf of an address_space without any particular mm being involved. There will usually be a calling task, though. But this is all very memcg-specific. What about other cgroups? I/O is per-task, right? Scheduling is definitely per-task. > > While a mm_struct technically may not map to a process, it is a very > close approxmiation which is hardly ever broken in practice. > >> While it's certainly easier to think about assigning processes to >> cgroups, and I certainly agree that, in the common case, it's the >> right thing to do, I don't see why requiring it is a good idea. Can >> we turn this around: what actually goes wrong if cgroup v2 were to >> allow assigning individual threads if a user specifically requests it? > > Consider the scenario where you have somebody faulting on behalf of a > foreign VMA, but the thread who created and is actively using that VMA > is in a different cgroup than the process leader. Who are we going to > charge? All possible answers seem erratic. > Indeed, and this problem is probably not solvable in practice unless you charge all involved cgroups. But the caller's *mm* is entirely irrelevant here, so I don't see how this implies that cgroups need to keep tasks in the same process together. The relevant entities are the calling *task* and the target mm, and you're going to be hard-pressed to ensure that they belong to the same cgroup, so I think you need to be able handle weird cases in which there isn't an obviously correct cgroup to charge. >> > there are other reasons to enforce process granularity. One >> > important one is isolating system-level management operations from >> > in-process application operations. The cgroup interface, being a >> > virtual filesystem, is very unfit for multiple independent >> > operations taking place at the same time as most operations have to >> > be multi-step and there is no way to synchronize multiple accessors. >> > See also [5] Documentation/cgroup-v2.txt, "R-2. Thread Granularity" >> >> I don't buy this argument at all. System-level code is likely to >> assign single process *trees*, which are a different beast entirely. >> I.e. you fork, move the child into a cgroup, and that child and its >> children stay in that cgroup. I don't see how the thread/process >> distinction matters. > > Good point on the multi-process issue, this is something which nagged > me a bit while working on rgroup, although I have to point out that > the issue here is one of not going far enough rather than the approach > being wrong. There are limitations to scoping it to individual > processes but that doesn't negate the underlying problem or the > usefulness of in-process control. > > For system-level and process-level operations to not step on each > other's toes, they need to agree on the granularity boundary - > system-level should be able to treat an application hierarchy as a > single unit. A possible solution is allowing rgroup hirearchies to > span across process boundaries and implementing cgroup migration > operations which treat such hierarchies as a single unit. I'm not yet > sure whether the boundary should be at program groups or rgroups. I think that, if the system cgroup manager is moving processes around after starting them and execing the final binary, there will be races and confusion, and no about of granularity fiddling will fix that. I know nothing about rgroups. Are they upstream? > >> > 2-1-2. No Internal Process Constraint >> > >> > cgroup v2 does not allow processes to belong to any cgroup which has >> > child cgroups when resource controllers are enabled on it (the >> > notable exception being the root cgroup itself). >> >> Can you elaborate on this exception? How do you get any of the >> supposed benefits of not having processes and cgroups exist as >> siblings when you make an exception for the root? Similarly, if you >> make an exception for the root, what do you do about cgroup namespaces >> where the apparent root isn't the global root? > > Having a special case doesn't necessarily get in the way of benefiting > from a set of general rules. The root cgroup is inherently special as > it has to be the catch-all scope for entities and resource > consumptions which can't be tied to any specific consumer - irq > handling, packet rx, journal writes, memory reclaim from global memory > pressure and so on. None of sub-cgroups have to worry about them. > > These base-system operations are special regardless of cgroup and we > already have sometimes crude ways to affect their behaviors where > necessary through sysctl knobs, priorities on specific kernel threads > and so on. cgroup doesn't change the situation all that much. What > gets left in the root cgroup usually are the base-system operations > which are outside the scope of cgroup resource control in the first > place and cgroup resource graph can treat the root as an opaque anchor > point. This seems to explain why the controllers need to be able to handle things being charged to the root cgroup (or to an unidentifiable cgroup, anyway). That isn't quite the same thing as allowing, from an ABI point of view, the root cgroup to contain processes and cgroups but not allowing other cgroups to do the same thing. Consider: suppose that systemd (or some competing cgroup manager) is designed to run in the root cgroup namespace. It presumably expects *itself* to be in the root cgroup. Now try to run it using cgroups v2 in a non-root namespace. I don't see how it can possibly work if it the hierarchy constraints don't permit it to create sub-cgroups while it's still in the root. In fact, this seems impossible to fix even with user code changes. The manager would need to simultaneously create a new child cgroup to contain itself and assign itself to that child cgroup, because the intermediate state is illegal. I really, really think that cgroup v2 should supply the same *interface* inside and outside of a non-root namespace. If this is impossible due to ABI compatibility, then you could, in the worst case, introduce cgroup v3, fix it there, and remove cgroup v2, since apparently cgroup v2 isn't in use right now in mainline kernels. (To be clear, I think either decision -- allowing tasks and cgroups to be siblings or disallowing it -- is okay, but I think that the interface should apply the same constraint at all levels.) --Andy
[toc] | [prev] | [next] | [standalone]
| From | Mike Galbraith <umgwanakikbuti@gmail.com> |
|---|---|
| Date | 2016-08-22 12:20 +0200 |
| Message-ID | <s8UuK-4YJ-3@gated-at.bofh.it> |
| In reply to | #1466824 |
On Sat, 2016-08-20 at 11:56 -0400, Tejun Heo wrote: > > > there are other reasons to enforce process granularity. One > > > important one is isolating system-level management operations from > > > in-process application operations. The cgroup interface, being a > > > virtual filesystem, is very unfit for multiple independent > > > operations taking place at the same time as most operations have to > > > be multi-step and there is no way to synchronize multiple accessors. > > > See also [5] Documentation/cgroup-v2.txt, "R-2. Thread Granularity" > > > > I don't buy this argument at all. System-level code is likely to > > assign single process *trees*, which are a different beast entirely. > > I.e. you fork, move the child into a cgroup, and that child and its > > children stay in that cgroup. I don't see how the thread/process > > distinction matters. > > Good point on the multi-process issue, this is something which nagged > me a bit while working on rgroup, although I have to point out that > the issue here is one of not going far enough rather than the approach > being wrong. There are limitations to scoping it to individual > processes but that doesn't negate the underlying problem or the > usefulness of in-process control. > > For system-level and process-level operations to not step on each > other's toes, they need to agree on the granularity boundary - > system-level should be able to treat an application hierarchy as a > single unit. A possible solution is allowing rgroup hirearchies to > span across process boundaries and implementing cgroup migration > operations which treat such hierarchies as a single unit. I'm not yet > sure whether the boundary should be at program groups or rgroups. Why is it not viable to predicate contentious lowest common denominator restrictions upon the set of enabled controllers? If only thread granularity controllers are enabled, from that point onward, v2 restrictions cease to make any sense, thus could be lifted, leaving nobody cast adrift in a leaky v1 lifeboat when v2 sets sail. Or? -Mike
[toc] | [prev] | [next] | [standalone]
| From | James Bottomley <James.Bottomley@HansenPartnership.com> |
|---|---|
| Date | 2016-08-21 07:40 +0200 |
| Message-ID | <s8tEd-4H7-7@gated-at.bofh.it> |
| In reply to | #1464779 |
On Wed, 2016-08-17 at 13:18 -0700, Andy Lutomirski wrote: > On Aug 5, 2016 7:07 PM, "Tejun Heo" <tj@kernel.org> wrote: [...] > > 2. Disagreements and Arguments > > > > There have been several lengthy discussion threads [3][4] on LKML > > around the structural constraints of cgroup v2. The two that > > affect the CPU controller are process granularity and no internal > > process constraint. Both arise primarily from the need for common > > resource domain definition across different resources. > > > > The common resource domain is a powerful concept in cgroup v2 that > > allows controllers to make basic assumptions about the structural > > organization of processes and controllers inside the cgroup > > hierarchy, and thus solve problems spanning multiple types of > > resources. The prime example for this is page cache writeback: > > dirty page cache is regulated through throttling buffered writers > > based on memory availability, and initiating batched write outs to > > the disk based on IO capacity. Tracking and controlling writeback > > inside a cgroup thus requires the direct cooperation of the memory > > and the IO controller. > > > > This easily extends to other areas, such as CPU cycles consumed > > while performing memory reclaim or IO encryption. > > > > > > 2-1. Contentious Restrictions > > > > For controllers of different resources to work together, they must > > agree on a common organization. This uniform model across > > controllers imposes two contentious restrictions on the CPU > > controller: process granularity and the no-internal-process > > constraint. > > > > > > 2-1-1. Process Granularity > > > > For memory, because an address space is shared between all > > threads > > of a process, the terminal consumer is a process, not a thread. > > Separating the threads of a single process into different memory > > control domains doesn't make semantical sense. cgroup v2 ensures > > that all controller can agree on the same organization by > > requiring > > that threads of the same process belong to the same cgroup. > > I haven't followed all of the history here, but it seems to me that > this argument is less accurate than it appears. Linux, for better or > for worse, has somewhat orthogonal concepts of thread groups > (processes), mms, and file tables. An mm has VMAs in it, and VMAs > can reference things (files, etc) that hold resources. (Two mms can > share resources by mapping the same thing or using fork().) File > tables hold files, and files can use resources. Both of these are, > at best, moderately good approximations of what actually holds > resources. Meanwhile, threads (tasks) do syscalls, take page faults, > *allocate* resources, etc. > > So I think it's not really true to say that the "terminal consumer" > of anything is a process, not a thread. > > While it's certainly easier to think about assigning processes to > cgroups, and I certainly agree that, in the common case, it's the > right thing to do, I don't see why requiring it is a good idea. Can > we turn this around: what actually goes wrong if cgroup v2 were to > allow assigning individual threads if a user specifically requests > it? A similar point from a different consumer: from the unprivileged containers point of view, I'm interested in a thread based interface as well. The principle utility of unprivileged containers is to allow applications that wish to to use container properties (effectively to become self-containerising). Some that use the producer/consumer model do use process pools (apache springs to mind instantly) but some use thread pools. It is useful to the latter to preserve the concept of a thread as being the entity inhabiting the cgroup (but only where the granularity of the cgroup permits threads to participate) so we can easily modify them to be self containerising without forcing them to switch back from a thread pool model to a process pool model. I can see that process based is conceptually easier in v2 because you begin with a process tree, but it would really be a pity to lose the thread based controls we have now and permanently lose the ability to create more as we find uses for them. I can't really see how improving "common resource domain" is a good tradeoff for this. James
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web