Re: [Update][PATCH v3 3/4] cpufreq: schedutil: Simplify iowait boosting

2019-04-07 Thread Viresh Kumar
On 28-03-19, 11:33, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > There is not reason for the minimum iowait boost value in the > schedutil cpufreq governor to depend on the available range of CPU > frequencies. In fact, that dependency is generally confusing, > because it causes the

Re: [Update][PATCH v3 3/4] cpufreq: schedutil: Simplify iowait boosting

2019-04-01 Thread Peter Zijlstra
On Mon, Apr 01, 2019 at 11:27:25AM +0200, Rafael J. Wysocki wrote: > On Thursday, March 28, 2019 11:33:21 AM CEST Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > There is not reason for the minimum iowait boost value in the > > schedutil cpufreq governor to depend on the available

Re: [Update][PATCH v3 3/4] cpufreq: schedutil: Simplify iowait boosting

2019-04-01 Thread Rafael J. Wysocki
On Thursday, March 28, 2019 11:33:21 AM CEST Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > There is not reason for the minimum iowait boost value in the > schedutil cpufreq governor to depend on the available range of CPU > frequencies. In fact, that dependency is generally confusing,

[Update][PATCH v3 3/4] cpufreq: schedutil: Simplify iowait boosting

2019-03-28 Thread Rafael J. Wysocki
From: Rafael J. Wysocki There is not reason for the minimum iowait boost value in the schedutil cpufreq governor to depend on the available range of CPU frequencies. In fact, that dependency is generally confusing, because it causes the iowait boost to behave somewhat differently on CPUs with

Re: [PATCH v3 3/4] cpufreq: schedutil: Simplify iowait boosting

2019-03-28 Thread Rafael J. Wysocki
On Thu, Mar 28, 2019 at 10:48 AM Quentin Perret wrote: > > Hi Rafael, > > On Tuesday 26 Mar 2019 at 12:18:00 (+0100), Rafael J. Wysocki wrote: > > @@ -13,6 +13,8 @@ > > #include > > #include > > > > +#define IOWAIT_BOOST_MIN (SCHED_CAPACITY_SCALE / 8) > > + > > struct sugov_tunables { >

Re: [PATCH v3 3/4] cpufreq: schedutil: Simplify iowait boosting

2019-03-28 Thread Quentin Perret
Hi Rafael, On Tuesday 26 Mar 2019 at 12:18:00 (+0100), Rafael J. Wysocki wrote: > @@ -13,6 +13,8 @@ > #include > #include > > +#define IOWAIT_BOOST_MIN (SCHED_CAPACITY_SCALE / 8) > + > struct sugov_tunables { > struct gov_attr_set attr_set; > unsigned int

[PATCH v3 3/4] cpufreq: schedutil: Simplify iowait boosting

2019-03-26 Thread Rafael J. Wysocki
From: Rafael J. Wysocki There is not reason for the minimum iowait boost value in the schedutil cpufreq governor to depend on the available range of CPU frequencies. In fact, that dependency is generally confusing, because it causes the iowait boost to behave somewhat differently on CPUs with