* [net-next PATCH] net: rtnetlink: fdb dflt dump must set idx used for cb->arg[0]
@ 2013-03-29 18:18 John Fastabend
2013-03-29 19:27 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: John Fastabend @ 2013-03-29 18:18 UTC (permalink / raw)
To: netdev; +Cc: vyasevich
In rtnl_fdb_dump() when the fdb_dump ndo op is not populated
we never set the idx value so that cb->arg[0] is always 0.
Resulting in a endless loop of messages.
Introduced with this commit,
commit 090096bf3db1c281ddd034573260045888a68fea
Author: Vlad Yasevich <vyasevic@redhat.com>
Date: Wed Mar 6 15:39:42 2013 +0000
net: generic fdb support for drivers without ndo_fdb_<op>
CC: Vlad Yasevich <vyasevic@redhat.com>
Signed-off-by: John Fastabend <john.r.fastabend@intel.com>
---
net/core/rtnetlink.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c
index 6fdfac8..d2322d7 100644
--- a/net/core/rtnetlink.c
+++ b/net/core/rtnetlink.c
@@ -2262,7 +2262,7 @@ skip:
* @dev: netdevice
*
* Default netdevice operation to dump the existing unicast address list.
- * Returns zero on success.
+ * Returns number of addresses from list put in skb.
*/
int ndo_dflt_fdb_dump(struct sk_buff *skb,
struct netlink_callback *cb,
@@ -2303,7 +2303,7 @@ static int rtnl_fdb_dump(struct sk_buff *skb, struct netlink_callback *cb)
if (dev->netdev_ops->ndo_fdb_dump)
idx = dev->netdev_ops->ndo_fdb_dump(skb, cb, dev, idx);
else
- ndo_dflt_fdb_dump(skb, cb, dev, idx);
+ idx = ndo_dflt_fdb_dump(skb, cb, dev, idx);
}
rcu_read_unlock();
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [net-next PATCH] net: rtnetlink: fdb dflt dump must set idx used for cb->arg[0]
2013-03-29 18:18 [net-next PATCH] net: rtnetlink: fdb dflt dump must set idx used for cb->arg[0] John Fastabend
@ 2013-03-29 19:27 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2013-03-29 19:27 UTC (permalink / raw)
To: john.fastabend; +Cc: netdev, vyasevich
From: John Fastabend <john.fastabend@gmail.com>
Date: Fri, 29 Mar 2013 11:18:37 -0700
> In rtnl_fdb_dump() when the fdb_dump ndo op is not populated
> we never set the idx value so that cb->arg[0] is always 0.
> Resulting in a endless loop of messages.
>
> Introduced with this commit,
>
> commit 090096bf3db1c281ddd034573260045888a68fea
> Author: Vlad Yasevich <vyasevic@redhat.com>
> Date: Wed Mar 6 15:39:42 2013 +0000
>
> net: generic fdb support for drivers without ndo_fdb_<op>
>
> CC: Vlad Yasevich <vyasevic@redhat.com>
> Signed-off-by: John Fastabend <john.r.fastabend@intel.com>
Applied, thanks John.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-03-29 19:27 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-29 18:18 [net-next PATCH] net: rtnetlink: fdb dflt dump must set idx used for cb->arg[0] John Fastabend
2013-03-29 19:27 ` David Miller
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.