Merge branch 'reciprocal'

Hannes Frederic Sowa says:

====================
reciprocal_divide update

This patch is on top of aee636c4809fa5 ("bpf: do not use reciprocal
divide") from Eric that sits in net tree. It will not create a merge
conflict, but it depends on this one, so we suggest, if possible, to
merge net into net-next.

We are proposing this change with only small modifications from the
v2 version, namely updating the name of trim to reciprocal_scale
(as commented on by Ben Hutchings and Eric Dumazet, thanks!).

We thought about introducing the reciprocal_divide algorithm in
parallel to the one already used by the kernel but faced organizational
issues, leading us to the conclusion that it is best to just replace
the old one: We could not come up with names for the different
implementations and also with a way to describe the differences to
guide developers which one to choose in which situation. This is
because we cannot specify the correct semantics for the version
which is currently used by the kernel. Altough it seems to not be
causing problems in the kernel, we cannot surely say so in the
case of flex_array for the future. Current usage seems ok, but
future users could run into problems.

Changelog:

v1->v2:
 - changed name to prandom_u32_max in p1
 - changed name to trim in p2
 - reworked code in p3
v2->v3:
 - p1 and p3 stays unchanged, only small update in commit
   message in p3
 - changed name to reciprocal_scale in p2
 - fixed kernel doc format
v3->v4:
 - pseduo -> pseudo (thanks to Tilman Schmidt)
v4->v5:
 - fix pseduo -> pseudo for real now, sorry for the noise
====================

Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 17 insertions, 7 deletions

diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c
index 3220b488dd1e..f100bd958b88 100644
--- a/drivers/net/bonding/bond_main.c
+++ b/drivers/net/bonding/bond_main.c
@@ -79,7 +79,6 @@
 #include <net/pkt_sched.h>
 #include <linux/rculist.h>
 #include <net/flow_keys.h>
-#include <linux/reciprocal_div.h>
 #include "bonding.h"
 #include "bond_3ad.h"
 #include "bond_alb.h"
@@ -3596,8 +3595,9 @@ static void bond_xmit_slave_id(struct bonding *bond, struct sk_buff *skb, int sl
  */
 static u32 bond_rr_gen_slave_id(struct bonding *bond)
 {
-        int packets_per_slave = bond->params.packets_per_slave;
         u32 slave_id;
+        struct reciprocal_value reciprocal_packets_per_slave;
+        int packets_per_slave = bond->params.packets_per_slave;
 
         switch (packets_per_slave) {
         case 0:
@@ -3607,8 +3607,10 @@ static u32 bond_rr_gen_slave_id(struct bonding *bond)
                 slave_id = bond->rr_tx_counter;
                 break;
         default:
+                reciprocal_packets_per_slave =
+                        bond->params.reciprocal_packets_per_slave;
                 slave_id = reciprocal_divide(bond->rr_tx_counter,
-                                             packets_per_slave);
+                                             reciprocal_packets_per_slave);
                 break;
         }
         bond->rr_tx_counter++;
@@ -4343,10 +4345,18 @@ static int bond_check_params(struct bond_params *params)
         params->resend_igmp = resend_igmp;
         params->min_links = min_links;
         params->lp_interval = lp_interval;
-        if (packets_per_slave > 1)
-                params->packets_per_slave = reciprocal_value(packets_per_slave);
-        else
-                params->packets_per_slave = packets_per_slave;
+        params->packets_per_slave = packets_per_slave;
+        if (packets_per_slave > 0) {
+                params->reciprocal_packets_per_slave =
+                        reciprocal_value(packets_per_slave);
+        } else {
+                /* reciprocal_packets_per_slave is unused if
+                 * packets_per_slave is 0 or 1, just initialize it
+                 */
+                params->reciprocal_packets_per_slave =
+                        (struct reciprocal_value) { 0 };
+        }
+
         if (primary) {
                 strncpy(params->primary, primary, IFNAMSIZ);
                 params->primary[IFNAMSIZ - 1] = 0;