Commit 6ee3c393 authored by Sven Eckelmann's avatar Sven Eckelmann Committed by Simon Wunderlich

batman-adv: Demote batadv-on-batadv skip error message

The error message "Cannot find parent device" was shown for users of
macvtap (on batadv devices) whenever the macvtap was moved to a different
netns. This happens because macvtap doesn't provide an implementation for
rtnl_link_ops->get_link_net.

The situation for which this message is printed is actually not an error
but just a warning that the optional sanity check was skipped. So demote
the message from error to warning and adjust the text to better explain
what happened.
Reported-by: default avatarLeonardo Mörlein <freifunk@irrelefant.net>
Signed-off-by: default avatarSven Eckelmann <sven@narfation.org>
Signed-off-by: default avatarSimon Wunderlich <sw@simonwunderlich.de>
parent eb7da4f1
...@@ -165,9 +165,9 @@ static bool batadv_is_on_batman_iface(const struct net_device *net_dev) ...@@ -165,9 +165,9 @@ static bool batadv_is_on_batman_iface(const struct net_device *net_dev)
/* recurse over the parent device */ /* recurse over the parent device */
parent_dev = __dev_get_by_index((struct net *)parent_net, parent_dev = __dev_get_by_index((struct net *)parent_net,
dev_get_iflink(net_dev)); dev_get_iflink(net_dev));
/* if we got a NULL parent_dev there is something broken.. */
if (!parent_dev) { if (!parent_dev) {
pr_err("Cannot find parent device\n"); pr_warn("Cannot find parent device. Skipping batadv-on-batadv check for %s\n",
net_dev->name);
return false; return false;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment