netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [net PATCH] rtnetlink: Use nlmsg type RTM_NEWNEIGH from dflt fdb dump
@ 2012-11-02  2:23 John Fastabend
  2012-11-03 19:27 ` David Miller
  0 siblings, 1 reply; 2+ messages in thread
From: John Fastabend @ 2012-11-02  2:23 UTC (permalink / raw)
  To: davem; +Cc: netdev

Change the dflt fdb dump handler to use RTM_NEWNEIGH to
be compatible with bridge dump routines.

The dump reply from the network driver handlers should
match the reply from bridge handler. The fact they were
not in the ixgbe case was effectively a bug. This patch
resolves it.

Applications that were not checking the nlmsg type will
continue to work. And now applications that do check
the type will work as expected.

Signed-off-by: John Fastabend <john.r.fastabend@intel.com>
---

 net/core/rtnetlink.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c
index 76d4c2c..fad649a 100644
--- a/net/core/rtnetlink.c
+++ b/net/core/rtnetlink.c
@@ -2192,7 +2192,8 @@ static int nlmsg_populate_fdb(struct sk_buff *skb,
 			goto skip;
 
 		err = nlmsg_populate_fdb_fill(skb, dev, ha->addr,
-					      portid, seq, 0, NTF_SELF);
+					      portid, seq,
+					      RTM_NEWNEIGH, NTF_SELF);
 		if (err < 0)
 			return err;
 skip:

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [net PATCH] rtnetlink: Use nlmsg type RTM_NEWNEIGH from dflt fdb dump
  2012-11-02  2:23 [net PATCH] rtnetlink: Use nlmsg type RTM_NEWNEIGH from dflt fdb dump John Fastabend
@ 2012-11-03 19:27 ` David Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2012-11-03 19:27 UTC (permalink / raw)
  To: john.r.fastabend; +Cc: netdev

From: John Fastabend <john.r.fastabend@intel.com>
Date: Thu, 01 Nov 2012 19:23:10 -0700

> Change the dflt fdb dump handler to use RTM_NEWNEIGH to
> be compatible with bridge dump routines.
> 
> The dump reply from the network driver handlers should
> match the reply from bridge handler. The fact they were
> not in the ixgbe case was effectively a bug. This patch
> resolves it.
> 
> Applications that were not checking the nlmsg type will
> continue to work. And now applications that do check
> the type will work as expected.
> 
> Signed-off-by: John Fastabend <john.r.fastabend@intel.com>

Applied.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-11-03 19:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-02  2:23 [net PATCH] rtnetlink: Use nlmsg type RTM_NEWNEIGH from dflt fdb dump John Fastabend
2012-11-03 19:27 ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).