lkml.org 
[lkml]   [2011]   [Mar]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[RFC][PATCH 12/14] sched: Simplify NODE/ALLNODES domain creation
    Don't treat ALLNODES/NODE different for difference's sake. Simply
    always create the ALLNODES domain and let the sd_degenerate() checks
    kill it when its redundant. This simplifies the code flow.

    Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
    LKML-Reference: <new-submission>
    ---
    kernel/sched.c | 40 ++++++++++++++++++++++------------------
    1 file changed, 22 insertions(+), 18 deletions(-)

    Index: linux-2.6/kernel/sched.c
    ===================================================================
    --- linux-2.6.orig/kernel/sched.c
    +++ linux-2.6/kernel/sched.c
    @@ -6814,9 +6814,6 @@ struct sd_data {
    };

    struct s_data {
    -#ifdef CONFIG_NUMA
    - int sd_allnodes;
    -#endif
    cpumask_var_t nodemask;
    cpumask_var_t send_covered;
    struct sched_domain ** __percpu sd;
    @@ -7088,30 +7085,35 @@ static void claim_allocations(int cpu, s
    }
    }

    -static struct sched_domain *__build_numa_sched_domains(struct s_data *d,
    - const struct cpumask *cpu_map, struct sched_domain_attr *attr, int i)
    +static struct sched_domain *__build_allnodes_sched_domain(struct s_data *d,
    + const struct cpumask *cpu_map, struct sched_domain_attr *attr,
    + struct sched_domain *parent, int i)
    {
    struct sched_domain *sd = NULL;
    #ifdef CONFIG_NUMA
    - struct sched_domain *parent;
    -
    - d->sd_allnodes = 0;
    - if (cpumask_weight(cpu_map) >
    - SD_NODES_PER_DOMAIN * cpumask_weight(d->nodemask)) {
    - sd = sd_init_ALLNODES(d, i);
    - set_domain_attribute(sd, attr);
    - cpumask_copy(sched_domain_span(sd), cpu_map);
    - d->sd_allnodes = 1;
    - }
    - parent = sd;
    + sd = sd_init_ALLNODES(d, i);
    + set_domain_attribute(sd, attr);
    + cpumask_copy(sched_domain_span(sd), cpu_map);
    + sd->parent = parent;
    + if (parent)
    + parent->child = sd;
    +#endif
    + return sd;
    +}

    +static struct sched_domain *__build_node_sched_domain(struct s_data *d,
    + const struct cpumask *cpu_map, struct sched_domain_attr *attr,
    + struct sched_domain *parent, int i)
    +{
    + struct sched_domain *sd = NULL;
    +#ifdef CONFIG_NUMA
    sd = sd_init_NODE(d, i);
    set_domain_attribute(sd, attr);
    sched_domain_node_span(cpu_to_node(i), sched_domain_span(sd));
    + cpumask_and(sched_domain_span(sd), sched_domain_span(sd), cpu_map);
    sd->parent = parent;
    if (parent)
    parent->child = sd;
    - cpumask_and(sched_domain_span(sd), sched_domain_span(sd), cpu_map);
    #endif
    return sd;
    }
    @@ -7196,7 +7198,9 @@ static int __build_sched_domains(const s
    cpumask_and(d.nodemask, cpumask_of_node(cpu_to_node(i)),
    cpu_map);

    - sd = __build_numa_sched_domains(&d, cpu_map, attr, i);
    + sd = NULL;
    + sd = __build_allnodes_sched_domain(&d, cpu_map, attr, sd, i);
    + sd = __build_node_sched_domain(&d, cpu_map, attr, sd, i);
    sd = __build_cpu_sched_domain(&d, cpu_map, attr, sd, i);
    sd = __build_book_sched_domain(&d, cpu_map, attr, sd, i);
    sd = __build_mc_sched_domain(&d, cpu_map, attr, sd, i);



    \
     
     \ /
      Last update: 2011-03-14 16:31    [W:0.021 / U:0.352 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site