From patchwork Wed Jul 25 03:41:08 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Len Brown X-Patchwork-Id: 1235191 Return-Path: X-Original-To: patchwork-linux-acpi@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id D802F3FD4F for ; Wed, 25 Jul 2012 03:53:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932806Ab2GYDxP (ORCPT ); Tue, 24 Jul 2012 23:53:15 -0400 Received: from mail-qa0-f46.google.com ([209.85.216.46]:62253 "EHLO mail-qa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932754Ab2GYDmY (ORCPT ); Tue, 24 Jul 2012 23:42:24 -0400 Received: by mail-qa0-f46.google.com with SMTP id b17so2268265qad.19 for ; Tue, 24 Jul 2012 20:42:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:cc:subject:date:message-id:x-mailer:in-reply-to :references:in-reply-to:references:reply-to:organization; bh=Dtcpp1YCCUk983o8e5m7j79LkMcXM4NOJQY/z50Lav4=; b=JehRLIt04K0Mte+1JCgiOHIX6YK+zh4CS0Kpy20VCAp96nEjXMQkNVcBLNBgPXJmBg DbbCdUrfHMW3J1K850Gaq4bMsOjCE1AGKBT+S/ab+Rge0VKYqJdCpWaBa84zY2pa5K9X 91Yp0ptJAv204pUtzkfTqnHpDKaILdgm9hHOqYGMKTPVWYNFvcnuImJkbOD+Fw7/DSKC v9guxHSXtDN5fe2JGrD5qwgs8C9r7adIC6A+IDfk69txJBvkN5MXB6Tjlj143CvsWWSh 7v5nluR06MEbDF/eWCPDgdNILS4pD0V7Cd8zMZlUZvZlXREC04qkxvdO8m6PjmnTAdoR K6Sg== Received: by 10.224.176.69 with SMTP id bd5mr30149061qab.66.1343187744017; Tue, 24 Jul 2012 20:42:24 -0700 (PDT) Received: from x980.localdomain6 (h184-61-125-197.altnnh.dsl.dynamic.tds.net. [184.61.125.197]) by mx.google.com with ESMTPS id et6sm15489186qab.8.2012.07.24.20.42.22 (version=SSLv3 cipher=OTHER); Tue, 24 Jul 2012 20:42:23 -0700 (PDT) From: Len Brown To: linux-acpi@vger.kernel.org, linux-pm@lists.linux-foundation.org Cc: linux-kernel@vger.kernel.org, Sergey Senozhatsky , Kevin Hilman , Trinabh Gupta , Deepthi Dharwar , Arjan van de Ven , Andrew Morton , Len Brown Subject: [PATCH 12/52] cpuidle: remove unused hrtimer_peek_ahead_timers() call Date: Tue, 24 Jul 2012 23:41:08 -0400 Message-Id: <0aeb9cac6f8a6fc68acfb07d30b62ad6106a6384.1343187617.git.len.brown@intel.com> X-Mailer: git-send-email 1.7.12.rc0 In-Reply-To: <1343187708-19532-1-git-send-email-lenb@kernel.org> References: <1343187708-19532-1-git-send-email-lenb@kernel.org> In-Reply-To: <6af1c4fc5227af65092ebc848989693562bfa3e8.1343187617.git.len.brown@intel.com> References: <6af1c4fc5227af65092ebc848989693562bfa3e8.1343187617.git.len.brown@intel.com> Reply-To: Len Brown Organization: Intel Open Source Technology Center Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org From: Sergey Senozhatsky commit 9a6558371bcd01c2973b7638181db4ccc34eab4f Author: Arjan van de Ven Date: Sun Nov 9 12:45:10 2008 -0800 regression: disable timer peek-ahead for 2.6.28 It's showing up as regressions; disabling it very likely just papers over an underlying issue, but time is running out for 2.6.28, lets get back to this for 2.6.29 Many years has passed since 2008, so it seems ok to remove whole `#if 0' block. Signed-off-by: Sergey Senozhatsky Cc: Kevin Hilman Cc: Trinabh Gupta Cc: Deepthi Dharwar Cc: Arjan van de Ven Signed-off-by: Andrew Morton Signed-off-by: Len Brown --- drivers/cpuidle/cpuidle.c | 9 --------- 1 file changed, 9 deletions(-) diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index 2f0083a..588b44a 100644 --- a/drivers/cpuidle/cpuidle.c +++ b/drivers/cpuidle/cpuidle.c @@ -124,15 +124,6 @@ int cpuidle_idle_call(void) if (!dev || !dev->enabled) return -EBUSY; -#if 0 - /* shows regressions, re-enable for 2.6.29 */ - /* - * run any timers that can be run now, at this point - * before calculating the idle duration etc. - */ - hrtimer_peek_ahead_timers(); -#endif - /* ask the governor for the next state */ next_state = cpuidle_curr_governor->select(drv, dev); if (need_resched()) {