diff mbox

[net-next] net: make net_device members garp_port and mrp_port conditional

Message ID 20170210154350.11694-1-tklauser@distanz.ch
State Accepted, archived
Delegated to: David Miller
Headers show

Commit Message

Tobias Klauser Feb. 10, 2017, 3:43 p.m. UTC
garp_port is only used in net/802/garp.c which is only compiled with
CONFIG_GARP enabled. Same goes for mrp_port which is only used in
net/802/mrp.c with CONFIG_MRP enabled.

Only include the two members in struct net_device if their respective
CONFIG_* is enabled. This saves a few bytes in struct net_device in case
CONFIG_GARP or CONFIG_MRP are not enabled.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
---
 include/linux/netdevice.h | 4 ++++
 1 file changed, 4 insertions(+)

Comments

David Miller Feb. 14, 2017, 3:03 a.m. UTC | #1
From: Tobias Klauser <tklauser@distanz.ch>
Date: Fri, 10 Feb 2017 16:43:50 +0100

> garp_port is only used in net/802/garp.c which is only compiled with
> CONFIG_GARP enabled. Same goes for mrp_port which is only used in
> net/802/mrp.c with CONFIG_MRP enabled.
> 
> Only include the two members in struct net_device if their respective
> CONFIG_* is enabled. This saves a few bytes in struct net_device in case
> CONFIG_GARP or CONFIG_MRP are not enabled.
> 
> Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

Applied, thanks.
diff mbox

Patch

diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
index 58afbd1cc659..7bb38f2c65c2 100644
--- a/include/linux/netdevice.h
+++ b/include/linux/netdevice.h
@@ -1863,8 +1863,12 @@  struct net_device {
 		struct pcpu_vstats __percpu		*vstats;
 	};
 
+#if IS_ENABLED(CONFIG_GARP)
 	struct garp_port __rcu	*garp_port;
+#endif
+#if IS_ENABLED(CONFIG_MRP)
 	struct mrp_port __rcu	*mrp_port;
+#endif
 
 	struct device		dev;
 	const struct attribute_group *sysfs_groups[4];