Re: [PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-13 Thread David Miller
From: Vlad Yasevich Date: Tue, 13 Aug 2013 19:06:37 -0400 > ast explained it in his header message (Bridge VLAN kernel/iproute2 > incompatibility) That's not a header message. Header messages have a subject prefix of the form "[PATCH 0/N]". If he had done this I wouldn't have had to ask such

Re: [PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-13 Thread Vlad Yasevich
On 08/13/2013 06:54 PM, David Miller wrote: From: Vlad Yasevich Date: Mon, 12 Aug 2013 15:57:29 -0400 On 08/12/2013 12:30 PM, Asbjoern Sloth Toennesen wrote: Fix the iproute2 command `bridge vlan show`, after switching from rtgenmsg to ifinfomsg. Signed-off-by: Asbjoern Sloth Toennesen

Re: [PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-13 Thread David Miller
From: Vlad Yasevich Date: Mon, 12 Aug 2013 15:57:29 -0400 > On 08/12/2013 12:30 PM, Asbjoern Sloth Toennesen wrote: >> Fix the iproute2 command `bridge vlan show`, after switching from >> rtgenmsg to ifinfomsg. >> >> Signed-off-by: Asbjoern Sloth Toennesen > > > Thanks.. I've still been

Re: [PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-13 Thread David Miller
From: Vlad Yasevich vyase...@redhat.com Date: Mon, 12 Aug 2013 15:57:29 -0400 On 08/12/2013 12:30 PM, Asbjoern Sloth Toennesen wrote: Fix the iproute2 command `bridge vlan show`, after switching from rtgenmsg to ifinfomsg. Signed-off-by: Asbjoern Sloth Toennesen a...@fiberby.net

Re: [PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-13 Thread Vlad Yasevich
On 08/13/2013 06:54 PM, David Miller wrote: From: Vlad Yasevich vyase...@redhat.com Date: Mon, 12 Aug 2013 15:57:29 -0400 On 08/12/2013 12:30 PM, Asbjoern Sloth Toennesen wrote: Fix the iproute2 command `bridge vlan show`, after switching from rtgenmsg to ifinfomsg. Signed-off-by: Asbjoern

Re: [PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-13 Thread David Miller
From: Vlad Yasevich vyase...@redhat.com Date: Tue, 13 Aug 2013 19:06:37 -0400 ast explained it in his header message (Bridge VLAN kernel/iproute2 incompatibility) That's not a header message. Header messages have a subject prefix of the form [PATCH 0/N]. If he had done this I wouldn't have

Re: [PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-12 Thread Vlad Yasevich
On 08/12/2013 12:30 PM, Asbjoern Sloth Toennesen wrote: Fix the iproute2 command `bridge vlan show`, after switching from rtgenmsg to ifinfomsg. Signed-off-by: Asbjoern Sloth Toennesen Thanks.. I've still been using an older iproute version and didn't see this. Reviewed-by: Vlad

[PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-12 Thread Asbjoern Sloth Toennesen
Fix the iproute2 command `bridge vlan show`, after switching from rtgenmsg to ifinfomsg. Signed-off-by: Asbjoern Sloth Toennesen --- net/core/rtnetlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c index 3de7408..a043171 100644

[PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-12 Thread Asbjoern Sloth Toennesen
Fix the iproute2 command `bridge vlan show`, after switching from rtgenmsg to ifinfomsg. Signed-off-by: Asbjoern Sloth Toennesen a...@fiberby.net --- net/core/rtnetlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c index

Re: [PATCH] rtnetlink: rtnl_bridge_getlink: Call nlmsg_find_attr() with ifinfomsg header

2013-08-12 Thread Vlad Yasevich
On 08/12/2013 12:30 PM, Asbjoern Sloth Toennesen wrote: Fix the iproute2 command `bridge vlan show`, after switching from rtgenmsg to ifinfomsg. Signed-off-by: Asbjoern Sloth Toennesen a...@fiberby.net Thanks.. I've still been using an older iproute version and didn't see this.