From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stable-owner@vger.kernel.org>
Received: from mail.linuxfoundation.org ([140.211.169.12]:44176 "EHLO
        mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S964967AbeGBIei (ORCPT
        <rfc822;stable@vger.kernel.org>); Mon, 2 Jul 2018 04:34:38 -0400
Date: Mon, 2 Jul 2018 10:34:34 +0200
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Marc Zyngier <marc.zyngier@arm.com>
Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org,
        Yang Yingliang <yangyingliang@huawei.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Jason Cooper <jason@lakedaemon.net>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Sumit Garg <sumit.garg@linaro.org>
Subject: Re: [PATCH 4.14 088/157] irqchip/gic-v3-its: Dont bind LPI to
 unavailable NUMA node
Message-ID: <20180702083434.GA20652@kroah.com>
References: <20180701160852.287307684@linuxfoundation.org>
 <20180701160856.970708088@linuxfoundation.org>
 <b7959075-2874-c774-9e01-221cb5e4e18c@arm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <b7959075-2874-c774-9e01-221cb5e4e18c@arm.com>
Sender: stable-owner@vger.kernel.org
List-ID: <stable.vger.kernel.org>

On Mon, Jul 02, 2018 at 08:44:05AM +0100, Marc Zyngier wrote:
> On 01/07/18 17:22, Greg Kroah-Hartman wrote:
> > 4.14-stable review patch.  If anyone has any objections, please let me know.
> > 
> > ------------------
> > 
> > From: Yang Yingliang <yangyingliang@huawei.com>
> > 
> > commit c1797b11a09c8323c92b074fd48b89a936c991d0 upstream.
> > 
> > On a NUMA system, if an ITS is local to an offline node, the ITS driver may
> > pick an offline CPU to bind the LPI.  In this case, pick an online CPU (and
> > the first one will do).
> > 
> > But on some systems, binding an LPI to non-local node CPU may cause
> > deadlock (see Cavium erratum 23144).  In this case, just fail the activate
> > and return an error code.
> > 
> > Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
> > Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
> > Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
> > Cc: Jason Cooper <jason@lakedaemon.net>
> > Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> > Cc: Sumit Garg <sumit.garg@linaro.org>
> > Cc: stable@vger.kernel.org
> > Link: https://lkml.kernel.org/r/20180622095254.5906-5-marc.zyngier@arm.com
> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > 
> > ---
> >  drivers/irqchip/irq-gic-v3-its.c |    9 ++++++++-
> >  1 file changed, 8 insertions(+), 1 deletion(-)
> > 
> > --- a/drivers/irqchip/irq-gic-v3-its.c
> > +++ b/drivers/irqchip/irq-gic-v3-its.c
> > @@ -2221,7 +2221,14 @@ static void its_irq_domain_activate(stru
> >  		cpu_mask = cpumask_of_node(its_dev->its->numa_node);
> >  
> >  	/* Bind the LPI to the first possible CPU */
> > -	cpu = cpumask_first(cpu_mask);
> > +	cpu = cpumask_first_and(cpu_mask, cpu_online_mask);
> > +	if (cpu >= nr_cpu_ids) {
> > +		if (its_dev->its->flags & ITS_FLAGS_WORKAROUND_CAVIUM_23144)
> > +			return -EINVAL;
> 
> As reported by the kbuild robot, you need to drop the return value here
> (the prototype was changed post 4.14). A WARN_ON() could be useful to
> detect the case though.

Thanks, now done (the return change, not WARN_ON, those are just
ignored).   I've pushed out a -rc2 with that fix in it.

greg k-h