From patchwork Sun Mar 31 14:31:47 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Viresh Kumar X-Patchwork-Id: 2368511 Return-Path: X-Original-To: patchwork-linux-fbdev@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id AE5AEDF264 for ; Sun, 31 Mar 2013 14:33:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755845Ab3CaOcp (ORCPT ); Sun, 31 Mar 2013 10:32:45 -0400 Received: from mail-pa0-f43.google.com ([209.85.220.43]:39773 "EHLO mail-pa0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755286Ab3CaOcn (ORCPT ); Sun, 31 Mar 2013 10:32:43 -0400 Received: by mail-pa0-f43.google.com with SMTP id hz11so952610pad.2 for ; Sun, 31 Mar 2013 07:32:42 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-received:from:to:cc:subject:date:message-id:x-mailer:in-reply-to :references:in-reply-to:references:x-gm-message-state; bh=tbArIdST0tEnhUK4AcKAPS+PQMcrq5DZl1x6J28siHM=; b=CWhbcbrgKSVHqDTYYnnfrlJddwHChhCht8U96SDxEenJkKZcIcGIzPyWbzjoPQ6npp ju+QTGj7N+aaWDShPcKbOpEbHmAUUOaCiEVGqzEnPys2NQWoeTil/H3VqIAr11IgDILk 0tn+Q/FSvykZTGEbv0DO1r1pNY4AGzvfaTrfh27yGkPJYw7Mu/pdAQp7Rum6tbHYMmJm zlwVSyAm2lE9Mliky0cw0VoOnvtGaMfFDIN7Wt8G6i3ER9ZVnKkwBg3d6eSe6b51Cdux 7MwgZoDydt8bacD6q18CT8Y8qBssiH0FCy9imbkCT4Q3uG8T6BtxB8j9zzMJflInpgVD V3Vg== X-Received: by 10.68.232.72 with SMTP id tm8mr13570548pbc.108.1364740362745; Sun, 31 Mar 2013 07:32:42 -0700 (PDT) Received: from localhost ([122.167.73.68]) by mx.google.com with ESMTPS id ky10sm11366192pab.23.2013.03.31.07.32.36 (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Sun, 31 Mar 2013 07:32:42 -0700 (PDT) From: Viresh Kumar To: tj@kernel.org Cc: linaro-kernel@lists.linaro.org, patches@linaro.org, robin.randhawa@arm.com, Steve.Bannister@arm.com, Liviu.Dudau@arm.com, charles.garcia-tobin@arm.com, arvind.chauhan@arm.com, davem@davemloft.net, airlied@redhat.com, axboe@kernel.dk, tglx@linutronix.de, peterz@infradead.org, mingo@redhat.com, rostedt@goodmis.org, linux-rt-users@vger.kernel.org, linux-kernel@vger.kernel.org, Viresh Kumar , linux-fbdev@vger.kernel.org Subject: [PATCH V4 4/4] fbcon: queue work on unbound wq Date: Sun, 31 Mar 2013 20:01:47 +0530 Message-Id: <2a30b6c081a7899fbce88bddf20b94c348eb4654.1364740180.git.viresh.kumar@linaro.org> X-Mailer: git-send-email 1.7.12.rc2.18.g61b472e In-Reply-To: References: In-Reply-To: References: X-Gm-Message-State: ALoCoQnmtE7XWQ1yN+gN7I41bdUBGwu3FT70vCBBGqGoL6NkLBlBRZcscIKnbx57MCsRTe6L8hOx Sender: linux-fbdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-fbdev@vger.kernel.org fbcon uses workqueues and it has no real dependency of scheduling these on the cpu which scheduled them. On a idle system, it is observed that and idle cpu wakes up many times just to service this work. It would be better if we can schedule it on a cpu which the scheduler believes to be the most appropriate one. This patch replaces system_wq with system_unbound_wq. Cc: Dave Airlie Cc: linux-fbdev@vger.kernel.org Signed-off-by: Viresh Kumar --- drivers/video/console/fbcon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/console/fbcon.c b/drivers/video/console/fbcon.c index 3cd6759..6c1f7c3 100644 --- a/drivers/video/console/fbcon.c +++ b/drivers/video/console/fbcon.c @@ -404,7 +404,7 @@ static void cursor_timer_handler(unsigned long dev_addr) struct fb_info *info = (struct fb_info *) dev_addr; struct fbcon_ops *ops = info->fbcon_par; - schedule_work(&info->queue); + queue_work(system_unbound_wq, &info->queue); mod_timer(&ops->cursor_timer, jiffies + HZ/5); }