Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1706973

Re: [PATCH] powerpc: xive: ensure active irqd when setting affinity

Show all headers | View raw

Michael Ellerman [mpe@ellerman.id.au] wrote:
> Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> writes:
> 
> > From fd0abf5c61b6041fdb75296e8580b86dc91d08d6 Mon Sep 17 00:00:00 2001
> > From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> > Date: Tue, 1 Aug 2017 20:54:41 -0500
> > Subject: [PATCH] powerpc: xive: ensure active irqd when setting affinity
> >
> > Ensure irqd is active before attempting to set affinity. This should
> > make the set affinity code more robust. For instance, this prevents
> > these messages seen on a 4.12 based kernel when taking cpus offline:
> >
> >    [  123.053037264,3] XIVE[ IC 00  ] ISN 2 lead to invalid IVE !
> >    [   77.885859] xive: Error -6 reconfiguring irq 17
> >    [   77.885862] IRQ17: set affinity failed(-6).
> >
> > The underlying problem with taking cpus offline was fixed in 4.13-rc1 by:
> >
> >    commit 91f26cb4cd3c ("genirq/cpuhotplug: Do not migrated shutdown irqs")
> 
> So do we still need this? Or is the above only a partial fix?

It would be good to have this fix.

Commit 91f26cb4cd3c fixes the problem, so we wont see the errors with
that commit applied. But if such a problem were to show up again, xive
will handle them earlier before hitting those errors.

Sukadev

> 
> I'm a bit confused.
> 
> cheers

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] powerpc: xive: ensure active irqd when setting affinity Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> - 2017-08-03 03:40 +0200
  Re: [PATCH] powerpc: xive: ensure active irqd when setting affinity Michael Ellerman <mpe@ellerman.id.au> - 2017-08-08 12:50 +0200
    Re: [PATCH] powerpc: xive: ensure active irqd when setting affinity Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> - 2017-08-09 02:10 +0200
      Re: [PATCH] powerpc: xive: ensure active irqd when setting affinity Michael Ellerman <mpe@ellerman.id.au> - 2017-08-09 08:20 +0200
        Re: [PATCH] powerpc: xive: ensure active irqd when setting affinity Benjamin Herrenschmidt <benh@kernel.crashing.org> - 2017-08-09 09:40 +0200
  Re: powerpc: xive: ensure active irqd when setting affinity Michael Ellerman <patch-notifications@ellerman.id.au> - 2017-08-11 14:30 +0200

csiph-web