From patchwork Wed Nov 9 17:43:30 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sudeep Holla X-Patchwork-Id: 81541 Delivered-To: patch@linaro.org Received: by 10.140.97.165 with SMTP id m34csp319637qge; Wed, 9 Nov 2016 09:43:40 -0800 (PST) X-Received: by 10.98.57.144 with SMTP id u16mr1299630pfj.142.1478713419935; Wed, 09 Nov 2016 09:43:39 -0800 (PST) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id r6si461864pgf.203.2016.11.09.09.43.39; Wed, 09 Nov 2016 09:43:39 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-pm-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752567AbcKIRni (ORCPT + 13 others); Wed, 9 Nov 2016 12:43:38 -0500 Received: from foss.arm.com ([217.140.101.70]:60088 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751420AbcKIRnh (ORCPT ); Wed, 9 Nov 2016 12:43:37 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id DD9B6F; Wed, 9 Nov 2016 09:43:36 -0800 (PST) Received: from e107155-lin.cambridge.arm.com (unknown [10.1.210.28]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 98E183F318; Wed, 9 Nov 2016 09:43:35 -0800 (PST) From: Sudeep Holla To: linux-pm@vger.kernel.org, "Rafael J . Wysocki" Cc: Sudeep Holla , linux-kernel@vger.kernel.org, Daniel Lezcano , Lorenzo Pieralisi , Andy Gross , Vincent Guittot Subject: [PATCH] drivers: cpuidle: assign enter_freeze to same as enter callback function Date: Wed, 9 Nov 2016 17:43:30 +0000 Message-Id: <1478713410-10727-1-git-send-email-sudeep.holla@arm.com> X-Mailer: git-send-email 2.7.4 Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org enter_freeze() callback is expected atleast to do the same as enter() but it has to guarantee that interrupts aren't enabled at any point in its execution, as the tick is frozen. CPUs execute ->enter_freeze with the local tick or entire timekeeping suspended, so it must not re-enable interrupts at any point (even temporarily) or attempt to change states of clock event devices. It will be called when the system goes to suspend-to-idle and will reduce power usage because CPUs won't be awaken for unnecessary IRQs (i.e. woken up only on IRQs from "wakeup sources") Since for all the states that have CPUIDLE_FLAG_TIMER_STOP flag set, local tick is stopped, we can reuse the same code for both the enter() and enter_freeze() callbacks. Only "coupled" cpuidle mechanism enables interrupts and doing that with timekeeping suspended is generally not safe. Since this generic DT based idle driver doesn't support "coupled" states, it is safe to assume that the interrupts are not re-enabled. This patch assign enter_freeze to same as enter callback function which helps to save power without any intermittent spurious wakeups from suspend-to-idle. Signed-off-by: Sudeep Holla --- drivers/cpuidle/dt_idle_states.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/cpuidle/dt_idle_states.c b/drivers/cpuidle/dt_idle_states.c index a5c111b67f37..5a087d108475 100644 --- a/drivers/cpuidle/dt_idle_states.c +++ b/drivers/cpuidle/dt_idle_states.c @@ -79,8 +79,17 @@ static int init_state_node(struct cpuidle_state *idle_state, desc = state_node->name; idle_state->flags = 0; - if (of_property_read_bool(state_node, "local-timer-stop")) + if (of_property_read_bool(state_node, "local-timer-stop")) { idle_state->flags |= CPUIDLE_FLAG_TIMER_STOP; + /* + * CPUIDLE_FLAG_TIMER_STOP guarantees that the local tick is + * stopped and since this is not a "coupled" state interrupts + * won't be enabled when it exits allowing the tick to be + * frozen safely. So enter() can be also enter_freeze() + * callback. + */ + idle_state->enter_freeze = match_id->data; + } /* * TODO: * replace with kstrdup and pointer assignment when name