From mboxrd@z Thu Jan 1 00:00:00 1970 From: robherring2@gmail.com (Rob Herring) Date: Thu, 02 Feb 2012 16:49:43 -0600 Subject: [PATCH RFC idle 2/3] arm: Avoid invoking RCU when CPU is idle In-Reply-To: <87obtgc1xx.fsf@ti.com> References: <20120202004253.GA10946@linux.vnet.ibm.com> <1328143404-11038-1-git-send-email-paulmck@linux.vnet.ibm.com> <1328143404-11038-2-git-send-email-paulmck@linux.vnet.ibm.com> <20120202044439.GD2435@linux.vnet.ibm.com> <20120202174337.GS2518@linux.vnet.ibm.com> <20120202190708.GE2518@linux.vnet.ibm.com> <87obtgc1xx.fsf@ti.com> Message-ID: <4F2B1307.5010207@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 02/02/2012 04:20 PM, Kevin Hilman wrote: > "Paul E. McKenney" writes: > > [...] > >>>> The two options I see are: >>>> >>>> 1. Rip tracing out of the inner idle loops and everything that >>>> they invoke. >>> >>> What I suggested above. But as I said I know sh*t about that tracing >>> implementation so that's an easy suggestion for me to make. >> >> Works for me as well. ;-) > > While I must admit not having a better suggestion, I for one would vote > strongly against removing tracing from the idle path. > > Being a PM developer and maintainer, much of the code I work on and > maintain happens to be run in the bowels of the idle path. Not having > the ability to trace this code would be a major step backwards IMO. How is it a step backwards if it is already broken. Obviously you haven't actually used any tracing here because it doesn't work right with things as is. What exactly do you want to trace at this level. By the point you are in this code, the path is somewhat known and problems you have are likely h/w issues. If you are trying to go thru a very precise sequence of saving cpu state and flushing caches, you don't want calls out to tracing code that could very easily change the behavior. It would be nice to understand a real example of how to hit this problem. Why type of tracing? Are there other known examples? Rob