From a090c4f2cd2cfeb54eaf8ad1e726a6c485f3abc3 Mon Sep 17 00:00:00 2001 From: Peter Zijlstra Date: Mon, 21 Aug 2017 15:42:52 +0200 Subject: [PATCH] sched/topology: Improve comments Mike provided a better comment for destroy_sched_domain() ... Signed-off-by: Peter Zijlstra (Intel) Cc: Linus Torvalds Cc: Mike Galbraith Cc: Peter Zijlstra Cc: Thomas Gleixner Signed-off-by: Ingo Molnar --- kernel/sched/topology.c | 5 +++-- 1 file 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); -- 2.20.1