* [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it
@ 2013-02-12 15:21 Antonio Quartulli
2013-02-12 15:21 ` [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: remove interfacing enslaving loop private check Antonio Quartulli
2013-02-16 12:22 ` [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it Marek Lindner
0 siblings, 2 replies; 6+ messages in thread
From: Antonio Quartulli @ 2013-02-12 15:21 UTC (permalink / raw)
To: b.a.t.m.a.n
When adding a new hard interface (e.h. wlan0) to a soft interface (e.g. bat0)
and the former is already enslaved in another virtual interface (e.g. a software
bridge) batman-adv has to free the it first and then continue with the
adding mechanism.
In this way the behaviour becomes consistent with what "ip link set master"
does. At the moment batman-adv enslaves the hard interface without checking for
the master device, possibly causing strange behaviours which are never wanted by
the users.
Reported-by: Marek Lindner <lindner_marek@yahoo.de>
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
---
** Work based on top of the 6 patches adding the new rtnl interface for
interface enslaving.
Cheers,
hard-interface.c | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/hard-interface.c b/hard-interface.c
index c08e39e..fd99e42 100644
--- a/hard-interface.c
+++ b/hard-interface.c
@@ -311,7 +311,7 @@ int batadv_hardif_enable_interface(struct batadv_hard_iface *hard_iface,
const char *iface_name)
{
struct batadv_priv *bat_priv;
- struct net_device *soft_iface;
+ struct net_device *soft_iface, *master;
__be16 ethertype = __constant_htons(ETH_P_BATMAN);
int ret;
@@ -321,11 +321,6 @@ int batadv_hardif_enable_interface(struct batadv_hard_iface *hard_iface,
if (!atomic_inc_not_zero(&hard_iface->refcount))
goto out;
- /* hard-interface is part of a bridge */
- if (hard_iface->net_dev->priv_flags & IFF_BRIDGE_PORT)
- pr_err("You are about to enable batman-adv on '%s' which already is part of a bridge. Unless you know exactly what you are doing this is probably wrong and won't work the way you think it would.\n",
- hard_iface->net_dev->name);
-
soft_iface = dev_get_by_name(&init_net, iface_name);
if (!soft_iface) {
@@ -347,6 +342,13 @@ int batadv_hardif_enable_interface(struct batadv_hard_iface *hard_iface,
goto err_dev;
}
+ /* check if the interface is enslaved in another virtual one and
+ * in that case unlink it first
+ */
+ master = netdev_master_upper_dev_get(hard_iface->net_dev);
+ if (master)
+ netdev_upper_dev_unlink(hard_iface->net_dev, master);
+
hard_iface->soft_iface = soft_iface;
bat_priv = netdev_priv(hard_iface->soft_iface);
--
1.8.1.2
^ permalink raw reply related [flat|nested] 6+ messages in thread* [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: remove interfacing enslaving loop private check
2013-02-12 15:21 [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it Antonio Quartulli
@ 2013-02-12 15:21 ` Antonio Quartulli
2013-02-16 12:22 ` [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it Marek Lindner
1 sibling, 0 replies; 6+ messages in thread
From: Antonio Quartulli @ 2013-02-12 15:21 UTC (permalink / raw)
To: b.a.t.m.a.n
Now that the new dev_upper API is used, the private check for enslaving loops is
not needed anymore as this is performed by the new API itself.
Reported-by: Marek Lindner <lindner_marek@yahoo.de>
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
---
hard-interface.c | 43 -------------------------------------------
1 file changed, 43 deletions(-)
diff --git a/hard-interface.c b/hard-interface.c
index fd99e42..b96f819 100644
--- a/hard-interface.c
+++ b/hard-interface.c
@@ -60,45 +60,6 @@ out:
return hard_iface;
}
-/**
- * batadv_is_on_batman_iface - check if a device is a batman iface descendant
- * @net_dev: the device to check
- *
- * If the user creates any virtual device on top of a batman-adv interface, it
- * is important to prevent this new interface to be used to create a new mesh
- * network (this behaviour would lead to a batman-over-batman configuration).
- * This function recursively checks all the fathers of the device passed as
- * argument looking for a batman-adv soft interface.
- *
- * Returns true if the device is descendant of a batman-adv mesh interface (or
- * if it is a batman-adv interface itself), false otherwise
- */
-static bool batadv_is_on_batman_iface(const struct net_device *net_dev)
-{
- struct net_device *parent_dev;
- bool ret;
-
- /* check if this is a batman-adv mesh interface */
- if (batadv_softif_is_valid(net_dev))
- return true;
-
- /* no more parents..stop recursion */
- if (net_dev->iflink == net_dev->ifindex)
- return false;
-
- /* recurse over the parent device */
- parent_dev = dev_get_by_index(&init_net, net_dev->iflink);
- /* if we got a NULL parent_dev there is something broken.. */
- if (WARN(!parent_dev, "Cannot find parent device"))
- return false;
-
- ret = batadv_is_on_batman_iface(parent_dev);
-
- if (parent_dev)
- dev_put(parent_dev);
- return ret;
-}
-
static int batadv_is_valid_iface(const struct net_device *net_dev)
{
if (net_dev->flags & IFF_LOOPBACK)
@@ -110,10 +71,6 @@ static int batadv_is_valid_iface(const struct net_device *net_dev)
if (net_dev->addr_len != ETH_ALEN)
return 0;
- /* no batman over batman */
- if (batadv_is_on_batman_iface(net_dev))
- return 0;
-
return 1;
}
--
1.8.1.2
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it
2013-02-12 15:21 [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it Antonio Quartulli
2013-02-12 15:21 ` [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: remove interfacing enslaving loop private check Antonio Quartulli
@ 2013-02-16 12:22 ` Marek Lindner
2013-02-16 12:27 ` Marek Lindner
2013-02-16 12:35 ` Antonio Quartulli
1 sibling, 2 replies; 6+ messages in thread
From: Marek Lindner @ 2013-02-16 12:22 UTC (permalink / raw)
To: The list for a Better Approach To Mobile Ad-hoc Networking
On Tuesday, February 12, 2013 23:21:55 Antonio Quartulli wrote:
> @@ -347,6 +342,13 @@ int batadv_hardif_enable_interface(struct
> batadv_hard_iface *hard_iface, goto err_dev;
> }
>
> + /* check if the interface is enslaved in another virtual one and
> + * in that case unlink it first
> + */
> + master = netdev_master_upper_dev_get(hard_iface->net_dev);
> + if (master)
> + netdev_upper_dev_unlink(hard_iface->net_dev, master);
> +
> hard_iface->soft_iface = soft_iface;
> bat_priv = netdev_priv(hard_iface->soft_iface);
Seems we need some compat code here:
hard-interface.c:348:2: error: implicit declaration of function
'netdev_master_upper_dev_get'
Cheers,
Marek
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it
2013-02-16 12:22 ` [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it Marek Lindner
@ 2013-02-16 12:27 ` Marek Lindner
2013-02-16 12:34 ` Antonio Quartulli
2013-02-16 12:35 ` Antonio Quartulli
1 sibling, 1 reply; 6+ messages in thread
From: Marek Lindner @ 2013-02-16 12:27 UTC (permalink / raw)
To: The list for a Better Approach To Mobile Ad-hoc Networking
On Saturday, February 16, 2013 20:22:37 Marek Lindner wrote:
> On Tuesday, February 12, 2013 23:21:55 Antonio Quartulli wrote:
> > @@ -347,6 +342,13 @@ int batadv_hardif_enable_interface(struct
> > batadv_hard_iface *hard_iface, goto err_dev;
> >
> > }
> >
> > + /* check if the interface is enslaved in another virtual one and
> > + * in that case unlink it first
> > + */
> > + master = netdev_master_upper_dev_get(hard_iface->net_dev);
> > + if (master)
> > + netdev_upper_dev_unlink(hard_iface->net_dev, master);
> > +
> >
> > hard_iface->soft_iface = soft_iface;
> > bat_priv = netdev_priv(hard_iface->soft_iface);
>
> Seems we need some compat code here:
>
> hard-interface.c:348:2: error: implicit declaration of function
> 'netdev_master_upper_dev_get'
I wouldn't be surprised if netdev_upper_dev_unlink() also needed some compat
code ...
Cheers,
Marek
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it
2013-02-16 12:27 ` Marek Lindner
@ 2013-02-16 12:34 ` Antonio Quartulli
0 siblings, 0 replies; 6+ messages in thread
From: Antonio Quartulli @ 2013-02-16 12:34 UTC (permalink / raw)
To: The list for a Better Approach To Mobile Ad-hoc Networking
[-- Attachment #1: Type: text/plain, Size: 1238 bytes --]
On Sat, Feb 16, 2013 at 08:27:24PM +0800, Marek Lindner wrote:
> On Saturday, February 16, 2013 20:22:37 Marek Lindner wrote:
> > On Tuesday, February 12, 2013 23:21:55 Antonio Quartulli wrote:
> > > @@ -347,6 +342,13 @@ int batadv_hardif_enable_interface(struct
> > > batadv_hard_iface *hard_iface, goto err_dev;
> > >
> > > }
> > >
> > > + /* check if the interface is enslaved in another virtual one and
> > > + * in that case unlink it first
> > > + */
> > > + master = netdev_master_upper_dev_get(hard_iface->net_dev);
> > > + if (master)
> > > + netdev_upper_dev_unlink(hard_iface->net_dev, master);
> > > +
> > >
> > > hard_iface->soft_iface = soft_iface;
> > > bat_priv = netdev_priv(hard_iface->soft_iface);
> >
> > Seems we need some compat code here:
> >
> > hard-interface.c:348:2: error: implicit declaration of function
> > 'netdev_master_upper_dev_get'
>
> I wouldn't be surprised if netdev_upper_dev_unlink() also needed some compat
> code ...
netdev_upper_dev_unlink() is already in the compat code as it was needed by the
patcheset you have already merged this morning..
Cheers,
--
Antonio Quartulli
..each of us alone is worth nothing..
Ernesto "Che" Guevara
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it
2013-02-16 12:22 ` [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it Marek Lindner
2013-02-16 12:27 ` Marek Lindner
@ 2013-02-16 12:35 ` Antonio Quartulli
1 sibling, 0 replies; 6+ messages in thread
From: Antonio Quartulli @ 2013-02-16 12:35 UTC (permalink / raw)
To: The list for a Better Approach To Mobile Ad-hoc Networking
[-- Attachment #1: Type: text/plain, Size: 1035 bytes --]
On Sat, Feb 16, 2013 at 08:22:37PM +0800, Marek Lindner wrote:
> On Tuesday, February 12, 2013 23:21:55 Antonio Quartulli wrote:
> > @@ -347,6 +342,13 @@ int batadv_hardif_enable_interface(struct
> > batadv_hard_iface *hard_iface, goto err_dev;
> > }
> >
> > + /* check if the interface is enslaved in another virtual one and
> > + * in that case unlink it first
> > + */
> > + master = netdev_master_upper_dev_get(hard_iface->net_dev);
> > + if (master)
> > + netdev_upper_dev_unlink(hard_iface->net_dev, master);
> > +
> > hard_iface->soft_iface = soft_iface;
> > bat_priv = netdev_priv(hard_iface->soft_iface);
>
> Seems we need some compat code here:
>
> hard-interface.c:348:2: error: implicit declaration of function
> 'netdev_master_upper_dev_get'
Correct. I'm not used to test code for master on kernel older than net-next.
I'll send v2 with the proper support for ancient kernels.
Cheers (:D),
--
Antonio Quartulli
..each of us alone is worth nothing..
Ernesto "Che" Guevara
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-02-16 12:35 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-12 15:21 [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it Antonio Quartulli
2013-02-12 15:21 ` [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: remove interfacing enslaving loop private check Antonio Quartulli
2013-02-16 12:22 ` [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: free an hard-interface before adding it Marek Lindner
2013-02-16 12:27 ` Marek Lindner
2013-02-16 12:34 ` Antonio Quartulli
2013-02-16 12:35 ` Antonio Quartulli
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.