sched/topology: Improve comments
authorPeter Zijlstra <peterz@infradead.org>
Mon, 21 Aug 2017 13:42:52 +0000 (15:42 +0200)
committerIngo Molnar <mingo@kernel.org>
Fri, 25 Aug 2017 09:12:19 +0000 (11:12 +0200)
Mike provided a better comment for destroy_sched_domain() ...

Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
kernel/sched/topology.c

index 4197f13..179b90b 100644 (file)
@@ -344,8 +344,9 @@ static void free_sched_groups(struct sched_group *sg, int free_sgc)
 static void destroy_sched_domain(struct sched_domain *sd)
 {
        /*
-        * A sched domain has many groups' reference, and an overlapping
-        * domain has private groups, iterate and nuke them all.
+        * A normal sched domain may have multiple group references, an
+        * overlapping domain, having private groups, only one.  Iterate,
+        * dropping group/capacity references, freeing where none remain.
         */
        free_sched_groups(sd->groups, 1);