Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-07 Thread Daniel Lezcano
On 11/07/2014 05:29 AM, Preeti U Murthy wrote: On 11/06/2014 07:12 PM, Daniel Lezcano wrote: Preeti, I am wondering if we aren't going to a false debate. If the latency_req is 0, we should just poll and not enter in any idle state even if one has zero exit latency. With a zero latency req,

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-07 Thread Daniel Lezcano
On 11/07/2014 05:29 AM, Preeti U Murthy wrote: On 11/06/2014 07:12 PM, Daniel Lezcano wrote: Preeti, I am wondering if we aren't going to a false debate. If the latency_req is 0, we should just poll and not enter in any idle state even if one has zero exit latency. With a zero latency req,

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-06 Thread Preeti U Murthy
On 11/06/2014 07:12 PM, Daniel Lezcano wrote: > > Preeti, > > I am wondering if we aren't going to a false debate. > > If the latency_req is 0, we should just poll and not enter in any idle > state even if one has zero exit latency. With a zero latency req, we > want full reactivity on the

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-06 Thread Preeti U Murthy
On 11/06/2014 05:57 PM, Daniel Lezcano wrote: > On 11/06/2014 05:08 AM, Preeti U Murthy wrote: >> On 11/05/2014 07:58 PM, Daniel Lezcano wrote: >>> On 10/29/2014 03:01 AM, Preeti U Murthy wrote: On 10/29/2014 12:29 AM, Daniel Lezcano wrote: > On 10/28/2014 04:51 AM, Preeti Murthy wrote:

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-06 Thread Daniel Lezcano
Preeti, I am wondering if we aren't going to a false debate. If the latency_req is 0, we should just poll and not enter in any idle state even if one has zero exit latency. With a zero latency req, we want full reactivity on the system, not enter an idle state with all the computation in

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-06 Thread Daniel Lezcano
On 11/06/2014 05:08 AM, Preeti U Murthy wrote: On 11/05/2014 07:58 PM, Daniel Lezcano wrote: On 10/29/2014 03:01 AM, Preeti U Murthy wrote: On 10/29/2014 12:29 AM, Daniel Lezcano wrote: On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-06 Thread Preeti U Murthy
On 11/06/2014 05:57 PM, Daniel Lezcano wrote: On 11/06/2014 05:08 AM, Preeti U Murthy wrote: On 11/05/2014 07:58 PM, Daniel Lezcano wrote: On 10/29/2014 03:01 AM, Preeti U Murthy wrote: On 10/29/2014 12:29 AM, Daniel Lezcano wrote: On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-06 Thread Preeti U Murthy
On 11/06/2014 07:12 PM, Daniel Lezcano wrote: Preeti, I am wondering if we aren't going to a false debate. If the latency_req is 0, we should just poll and not enter in any idle state even if one has zero exit latency. With a zero latency req, we want full reactivity on the system, not

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-06 Thread Daniel Lezcano
On 11/06/2014 05:08 AM, Preeti U Murthy wrote: On 11/05/2014 07:58 PM, Daniel Lezcano wrote: On 10/29/2014 03:01 AM, Preeti U Murthy wrote: On 10/29/2014 12:29 AM, Daniel Lezcano wrote: On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-06 Thread Daniel Lezcano
Preeti, I am wondering if we aren't going to a false debate. If the latency_req is 0, we should just poll and not enter in any idle state even if one has zero exit latency. With a zero latency req, we want full reactivity on the system, not enter an idle state with all the computation in

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-05 Thread Preeti U Murthy
On 11/05/2014 07:58 PM, Daniel Lezcano wrote: > On 10/29/2014 03:01 AM, Preeti U Murthy wrote: >> On 10/29/2014 12:29 AM, Daniel Lezcano wrote: >>> On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano wrote: > When the

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-05 Thread Daniel Lezcano
On 11/05/2014 10:57 PM, Rafael J. Wysocki wrote: On Thursday, October 23, 2014 11:01:17 AM Daniel Lezcano wrote: When the pmqos latency requirement is set to zero that means "poll in all the cases". That is correctly implemented on x86 but not on the other archs. As how is written the code,

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-05 Thread Rafael J. Wysocki
On Thursday, October 23, 2014 11:01:17 AM Daniel Lezcano wrote: > When the pmqos latency requirement is set to zero that means "poll in all the > cases". > > That is correctly implemented on x86 but not on the other archs. > > As how is written the code, if the latency request is zero, the

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-05 Thread Daniel Lezcano
On 10/29/2014 03:01 AM, Preeti U Murthy wrote: On 10/29/2014 12:29 AM, Daniel Lezcano wrote: On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano wrote: When the pmqos latency requirement is set to zero that means "poll in all the cases".

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-05 Thread Daniel Lezcano
On 10/29/2014 03:01 AM, Preeti U Murthy wrote: On 10/29/2014 12:29 AM, Daniel Lezcano wrote: On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: When the pmqos latency requirement is set to zero that means

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-05 Thread Rafael J. Wysocki
On Thursday, October 23, 2014 11:01:17 AM Daniel Lezcano wrote: When the pmqos latency requirement is set to zero that means poll in all the cases. That is correctly implemented on x86 but not on the other archs. As how is written the code, if the latency request is zero, the governor will

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-05 Thread Daniel Lezcano
On 11/05/2014 10:57 PM, Rafael J. Wysocki wrote: On Thursday, October 23, 2014 11:01:17 AM Daniel Lezcano wrote: When the pmqos latency requirement is set to zero that means poll in all the cases. That is correctly implemented on x86 but not on the other archs. As how is written the code, if

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-11-05 Thread Preeti U Murthy
On 11/05/2014 07:58 PM, Daniel Lezcano wrote: On 10/29/2014 03:01 AM, Preeti U Murthy wrote: On 10/29/2014 12:29 AM, Daniel Lezcano wrote: On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: When the pmqos

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-10-28 Thread Preeti U Murthy
On 10/29/2014 12:29 AM, Daniel Lezcano wrote: > On 10/28/2014 04:51 AM, Preeti Murthy wrote: >> Hi Daniel, >> >> On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano >> wrote: >>> When the pmqos latency requirement is set to zero that means "poll in >>> all the >>> cases". >>> >>> That is correctly

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-10-28 Thread Daniel Lezcano
On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano wrote: When the pmqos latency requirement is set to zero that means "poll in all the cases". That is correctly implemented on x86 but not on the other archs. As how is written the code,

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-10-28 Thread Preeti Murthy
Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano wrote: > When the pmqos latency requirement is set to zero that means "poll in all the > cases". > > That is correctly implemented on x86 but not on the other archs. > > As how is written the code, if the latency request is zero, the

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-10-28 Thread Preeti Murthy
Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: When the pmqos latency requirement is set to zero that means poll in all the cases. That is correctly implemented on x86 but not on the other archs. As how is written the code, if the latency request

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-10-28 Thread Daniel Lezcano
On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: When the pmqos latency requirement is set to zero that means poll in all the cases. That is correctly implemented on x86 but not on the other archs. As how

Re: [PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-10-28 Thread Preeti U Murthy
On 10/29/2014 12:29 AM, Daniel Lezcano wrote: On 10/28/2014 04:51 AM, Preeti Murthy wrote: Hi Daniel, On Thu, Oct 23, 2014 at 2:31 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: When the pmqos latency requirement is set to zero that means poll in all the cases. That is correctly

[PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-10-23 Thread Daniel Lezcano
When the pmqos latency requirement is set to zero that means "poll in all the cases". That is correctly implemented on x86 but not on the other archs. As how is written the code, if the latency request is zero, the governor will return zero, so corresponding, for x86, to the poll function, but

[PATCH V2 1/5] sched: idle: cpuidle: Check the latency req before idle

2014-10-23 Thread Daniel Lezcano
When the pmqos latency requirement is set to zero that means poll in all the cases. That is correctly implemented on x86 but not on the other archs. As how is written the code, if the latency request is zero, the governor will return zero, so corresponding, for x86, to the poll function, but for