From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: Regression with suspicious RCU usage splats with cpu_pm change Date: Sun, 16 Jul 2017 23:08:07 -0700 Message-ID: <20170717060806.GC10026@atomide.com> References: <20170713070749.GE16509@atomide.com> <20170713124318.GK2393@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Alex Shi Cc: paulmck@linux.vnet.ibm.com, "Rafael J. Wysocki" , Linux Kernel Mailing List , Linux PM , Linux OMAP Mailing List , Catalin Marinas , Will Deacon List-Id: linux-omap@vger.kernel.org * Alex Shi [170716 16:25]: > I reused the rcu_irq_enter_irqson() from RCU_NONIDLE to avoid this issue. > It works fine. > > Tony, Could you like to give a tested-by if this patch works for you. Yeah that keeps things booting for me with no splats so: Tested-by: Tony Lindgren In general, it seems we're missing the knowledge in Linux kernel of when the entire system is idle. Right now it seems that only cpuidle_coupled knows that? We could probably simplify things by adding some PM state for entire system idle. Then cpuidle code and timer code could use that to test when it's safe to do whatever the SoC needs to do to enter deeper power states. If we already have something like that, please do let me know :) Regards, Tony