From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 14 Sep 2017 16:32:28 +1000 From: Nicholas Piggin To: Benjamin Herrenschmidt Cc: linuxppc-dev@lists.ozlabs.org, skiboot@lists.ozlabs.org, Alistair Popple Subject: Re: [RFC PATCH 2/2] powerpc/powernv: implement NMI IPIs with OPAL_SIGNAL_SYSTEM_RESET Message-ID: <20170914163228.0510251a@roar.ozlabs.ibm.com> In-Reply-To: <1505355889.12628.177.camel@kernel.crashing.org> References: <20170912160553.13422-1-npiggin@gmail.com> <20170912160553.13422-3-npiggin@gmail.com> <20170913231354.6b249fd0@roar.ozlabs.ibm.com> <1505355889.12628.177.camel@kernel.crashing.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 14 Sep 2017 12:24:49 +1000 Benjamin Herrenschmidt wrote: > On Wed, 2017-09-13 at 23:13 +1000, Nicholas Piggin wrote: > > On Wed, 13 Sep 2017 02:05:53 +1000 > > Nicholas Piggin wrote: > > > > > There are two complications. The first is that sreset from stop states > > > come in with SRR1 set to do a powersave wakeup, with an sreset reason > > > encoded. > > > > > > The second is that threads on the same core can't be signalled directly > > > so we must designate a bounce CPU to reflect the IPI back. > > > > Here is an updated Linux patch for the latest OPAL patch. This has > > a few assorted fixes as well to make it work nicely, I roll them into > > one patch here to make it easy to apply for testing the OPAL patch. > > Why can't you sreset threads of the same core on P9 ? It looks like we can, I think I had some other bugs still not ironed out when I previously tested it. That simplifies things a lot on the Linux side. It may be that the bounce is still required if we implement it on POWER8 using ramming, but I'll get the POWER9 code in first. Thanks, Nick