Re: [PATCH 5/7] sched: loadavg: make calc_load_n() public

2018-05-10 Thread Johannes Weiner
On Wed, May 09, 2018 at 11:49:06AM +0200, Peter Zijlstra wrote:
> On Mon, May 07, 2018 at 05:01:33PM -0400, Johannes Weiner wrote:
> > +static inline unsigned long
> > +fixed_power_int(unsigned long x, unsigned int frac_bits, unsigned int n)
> > +{
> > +   unsigned long result = 1UL << frac_bits;
> > +
> > +   if (n) {
> > +   for (;;) {
> > +   if (n & 1) {
> > +   result *= x;
> > +   result += 1UL << (frac_bits - 1);
> > +   result >>= frac_bits;
> > +   }
> > +   n >>= 1;
> > +   if (!n)
> > +   break;
> > +   x *= x;
> > +   x += 1UL << (frac_bits - 1);
> > +   x >>= frac_bits;
> > +   }
> > +   }
> > +
> > +   return result;
> > +}
> 
> No real objection; but that does look a wee bit fat for an inline I
> suppose.

Fair enough, I'll put these back where I found them and make
calc_load_n() extern instead.


Re: [PATCH 5/7] sched: loadavg: make calc_load_n() public

2018-05-09 Thread Peter Zijlstra
On Mon, May 07, 2018 at 05:01:33PM -0400, Johannes Weiner wrote:
> +static inline unsigned long
> +fixed_power_int(unsigned long x, unsigned int frac_bits, unsigned int n)
> +{
> + unsigned long result = 1UL << frac_bits;
> +
> + if (n) {
> + for (;;) {
> + if (n & 1) {
> + result *= x;
> + result += 1UL << (frac_bits - 1);
> + result >>= frac_bits;
> + }
> + n >>= 1;
> + if (!n)
> + break;
> + x *= x;
> + x += 1UL << (frac_bits - 1);
> + x >>= frac_bits;
> + }
> + }
> +
> + return result;
> +}

No real objection; but that does look a wee bit fat for an inline I
suppose.


[PATCH 5/7] sched: loadavg: make calc_load_n() public

2018-05-07 Thread Johannes Weiner
It's going to be used in the following patch. Keep the churn separate.

Signed-off-by: Johannes Weiner 
---
 include/linux/sched/loadavg.h | 69 +++
 kernel/sched/loadavg.c| 69 ---
 2 files changed, 69 insertions(+), 69 deletions(-)

diff --git a/include/linux/sched/loadavg.h b/include/linux/sched/loadavg.h
index cc9cc62bb1f8..0e4c24978751 100644
--- a/include/linux/sched/loadavg.h
+++ b/include/linux/sched/loadavg.h
@@ -37,6 +37,75 @@ calc_load(unsigned long load, unsigned long exp, unsigned 
long active)
return newload / FIXED_1;
 }
 
