diff options
| author | Hannes Frederic Sowa <hannes@stressinduktion.org> | 2016-04-18 19:19:47 (GMT) |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-04-21 19:35:44 (GMT) |
| commit | b7aade15485a660cbf5161962c284131324a9534 (patch) | |
| tree | 4030abc9519f6497fc7f937dae808da4d25fad05 /include/net/vxlan.h | |
| parent | 50d65d78897ff9785b7debbdca0030967cd5772d (diff) | |
| download | linux-b7aade15485a660cbf5161962c284131324a9534.tar.xz | |
vxlan: break dependency with netdev drivers
Currently all drivers depend and autoload the vxlan module because how
vxlan_get_rx_port is linked into them. Remove this dependency:
By using a new event type in the netdevice notifier call chain we proxy
the request from the drivers to flush and resetup the vxlan ports not
directly via function call but by the already existing netdevice
notifier call chain.
I added a separate new event type, NETDEV_OFFLOAD_PUSH_VXLAN, to do so.
We don't need to save those ids, as the event type field is an unsigned
long and using specialized event types for this purpose seemed to be a
more elegant way. This also comes in beneficial if in future we want to
add offloading knobs for vxlan.
Cc: Jesse Gross <jesse@kernel.org>
Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/vxlan.h')
| -rw-r--r-- | include/net/vxlan.h | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/include/net/vxlan.h b/include/net/vxlan.h index d442eb3..673e9f9 100644 --- a/include/net/vxlan.h +++ b/include/net/vxlan.h @@ -390,13 +390,11 @@ static inline __be32 vxlan_compute_rco(unsigned int start, unsigned int offset) return vni_field; } -#if IS_ENABLED(CONFIG_VXLAN) -void vxlan_get_rx_port(struct net_device *netdev); -#else static inline void vxlan_get_rx_port(struct net_device *netdev) { + ASSERT_RTNL(); + call_netdevice_notifiers(NETDEV_OFFLOAD_PUSH_VXLAN, netdev); } -#endif static inline unsigned short vxlan_get_sk_family(struct vxlan_sock *vs) { |