xfrm: place af number into xfrm_mode struct

This will be useful to know if we're supposed to decode ipv4 or ipv6.

While at it, make the unregister function return void, all module_exit
functions did just BUG(); there is never a point in doing error checks
if there is no way to handle such error.

Signed-off-by: Florian Westphal <fw@strlen.de>
Reviewed-by: Sabrina Dubroca <sd@queasysnail.net>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>

diff --git a/include/net/xfrm.h b/include/net/xfrm.h
index 85386becbaea211504eaeae6a549e96d204afc75..9a155063c25fcd97d8b28e7f60d52f98409d7479 100644 (file)
--- a/include/net/xfrm.h
+++ b/include/net/xfrm.h
@@ -482,8 +482,9 @@ struct xfrm_mode {
 
        struct xfrm_state_afinfo *afinfo;
        struct module *owner;
-       unsigned int encap;
-       int flags;
+       u8 encap;
+       u8 family;
+       u8 flags;
 };
 
 /* Flags for xfrm_mode. */
@@ -491,8 +492,8 @@ enum {
        XFRM_MODE_FLAG_TUNNEL = 1,
 };
 
-int xfrm_register_mode(struct xfrm_mode *mode, int family);
-int xfrm_unregister_mode(struct xfrm_mode *mode, int family);
+int xfrm_register_mode(struct xfrm_mode *mode);
+void xfrm_unregister_mode(struct xfrm_mode *mode);
 
 static inline int xfrm_af2proto(unsigned int family)
 {

diff --git a/net/ipv4/xfrm4_mode_beet.c b/net/ipv4/xfrm4_mode_beet.c
index 856d2dfdb44b6be0d9d9be732d1e26f7db69792c..a2e3b52ae46c87bab92acb9da12996711ba7199a 100644 (file)
--- a/net/ipv4/xfrm4_mode_beet.c
+++ b/net/ipv4/xfrm4_mode_beet.c
@@ -134,19 +134,17 @@ static struct xfrm_mode xfrm4_beet_mode = {
        .owner = THIS_MODULE,
        .encap = XFRM_MODE_BEET,
        .flags = XFRM_MODE_FLAG_TUNNEL,
+       .family = AF_INET,
 };
 
 static int __init xfrm4_beet_init(void)
 {
-       return xfrm_register_mode(&xfrm4_beet_mode, AF_INET);
+       return xfrm_register_mode(&xfrm4_beet_mode);
 }
 
 static void __exit xfrm4_beet_exit(void)
 {
-       int err;
-
-       err = xfrm_unregister_mode(&xfrm4_beet_mode, AF_INET);
-       BUG_ON(err);
+       xfrm_unregister_mode(&xfrm4_beet_mode);
 }
 
 module_init(xfrm4_beet_init);

diff --git a/net/ipv4/xfrm4_mode_transport.c b/net/ipv4/xfrm4_mode_transport.c
index 1ad2c2c4e250f84b1ad73020c727ea8b68b3e0d3..7c5443f797cfcb9d580e194ea35dfa523bf66edf 100644 (file)
--- a/net/ipv4/xfrm4_mode_transport.c
+++ b/net/ipv4/xfrm4_mode_transport.c
@@ -93,19 +93,17 @@ static struct xfrm_mode xfrm4_transport_mode = {
        .xmit = xfrm4_transport_xmit,
        .owner = THIS_MODULE,
        .encap = XFRM_MODE_TRANSPORT,
+       .family = AF_INET,
 };
 
 static int __init xfrm4_transport_init(void)
 {
-       return xfrm_register_mode(&xfrm4_transport_mode, AF_INET);
+       return xfrm_register_mode(&xfrm4_transport_mode);
 }
 
 static void __exit xfrm4_transport_exit(void)
 {
-       int err;
-
-       err = xfrm_unregister_mode(&xfrm4_transport_mode, AF_INET);
-       BUG_ON(err);
+       xfrm_unregister_mode(&xfrm4_transport_mode);
 }
 
 module_init(xfrm4_transport_init);

diff --git a/net/ipv4/xfrm4_mode_tunnel.c b/net/ipv4/xfrm4_mode_tunnel.c
index 2a9764bd17196966b41755269e03f487cb757288..cfc6b6d397554b0e60d781661de594b213449546 100644 (file)
--- a/net/ipv4/xfrm4_mode_tunnel.c
+++ b/net/ipv4/xfrm4_mode_tunnel.c
@@ -131,19 +131,17 @@ static struct xfrm_mode xfrm4_tunnel_mode = {
        .owner = THIS_MODULE,
        .encap = XFRM_MODE_TUNNEL,
        .flags = XFRM_MODE_FLAG_TUNNEL,
+       .family = AF_INET,
 };
 
 static int __init xfrm4_mode_tunnel_init(void)
 {
-       return xfrm_register_mode(&xfrm4_tunnel_mode, AF_INET);
+       return xfrm_register_mode(&xfrm4_tunnel_mode);
 }
 
 static void __exit xfrm4_mode_tunnel_exit(void)
 {
-       int err;
-
-       err = xfrm_unregister_mode(&xfrm4_tunnel_mode, AF_INET);
-       BUG_ON(err);
+       xfrm_unregister_mode(&xfrm4_tunnel_mode);
 }
 
 module_init(xfrm4_mode_tunnel_init);

diff --git a/net/ipv6/xfrm6_mode_beet.c b/net/ipv6/xfrm6_mode_beet.c
index 57fd314ec2b89f69a871c46e7e3f2ad2d8a520c6..0d440e3a13f8d3f31f41095212ca1479d23e5f56 100644 (file)
--- a/net/ipv6/xfrm6_mode_beet.c
+++ b/net/ipv6/xfrm6_mode_beet.c
@@ -110,19 +110,17 @@ static struct xfrm_mode xfrm6_beet_mode = {
        .owner = THIS_MODULE,
        .encap = XFRM_MODE_BEET,
        .flags = XFRM_MODE_FLAG_TUNNEL,
+       .family = AF_INET6,
 };
 
 static int __init xfrm6_beet_init(void)
 {
-       return xfrm_register_mode(&xfrm6_beet_mode, AF_INET6);
+       return xfrm_register_mode(&xfrm6_beet_mode);
 }
 
 static void __exit xfrm6_beet_exit(void)
 {
-       int err;
-
-       err = xfrm_unregister_mode(&xfrm6_beet_mode, AF_INET6);
-       BUG_ON(err);
+       xfrm_unregister_mode(&xfrm6_beet_mode);
 }
 
 module_init(xfrm6_beet_init);

diff --git a/net/ipv6/xfrm6_mode_ro.c b/net/ipv6/xfrm6_mode_ro.c
index da28e4407b8faa91eeb7517e87e9b644ae3f841a..0408547d01ab951af0b025e960416410211b5234 100644 (file)
--- a/net/ipv6/xfrm6_mode_ro.c
+++ b/net/ipv6/xfrm6_mode_ro.c
@@ -64,19 +64,17 @@ static struct xfrm_mode xfrm6_ro_mode = {
        .output = xfrm6_ro_output,
        .owner = THIS_MODULE,
        .encap = XFRM_MODE_ROUTEOPTIMIZATION,
+       .family = AF_INET6,
 };
 
 static int __init xfrm6_ro_init(void)
 {
-       return xfrm_register_mode(&xfrm6_ro_mode, AF_INET6);
+       return xfrm_register_mode(&xfrm6_ro_mode);
 }
 
 static void __exit xfrm6_ro_exit(void)
 {
-       int err;
-
-       err = xfrm_unregister_mode(&xfrm6_ro_mode, AF_INET6);
-       BUG_ON(err);
+       xfrm_unregister_mode(&xfrm6_ro_mode);
 }
 
 module_init(xfrm6_ro_init);

diff --git a/net/ipv6/xfrm6_mode_transport.c b/net/ipv6/xfrm6_mode_transport.c
index 3c29da5defe6c357ff04ca4adead1a9fee208f08..66ae79218bdf57f49ee7dc93048f75f5a042a881 100644 (file)
--- a/net/ipv6/xfrm6_mode_transport.c
+++ b/net/ipv6/xfrm6_mode_transport.c
@@ -100,19 +100,17 @@ static struct xfrm_mode xfrm6_transport_mode = {
        .xmit = xfrm6_transport_xmit,
        .owner = THIS_MODULE,
        .encap = XFRM_MODE_TRANSPORT,
+       .family = AF_INET6,
 };
 
 static int __init xfrm6_transport_init(void)
 {
-       return xfrm_register_mode(&xfrm6_transport_mode, AF_INET6);
+       return xfrm_register_mode(&xfrm6_transport_mode);
 }
 
 static void __exit xfrm6_transport_exit(void)
 {
-       int err;
-
-       err = xfrm_unregister_mode(&xfrm6_transport_mode, AF_INET6);
-       BUG_ON(err);
+       xfrm_unregister_mode(&xfrm6_transport_mode);
 }
 
 module_init(xfrm6_transport_init);

diff --git a/net/ipv6/xfrm6_mode_tunnel.c b/net/ipv6/xfrm6_mode_tunnel.c
index de1b0b8c53b0ba26836d40a53dfa6fe077c7ebef..6cf12e961ea55719a0a9c819b9bcb7d436c80791 100644 (file)
--- a/net/ipv6/xfrm6_mode_tunnel.c
+++ b/net/ipv6/xfrm6_mode_tunnel.c
@@ -130,19 +130,17 @@ static struct xfrm_mode xfrm6_tunnel_mode = {
        .owner = THIS_MODULE,
        .encap = XFRM_MODE_TUNNEL,
        .flags = XFRM_MODE_FLAG_TUNNEL,
+       .family = AF_INET6,
 };
 
 static int __init xfrm6_mode_tunnel_init(void)
 {
-       return xfrm_register_mode(&xfrm6_tunnel_mode, AF_INET6);
+       return xfrm_register_mode(&xfrm6_tunnel_mode);
 }
 
 static void __exit xfrm6_mode_tunnel_exit(void)
 {
-       int err;
-
-       err = xfrm_unregister_mode(&xfrm6_tunnel_mode, AF_INET6);
-       BUG_ON(err);
+       xfrm_unregister_mode(&xfrm6_tunnel_mode);
 }
 
 module_init(xfrm6_mode_tunnel_init);

diff --git a/net/xfrm/xfrm_state.c b/net/xfrm/xfrm_state.c
index 1bb971f46fc6f9096f59c99740d6e832276c3b34..c32394b59776b12e1e9b946f8d80e32b4dbbd314 100644 (file)
--- a/net/xfrm/xfrm_state.c
+++ b/net/xfrm/xfrm_state.c
@@ -331,7 +331,7 @@ static void xfrm_put_type_offload(const struct xfrm_type_offload *type)
 }
 
 static DEFINE_SPINLOCK(xfrm_mode_lock);
