From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [Patch v2 08/14] genirq: Introduce helper function irq_data_get_affinity_mask() Date: Wed, 20 May 2015 20:34:26 +0100 Message-ID: <20150520193426.GT2067@n2100.arm.linux.org.uk> References: <1432114845-24304-1-git-send-email-jiang.liu@linux.intel.com> <1432114845-24304-9-git-send-email-jiang.liu@linux.intel.com> Mime-Version: 1.0 Return-path: DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=arm.linux.org.uk; s=pandora-2014; h=Sender:In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date; bh=bsCq3ygpoZHvRFo7OZqofcWGuX8p4fLIfKDPcNJnAjo=; b=OL+zzLPYaMy9fejRiic2vNNBH9n8nhgOE1kb0q3nwikwfil7FO+8tJIiEkk90IzUvKBMtax7H/U9u4Ct5oyg7pvyhIDQ52eiSIuX0vLE6g76Wl+RhbBDdiDtLdtJAv4qOGfUPt0Dsv1xepH9fVAr3W5R+EZ4CL4LKQM91bv3Ohg=; Content-Disposition: inline In-Reply-To: <1432114845-24304-9-git-send-email-jiang.liu@linux.intel.com> Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: List-subscribe: List-owner: List-post: List-archive: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Jiang Liu Cc: Thomas Gleixner , Bjorn Helgaas , Benjamin Herrenschmidt , Ingo Molnar , "H. Peter Anvin" , "Rafael J. Wysocki" , Randy Dunlap , Yinghai Lu , Borislav Petkov , Richard Henderson , Ivan Kokshaysky , Matt Turner , Catalin Marinas , Will Deacon , Steven Miao , Tony Luck , Fenghua Yu , James Hogan , Ralf Baechle , David Howells On Wed, May 20, 2015 at 05:40:39PM +0800, Jiang Liu wrote: > diff --git a/arch/arm/kernel/irq.c b/arch/arm/kernel/irq.c > index 350f188c92d2..baf8edebe26f 100644 > --- a/arch/arm/kernel/irq.c > +++ b/arch/arm/kernel/irq.c > @@ -140,7 +140,7 @@ int __init arch_probe_nr_irqs(void) > static bool migrate_one_irq(struct irq_desc *desc) > { > struct irq_data *d = irq_desc_get_irq_data(desc); > - const struct cpumask *affinity = d->affinity; > + const struct cpumask *affinity = irq_data_get_affinity_mask(d); > struct irq_chip *c; > bool ret = false; > > @@ -160,7 +160,7 @@ static bool migrate_one_irq(struct irq_desc *desc) > if (!c->irq_set_affinity) > pr_debug("IRQ%u: unable to set affinity\n", d->irq); > else if (c->irq_set_affinity(d, affinity, false) == IRQ_SET_MASK_OK && ret) > - cpumask_copy(d->affinity, affinity); > + cpumask_copy(irq_data_get_affinity_mask(d), affinity); > > return ret; > } > diff --git a/include/linux/irq.h b/include/linux/irq.h > index 43581e166298..2eb82257aaee 100644 > --- a/include/linux/irq.h > +++ b/include/linux/irq.h > @@ -650,6 +650,18 @@ static inline int irq_data_get_node(struct irq_data *d) > return irq_common_data_get_node(d->common); > } > > +static inline struct cpumask *irq_get_affinity_mask(int irq) > +{ > + struct irq_data *d = irq_get_irq_data(irq); > + > + return d ? d->affinity : NULL; > +} > + > +static inline struct cpumask *irq_data_get_affinity_mask(struct irq_data *d) > +{ > + return d->affinity; > +} > + > unsigned int arch_dynirq_lower_bound(unsigned int from); > > int __irq_alloc_descs(int irq, unsigned int from, unsigned int cnt, int node, For the above only, Acked-by: Russell King Thanks. -- FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up according to speedtest.net.