From patchwork Thu Jan 30 22:43:34 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Zoran Markovic X-Patchwork-Id: 3559711 Return-Path: X-Original-To: patchwork-linux-wireless@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id F3C04C02DC for ; Thu, 30 Jan 2014 22:44:32 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 2CB602017E for ; Thu, 30 Jan 2014 22:44:32 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 4F68220160 for ; Thu, 30 Jan 2014 22:44:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753649AbaA3WoF (ORCPT ); Thu, 30 Jan 2014 17:44:05 -0500 Received: from mail-pa0-f46.google.com ([209.85.220.46]:34358 "EHLO mail-pa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752733AbaA3WoE (ORCPT ); Thu, 30 Jan 2014 17:44:04 -0500 Received: by mail-pa0-f46.google.com with SMTP id rd3so3672146pab.5 for ; Thu, 30 Jan 2014 14:44:03 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=2LLtpG3E4S3zUNAl7/fUURlUxBbi3JUn3/LVkUEu5jY=; b=b0o7gCuWiFk2D+KeoloWbGvnzsWXHCp6Fj3KwoBEbJIvudaWOeBGyYrCV2/6C5yyTz 52dmaWh2ROjI1Xs2bzPnkoRNnL/KSCkBmE/CS6p+S9AvLWMlGRC44oGKXznjC608661o lqyrKE5OBBkfcLXbk152aqqR0sE8Kkyd3+TUy1E5ethor9D3pDuRwnO7InF8N63CcD/l a0rstbGkYn2ZlC/Ka9KKkc2l/sCvVxI8piYEe2Ol41r6e3m+t+9MXg9mJhz5GhkalvhW 7uHiVD/Ef8TojmhXWmd7xBmhzCltJZ2knFP4N3N2ca36h41NxEVIu6BlHJSLBRK56rpM gxEQ== X-Gm-Message-State: ALoCoQlSyNhj+k+FLXYphc2KIsMgMT89Hd6sbPrjaO+86YyeVu5QYgqubvBd7j5SuMsFZ9ML61u3 X-Received: by 10.68.44.71 with SMTP id c7mr7249301pbm.24.1391121843090; Thu, 30 Jan 2014 14:44:03 -0800 (PST) Received: from vb-linaro.ric.broadcom.com ([216.31.219.19]) by mx.google.com with ESMTPSA id qq5sm21000149pbb.24.2014.01.30.14.44.01 for (version=TLSv1.1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Thu, 30 Jan 2014 14:44:02 -0800 (PST) From: Zoran Markovic To: linux-kernel@vger.kernel.org Cc: linux-wireless@vger.kernel.org, netdev@vger.kernel.org, Shaibal Dutta , Johannes Berg , "John W. Linville" , "David S. Miller" , Zoran Markovic Subject: [RFC PATCH] net: rfkill: move poll work to power efficient workqueue Date: Thu, 30 Jan 2014 14:43:34 -0800 Message-Id: <1391121814-6026-1-git-send-email-zoran.markovic@linaro.org> X-Mailer: git-send-email 1.7.9.5 Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org X-Spam-Status: No, score=-7.3 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Shaibal Dutta This patch moves the rfkill poll_work to the power efficient workqueue. This work does not have to be bound to the CPU that scheduled it, hence the selection of CPU that executes it would be left to the scheduler. Net result is that CPU idle times would be extended, resulting in power savings. This behaviour is enabled when CONFIG_WQ_POWER_EFFICIENT is selected. Cc: Johannes Berg Cc: "John W. Linville" Cc: "David S. Miller" Signed-off-by: Shaibal Dutta [zoran.markovic@linaro.org: Rebased to latest kernel, added commit message. Fixed workqueue selection after suspend/resume cycle.] Signed-off-by: Zoran Markovic --- net/rfkill/core.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/net/rfkill/core.c b/net/rfkill/core.c index ed7e0b4..b3b16c0 100644 --- a/net/rfkill/core.c +++ b/net/rfkill/core.c @@ -789,7 +789,8 @@ void rfkill_resume_polling(struct rfkill *rfkill) if (!rfkill->ops->poll) return; - schedule_work(&rfkill->poll_work.work); + queue_delayed_work(system_power_efficient_wq, + &rfkill->poll_work, 0); } EXPORT_SYMBOL(rfkill_resume_polling); @@ -894,7 +895,8 @@ static void rfkill_poll(struct work_struct *work) */ rfkill->ops->poll(rfkill, rfkill->data); - schedule_delayed_work(&rfkill->poll_work, + queue_delayed_work(system_power_efficient_wq, + &rfkill->poll_work, round_jiffies_relative(POLL_INTERVAL)); } @@ -958,7 +960,8 @@ int __must_check rfkill_register(struct rfkill *rfkill) INIT_WORK(&rfkill->sync_work, rfkill_sync_work); if (rfkill->ops->poll) - schedule_delayed_work(&rfkill->poll_work, + queue_delayed_work(system_power_efficient_wq, + &rfkill->poll_work, round_jiffies_relative(POLL_INTERVAL)); if (!rfkill->persistent || rfkill_epo_lock_active) {