From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kroah-Hartman Subject: [ 094/122] alpha: Add missing RCU idle APIs on idle loop Date: Thu, 11 Oct 2012 07:54:20 +0900 Message-ID: <20121010225354.969129613@linuxfoundation.org> References: <20121010225337.989799482@linuxfoundation.org> Return-path: DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= messagingengine.com; h=from:to:cc:subject:date:message-id :in-reply-to:references; s=smtpout; bh=lepkfwiLQ86TApPklaWxUcLgF bY=; b=TWJ7Md+6ES3mzp2vNj/N6Okyw+DMmL4ppshaBWCBSEQOeYgkJ6xjv9ash MRTw1pSJxGN5Aaz2a/jDbgqxDphoKt2z7HLH2t6NooAPM3xwhfjtzkTiC8STW0sp fnC6+EZI3ahPWi1Uv3CkhqIcD1xH0ri9MJJh8UKoxCPykYKEFQ= In-Reply-To: <20121010225337.989799482@linuxfoundation.org> Sender: linux-alpha-owner@vger.kernel.org List-ID: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Greg Kroah-Hartman , alan@lxorguk.ukuu.org.uk, "Paul E. McKenney" , Frederic Weisbecker , Michael Cree , Richard Henderson , Ivan Kokshaysky , Matt Turner , alpha , Josh Triplett 3.6-stable review patch. If anyone has any objections, please let me know. ------------------ From: Frederic Weisbecker commit 4c94cada48f7c660eca582be6032427a5e367117 upstream. In the old times, the whole idle task was considered as an RCU quiescent state. But as RCU became more and more successful overtime, some RCU read side critical section have been added even in the code of some architectures idle tasks, for tracing for example. So nowadays, rcu_idle_enter() and rcu_idle_exit() must be called by the architecture to tell RCU about the part in the idle loop that doesn't make use of rcu read side critical sections, typically the part that puts the CPU in low power mode. This is necessary for RCU to find the quiescent states in idle in order to complete grace periods. Add this missing pair of calls in the Alpha's idle loop. Reported-by: Paul E. McKenney Signed-off-by: Frederic Weisbecker Tested-by: Michael Cree Cc: Richard Henderson Cc: Ivan Kokshaysky Cc: Matt Turner Cc: alpha Cc: Paul E. McKenney Reviewed-by: Josh Triplett Signed-off-by: Greg Kroah-Hartman --- arch/alpha/kernel/process.c | 3 +++ 1 file changed, 3 insertions(+) --- a/arch/alpha/kernel/process.c +++ b/arch/alpha/kernel/process.c @@ -28,6 +28,7 @@ #include #include #include +#include #include #include @@ -54,8 +55,10 @@ cpu_idle(void) /* FIXME -- EV6 and LCA45 know how to power down the CPU. */ + rcu_idle_enter(); while (!need_resched()) cpu_relax(); + rcu_idle_exit(); schedule(); } }