From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2997265AbdD1Ne1 (ORCPT ); Fri, 28 Apr 2017 09:34:27 -0400 Received: from bombadil.infradead.org ([65.50.211.133]:46453 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1162704AbdD1Ncg (ORCPT ); Fri, 28 Apr 2017 09:32:36 -0400 Message-Id: <20170428132502.699851310@infradead.org> User-Agent: quilt/0.63-1 Date: Fri, 28 Apr 2017 15:20:05 +0200 From: Peter Zijlstra To: mingo@kernel.org, lvenanci@redhat.com Cc: lwang@redhat.com, riel@redhat.com, efault@gmx.de, tglx@linutronix.de, linux-kernel@vger.kernel.org, peterz@infradead.org Subject: [PATCH 07/14] sched/topology: Optimize build_group_mask() References: <20170428131958.893188882@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline; filename=lauro_ramos_venancio-sched_topology-optimize_build_group_mask.patch Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The group mask is always used in intersection with the group cpus. So, when building the group mask, we don't have to care about cpus that are not part of the group. Cc: Ingo Molnar Cc: Mike Galbraith Cc: Thomas Gleixner Cc: riel@redhat.com Signed-off-by: Lauro Ramos Venancio Signed-off-by: Peter Zijlstra (Intel) Link: http://lkml.kernel.org/r/1492717903-5195-2-git-send-email-lvenanci@redhat.com --- kernel/sched/topology.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/kernel/sched/topology.c +++ b/kernel/sched/topology.c @@ -506,12 +506,12 @@ enum s_alloc { */ static void build_group_mask(struct sched_domain *sd, struct sched_group *sg) { - const struct cpumask *span = sched_domain_span(sd); + const struct cpumask *sg_span = sched_group_cpus(sg); struct sd_data *sdd = sd->private; struct sched_domain *sibling; int i; - for_each_cpu(i, span) { + for_each_cpu(i, sg_span) { sibling = *per_cpu_ptr(sdd->sd, i); if (!cpumask_test_cpu(i, sched_domain_span(sibling))) continue;