| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <20221101181225.1272144-1-andy.ren@getcruise.com>
Date: Tue, 1 Nov 2022 11:12:25 -0700
From: Andy Ren <andy.ren@...cruise.com>
To: netdev@...r.kernel.org, davem@...emloft.com, kuba@...nel.org,
pabeni@...hat.com, corbet@....net
Cc: linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
roman.gushchin@...ux.dev, Andy Ren <andy.ren@...cruise.com>
Subject: [PATCH net-next] netconsole: Enable live renaming for network interfaces used by netconsole
This patch enables support for live renaming of network interfaces
initialized by netconsole.
This resolves an issue seen when netconsole is configured to boot as a
built-in kernel module with a kernel boot argument. As stated in the
kernel man page - As a built-in, netconsole initializes immediately
after NIC cards and will bring up the specified interface as soon as
possible. Consequently, the renaming of specified interfaces will fail
and return EBUSY. This is because by default, the kernel disallows live
renaming unless the device explicitly sets a priv_flags bit
(IFF_LIVE_RENAME_OK), and renaming after a network interface is up
returns EBUSY.
The changes to the kernel are as of following:
- Addition of a iface_live_renaming boolean flag to the netpoll struct,
used to enable/disable interface live renaming. False by default
- Changes to check for the aforementioned flag in network and ethernet
driver interface renaming code
- Adds a new optional "*" parameter to the netconsole configuration string
after interface device name that enables interface live renaming when
included (e.g. "eth0*"). When this parameter is included,
"iface_live_renaming" is set to true.
---
Documentation/networking/netconsole.rst | 5 +++--
include/linux/netpoll.h | 3 +++
net/core/dev.c | 3 ++-
net/core/netpoll.c | 18 +++++++++++++++++-
net/ethernet/eth.c | 5 ++++-
5 files changed, 29 insertions(+), 5 deletions(-)
diff --git a/Documentation/networking/netconsole.rst b/Documentation/networking/netconsole.rst
index 1f5c4a04027c..c1255a4d9c9b 100644
--- a/Documentation/networking/netconsole.rst
+++ b/Documentation/networking/netconsole.rst
@@ -34,20 +34,21 @@ Sender and receiver configuration:
It takes a string configuration parameter "netconsole" in the
following format::
- netconsole=[+][src-port]@[src-ip]/[<dev>],[tgt-port]@<tgt-ip>/[tgt-macaddr]
+ netconsole=[+][src-port]@[src-ip]/[<dev>][*],[tgt-port]@<tgt-ip>/[tgt-macaddr]
where
+ if present, enable extended console support
src-port source for UDP packets (defaults to 6665)
src-ip source IP to use (interface address)
dev network interface (eth0)
+ * if present, allow runtime network interface renaming
tgt-port port for logging agent (6666)
tgt-ip IP address for logging agent
tgt-macaddr ethernet MAC address for logging agent (broadcast)
Examples::
- linux netconsole=4444@...0.0.1/eth1,9353@...0.0.2/12:34:56:78:9a:bc
+ linux netconsole=4444@...0.0.1/eth1*,9353@...0.0.2/12:34:56:78:9a:bc
or::
diff --git a/include/linux/netpoll.h b/include/linux/netpoll.h
index bd19c4b91e31..f2ebdabf0959 100644
--- a/include/linux/netpoll.h
+++ b/include/linux/netpoll.h
@@ -32,6 +32,7 @@ struct netpoll {
bool ipv6;
u16 local_port, remote_port;
u8 remote_mac[ETH_ALEN];
+ bool iface_live_renaming;
};
struct netpoll_info {
@@ -51,9 +52,11 @@ struct netpoll_info {
void netpoll_poll_dev(struct net_device *dev);
void netpoll_poll_disable(struct net_device *dev);
void netpoll_poll_enable(struct net_device *dev);
+bool netpoll_live_renaming_enabled(struct net_device *dev);
#else
static inline void netpoll_poll_disable(struct net_device *dev) { return; }
static inline void netpoll_poll_enable(struct net_device *dev) { return; }
+static inline bool netpoll_live_renaming_enabled(struct net_device *dev) { return false; }
#endif
void netpoll_send_udp(struct netpoll *np, const char *msg, int len);
diff --git a/net/core/dev.c b/net/core/dev.c
index cfb68db040a4..67430bec1f33 100644
--- a/net/core/dev.c
+++ b/net/core/dev.c
@@ -1176,7 +1176,8 @@ int dev_change_name(struct net_device *dev, const char *newname)
* directly.
*/
if (dev->flags & IFF_UP &&
- likely(!(dev->priv_flags & IFF_LIVE_RENAME_OK)))
+ likely(!(dev->priv_flags & IFF_LIVE_RENAME_OK) &&
+ !netpoll_live_renaming_enabled(dev)))
return -EBUSY;
down_write(&devnet_rename_sem);
diff --git a/net/core/netpoll.c b/net/core/netpoll.c
index 9be762e1d042..fcea265290c3 100644
--- a/net/core/netpoll.c
+++ b/net/core/netpoll.c
@@ -224,6 +224,15 @@ void netpoll_poll_enable(struct net_device *dev)
}
EXPORT_SYMBOL(netpoll_poll_enable);
+bool netpoll_live_renaming_enabled(struct net_device *dev)
+{
+ if (dev->npinfo && dev->npinfo->netpoll->iface_live_renaming)
+ return true;
+
+ return false;
+}
+EXPORT_SYMBOL(netpoll_live_renaming_enabled);
+
static void refill_skbs(void)
{
struct sk_buff *skb;
@@ -523,7 +532,7 @@ static int netpoll_parse_ip_addr(const char *str, union inet_addr *addr)
int netpoll_parse_options(struct netpoll *np, char *opt)
{
- char *cur=opt, *delim;
+ char *cur = opt, *delim, *asterisk;
int ipv6;
bool ipversion_set = false;
@@ -556,6 +565,13 @@ int netpoll_parse_options(struct netpoll *np, char *opt)
if ((delim = strchr(cur, ',')) == NULL)
goto parse_failed;
*delim = 0;
+
+ asterisk = strchr(cur, '*');
+ if (asterisk) {
+ np->iface_live_renaming = true;
+ *asterisk = 0;
+ }
+
strscpy(np->dev_name, cur, sizeof(np->dev_name));
cur = delim;
}
diff --git a/net/ethernet/eth.c b/net/ethernet/eth.c
index e02daa74e833..bb341acfcf05 100644
--- a/net/ethernet/eth.c
+++ b/net/ethernet/eth.c
@@ -62,6 +62,7 @@
#include <net/gro.h>
#include <linux/uaccess.h>
#include <net/pkt_sched.h>
+#include <linux/netpoll.h>
/**
* eth_header - create the Ethernet header
@@ -288,8 +289,10 @@ int eth_prepare_mac_addr_change(struct net_device *dev, void *p)
{
struct sockaddr *addr = p;
- if (!(dev->priv_flags & IFF_LIVE_ADDR_CHANGE) && netif_running(dev))
+ if (!(dev->priv_flags & IFF_LIVE_ADDR_CHANGE) && netif_running(dev) &&
+ !netpoll_live_renaming_enabled(dev))
return -EBUSY;
+
if (!is_valid_ether_addr(addr->sa_data))
return -EADDRNOTAVAIL;
return 0;
--
2.38.1
Powered by blists - more mailing lists