There are multiple instances of static const arrays for broadcast and zero ethernet addresses used for various purposes.
Add const u8 ether_<foo>_addr[ETH_ALEN] globals to consolidate these uses. Miscellanea: o Move and rename the eth_reserved_addr_base declaration to this file and declare it extern to avoid possible multiple static definitions o Add compilation to the Makefile Signed-off-by: Joe Perches <j...@perches.com> --- Not sure this is the best place for this. Better ideas welcomed. include/linux/etherdevice.h | 12 ++++++++---- net/core/Makefile | 3 ++- net/core/ether_addrs.c | 16 ++++++++++++++++ 3 files changed, 26 insertions(+), 5 deletions(-) create mode 100644 net/core/ether_addrs.c diff --git a/include/linux/etherdevice.h b/include/linux/etherdevice.h index 79563840c295..85d2486b2959 100644 --- a/include/linux/etherdevice.h +++ b/include/linux/etherdevice.h @@ -63,10 +63,14 @@ struct sk_buff **eth_gro_receive(struct sk_buff **head, struct sk_buff *skb); int eth_gro_complete(struct sk_buff *skb, int nhoff); +/* Generic Ethernet addresses */ +extern const u8 ether_broadcast_addr[ETH_ALEN]; /* all 0xff */ +extern const u8 ether_zero_addr[ETH_ALEN]; /* all zeros */ + /* Reserved Ethernet Addresses per IEEE 802.1Q */ -static const u8 eth_reserved_addr_base[ETH_ALEN] __aligned(2) = -{ 0x01, 0x80, 0xc2, 0x00, 0x00, 0x00 }; -#define eth_stp_addr eth_reserved_addr_base +extern const u8 ether_reserved_addr_base[ETH_ALEN]; + +#define eth_stp_addr ether_reserved_addr_base /** * is_link_local_ether_addr - Determine if given Ethernet address is link-local @@ -80,7 +84,7 @@ static const u8 eth_reserved_addr_base[ETH_ALEN] __aligned(2) = static inline bool is_link_local_ether_addr(const u8 *addr) { __be16 *a = (__be16 *)addr; - static const __be16 *b = (const __be16 *)eth_reserved_addr_base; + static const __be16 *b = (const __be16 *)ether_reserved_addr_base; static const __be16 m = cpu_to_be16(0xfff0); #if defined(CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS) diff --git a/net/core/Makefile b/net/core/Makefile index 6dbbba8c57ae..7fb941243cc3 100644 --- a/net/core/Makefile +++ b/net/core/Makefile @@ -4,7 +4,8 @@ # obj-y := sock.o request_sock.o skbuff.o datagram.o stream.o scm.o \ - gen_stats.o gen_estimator.o net_namespace.o secure_seq.o flow_dissector.o + gen_stats.o gen_estimator.o net_namespace.o secure_seq.o \ + flow_dissector.o ether_addrs.o obj-$(CONFIG_SYSCTL) += sysctl_net_core.o diff --git a/net/core/ether_addrs.c b/net/core/ether_addrs.c new file mode 100644 index 000000000000..18eedabe73fa --- /dev/null +++ b/net/core/ether_addrs.c @@ -0,0 +1,16 @@ +// SPDX-License-Identifier: GPL-2.0 +/* Various const global Ethernet addresses */ + +#include <linux/etherdevice.h> + +const u8 ether_broadcast_addr[ETH_ALEN] __aligned(2) = { + 0xff, 0xff, 0xff, 0xff, 0xff, 0xff +}; + +const u8 ether_zero_addr[ETH_ALEN] __aligned(2) = { + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 +}; + +const u8 ether_reserved_addr_base[ETH_ALEN] __aligned(2) = { + 0x01, 0x80, 0xc2, 0x00, 0x00, 0x00 +}; -- 2.15.0