From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Zijlstra Subject: Re: [PATCH 0/9] setting the table for integration of cpuidle with the scheduler Date: Mon, 27 Jan 2014 13:47:31 +0100 Message-ID: <20140127124731.GA4941@twins.programming.kicks-ass.net> References: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Nicolas Pitre Cc: linaro-kernel@lists.linaro.org, Russell King , linux-sh@vger.kernel.org, Catalin Marinas , linux-pm@vger.kernel.org, Daniel Lezcano , "Rafael J. Wysocki" , linux-kernel@vger.kernel.org, Paul Mundt , Benjamin Herrenschmidt , Thomas Gleixner , linuxppc-dev@lists.ozlabs.org, Ingo Molnar , linux-arm-kernel@lists.infradead.org List-Id: linux-pm@vger.kernel.org On Mon, Jan 27, 2014 at 01:08:15AM -0500, Nicolas Pitre wrote: > As everyone should know by now, we want to integrate the cpuidle > governor with the scheduler for a more efficient idling of CPUs. > In order to help the transition, this small patch series moves the > existing interaction with cpuidle from architecture code to generic > core code. No functional change should have occurred yet. > > The ARM, PPC, SH and X86 architectures are concerned. Small cleanups > to ARM and ARM64 are also included. I don't know yet the best path for > those patches to get into mainline, but it is probably best if they > stay together. So ACKs from architecture maintainers would be greatly > appreciated. > > > arch/arm/kernel/process.c | 21 +++--------- > arch/arm/kernel/setup.c | 7 ++++ > arch/arm64/kernel/process.c | 5 --- > arch/arm64/kernel/setup.c | 7 ++++ > arch/powerpc/platforms/pseries/processor_idle.c | 5 +++ > arch/powerpc/platforms/pseries/setup.c | 34 ++++++++----------- > arch/sh/kernel/idle.c | 4 +-- > arch/x86/kernel/process.c | 5 +-- > include/linux/cpu.h | 1 - > kernel/Makefile | 1 - > kernel/cpu/Makefile | 1 - > kernel/sched/Makefile | 2 +- > kernel/{cpu => sched}/idle.c | 6 ++-- > 13 files changed, 44 insertions(+), 55 deletions(-) Thomas, any objections to this? It looks like a sensible thing to do. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Zijlstra Date: Mon, 27 Jan 2014 12:47:31 +0000 Subject: Re: [PATCH 0/9] setting the table for integration of cpuidle with the scheduler Message-Id: <20140127124731.GA4941@twins.programming.kicks-ass.net> List-Id: References: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> In-Reply-To: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On Mon, Jan 27, 2014 at 01:08:15AM -0500, Nicolas Pitre wrote: > As everyone should know by now, we want to integrate the cpuidle > governor with the scheduler for a more efficient idling of CPUs. > In order to help the transition, this small patch series moves the > existing interaction with cpuidle from architecture code to generic > core code. No functional change should have occurred yet. > > The ARM, PPC, SH and X86 architectures are concerned. Small cleanups > to ARM and ARM64 are also included. I don't know yet the best path for > those patches to get into mainline, but it is probably best if they > stay together. So ACKs from architecture maintainers would be greatly > appreciated. > > > arch/arm/kernel/process.c | 21 +++--------- > arch/arm/kernel/setup.c | 7 ++++ > arch/arm64/kernel/process.c | 5 --- > arch/arm64/kernel/setup.c | 7 ++++ > arch/powerpc/platforms/pseries/processor_idle.c | 5 +++ > arch/powerpc/platforms/pseries/setup.c | 34 ++++++++----------- > arch/sh/kernel/idle.c | 4 +-- > arch/x86/kernel/process.c | 5 +-- > include/linux/cpu.h | 1 - > kernel/Makefile | 1 - > kernel/cpu/Makefile | 1 - > kernel/sched/Makefile | 2 +- > kernel/{cpu => sched}/idle.c | 6 ++-- > 13 files changed, 44 insertions(+), 55 deletions(-) Thomas, any objections to this? It looks like a sensible thing to do. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from merlin.infradead.org (unknown [IPv6:2001:4978:20e::2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id F03232C0092 for ; Mon, 27 Jan 2014 23:47:59 +1100 (EST) Date: Mon, 27 Jan 2014 13:47:31 +0100 From: Peter Zijlstra To: Nicolas Pitre Subject: Re: [PATCH 0/9] setting the table for integration of cpuidle with the scheduler Message-ID: <20140127124731.GA4941@twins.programming.kicks-ass.net> References: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> Cc: linaro-kernel@lists.linaro.org, Russell King , linux-sh@vger.kernel.org, Catalin Marinas , linux-pm@vger.kernel.org, Daniel Lezcano , "Rafael J. Wysocki" , linux-kernel@vger.kernel.org, Paul Mundt , Thomas Gleixner , linuxppc-dev@lists.ozlabs.org, Ingo Molnar , linux-arm-kernel@lists.infradead.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, Jan 27, 2014 at 01:08:15AM -0500, Nicolas Pitre wrote: > As everyone should know by now, we want to integrate the cpuidle > governor with the scheduler for a more efficient idling of CPUs. > In order to help the transition, this small patch series moves the > existing interaction with cpuidle from architecture code to generic > core code. No functional change should have occurred yet. > > The ARM, PPC, SH and X86 architectures are concerned. Small cleanups > to ARM and ARM64 are also included. I don't know yet the best path for > those patches to get into mainline, but it is probably best if they > stay together. So ACKs from architecture maintainers would be greatly > appreciated. > > > arch/arm/kernel/process.c | 21 +++--------- > arch/arm/kernel/setup.c | 7 ++++ > arch/arm64/kernel/process.c | 5 --- > arch/arm64/kernel/setup.c | 7 ++++ > arch/powerpc/platforms/pseries/processor_idle.c | 5 +++ > arch/powerpc/platforms/pseries/setup.c | 34 ++++++++----------- > arch/sh/kernel/idle.c | 4 +-- > arch/x86/kernel/process.c | 5 +-- > include/linux/cpu.h | 1 - > kernel/Makefile | 1 - > kernel/cpu/Makefile | 1 - > kernel/sched/Makefile | 2 +- > kernel/{cpu => sched}/idle.c | 6 ++-- > 13 files changed, 44 insertions(+), 55 deletions(-) Thomas, any objections to this? It looks like a sensible thing to do. From mboxrd@z Thu Jan 1 00:00:00 1970 From: peterz@infradead.org (Peter Zijlstra) Date: Mon, 27 Jan 2014 13:47:31 +0100 Subject: [PATCH 0/9] setting the table for integration of cpuidle with the scheduler In-Reply-To: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> References: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> Message-ID: <20140127124731.GA4941@twins.programming.kicks-ass.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, Jan 27, 2014 at 01:08:15AM -0500, Nicolas Pitre wrote: > As everyone should know by now, we want to integrate the cpuidle > governor with the scheduler for a more efficient idling of CPUs. > In order to help the transition, this small patch series moves the > existing interaction with cpuidle from architecture code to generic > core code. No functional change should have occurred yet. > > The ARM, PPC, SH and X86 architectures are concerned. Small cleanups > to ARM and ARM64 are also included. I don't know yet the best path for > those patches to get into mainline, but it is probably best if they > stay together. So ACKs from architecture maintainers would be greatly > appreciated. > > > arch/arm/kernel/process.c | 21 +++--------- > arch/arm/kernel/setup.c | 7 ++++ > arch/arm64/kernel/process.c | 5 --- > arch/arm64/kernel/setup.c | 7 ++++ > arch/powerpc/platforms/pseries/processor_idle.c | 5 +++ > arch/powerpc/platforms/pseries/setup.c | 34 ++++++++----------- > arch/sh/kernel/idle.c | 4 +-- > arch/x86/kernel/process.c | 5 +-- > include/linux/cpu.h | 1 - > kernel/Makefile | 1 - > kernel/cpu/Makefile | 1 - > kernel/sched/Makefile | 2 +- > kernel/{cpu => sched}/idle.c | 6 ++-- > 13 files changed, 44 insertions(+), 55 deletions(-) Thomas, any objections to this? It looks like a sensible thing to do. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753799AbaA0MsH (ORCPT ); Mon, 27 Jan 2014 07:48:07 -0500 Received: from merlin.infradead.org ([205.233.59.134]:55186 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751431AbaA0MsE (ORCPT ); Mon, 27 Jan 2014 07:48:04 -0500 Date: Mon, 27 Jan 2014 13:47:31 +0100 From: Peter Zijlstra To: Nicolas Pitre Cc: Russell King , Catalin Marinas , Benjamin Herrenschmidt , Paul Mundt , Thomas Gleixner , Ingo Molnar , "Rafael J. Wysocki" , Daniel Lezcano , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linuxppc-dev@lists.ozlabs.org, linux-sh@vger.kernel.org, linux-pm@vger.kernel.org, linaro-kernel@lists.linaro.org Subject: Re: [PATCH 0/9] setting the table for integration of cpuidle with the scheduler Message-ID: <20140127124731.GA4941@twins.programming.kicks-ass.net> References: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1390802904-28399-1-git-send-email-nicolas.pitre@linaro.org> User-Agent: Mutt/1.5.21 (2012-12-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jan 27, 2014 at 01:08:15AM -0500, Nicolas Pitre wrote: > As everyone should know by now, we want to integrate the cpuidle > governor with the scheduler for a more efficient idling of CPUs. > In order to help the transition, this small patch series moves the > existing interaction with cpuidle from architecture code to generic > core code. No functional change should have occurred yet. > > The ARM, PPC, SH and X86 architectures are concerned. Small cleanups > to ARM and ARM64 are also included. I don't know yet the best path for > those patches to get into mainline, but it is probably best if they > stay together. So ACKs from architecture maintainers would be greatly > appreciated. > > > arch/arm/kernel/process.c | 21 +++--------- > arch/arm/kernel/setup.c | 7 ++++ > arch/arm64/kernel/process.c | 5 --- > arch/arm64/kernel/setup.c | 7 ++++ > arch/powerpc/platforms/pseries/processor_idle.c | 5 +++ > arch/powerpc/platforms/pseries/setup.c | 34 ++++++++----------- > arch/sh/kernel/idle.c | 4 +-- > arch/x86/kernel/process.c | 5 +-- > include/linux/cpu.h | 1 - > kernel/Makefile | 1 - > kernel/cpu/Makefile | 1 - > kernel/sched/Makefile | 2 +- > kernel/{cpu => sched}/idle.c | 6 ++-- > 13 files changed, 44 insertions(+), 55 deletions(-) Thomas, any objections to this? It looks like a sensible thing to do.