From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Pihet Subject: Re: get_irq_regs() from soft IRQ Date: Mon, 29 Jun 2009 18:12:51 +0200 Message-ID: <200906291812.51794.jpihet@mvista.com> References: <200906291631.19562.jpihet@mvista.com> <200906291735.38290.jpihet@mvista.com> <20090629160744.GA11523@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from gateway-1237.mvista.com ([63.81.120.158]:4650 "EHLO gateway-1237.mvista.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759531AbZF2QNA (ORCPT ); Mon, 29 Jun 2009 12:13:00 -0400 In-Reply-To: <20090629160744.GA11523@n2100.arm.linux.org.uk> Content-Disposition: inline Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Russell King - ARM Linux Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.arm.linux.org.uk, oprofile-list@lists.sourceforge.net On Monday 29 June 2009 18:07:44 Russell King - ARM Linux wrote: > On Mon, Jun 29, 2009 at 05:35:37PM +0200, Jean Pihet wrote: > > On Monday 29 June 2009 17:19:31 Russell King - ARM Linux wrote: > > > It's one of these things that nests itself - when you have several IRQs > > > being processed on one CPU, there are several register contexts saved, > > > and get_irq_regs() returns the most recent one. > > > > > > > The use case is that the performance unit (PMNC) of the Cortex A8 has > > > > some serious bug, in short the performance counters overflow IRQ is > > > > to be avoided. > > > > > > I don't follow. None of the PMNC support code in the mainline kernel > > > uses get_irq_regs() outside of IRQ context. > > > > That is correct. The Cortex A8 needs some special treatment. > > The errata says that if the counters are overflowing at the same time as > > a coprocessor access is performed, the perf unit gets reset and/or locks > > up. In short the counters overflow is to be avoided and so the PMNC IRQ. > > Are you talking about 628216? Yes that is the one. Sorry not to mention it sooner. Jean