. Remove the constraint of having to disable tickless idle on the broadcast
CPU by queueing a hrtimer dedicated to do broadcast.
V1 posting: https://lkml.org/lkml/2013/7/25/740.
1. Added the infrastructure to wakeup CPUs in deep idle states in which the
local timers stop.
---
Preeti U Murthy (4
...@linux.vnet.ibm.com
Signed-off-by: Preeti U. Murthy pre...@linux.vnet.ibm.com
Acked-by: Geoff Levand ge...@infradead.org [For the PS3 part]
---
arch/powerpc/include/asm/smp.h |2 +-
arch/powerpc/kernel/smp.c | 12 +---
arch/powerpc/platforms/cell/interrupt.c |2
-by: Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com
[Functions renamed to tick_broadcast* and Changelog modified by
Preeti U. Murthypre...@linux.vnet.ibm.com]
Signed-off-by: Preeti U. Murthy pre...@linux.vnet.ibm.com
Acked-by: Geoff Levand ge...@infradead.org [For the PS3 part]
---
arch
timers to
directly call into __timer_interupt(). One of the use cases of this is the
tick broadcast IPI handling in which the sleeping CPUs need to handle the local
timers that have expired.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/kernel/time.c | 73
idle state.
Signed-off-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com
[Changelog modified by Preeti U. Murthy pre...@linux.vnet.ibm.com]
Signed-off-by: Preeti U. Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/include/asm/processor.h |1 +
arch/powerpc/kernel/exceptions-64s.S | 10
in the broadcast mask. This newly nominated bc_cpu is woken up by
an IPI so as to queue the above mentioned hrtimer on it.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
include/linux/clockchips.h |4 -
kernel/time/clockevents.c|9 +-
kernel/time/tick-broadcast.c | 192
methods.
Signed-off-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com
Signed-off-by: Preeti U. Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/include/asm/opal.h|2 ++
arch/powerpc/kernel/exceptions-64s.S |2 +-
arch/powerpc/kernel/idle_power7.S
support
for fast sleep, enable it in the cpuidle framework on PowerNV.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/Kconfig |2 ++
arch/powerpc/kernel/time.c|2 +-
drivers/cpuidle/cpuidle-powernv.c | 39
Add deep idle states such as nap and fast sleep to the cpuidle state table
only if they are discovered from the device tree during cpuidle initialization.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
drivers/cpuidle/cpuidle-powernv.c | 81
Hi Paul,
On 01/15/2014 08:59 PM, Paul Gortmaker wrote:
On 14-01-15 03:07 AM, Preeti U Murthy wrote:
[...]
This patchset is based on mainline commit-id:8ae516aa8b8161254d3, and the
I figured I'd give this a quick sanity build test for a few
configs, but v3.13-rc1-141-g8ae516aa8b81
On 01/14/2014 01:07 PM, Srivatsa S. Bhat wrote:
> On 01/14/2014 12:30 PM, Srivatsa S. Bhat wrote:
>> On 01/14/2014 11:35 AM, Preeti U Murthy wrote:
>>> On PowerPC, in a particular test scenario, all the cpu idle states were
>>> disabled.
>>> Inspite of this
Hi Srivatsa,
On 01/14/2014 12:30 PM, Srivatsa S. Bhat wrote:
> On 01/14/2014 11:35 AM, Preeti U Murthy wrote:
>> On PowerPC, in a particular test scenario, all the cpu idle states were
>> disabled.
>> Inspite of this it was observed that the idle state count of the sha
Hi Srivatsa,
On 01/14/2014 12:30 PM, Srivatsa S. Bhat wrote:
On 01/14/2014 11:35 AM, Preeti U Murthy wrote:
On PowerPC, in a particular test scenario, all the cpu idle states were
disabled.
Inspite of this it was observed that the idle state count of the shallowest
idle state, snooze
On 01/14/2014 01:07 PM, Srivatsa S. Bhat wrote:
On 01/14/2014 12:30 PM, Srivatsa S. Bhat wrote:
On 01/14/2014 11:35 AM, Preeti U Murthy wrote:
On PowerPC, in a particular test scenario, all the cpu idle states were
disabled.
Inspite of this it was observed that the idle state count
the menu governor criteria to be chosen as the
next idle state.
This patch adds the code to indicate that a valid cpu idle state could not be
chosen by the menu governor and reports back to arch so that it can take some
default action.
Signed-off-by: Preeti U Murthy
---
drivers/cpuidle/cpuidle.c
the menu governor criteria to be chosen as the
next idle state.
This patch adds the code to indicate that a valid cpu idle state could not be
chosen by the menu governor and reports back to arch so that it can take some
default action.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
On 01/07/2014 06:01 PM, Vincent Guittot wrote:
> On 7 January 2014 11:39, Preeti U Murthy wrote:
>> On 01/07/2014 03:20 PM, Peter Zijlstra wrote:
>>> On Tue, Jan 07, 2014 at 03:10:21PM +0530, Preeti U Murthy wrote:
>>>> What if we want to add arch specific flags
On 01/07/2014 04:43 PM, Peter Zijlstra wrote:
> On Tue, Jan 07, 2014 at 04:09:39PM +0530, Preeti U Murthy wrote:
>> On 01/07/2014 03:20 PM, Peter Zijlstra wrote:
>>> On Tue, Jan 07, 2014 at 03:10:21PM +0530, Preeti U Murthy wrote:
>>>> What if we want to add arch spe
On 01/07/2014 03:20 PM, Peter Zijlstra wrote:
> On Tue, Jan 07, 2014 at 03:10:21PM +0530, Preeti U Murthy wrote:
>> What if we want to add arch specific flags to the NUMA domain? Currently
>> with Peter's patch:https://lkml.org/lkml/2013/11/5/239 and this patch,
>> the arch ca
r sharing) but it can become complex if we
> want to add more.
What if we want to add arch specific flags to the NUMA domain? Currently
with Peter's patch:https://lkml.org/lkml/2013/11/5/239 and this patch,
the arch can modify the sd flags of the topology levels till just before
the NUMA domain.
to perhaps call into arch here to probe for
additional flags?
Thanks
Regards
Preeti U Murthy
Regards
Vincent
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo
On 01/07/2014 03:20 PM, Peter Zijlstra wrote:
On Tue, Jan 07, 2014 at 03:10:21PM +0530, Preeti U Murthy wrote:
What if we want to add arch specific flags to the NUMA domain? Currently
with Peter's patch:https://lkml.org/lkml/2013/11/5/239 and this patch,
the arch can modify the sd flags
On 01/07/2014 04:43 PM, Peter Zijlstra wrote:
On Tue, Jan 07, 2014 at 04:09:39PM +0530, Preeti U Murthy wrote:
On 01/07/2014 03:20 PM, Peter Zijlstra wrote:
On Tue, Jan 07, 2014 at 03:10:21PM +0530, Preeti U Murthy wrote:
What if we want to add arch specific flags to the NUMA domain? Currently
On 01/07/2014 06:01 PM, Vincent Guittot wrote:
On 7 January 2014 11:39, Preeti U Murthy pre...@linux.vnet.ibm.com wrote:
On 01/07/2014 03:20 PM, Peter Zijlstra wrote:
On Tue, Jan 07, 2014 at 03:10:21PM +0530, Preeti U Murthy wrote:
What if we want to add arch specific flags to the NUMA domain
mode to
periodic.
Signed-off-by: Preeti U Murthy
---
include/linux/clockchips.h |4 -
kernel/time/clockevents.c|8 +-
kernel/time/tick-broadcast.c | 180 ++
kernel/time/tick-internal.h |8 +-
4 files changed, 173 insertions(+), 27
us of the lower domains. As far as I see, this patch does not change
these assumptions. Hence I am unable to imagine a scenario when the
parent might not include all cpus of its children domain. Do you have
such a scenario in mind which can arise due to this patch ?
Thanks
Regards
Preeti U Murthy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
these assumptions. Hence I am unable to imagine a scenario when the
parent might not include all cpus of its children domain. Do you have
such a scenario in mind which can arise due to this patch ?
Thanks
Regards
Preeti U Murthy
--
To unsubscribe from this list: send the line unsubscribe linux
mode to
periodic.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
include/linux/clockchips.h |4 -
kernel/time/clockevents.c|8 +-
kernel/time/tick-broadcast.c | 180 ++
kernel/time/tick-internal.h |8 +-
4 files changed, 173
Hi,
The patch had some compile time fixes to be done. It was accidentally mailed
out before doing so. Below is the right patch. Apologies for the same.
Thanks
Regards
Preeti U Murthy
-
time: Support in tick broadcast
Hi,
The patch had some compile time fixes to be done. It was accidentally mailed
out before doing so. Below is the right patch. Apologies for the same.
Thanks
Regards
Preeti U Murthy
-
time: Support in tick broadcast
Hi Ben,
On 12/13/2013 10:47 AM, Benjamin Herrenschmidt wrote:
> On Fri, 2013-12-13 at 09:49 +0530, Preeti U Murthy wrote:
>> On some architectures, in certain CPU deep idle states the local timers stop.
>> An external clock device is used to wakeup these CPUs. The
bc_cpu is woken up by
an IPI so as to queue the above mentioned hrtimer on itself.
This patch is compile tested only.
Signed-off-by: Preeti U Murthy
---
include/linux/clockchips.h |4 +
kernel/time/clockevents.c|8 +-
kernel/time/tick-broadcast.c | 157
bc_cpu is woken up by
an IPI so as to queue the above mentioned hrtimer on itself.
This patch is compile tested only.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
include/linux/clockchips.h |4 +
kernel/time/clockevents.c|8 +-
kernel/time/tick-broadcast.c | 157
Hi Ben,
On 12/13/2013 10:47 AM, Benjamin Herrenschmidt wrote:
On Fri, 2013-12-13 at 09:49 +0530, Preeti U Murthy wrote:
On some architectures, in certain CPU deep idle states the local timers stop.
An external clock device is used to wakeup these CPUs. The kernel support
for the
wakeup
s through some initialization steps after wakeup,
it will be in a position to acknowledge the external interrupt and
service it accordingly.
Ideally the interrupt handler of this external interrupt should be that
of the local timer itself since it was meant to act on the behalf of the
local timer interrupt.
Thanks
to acknowledge the external interrupt and
service it accordingly.
Ideally the interrupt handler of this external interrupt should be that
of the local timer itself since it was meant to act on the behalf of the
local timer interrupt.
Thanks
Regards
Preeti U Murthy
--
To unsubscribe from this list: send
Hi Thomas,
On 11/29/2013 05:28 PM, Thomas Gleixner wrote:
> On Fri, 29 Nov 2013, Preeti U Murthy wrote:
>> diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
>> index b44b52c..cafa788 100644
>> --- a/arch/powerpc/Kconfig
>> +++ b/arch/powerpc/Kconfig
>&g
Hi Thomas,
On 11/29/2013 08:09 PM, Thomas Gleixner wrote:
> On Fri, 29 Nov 2013, Preeti U Murthy wrote:
>> +static enum hrtimer_restart handle_broadcast(struct hrtimer *hrtimer)
>> +{
>> +struct clock_event_device *bc_evt = _timer;
>> +ktime_t interval, next_bc_
Hi Thomas,
On 11/29/2013 08:09 PM, Thomas Gleixner wrote:
On Fri, 29 Nov 2013, Preeti U Murthy wrote:
+static enum hrtimer_restart handle_broadcast(struct hrtimer *hrtimer)
+{
+struct clock_event_device *bc_evt = bc_timer;
+ktime_t interval, next_bc_tick, now;
+
+now
Hi Thomas,
On 11/29/2013 05:28 PM, Thomas Gleixner wrote:
On Fri, 29 Nov 2013, Preeti U Murthy wrote:
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index b44b52c..cafa788 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -129,6 +129,8 @@ config PPC
select
Signed-off-by: Preeti U Murthy
---
arch/powerpc/Kconfig|2 +
arch/powerpc/include/asm/time.h |1 +
arch/powerpc/kernel/time.c | 58 ++-
3 files changed, 60 insertions(+), 1 deletion(-)
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kco
so as to not
miss wakeups under such scenarios.
Signed-off-by: Preeti U Murthy
---
arch/powerpc/include/asm/time.h |1 +
arch/powerpc/kernel/time.c |1 +
drivers/cpuidle/cpuidle-powerpc-book3s.c | 22 ++
3 files changed, 24 insertions
repeats.
Protect the region of nomination,de-nomination and check for existence of
broadcast
CPU with a lock to ensure synchronization between them.
Signed-off-by: Preeti U Murthy
---
arch/powerpc/include/asm/time.h |1
arch/powerpc/kernel/time.c |2
drivers
Add deep idle states such as nap and fast sleep to the cpuidle state table
only if they are discovered from the device tree during cpuidle initialization.
Signed-off-by: Preeti U. Murthy
---
drivers/cpuidle/cpuidle-powerpc-book3s.c | 81 --
1 file changed, 64
Signed-off-by: Vaidyanathan Srinivasan
Signed-off-by: Preeti U. Murthy
---
arch/powerpc/include/asm/opal.h|2 ++
arch/powerpc/kernel/exceptions-64s.S |2 +-
arch/powerpc/kernel/idle_power7.S | 27
arch/powerpc/platforms/po
-by: Vaidyanathan Srinivasan
[Changelog modified by Preeti U. Murthy ]
Signed-off-by: Preeti U. Murthy
---
arch/powerpc/include/asm/processor.h |1 +
arch/powerpc/kernel/exceptions-64s.S | 10 -
arch/powerpc/kernel/idle_power7.S| 63 --
3 files
timers to
directly call into __timer_interupt(). One of the use cases of this is the
tick broadcast IPI handling in which the sleeping CPUs need to handle the local
timers that have expired.
Signed-off-by: Preeti U Murthy
---
arch/powerpc/kernel/time.c | 73
oadcast CPU, instead of having a dedicated one.
2. Remove the constraint of having to disable tickless idle on the broadcast
CPU by queueing a hrtimer dedicated to do broadcast.
V1 posting: https://lkml.org/lkml/2013/7/25/740.
1. Added the infrastructure to wakeup CPUs in deep idle states in which the
loc
[Functions renamed to tick_broadcast* and Changelog modified by
Preeti U. Murthy]
Signed-off-by: Preeti U. Murthy
Acked-by: Geoff Levand [For the PS3 part]
---
arch/powerpc/include/asm/smp.h |2 +-
arch/powerpc/include/asm/time.h |1 +
arch/powerpc/kernel/smp.c
slots are available).
So, implement the functionality of PPC_MSG_CALL_FUNC_SINGLE using
PPC_MSG_CALL_FUNC itself and release its IPI message slot, so that it can be
used for something else in the future, if desired.
Signed-off-by: Srivatsa S. Bhat
Signed-off-by: Preeti U. Murthy
Acked-by: Geoff
...@linux.vnet.ibm.com
Signed-off-by: Preeti U. Murthy pre...@linux.vnet.ibm.com
Acked-by: Geoff Levand ge...@infradead.org [For the PS3 part]
---
arch/powerpc/include/asm/smp.h |2 +-
arch/powerpc/kernel/smp.c | 12 +---
arch/powerpc/platforms/cell/interrupt.c |2
.
2. Remove the constraint of having to disable tickless idle on the broadcast
CPU by queueing a hrtimer dedicated to do broadcast.
V1 posting: https://lkml.org/lkml/2013/7/25/740.
1. Added the infrastructure to wakeup CPUs in deep idle states in which the
local timers stop.
---
Preeti U Murthy
-by: Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com
[Functions renamed to tick_broadcast* and Changelog modified by
Preeti U. Murthypre...@linux.vnet.ibm.com]
Signed-off-by: Preeti U. Murthy pre...@linux.vnet.ibm.com
Acked-by: Geoff Levand ge...@infradead.org [For the PS3 part]
---
arch
idle state.
Signed-off-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com
[Changelog modified by Preeti U. Murthy pre...@linux.vnet.ibm.com]
Signed-off-by: Preeti U. Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/include/asm/processor.h |1 +
arch/powerpc/kernel/exceptions-64s.S | 10
timers to
directly call into __timer_interupt(). One of the use cases of this is the
tick broadcast IPI handling in which the sleeping CPUs need to handle the local
timers that have expired.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/kernel/time.c | 73
methods.
Signed-off-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com
Signed-off-by: Preeti U. Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/include/asm/opal.h|2 ++
arch/powerpc/kernel/exceptions-64s.S |2 +-
arch/powerpc/kernel/idle_power7.S
repeats.
Protect the region of nomination,de-nomination and check for existence of
broadcast
CPU with a lock to ensure synchronization between them.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/include/asm/time.h |1
arch/powerpc/kernel/time.c
Add deep idle states such as nap and fast sleep to the cpuidle state table
only if they are discovered from the device tree during cpuidle initialization.
Signed-off-by: Preeti U. Murthy pre...@linux.vnet.ibm.com
---
drivers/cpuidle/cpuidle-powerpc-book3s.c | 81
-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/Kconfig|2 +
arch/powerpc/include/asm/time.h |1 +
arch/powerpc/kernel/time.c | 58 ++-
3 files changed, 60 insertions(+), 1 deletion(-)
diff --git a/arch/powerpc/Kconfig b/arch
so as to not
miss wakeups under such scenarios.
Signed-off-by: Preeti U Murthy pre...@linux.vnet.ibm.com
---
arch/powerpc/include/asm/time.h |1 +
arch/powerpc/kernel/time.c |1 +
drivers/cpuidle/cpuidle-powerpc-book3s.c | 22 ++
3 files
Hi Peter,
On 11/14/2013 02:00 PM, Peter Zijlstra wrote:
> On Thu, Nov 14, 2013 at 11:36:27AM +0530, Preeti U Murthy wrote:
>> However I was thinking that a better fix would be to reorder the way we call
>> update_group_power() and cpu_attach_domain(). Why do we need to do
>&g
Hi Peter,
On 11/14/2013 02:00 PM, Peter Zijlstra wrote:
On Thu, Nov 14, 2013 at 11:36:27AM +0530, Preeti U Murthy wrote:
However I was thinking that a better fix would be to reorder the way we call
update_group_power() and cpu_attach_domain(). Why do we need to do
update_group_power
group_power()
to after cpu_attach_domain() so that it saves unnecessary iterations over
sched domains which could degenerate, and it fixes the issue that you have
brought out
as well. See below for the patch:
-----------
sched: Upd
:
---
sched: Update power of sched groups after sched domains have been attached to
CPUs
From: Preeti U Murthy pre...@linux.vnet.ibm.com
Avoid iterating unnecessarily over the sched domains which could potentially
degenerate
Commit-ID: 37dc6b50cee97954c4e6edcd5b1fa614b76038ee
Gitweb: http://git.kernel.org/tip/37dc6b50cee97954c4e6edcd5b1fa614b76038ee
Author: Preeti U Murthy
AuthorDate: Wed, 30 Oct 2013 08:42:52 +0530
Committer: Ingo Molnar
CommitDate: Wed, 6 Nov 2013 12:37:55 +0100
sched: Remove
Commit-ID: 37dc6b50cee97954c4e6edcd5b1fa614b76038ee
Gitweb: http://git.kernel.org/tip/37dc6b50cee97954c4e6edcd5b1fa614b76038ee
Author: Preeti U Murthy pre...@linux.vnet.ibm.com
AuthorDate: Wed, 30 Oct 2013 08:42:52 +0530
Committer: Ingo Molnar mi...@kernel.org
CommitDate: Wed, 6 Nov 2013
eried directly when required.
>>
>> While we are at it, we might as well change the nohz_idle parameter to be
>> updated at the sd_busy domain level alone and not the base domain level of a
>> CPU.
>> This will unify the concept of busy cpus at just one level of sched do
be incremented/decremented.
Resulting is nohz_kick_needed returning 0.
Right and it *should* return 0. There is no sibling domain that can
offload tasks from it. Therefore there is no point kicking nohz idle
balance.
Regards
Preeti U Murthy
Thanks,
Kamalesh.
--
To unsubscribe from this list
The changelog has missed mentioning the introduction of sd_asym per_cpu sched
domain.
Apologies for this. The patch with the changelog including mention of sd_asym is
pasted below.
Regards
Preeti U Murthy
---
sched: Remove un-necessary iteration over sched domains to update
at the sd_busy domain level alone and not the base domain level of a
CPU.
This will unify the concept of busy cpus at just one level of sched domain
where it is currently used.
Signed-off-by: Preeti U Murthy
---
kernel/sched/core.c |6 ++
kernel/sched/fair.c | 38
and() will not yield any set bits if this domain
has no idle cpu.
Hence, nr_busy check against group weight can be removed.
Reported-by: Michael Neuling
Signed-off-by: Vaidyanathan Srinivasan
Signed-off-by: Preeti U Murthy
Tested-by: Michael Neuling
---
kernel/sched/fair.c |2 +-
1 file chan
domain-sd_busy where it is relevant.
3. Introduce sd_asym to represent the sched domain where asymmetric load
balancing has to be done.
---
Preeti U Murthy (1):
sched: Remove un-necessary iteration over sched domains to update
nr_busy_cpus
Vaidyanathan Srinivasan (1):
sched: Fix
domain-sd_busy where it is relevant.
3. Introduce sd_asym to represent the sched domain where asymmetric load
balancing has to be done.
---
Preeti U Murthy (1):
sched: Remove un-necessary iteration over sched domains to update
nr_busy_cpus
Vaidyanathan Srinivasan (1):
sched: Fix
nr_busy_cpus parameter is used by nohz_kick_needed() to find out the number
of busy cpus in a sched domain which has SD_SHARE_PKG_RESOURCES flag set.
Therefore instead of updating nr_busy_cpus at every level of sched domain,
since it is irrelevant, we can update this parameter only at the parent
cpumask_first_and() will not yield any set bits if this domain
has no idle cpu.
Hence, nr_busy check against group weight can be removed.
Reported-by: Michael Neuling michael.neul...@au1.ibm.com
Signed-off-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com
Signed-off-by: Preeti U Murthy pre
The changelog has missed mentioning the introduction of sd_asym per_cpu sched
domain.
Apologies for this. The patch with the changelog including mention of sd_asym is
pasted below.
Regards
Preeti U Murthy
---
sched: Remove un-necessary iteration over sched domains to update
Hi Peter,
On 10/28/2013 09:23 PM, Peter Zijlstra wrote:
> On Mon, Oct 21, 2013 at 05:15:02PM +0530, Vaidyanathan Srinivasan wrote:
>> From: Preeti U Murthy
>>
>> The current logic in load balance is such that after picking the
>> busiest group, the load is attempted t
Hi Peter,
On 10/28/2013 07:20 PM, Peter Zijlstra wrote:
> On Thu, Oct 24, 2013 at 01:37:38PM +0530, Preeti U Murthy wrote:
>> kernel/sched/core.c |5 +
>> kernel/sched/fair.c | 38 --
>> kernel/sched/sched.h |1 +
&g
Hi Peter,
On 10/28/2013 07:20 PM, Peter Zijlstra wrote:
On Thu, Oct 24, 2013 at 01:37:38PM +0530, Preeti U Murthy wrote:
kernel/sched/core.c |5 +
kernel/sched/fair.c | 38 --
kernel/sched/sched.h |1 +
3 files changed, 26 insertions
Hi Peter,
On 10/28/2013 09:23 PM, Peter Zijlstra wrote:
On Mon, Oct 21, 2013 at 05:15:02PM +0530, Vaidyanathan Srinivasan wrote:
From: Preeti U Murthy pre...@linux.vnet.ibm.com
The current logic in load balance is such that after picking the
busiest group, the load is attempted to be moved
flags);
>> env.flags |= LBF_ALL_PINNED;
>> +if (share_pkg_res &&
>> +cpumask_intersects(cpus,
>> +to_cpumask(group
this just at the SIBLINGS level? Having the hyper
threads busy due to the scenario described in the changelog is bad for
performance.
Regards
Preeti U Murthy
___
Linuxppc-dev mailing list
linuxppc-...@lists.ozlabs.org
https://lists.ozlabs.org/listinfo
Hi Vincent,
I have addressed your comments and below is the fresh patch. This patch
applies on PATCH 2/3 posted in this thread.
Regards
Preeti U Murthy
sched:Remove un-necessary iterations over sched domains to update/query
nr_busy_cpus
From: Preeti U Murthy
nr_busy_cpus parameter is used
Hi Vincent,
I have addressed your comments and below is the fresh patch. This patch
applies on PATCH 2/3 posted in this thread.
Regards
Preeti U Murthy
sched:Remove un-necessary iterations over sched domains to update/query
nr_busy_cpus
From: Preeti U Murthy pre...@linux.vnet.ibm.com
flags);
>> env.flags |= LBF_ALL_PINNED;
>> +if (share_pkg_res &&
>> + cpumask_intersects(cpus,
>> +to_cpumask(group->
y does. sd_busy therefore is irrelevant for asymmetric load
balancing.
Regards
Preeti U Murthy
START_PATCH---
sched: Fix nohz_kick_needed()
---
kernel/sched/core.c |4
kernel/sched/fair.c | 40 ++--
. sd_busy therefore is irrelevant for asymmetric load
balancing.
Regards
Preeti U Murthy
START_PATCH---
sched: Fix nohz_kick_needed()
---
kernel/sched/core.c |4
kernel/sched/fair.c | 40 ++--
kernel/sched
this a thought and get back.
Regards
Preeti U Murthy
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
On 10/23/2013 09:30 AM, Preeti U Murthy wrote:
> Hi Peter,
>
> On 10/23/2013 03:41 AM, Peter Zijlstra wrote:
>> On Mon, Oct 21, 2013 at 05:14:42PM +0530, Vaidyanathan Srinivasan wrote:
>>> kernel/sched/fair.c | 19 +--
>>> 1 file chang
is done to know the total number of
busy cpus at a sched domain level which has the SD_SHARE_PKG_RESOURCES
set and not at a sched group level.
So why not move nr_busy to struct sched_domain and having the below
patch which just updates this parameter for the sched domain, sd_busy ?
This wil
ng without any pre-conditions.
In a single socket machine, there will be a CPU domain encompassing the
socket and the MC domain will encompass a core. nohz_idle load balancer
will kick in if both the threads in the core have tasks running on them.
This is fair enough because the threads share th
machine, there will be a CPU domain encompassing the
socket and the MC domain will encompass a core. nohz_idle load balancer
will kick in if both the threads in the core have tasks running on them.
This is fair enough because the threads share the resources of the core.
Regards
Preeti U Murthy
the scalability issue. We also don't need to get to
sd-parent to get the nr_busy parameter for the sake of nohz_kick_needed().
What do you think?
Regards
Preeti U Murthy
This nohz stuff really needs to be re-thought and made more scalable --
its a royal pain :/
kernel/sched/core.c | 4
On 10/23/2013 09:30 AM, Preeti U Murthy wrote:
Hi Peter,
On 10/23/2013 03:41 AM, Peter Zijlstra wrote:
On Mon, Oct 21, 2013 at 05:14:42PM +0530, Vaidyanathan Srinivasan wrote:
kernel/sched/fair.c | 19 +--
1 file changed, 13 insertions(+), 6 deletions(-)
diff --git
better power numbers can be obtained or at-least the default power
efficiency of the kernel will show up.
However adding the new patchsets like packing small tasks, heterogeneous
scheduling, power aware scheduling etc.. *should* then yield good and
consistent power savings since they now stand o
stable power driver.
Regards
Preeti U Murthy
Also, how does this proposal deal with cpufreq's fundamental broken
approach to SMP? Afaict nothing considers the effect of one cpu upon
another -- something which isn't true at all.
In fact, I don't see anything except a random bunch of hooks
the broadcast framework approves. The per cpu device does
not register itself with the broadcast framework. It merely programs
itself for the next broadcast event. Hence this fix will not hinder the
broadcast support on PPC.
>
> Thanks,
> Sören
>
>
Regards
Preeti U Murthy
m the clock
> device of another cpu.
>
> Soren, do let me know if the above approach described in the patch has not
> addressed any of the challenges
> that you see with having a per cpu clock device as the broadcast source.
>
> Regards
> Preeti U Murthy
>
>
>
as the broadcast source.
Regards
Preeti U Murthy
On Fri, Sep 13, 2013 at 1:55 PM, Daniel Lezcano
daniel.lezc...@linaro.orgwrote:
On 09/12/2013 10:30 PM, Thomas Gleixner wrote:
On Thu, 12 Sep 2013, Soren Brinkmann wrote:
From: Stephen Boyd sb...@codeaurora.org
On most ARM systems the per
with the broadcast framework. It merely programs
itself for the next broadcast event. Hence this fix will not hinder the
broadcast support on PPC.
Thanks,
Sören
Regards
Preeti U Murthy
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message
801 - 900 of 1248 matches
Mail list logo