lkml.org 
[lkml]   [2015]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[Patch v3 07/36] genirq: Move field 'node' from struct irq_data into struct irq_common_data
    Date
    NUMA node information is per-irq instead of per-irqchip, so move it into
    struct irq_common_data. Also use CONFIG_NUMA to guard irq_common_data.node.

    Signed-off-by: Jiang Liu <jiang.liu@linux.intel.com>
    ---
    include/linux/irq.h | 17 ++++++++++++++---
    kernel/irq/internals.h | 2 +-
    kernel/irq/irqdesc.c | 4 +++-
    kernel/irq/irqdomain.c | 1 -
    4 files changed, 18 insertions(+), 6 deletions(-)

    diff --git a/include/linux/irq.h b/include/linux/irq.h
    index b3b82a5344c8..bcedf9f952d4 100644
    --- a/include/linux/irq.h
    +++ b/include/linux/irq.h
    @@ -129,9 +129,13 @@ struct irq_domain;
    * struct irq_common_data - per irq data shared by all irqchips
    * @state_use_accessors: status information for irq chip functions.
    * Use accessor functions to deal with it
    + * @node: node index useful for balancing
    */
    struct irq_common_data {
    unsigned int state_use_accessors;
    +#ifdef CONFIG_NUMA
    + unsigned int node;
    +#endif
    };

    /**
    @@ -139,7 +143,6 @@ struct irq_common_data {
    * @mask: precomputed bitmask for accessing the chip registers
    * @irq: interrupt number
    * @hwirq: hardware interrupt number, local to the interrupt domain
    - * @node: node index useful for balancing
    * @common: point to data shared by all irqchips
    * @chip: low level interrupt hardware access
    * @domain: Interrupt translation domain; responsible for mapping
    @@ -160,7 +163,6 @@ struct irq_data {
    u32 mask;
    unsigned int irq;
    unsigned long hwirq;
    - unsigned int node;
    struct irq_common_data *common;
    struct irq_chip *chip;
    struct irq_domain *domain;
    @@ -640,9 +642,18 @@ static inline u32 irq_get_trigger_type(unsigned int irq)
    return d ? irqd_get_trigger_type(d) : 0;
    }

    -static inline int irq_data_get_node(struct irq_data *d)
    +static inline int irq_common_data_get_node(struct irq_common_data *d)
    {
    +#ifdef CONFIG_NUMA
    return d->node;
    +#else
    + return 0;
    +#endif
    +}
    +
    +static inline int irq_data_get_node(struct irq_data *d)
    +{
    + return irq_common_data_get_node(d->common);
    }

    unsigned int arch_dynirq_lower_bound(unsigned int from);
    diff --git a/kernel/irq/internals.h b/kernel/irq/internals.h
    index 4834ee828c41..b117e4c243d8 100644
    --- a/kernel/irq/internals.h
    +++ b/kernel/irq/internals.h
    @@ -199,7 +199,7 @@ static inline void kstat_incr_irqs_this_cpu(unsigned int irq, struct irq_desc *d

    static inline int irq_desc_get_node(struct irq_desc *desc)
    {
    - return irq_data_get_node(&desc->irq_data);
    + return irq_common_data_get_node(&desc->irq_common_data);
    }

    #ifdef CONFIG_PM_SLEEP
    diff --git a/kernel/irq/irqdesc.c b/kernel/irq/irqdesc.c
    index b18d3f1d73d9..0c3057e42906 100644
    --- a/kernel/irq/irqdesc.c
    +++ b/kernel/irq/irqdesc.c
    @@ -52,11 +52,13 @@ static int alloc_masks(struct irq_desc *desc, gfp_t gfp, int node)

    static void desc_smp_init(struct irq_desc *desc, int node)
    {
    - desc->irq_data.node = node;
    cpumask_copy(desc->irq_data.affinity, irq_default_affinity);
    #ifdef CONFIG_GENERIC_PENDING_IRQ
    cpumask_clear(desc->pending_mask);
    #endif
    +#ifdef CONFIG_NUMA
    + desc->irq_common_data.node = node;
    +#endif
    }

    #else
    diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c
    index 1b06dfed4574..93ef89059022 100644
    --- a/kernel/irq/irqdomain.c
    +++ b/kernel/irq/irqdomain.c
    @@ -836,7 +836,6 @@ static struct irq_data *irq_domain_insert_irq_data(struct irq_domain *domain,
    child->parent_data = irq_data;
    irq_data->irq = child->irq;
    irq_data->common = child->common;
    - irq_data->node = child->node;
    irq_data->domain = domain;
    }

    --
    1.7.10.4


    \
     
     \ /
      Last update: 2015-06-01 10:41    [W:4.213 / U:0.376 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site