diff options
| author |   Peter Zijlstra <peterz@infradead.org> | 2017-08-21 15:42:52 +0200 |
|---|---|---|
| committer |   Ingo Molnar <mingo@kernel.org> | 2017-08-25 11:12:19 +0200 |
| commit | a090c4f2cd2cfeb54eaf8ad1e726a6c485f3abc3 (patch) | |
| tree | bf53680ac0601a1abc107a9591989c1ee634038a /kernel/sched | |
| parent | sched/topology: Fix memory leak in __sdt_alloc() (diff) | |
| download | linux-dev-a090c4f2cd2cfeb54eaf8ad1e726a6c485f3abc3.tar.xz linux-dev-a090c4f2cd2cfeb54eaf8ad1e726a6c485f3abc3.zip | |
sched/topology: Improve comments
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>
Diffstat (limited to 'kernel/sched')
| -rw-r--r-- | kernel/sched/topology.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/kernel/sched/topology.c b/kernel/sched/topology.c index 4197f1346153..179b90b60ec6 100644 --- a/kernel/sched/topology.c +++ b/kernel/sched/topology.c @@ -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); |