batman-adv: Drop per algo GW section class code

This code was  only used in the past for the sysfs interface. But since
this was replace with netlink, it was never executed. The function pointer
was only checked to figure out whether the limit 255 (B.A.T.M.A.N. IV) or
2**32-1 (B.A.T.M.A.N. V) should be used as limit.

So instead of keeping the function pointer, just store the limits directly
in struct batadv_algo_gw_ops.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>

diff --git a/net/batman-adv/bat_iv_ogm.c b/net/batman-adv/bat_iv_ogm.c
index 828fb393ee94a31e8715b0b157b4c39b6337b19d..74b49c35ddc14db4efdb1ad4f4a265ff52a2b034 100644 (file)
--- a/net/batman-adv/bat_iv_ogm.c
+++ b/net/batman-adv/bat_iv_ogm.c
@@ -2516,6 +2516,7 @@ static struct batadv_algo_ops batadv_batman_iv __read_mostly = {
        },
        .gw = {
                .init_sel_class = batadv_iv_init_sel_class,
+               .sel_class_max = BATADV_TQ_MAX_VALUE,
                .get_best_gw_node = batadv_iv_gw_get_best_gw_node,
                .is_eligible = batadv_iv_gw_is_eligible,
                .dump = batadv_iv_gw_dump,

diff --git a/net/batman-adv/bat_v.c b/net/batman-adv/bat_v.c
index 54e41fc709c378aa8e0ef7fefea6047b1b04f2ec..ac11f1f08db0f93bdaee81a2ce8822a6eda8db04 100644 (file)
--- a/net/batman-adv/bat_v.c
+++ b/net/batman-adv/bat_v.c
@@ -14,6 +14,7 @@
 #include <linux/init.h>
 #include <linux/jiffies.h>
 #include <linux/kref.h>
+#include <linux/limits.h>
 #include <linux/list.h>
 #include <linux/minmax.h>
 #include <linux/netdevice.h>
@@ -34,7 +35,6 @@
 #include "bat_v_elp.h"
 #include "bat_v_ogm.h"
 #include "gateway_client.h"
-#include "gateway_common.h"
 #include "hard-interface.h"
 #include "hash.h"
 #include "log.h"
@@ -512,25 +512,6 @@ static void batadv_v_init_sel_class(struct batadv_priv *bat_priv)
        atomic_set(&bat_priv->gw.sel_class, 50);
 }
 
-static ssize_t batadv_v_store_sel_class(struct batadv_priv *bat_priv,
-                                       char *buff, size_t count)
-{
-       u32 old_class, class;
-
-       if (!batadv_parse_throughput(bat_priv->soft_iface, buff,
-                                    "B.A.T.M.A.N. V GW selection class",
-                                    &class))
-               return -EINVAL;
-
-       old_class = atomic_read(&bat_priv->gw.sel_class);
-       atomic_set(&bat_priv->gw.sel_class, class);
-
-       if (old_class != class)
-               batadv_gw_reselect(bat_priv);
-
-       return count;
-}
-
 /**
  * batadv_v_gw_throughput_get() - retrieve the GW-bandwidth for a given GW
  * @gw_node: the GW to retrieve the metric for
@@ -818,7 +799,7 @@ static struct batadv_algo_ops batadv_batman_v __read_mostly = {
        },
        .gw = {
                .init_sel_class = batadv_v_init_sel_class,
-               .store_sel_class = batadv_v_store_sel_class,
+               .sel_class_max = U32_MAX,
                .get_best_gw_node = batadv_v_gw_get_best_gw_node,
                .is_eligible = batadv_v_gw_is_eligible,
                .dump = batadv_v_gw_dump,

diff --git a/net/batman-adv/gateway_common.c b/net/batman-adv/gateway_common.c
index d9632607f92b6221b19df2fb7b5208366c801606..2dd36ef03c841981835ab619da6b143baabc8c9e 100644 (file)
--- a/net/batman-adv/gateway_common.c
+++ b/net/batman-adv/gateway_common.c
@@ -9,86 +9,14 @@
 
 #include <linux/atomic.h>
 #include <linux/byteorder/generic.h>
-#include <linux/kstrtox.h>
-#include <linux/limits.h>
-#include <linux/math64.h>
-#include <linux/netdevice.h>
 #include <linux/stddef.h>
-#include <linux/string.h>
+#include <linux/types.h>
 #include <uapi/linux/batadv_packet.h>
 #include <uapi/linux/batman_adv.h>
 
 #include "gateway_client.h"
-#include "log.h"
 #include "tvlv.h"
 
-/**
- * batadv_parse_throughput() - parse supplied string buffer to extract
- *  throughput information
- * @net_dev: the soft interface net device
- * @buff: string buffer to parse
- * @description: text shown when throughput string cannot be parsed
- * @throughput: pointer holding the returned throughput information
- *
- * Return: false on parse error and true otherwise.
- */
-bool batadv_parse_throughput(struct net_device *net_dev, char *buff,
-                            const char *description, u32 *throughput)
-{
-       enum batadv_bandwidth_units bw_unit_type = BATADV_BW_UNIT_KBIT;
-       u64 lthroughput;
-       char *tmp_ptr;
-       int ret;
-
-       if (strlen(buff) > 4) {
-               tmp_ptr = buff + strlen(buff) - 4;
-
-               if (strncasecmp(tmp_ptr, "mbit", 4) == 0)
-                       bw_unit_type = BATADV_BW_UNIT_MBIT;
-
-               if (strncasecmp(tmp_ptr, "kbit", 4) == 0 ||
-                   bw_unit_type == BATADV_BW_UNIT_MBIT)
-                       *tmp_ptr = '\0';
-       }
-
-       ret = kstrtou64(buff, 10, &lthroughput);
-       if (ret) {
-               batadv_err(net_dev,
-                          "Invalid throughput speed for %s: %s\n",
-                          description, buff);
-               return false;
-       }
-
-       switch (bw_unit_type) {
-       case BATADV_BW_UNIT_MBIT:
-               /* prevent overflow */
-               if (U64_MAX / 10 < lthroughput) {
-                       batadv_err(net_dev,
-                                  "Throughput speed for %s too large: %s\n",
-                                  description, buff);
-                       return false;
-               }
-
-               lthroughput *= 10;
-               break;
-       case BATADV_BW_UNIT_KBIT:
-       default:
-               lthroughput = div_u64(lthroughput, 100);
-               break;
-       }
-
-       if (lthroughput > U32_MAX) {
-               batadv_err(net_dev,
-                          "Throughput speed for %s too large: %s\n",
-                          description, buff);
-               return false;
-       }
-
-       *throughput = lthroughput;
-
-       return true;
-}
-
 /**
  * batadv_gw_tvlv_container_update() - update the gw tvlv container after
  *  gateway setting change

diff --git a/net/batman-adv/gateway_common.h b/net/batman-adv/gateway_common.h
index cb2e72d7ab141efb59ba7770d708b76beb623de0..5d097d6a1dd951dc422ac654c1401731aa5cd09e 100644 (file)
--- a/net/batman-adv/gateway_common.h
+++ b/net/batman-adv/gateway_common.h
@@ -9,9 +9,6 @@
 
 #include "main.h"
 
-#include <linux/netdevice.h>
-#include <linux/types.h>
-
 /**
  * enum batadv_bandwidth_units - bandwidth unit types
  */