+/**
+ * fixed_power_int - compute: x^n, in O(log n) time
+ *
+ * @x: base of the power
+ * @frac_bits: fractional bits of @x
+ * @n: power to raise @x to.
+ *
+ * By exploiting the relation between the definition of the natural power
+ * function: x^n := x*x*...*x (x multiplied by itself for n times), and
+ * the binary encoding of numbers used by computers: n := \Sum n_i * 2^i,
+ * (where: n_i \elem {0, 1}, the binary vector representing n),
+ * we find: x^n := x^(\Sum n_i * 2^i) := \Prod x^(n_i * 2^i), which is
+ * of course trivially computable in O(log_2 n), the length of our binary
+ * vector.
+ */
+static inline unsigned long
+fixed_power_int(unsigned long x, unsigned int frac_bits, unsigned int n)
+{
+   unsigned long result = 1UL << frac_bits;
+
+   if (n) {
+   for (;;) {
+   if (n & 1) {
+   result *= x;
+   result += 1UL << (frac_bits - 1);
+   result >>= frac_bits;
+   }
+   n >>= 1;
+   if (!n)
+   break;
+   x *= x;
+   x += 1UL << (frac_bits - 1);
+   x >>= frac_bits;
+   }
+   }
+
+   return result;
+}
+
+/*
+ * a1 = a0 * e + a * (1 - e)
+ *
+ * a2 = a1 * e + a * (1 - e)
+ *= (a0 * e + a * (1 - e)) * e + a * (1 - e)
+ *= a0 * e^2 + a * (1 - e) * (1 + e)
+ *
+ * a3 = a2 * e + a * (1 - e)
+ *= (a0 * e^2 + a * (1 - e) * (1 + e)) * e + a * (1 - e)
+ *= a0 * e^3 + a * (1 - e) * (1 + e + e^2)
+ *
+ *  ...
+ *
+ * an = a0 * e^n + a * (1 - e) * (1 + e + ... + e^n-1) [1]
+ *= a0 * e^n + a * (1 - e) * (1 - e^n)/(1 - e)
+ *= a0 * e^n + a * (1 - e^n)
+ *
+ * [1] application of the geometric series:
+ *
+ *  n 1 - x^(n+1)
+ * S_n := \Sum x^i = -
+ * i=0  1 - x
+ */
+static inline unsigned long
+calc_load_n(unsigned long load, unsigned long exp,
+   unsigned long active, unsigned int n)
+{
+   return calc_load(load, fixed_power_int(exp, FSHIFT, n), active);
+}
+
 #define LOAD_INT(x) ((x) >> FSHIFT)
 #define LOAD_FRAC(x) LOAD_INT(((x) & (FIXED_1-1)) * 100)
 
diff --git a/kernel/sched/loadavg.c b/kernel/sched/loadavg.c
index 54fbdfb2d86c..0736e349a54e 100644
--- a/kernel/sched/loadavg.c
+++ b/kernel/sched/loadavg.c
@@ -210,75 +210,6 @@ static long calc_load_nohz_fold(void)
return delta;
 }
 
-/**
- * fixed_power_int - compute: x^n, in O(log n) time
- *
- * @x: base of the power
- * @frac_bits: fractional bits of @x
- * @n: power to raise @x to.
- *
- * By exploiting the relation between the definition of the natural power
- * function: x^n := x*x*...*x (x multiplied by itself for n times), and
- * the binary encoding of numbers used by computers: n := \Sum n_i * 2^i,
- * (where: n_i \elem {0, 1}, the binary vector representing n),
- * we find: x^n := x^(\Sum n_i * 2^i) := \Prod x^(n_i * 2^i), which is
- * of course trivially computable in O(log_2 n), the length of our binary
- * vector.
- */
-static unsigned long
-fixed_power_int(unsigned long x, unsigned int frac_bits, unsigned int n)
-{
-   unsigned long result = 1UL << frac_bits;
-
-   if (n) {
-   for (;;) {
-   if (n & 1) {
-   result *= x;
-   result += 1UL << (frac_bits - 1);
-   result >>= frac_bits;
-   }
-   n >>= 1;
-   if (!n)
-   break;
-   x *= x;
-   x += 1UL << (frac_bits - 1);
-   x >>= frac_bits;
-   }
-   }
-
-   return result;
-}
-
-/*
- * a1 = a0 * e + a * (1 - e)
- *
- * a2 = a1 * e + a * (1 - e)
- *= (a0 * e + a * (1 - e)) * e + a * (1 - e)
- *= a0 * e^2 + a * (1 - e) * (1 + e)
- *
- * a3 = a2 * e + a * (1 - e)
- *= (a0 * e^2 + a * (1 - e) * (1 + e)) * e + a * (1 - e)
- *= a0 * e^3 + a * (1 - e) * (1 + e + e^2)
- *
- *  ...
- *
- * an = a0 * e^n + a * (1 - e) * (1 + e + ... + e^n-1) [1]
- *= a0 * e^n + a * (1 - e) * (1 - e^n)/(1 - e)
- *= a0 * e^n + a * (1 - e^n)
- *
- * [1] application of the geo