* [PATCH net-next] bridge: add address and vlan to fdb warning messages
@ 2016-10-11 21:33 Roopa Prabhu
2016-10-11 22:17 ` Stephen Hemminger
0 siblings, 1 reply; 3+ messages in thread
From: Roopa Prabhu @ 2016-10-11 21:33 UTC (permalink / raw)
To: davem, stephen; +Cc: nikolay, netdev
From: Roopa Prabhu <roopa@cumulusnetworks.com>
This patch adds vlan and address to warning messages printed
in the bridge fdb code for debuggability.
Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
---
net/bridge/br_fdb.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c
index 6b43c8c..1257a88 100644
--- a/net/bridge/br_fdb.c
+++ b/net/bridge/br_fdb.c
@@ -536,8 +536,8 @@ static int fdb_insert(struct net_bridge *br, struct net_bridge_port *source,
if (fdb->is_local)
return 0;
br_warn(br, "adding interface %s with same address "
- "as a received packet\n",
- source ? source->dev->name : br->dev->name);
+ "as a received packet (addr:%pM, vlan:%d)\n",
+ source ? source->dev->name : br->dev->name, addr, vid);
fdb_delete(br, fdb);
}
@@ -583,9 +583,10 @@ void br_fdb_update(struct net_bridge *br, struct net_bridge_port *source,
/* attempt to update an entry for a local interface */
if (unlikely(fdb->is_local)) {
if (net_ratelimit())
- br_warn(br, "received packet on %s with "
- "own address as source address\n",
- source->dev->name);
+ br_warn(br, "received packet on %s with own "
+ "address as source address "
+ "(addr:%pM, vlan:%d)\n",
+ source->dev->name, addr, vid);
} else {
/* fastpath: update of existing entry */
if (unlikely(source != fdb->dst)) {
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] bridge: add address and vlan to fdb warning messages
2016-10-11 21:33 [PATCH net-next] bridge: add address and vlan to fdb warning messages Roopa Prabhu
@ 2016-10-11 22:17 ` Stephen Hemminger
2016-10-11 22:30 ` Roopa Prabhu
0 siblings, 1 reply; 3+ messages in thread
From: Stephen Hemminger @ 2016-10-11 22:17 UTC (permalink / raw)
To: Roopa Prabhu; +Cc: davem, nikolay, netdev
On Tue, 11 Oct 2016 14:33:51 -0700
Roopa Prabhu <roopa@cumulusnetworks.com> wrote:
> From: Roopa Prabhu <roopa@cumulusnetworks.com>
>
> This patch adds vlan and address to warning messages printed
> in the bridge fdb code for debuggability.
>
> Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
> ---
> net/bridge/br_fdb.c | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c
> index 6b43c8c..1257a88 100644
> --- a/net/bridge/br_fdb.c
> +++ b/net/bridge/br_fdb.c
> @@ -536,8 +536,8 @@ static int fdb_insert(struct net_bridge *br, struct net_bridge_port *source,
> if (fdb->is_local)
> return 0;
> br_warn(br, "adding interface %s with same address "
> - "as a received packet\n",
> - source ? source->dev->name : br->dev->name);
> + "as a received packet (addr:%pM, vlan:%d)\n",
> + source ? source->dev->name : br->dev->name, addr, vid);
> fdb_delete(br, fdb);
> }
>
> @@ -583,9 +583,10 @@ void br_fdb_update(struct net_bridge *br, struct net_bridge_port *source,
> /* attempt to update an entry for a local interface */
> if (unlikely(fdb->is_local)) {
> if (net_ratelimit())
> - br_warn(br, "received packet on %s with "
> - "own address as source address\n",
> - source->dev->name);
> + br_warn(br, "received packet on %s with own "
> + "address as source address "
> + "(addr:%pM, vlan:%d)\n",
> + source->dev->name, addr, vid);
> } else {
Isn't vlan unsigned here so print with %u
Also it would be good to make string format on one line to allow for easy search of source.
br_warn(br,
"received packet on %s with own address %pM vlan %u",
source->dev->name, addr, vid);
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] bridge: add address and vlan to fdb warning messages
2016-10-11 22:17 ` Stephen Hemminger
@ 2016-10-11 22:30 ` Roopa Prabhu
0 siblings, 0 replies; 3+ messages in thread
From: Roopa Prabhu @ 2016-10-11 22:30 UTC (permalink / raw)
To: Stephen Hemminger; +Cc: davem, nikolay, netdev
On 10/11/16, 3:17 PM, Stephen Hemminger wrote:
> On Tue, 11 Oct 2016 14:33:51 -0700
> Roopa Prabhu <roopa@cumulusnetworks.com> wrote:
>
>> From: Roopa Prabhu <roopa@cumulusnetworks.com>
>>
>> This patch adds vlan and address to warning messages printed
>> in the bridge fdb code for debuggability.
>>
>> Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
>> ---
>> net/bridge/br_fdb.c | 11 ++++++-----
>> 1 file changed, 6 insertions(+), 5 deletions(-)
>>
>> diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c
>> index 6b43c8c..1257a88 100644
>> --- a/net/bridge/br_fdb.c
>> +++ b/net/bridge/br_fdb.c
>> @@ -536,8 +536,8 @@ static int fdb_insert(struct net_bridge *br, struct net_bridge_port *source,
>> if (fdb->is_local)
>> return 0;
>> br_warn(br, "adding interface %s with same address "
>> - "as a received packet\n",
>> - source ? source->dev->name : br->dev->name);
>> + "as a received packet (addr:%pM, vlan:%d)\n",
>> + source ? source->dev->name : br->dev->name, addr, vid);
>> fdb_delete(br, fdb);
>> }
>>
>> @@ -583,9 +583,10 @@ void br_fdb_update(struct net_bridge *br, struct net_bridge_port *source,
>> /* attempt to update an entry for a local interface */
>> if (unlikely(fdb->is_local)) {
>> if (net_ratelimit())
>> - br_warn(br, "received packet on %s with "
>> - "own address as source address\n",
>> - source->dev->name);
>> + br_warn(br, "received packet on %s with own "
>> + "address as source address "
>> + "(addr:%pM, vlan:%d)\n",
>> + source->dev->name, addr, vid);
>> } else {
> Isn't vlan unsigned here so print with %u
you are right. will fix it
> Also it would be good to make string format on one line to allow for easy search of source.
> br_warn(br,
> "received packet on %s with own address %pM vlan %u",
> source->dev->name, addr, vid);
ack, was debating abt this.
v2 coming..thanks
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-10-11 22:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-11 21:33 [PATCH net-next] bridge: add address and vlan to fdb warning messages Roopa Prabhu
2016-10-11 22:17 ` Stephen Hemminger
2016-10-11 22:30 ` Roopa Prabhu
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).