From: Florian Westphal fw@strlen.de
Replace all trans_start updates with netif_trans_update helper. change was done via spatch:
struct net_device *d; @@ - d->trans_start = jiffies + netif_trans_update(d)
Compile tested only.
Signed-off-by: Florian Westphal fw@strlen.de Signed-off-by: Sven Eckelmann sven.eckelmann@open-mesh.com --- Warning: untested
compat-include/linux/netdevice.h | 10 ++++++++++ net/batman-adv/soft-interface.c | 2 +- 2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/compat-include/linux/netdevice.h b/compat-include/linux/netdevice.h index f47f5c3..fb5b519 100644 --- a/compat-include/linux/netdevice.h +++ b/compat-include/linux/netdevice.h @@ -74,4 +74,14 @@
#endif /* < KERNEL_VERSION(4, 5, 0) */
+#if LINUX_VERSION_CODE < KERNEL_VERSION(4, 7, 0) + +#define netif_trans_update batadv_netif_trans_update +static inline void batadv_netif_trans_update(struct net_device *dev) +{ + dev->trans_start = jiffies; +} + +#endif /* < KERNEL_VERSION(4, 7, 0) */ + #endif /* _NET_BATMAN_ADV_COMPAT_LINUX_NETDEVICE_H_ */ diff --git a/net/batman-adv/soft-interface.c b/net/batman-adv/soft-interface.c index 31d1df2..b349287 100644 --- a/net/batman-adv/soft-interface.c +++ b/net/batman-adv/soft-interface.c @@ -207,7 +207,7 @@ static int batadv_interface_tx(struct sk_buff *skb, if (atomic_read(&bat_priv->mesh_state) != BATADV_MESH_ACTIVE) goto dropped;
- soft_iface->trans_start = jiffies; + netif_trans_update(soft_iface); vid = batadv_get_vid(skb, 0); ethhdr = eth_hdr(skb);
On Thu, May 05, 2016 at 09:44:42AM +0200, Sven Eckelmann wrote:
From: Florian Westphal fw@strlen.de
Replace all trans_start updates with netif_trans_update helper. change was done via spatch:
struct net_device *d; @@
- d->trans_start = jiffies
- netif_trans_update(d)
Compile tested only.
Signed-off-by: Florian Westphal fw@strlen.de Signed-off-by: Sven Eckelmann sven.eckelmann@open-mesh.com
Warning: untested
compat-include/linux/netdevice.h | 10 ++++++++++ net/batman-adv/soft-interface.c | 2 +- 2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/compat-include/linux/netdevice.h b/compat-include/linux/netdevice.h index f47f5c3..fb5b519 100644 --- a/compat-include/linux/netdevice.h +++ b/compat-include/linux/netdevice.h @@ -74,4 +74,14 @@
#endif /* < KERNEL_VERSION(4, 5, 0) */
+#if LINUX_VERSION_CODE < KERNEL_VERSION(4, 7, 0)
+#define netif_trans_update batadv_netif_trans_update +static inline void batadv_netif_trans_update(struct net_device *dev)
Sven,
do we really need to rename the function is the original one does not exist in kernels < 4.7 ? Or maybe it exists but does something else ?
Cheers,
On Thursday 05 May 2016 18:04:35 Antonio Quartulli wrote: [...]
+#if LINUX_VERSION_CODE < KERNEL_VERSION(4, 7, 0)
+#define netif_trans_update batadv_netif_trans_update +static inline void batadv_netif_trans_update(struct net_device *dev)
Sven,
do we really need to rename the function is the original one does not exist in kernels < 4.7 ? Or maybe it exists but does something else ?
Not necessarily but Linus' noticed for a different function that some distros (or stable maintainers) backported the function which we introduced. Thus we used the same trick to solve the conflict.
Kind regards, Sven
On Thu, May 05, 2016 at 12:36:54PM +0200, Sven Eckelmann wrote:
On Thursday 05 May 2016 18:04:35 Antonio Quartulli wrote: [...]
+#if LINUX_VERSION_CODE < KERNEL_VERSION(4, 7, 0)
+#define netif_trans_update batadv_netif_trans_update +static inline void batadv_netif_trans_update(struct net_device *dev)
Sven,
do we really need to rename the function is the original one does not exist in kernels < 4.7 ? Or maybe it exists but does something else ?
Not necessarily but Linus' noticed for a different function that some distros (or stable maintainers) backported the function which we introduced. Thus we used the same trick to solve the conflict.
Yup, thanks for reminding!
Cheers,
On Thursday, May 05, 2016 09:44:42 Sven Eckelmann wrote:
From: Florian Westphal fw@strlen.de
Replace all trans_start updates with netif_trans_update helper. change was done via spatch:
struct net_device *d; @@
- d->trans_start = jiffies
- netif_trans_update(d)
Compile tested only.
Signed-off-by: Florian Westphal fw@strlen.de Signed-off-by: Sven Eckelmann sven.eckelmann@open-mesh.com
Warning: untested
compat-include/linux/netdevice.h | 10 ++++++++++ net/batman-adv/soft-interface.c | 2 +- 2 files changed, 11 insertions(+), 1 deletion(-)
Applied in revision c5ebb1b.
Thanks, Marek
b.a.t.m.a.n@lists.open-mesh.org