From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] OMAP: don't trace functions called from sched_clock Date: Tue, 31 May 2011 04:50:40 -0700 Message-ID: <20110531115039.GZ11352@atomide.com> References: <1304846514-15641-1-git-send-email-rabin@rab.in> <87hb8fyj2p.fsf@ti.com> <8762ovvf5f.fsf@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-04-ewr.mailhop.org ([204.13.248.74]:19401 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754439Ab1EaLup (ORCPT ); Tue, 31 May 2011 07:50:45 -0400 Content-Disposition: inline In-Reply-To: <8762ovvf5f.fsf@ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Kevin Hilman Cc: Rabin Vincent , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org * Kevin Hilman [110527 17:28]: > Kevin Hilman writes: > > > Rabin Vincent writes: > > > >> omap_readl() is used from the sched_clock() implementations and so must > >> be marked notrace to avoid recursion in ftrace. Same thing with > >> mpu_read() for OMAP1. > >> > >> Signed-off-by: Rabin Vincent > > > > Acked-by: Kevin Hilman > > I also meant to suggest this should probably queue for .40-rc series. Took a quick look and we should get rid of the omap_readl usage instead like I commented in another mail. Tony From mboxrd@z Thu Jan 1 00:00:00 1970 From: tony@atomide.com (Tony Lindgren) Date: Tue, 31 May 2011 04:50:40 -0700 Subject: [PATCH] OMAP: don't trace functions called from sched_clock In-Reply-To: <8762ovvf5f.fsf@ti.com> References: <1304846514-15641-1-git-send-email-rabin@rab.in> <87hb8fyj2p.fsf@ti.com> <8762ovvf5f.fsf@ti.com> Message-ID: <20110531115039.GZ11352@atomide.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org * Kevin Hilman [110527 17:28]: > Kevin Hilman writes: > > > Rabin Vincent writes: > > > >> omap_readl() is used from the sched_clock() implementations and so must > >> be marked notrace to avoid recursion in ftrace. Same thing with > >> mpu_read() for OMAP1. > >> > >> Signed-off-by: Rabin Vincent > > > > Acked-by: Kevin Hilman > > I also meant to suggest this should probably queue for .40-rc series. Took a quick look and we should get rid of the omap_readl usage instead like I commented in another mail. Tony