net: openvswitch: Constify static struct genl_small_ops
authorRikard Falkeborn <rikard.falkeborn@gmail.com>
Sun, 4 Oct 2020 23:44:17 +0000 (01:44 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 5 Oct 2020 04:13:36 +0000 (21:13 -0700)
The only usage of these is to assign their address to the small_ops field
in the genl_family struct, which is a const pointer, and applying
ARRAY_SIZE() on them. Make them const to allow the compiler to put them
in read-only memory.

Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/openvswitch/conntrack.c
net/openvswitch/meter.c

index 18af10b..e6fe26a 100644 (file)
@@ -2231,7 +2231,7 @@ exit_err:
        return err;
 }
 
-static struct genl_small_ops ct_limit_genl_ops[] = {
+static const struct genl_small_ops ct_limit_genl_ops[] = {
        { .cmd = OVS_CT_LIMIT_CMD_SET,
                .validate = GENL_DONT_VALIDATE_STRICT | GENL_DONT_VALIDATE_DUMP,
                .flags = GENL_ADMIN_PERM, /* Requires CAP_NET_ADMIN
index 50541e8..8fbefd5 100644 (file)
@@ -672,7 +672,7 @@ bool ovs_meter_execute(struct datapath *dp, struct sk_buff *skb,
        return false;
 }
 
-static struct genl_small_ops dp_meter_genl_ops[] = {
+static const struct genl_small_ops dp_meter_genl_ops[] = {
        { .cmd = OVS_METER_CMD_FEATURES,
                .validate = GENL_DONT_VALIDATE_STRICT | GENL_DONT_VALIDATE_DUMP,
                .flags = 0,               /* OK for unprivileged users. */