From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolas Pitre Subject: Re: [PATCH 3/6] ARM: EXYNOS: add Exynos Dual Cluster Support Date: Mon, 7 Oct 2013 18:06:27 -0400 (EDT) Message-ID: References: <20131007140343.GA12030@localhost.localdomain> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Return-path: In-Reply-To: <20131007140343.GA12030@localhost.localdomain> Sender: linux-doc-owner@vger.kernel.org To: Dave Martin Cc: Mark Rutland , devicetree@vger.kernel.org, Kukjin Kim , Russell King , Ben Dooks , Pawel Moll , Ian Campbell , Daniel Lezcano , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, Rob Herring , linux-samsung-soc@vger.kernel.org, Vyacheslav Tyrtov , Rob Landley , Stephen Warren , Tarek Dakhran , Thomas Gleixner , Naour Romain , Mike Turquette , linux-arm-kernel@lists.infradead.org, Heiko Stuebner List-Id: devicetree@vger.kernel.org On Mon, 7 Oct 2013, Dave Martin wrote: > On Fri, Oct 04, 2013 at 03:51:31PM -0400, Nicolas Pitre wrote: > > No FIQs are supposed to ever race with this code. > > There is an anomaly though: FIQ and external abort don't seem to get > explicitly masked anywhere, either on the suspend or powerdown paths. > Sometimes either or both remains unmasked (I tried some trace in the > TC2 MCPM backend to confirm this.) > > Looks like a possible omission in the arch/arm/ suspend and shutdown > code, rather than a problem specific to MCPM. Possibly, yes. Feel free to post a patch. Nicolas