Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1629618
| From | Peter Zijlstra <peterz@infradead.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed |
| Date | 2017-04-24 16:30 +0200 |
| Message-ID | <tzNa1-87g-3@gated-at.bofh.it> (permalink) |
| References | <tyqpb-4ED-9@gated-at.bofh.it> <tyqpb-4ED-15@gated-at.bofh.it> <tzLUD-7pd-35@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Mon, Apr 24, 2017 at 03:03:26PM +0200, Peter Zijlstra wrote:
> Also, would it not make sense to re-order patch 2 to come after this,
> such that we _do_ have the group_mask available and don't have to jump
> through hoops in order to link up the sgc? Afaict we don't actually use
> the sgc until the above (reverse) loop computing the CPU capacities.
That is, if I force 4 on without 2, then doesn't something like the
below also do the right thing? (without duplicating part of the magic
already contained in build_group_mask)
---
--- a/kernel/sched/topology.c
+++ b/kernel/sched/topology.c
@@ -498,13 +498,16 @@ enum s_alloc {
*
* This function can only be used when all the groups are already built.
*/
-static void build_group_mask(struct sched_domain *sd, struct sched_group *sg)
+static void
+build_group_mask(struct sched_domain *sd, struct sched_group *sg, struct cpumask *mask)
{
const struct cpumask *sg_span = sched_group_cpus(sg);
struct sd_data *sdd = sd->private;
struct sched_domain *sibling;
int i;
+ cpumask_clear(mask);
+
for_each_cpu(i, sg_span) {
sibling = *per_cpu_ptr(sdd->sd, i);
@@ -514,7 +517,7 @@ static void build_group_mask(struct sche
if (!cpumask_equal(sg_span, sched_group_cpus(sibling->groups)))
continue;
- cpumask_set_cpu(i, sched_group_mask(sg));
+ cpumask_set_cpu(i, mask);
}
}
@@ -549,14 +552,19 @@ build_group_from_child_sched_domain(stru
}
static void init_overlap_sched_group(struct sched_domain *sd,
- struct sched_group *sg, int cpu)
+ struct sched_group *sg)
{
+ struct cpumask *mask = sched_domains_tmpmask;
struct sd_data *sdd = sd->private;
struct cpumask *sg_span;
+ int cpu;
+
+ build_group_mask(sd, sg, mask);
+ cpu = cpumask_first_and(sched_group_mask(sg), mask); /* balance cpu */
sg->sgc = *per_cpu_ptr(sdd->sgc, cpu);
if (atomic_inc_return(&sg->sgc->ref) == 1)
- build_group_mask(sd, sg);
+ cpumask_copy(sched_group_mask(sg), mask);
/*
* Initialize sgc->capacity such that even if we mess up the
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH 0/4] sched/topology: fix overlap group capacity and balance cpu Lauro Ramos Venancio <lvenanci@redhat.com> - 2017-04-20 22:00 +0200
[PATCH 1/4] sched/topology: optimize build_group_mask() Lauro Ramos Venancio <lvenanci@redhat.com> - 2017-04-20 22:00 +0200
[PATCH 3/4] sched/topology: move comment about asymmetric node setups Lauro Ramos Venancio <lvenanci@redhat.com> - 2017-04-20 22:00 +0200
Re: [PATCH 3/4] sched/topology: move comment about asymmetric node setups Peter Zijlstra <peterz@infradead.org> - 2017-04-21 19:50 +0200
[PATCH 2/4] sched/topology: all instances of a sched group must use the same sched_group_capacity Lauro Ramos Venancio <lvenanci@redhat.com> - 2017-04-20 22:00 +0200
[PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Lauro Ramos Venancio <lvenanci@redhat.com> - 2017-04-20 22:00 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-24 15:10 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-24 16:30 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-24 16:30 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Lauro Venancio <lvenanci@redhat.com> - 2017-04-24 17:30 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-25 00:30 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Lauro Venancio <lvenanci@redhat.com> - 2017-04-24 17:20 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-25 00:20 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-25 14:20 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Lauro Venancio <lvenanci@redhat.com> - 2017-04-25 16:40 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-25 17:30 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-25 17:50 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-25 18:00 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Lauro Venancio <lvenanci@redhat.com> - 2017-04-25 18:00 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-25 18:30 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-25 17:30 +0200
Re: [PATCH 4/4] sched/topology: the group balance cpu must be a cpu where the group is installed Peter Zijlstra <peterz@infradead.org> - 2017-04-25 17:30 +0200
Re: [PATCH 0/4] sched/topology: fix overlap group capacity and balance cpu Peter Zijlstra <peterz@infradead.org> - 2017-04-26 18:40 +0200
Re: [PATCH 0/4] sched/topology: fix overlap group capacity and balance cpu Lauro Venancio <lvenanci@redhat.com> - 2017-04-26 20:00 +0200
Re: [PATCH 0/4] sched/topology: fix overlap group capacity and balance cpu Peter Zijlstra <peterz@infradead.org> - 2017-04-27 00:50 +0200
Re: [PATCH 0/4] sched/topology: fix overlap group capacity and balance cpu Peter Zijlstra <peterz@infradead.org> - 2017-04-28 12:40 +0200
csiph-web