@@ -30,7 +27,5 @@ enum batadv_bandwidth_units {
 void batadv_gw_tvlv_container_update(struct batadv_priv *bat_priv);
 void batadv_gw_init(struct batadv_priv *bat_priv);
 void batadv_gw_free(struct batadv_priv *bat_priv);
-bool batadv_parse_throughput(struct net_device *net_dev, char *buff,
-                            const char *description, u32 *throughput);
 
 #endif /* _NET_BATMAN_ADV_GATEWAY_COMMON_H_ */

diff --git a/net/batman-adv/netlink.c b/net/batman-adv/netlink.c
index b6c512ce6704325bf2cbe60fa574d2e7f45b7e5a..d37872b34281d7654b27393988008dc529113a9a 100644 (file)
--- a/net/batman-adv/netlink.c
+++ b/net/batman-adv/netlink.c
@@ -548,15 +548,12 @@ static int batadv_netlink_set_mesh(struct sk_buff *skb, struct genl_info *info)
                 * algorithm in use implements the GW API
                 */
 
-               u32 sel_class_max = 0xffffffffu;
+               u32 sel_class_max = bat_priv->algo_ops->gw.sel_class_max;
                u32 sel_class;
 
                attr = info->attrs[BATADV_ATTR_GW_SEL_CLASS];
                sel_class = nla_get_u32(attr);
 
-               if (!bat_priv->algo_ops->gw.store_sel_class)
-                       sel_class_max = BATADV_TQ_MAX_VALUE;
-
                if (sel_class >= 1 && sel_class <= sel_class_max) {
                        atomic_set(&bat_priv->gw.sel_class, sel_class);
                        batadv_gw_reselect(bat_priv);

diff --git a/net/batman-adv/types.h b/net/batman-adv/types.h
index ca9449ec9836a17bad1455221af0016091e87bde..54c2b8fa48ccdf485dc83117a9db675c5e6275e8 100644 (file)
--- a/net/batman-adv/types.h
+++ b/net/batman-adv/types.h
@@ -2191,11 +2191,10 @@ struct batadv_algo_gw_ops {
        void (*init_sel_class)(struct batadv_priv *bat_priv);
 
        /**
-        * @store_sel_class: parse and stores a new GW selection class
-        *  (optional)
+        * @sel_class_max: maximum allowed GW selection class
         */
-       ssize_t (*store_sel_class)(struct batadv_priv *bat_priv, char *buff,
-                                  size_t count);
+       u32 sel_class_max;
+
        /**
         * @get_best_gw_node: select the best GW from the list of available
         *  nodes (optional)