Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Srivatsa S. Bhat
On 05/13/2013 08:55 PM, Daniel Lezcano wrote: > On 05/13/2013 11:04 AM, Srivatsa S. Bhat wrote: >> On 05/13/2013 12:22 PM, Lianwei Wang wrote: >>> Thank you. Patch is updated. >>> >>> From 2d0b4afb5461847dcdf08a87b02015d061b12e85 Mon Sep 17 00:00:00 2001 >>> From: Lianwei Wang >>> Date: Fri, 26

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Daniel Lezcano
On 05/13/2013 11:04 AM, Srivatsa S. Bhat wrote: > On 05/13/2013 12:22 PM, Lianwei Wang wrote: >> Thank you. Patch is updated. >> >> From 2d0b4afb5461847dcdf08a87b02015d061b12e85 Mon Sep 17 00:00:00 2001 >> From: Lianwei Wang >> Date: Fri, 26 Apr 2013 10:59:24 +0800 >> Subject: [PATCH] cpuidle:

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Srivatsa S. Bhat
On 05/13/2013 12:22 PM, Lianwei Wang wrote: > Thank you. Patch is updated. > > From 2d0b4afb5461847dcdf08a87b02015d061b12e85 Mon Sep 17 00:00:00 2001 > From: Lianwei Wang > Date: Fri, 26 Apr 2013 10:59:24 +0800 > Subject: [PATCH] cpuidle: wakeup processor on a smaller latency > > Checking the

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Lianwei Wang
smp_call_function_single may not good for quad or more cores because it just send one interrupt one time and wait there... Update the patch to wakeup the cpus at the same time by calling smp_call_function_many. diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Lianwei Wang
Thank you. Patch is updated. >From 2d0b4afb5461847dcdf08a87b02015d061b12e85 Mon Sep 17 00:00:00 2001 From: Lianwei Wang Date: Fri, 26 Apr 2013 10:59:24 +0800 Subject: [PATCH] cpuidle: wakeup processor on a smaller latency Checking the PM-Qos latency and cpu idle sleep latency, and only wakeup

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Lianwei Wang
Thank you. Patch is updated. From 2d0b4afb5461847dcdf08a87b02015d061b12e85 Mon Sep 17 00:00:00 2001 From: Lianwei Wang lianwei.w...@gmail.com Date: Fri, 26 Apr 2013 10:59:24 +0800 Subject: [PATCH] cpuidle: wakeup processor on a smaller latency Checking the PM-Qos latency and cpu idle sleep

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Lianwei Wang
smp_call_function_single may not good for quad or more cores because it just send one interrupt one time and wait there... Update the patch to wakeup the cpus at the same time by calling smp_call_function_many. diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Srivatsa S. Bhat
On 05/13/2013 12:22 PM, Lianwei Wang wrote: Thank you. Patch is updated. From 2d0b4afb5461847dcdf08a87b02015d061b12e85 Mon Sep 17 00:00:00 2001 From: Lianwei Wang lianwei.w...@gmail.com Date: Fri, 26 Apr 2013 10:59:24 +0800 Subject: [PATCH] cpuidle: wakeup processor on a smaller latency

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Daniel Lezcano
On 05/13/2013 11:04 AM, Srivatsa S. Bhat wrote: On 05/13/2013 12:22 PM, Lianwei Wang wrote: Thank you. Patch is updated. From 2d0b4afb5461847dcdf08a87b02015d061b12e85 Mon Sep 17 00:00:00 2001 From: Lianwei Wang lianwei.w...@gmail.com Date: Fri, 26 Apr 2013 10:59:24 +0800 Subject: [PATCH]

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-13 Thread Srivatsa S. Bhat
On 05/13/2013 08:55 PM, Daniel Lezcano wrote: On 05/13/2013 11:04 AM, Srivatsa S. Bhat wrote: On 05/13/2013 12:22 PM, Lianwei Wang wrote: Thank you. Patch is updated. From 2d0b4afb5461847dcdf08a87b02015d061b12e85 Mon Sep 17 00:00:00 2001 From: Lianwei Wang lianwei.w...@gmail.com Date: Fri,

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-09 Thread Daniel Lezcano
On 05/09/2013 09:14 AM, Lianwei Wang wrote: > Thank you very much. I have a quick updated patch based on your comments. > > diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c > index 2f0083a..cd1af4b 100644 > --- a/drivers/cpuidle/cpuidle.c > +++ b/drivers/cpuidle/cpuidle.c > @@

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-09 Thread Lianwei Wang
Thank you very much. I have a quick updated patch based on your comments. diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index 2f0083a..cd1af4b 100644 --- a/drivers/cpuidle/cpuidle.c +++ b/drivers/cpuidle/cpuidle.c @@ -18,6 +18,7 @@ #include #include #include +#include

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-09 Thread Lianwei Wang
Thank you very much. I have a quick updated patch based on your comments. diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index 2f0083a..cd1af4b 100644 --- a/drivers/cpuidle/cpuidle.c +++ b/drivers/cpuidle/cpuidle.c @@ -18,6 +18,7 @@ #include linux/ktime.h #include

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-09 Thread Daniel Lezcano
On 05/09/2013 09:14 AM, Lianwei Wang wrote: Thank you very much. I have a quick updated patch based on your comments. diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index 2f0083a..cd1af4b 100644 --- a/drivers/cpuidle/cpuidle.c +++ b/drivers/cpuidle/cpuidle.c @@ -18,6

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-08 Thread Daniel Lezcano
On 05/08/2013 04:44 AM, Lianwei Wang wrote: > When a PM-Qos is updated, the cpuidle driver will wakeup all the CPUs > no matter what a latency is set. But actually it only need to wakeup > the CPUs when a shorter latency is set. In this way we can reduce the > cpu wakeup count and save battery. I

Re: [linux-pm] [PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-08 Thread Daniel Lezcano
On 05/08/2013 04:44 AM, Lianwei Wang wrote: When a PM-Qos is updated, the cpuidle driver will wakeup all the CPUs no matter what a latency is set. But actually it only need to wakeup the CPUs when a shorter latency is set. In this way we can reduce the cpu wakeup count and save battery. I am

[PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-07 Thread Lianwei Wang
When a PM-Qos is updated, the cpuidle driver will wakeup all the CPUs no matter what a latency is set. But actually it only need to wakeup the CPUs when a shorter latency is set. In this way we can reduce the cpu wakeup count and save battery. So we can pass the prev_value to the notifier

[PATCH] cpuidle: don't wakeup processor when set a longer latency

2013-05-07 Thread Lianwei Wang
When a PM-Qos is updated, the cpuidle driver will wakeup all the CPUs no matter what a latency is set. But actually it only need to wakeup the CPUs when a shorter latency is set. In this way we can reduce the cpu wakeup count and save battery. So we can pass the prev_value to the notifier