-int xfrm_register_mode(struct xfrm_mode *mode, int family)
+int xfrm_register_mode(struct xfrm_mode *mode)
 {
        struct xfrm_state_afinfo *afinfo;
        struct xfrm_mode **modemap;
@@ -340,7 +340,7 @@ int xfrm_register_mode(struct xfrm_mode *mode, int family)
        if (unlikely(mode->encap >= XFRM_MODE_MAX))
                return -EINVAL;
 
-       afinfo = xfrm_state_get_afinfo(family);
+       afinfo = xfrm_state_get_afinfo(mode->family);
        if (unlikely(afinfo == NULL))
                return -EAFNOSUPPORT;
 
@@ -365,31 +365,24 @@ int xfrm_register_mode(struct xfrm_mode *mode, int family)
 }
 EXPORT_SYMBOL(xfrm_register_mode);
 
-int xfrm_unregister_mode(struct xfrm_mode *mode, int family)
+void xfrm_unregister_mode(struct xfrm_mode *mode)
 {
        struct xfrm_state_afinfo *afinfo;
        struct xfrm_mode **modemap;
-       int err;
-
-       if (unlikely(mode->encap >= XFRM_MODE_MAX))
-               return -EINVAL;
 
-       afinfo = xfrm_state_get_afinfo(family);
-       if (unlikely(afinfo == NULL))
-               return -EAFNOSUPPORT;
+       afinfo = xfrm_state_get_afinfo(mode->family);
+       if (WARN_ON_ONCE(!afinfo))
+               return;
 
-       err = -ENOENT;
        modemap = afinfo->mode_map;
        spin_lock_bh(&xfrm_mode_lock);
        if (likely(modemap[mode->encap] == mode)) {
                modemap[mode->encap] = NULL;
                module_put(mode->afinfo->owner);
-               err = 0;
        }
 
        spin_unlock_bh(&xfrm_mode_lock);
        rcu_read_unlock();
-       return err;
 }
 EXPORT_SYMBOL(xfrm_unregister_mode);