netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH net-next] net: bridge: correct debug message function name in br_fill_ifinfo
@ 2025-10-13 10:01 Alok Tiwari
  2025-10-13 12:11 ` Simon Horman
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Alok Tiwari @ 2025-10-13 10:01 UTC (permalink / raw)
  To: idosch, razor, davem, edumazet, kuba, pabeni, horms, bridge,
	netdev
  Cc: alok.a.tiwari

The debug message in br_fill_ifinfo() incorrectly refers to br_fill_info
instead of the actual function name. Update it for clarity in debugging
output.

Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
---
 net/bridge/br_netlink.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/bridge/br_netlink.c b/net/bridge/br_netlink.c
index 4e2d53b27221..0264730938f4 100644
--- a/net/bridge/br_netlink.c
+++ b/net/bridge/br_netlink.c
@@ -467,7 +467,7 @@ static int br_fill_ifinfo(struct sk_buff *skb,
 	else
 		br = netdev_priv(dev);
 
-	br_debug(br, "br_fill_info event %d port %s master %s\n",
+	br_debug(br, "br_fill_ifinfo event %d port %s master %s\n",
 		     event, dev->name, br->dev->name);
 
 	nlh = nlmsg_put(skb, pid, seq, event, sizeof(*hdr), flags);
-- 
2.50.1


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

* Re: [PATCH net-next] net: bridge: correct debug message function name in br_fill_ifinfo
  2025-10-13 10:01 [PATCH net-next] net: bridge: correct debug message function name in br_fill_ifinfo Alok Tiwari
@ 2025-10-13 12:11 ` Simon Horman
  2025-10-13 12:22 ` Nikolay Aleksandrov
  2025-10-14 20:40 ` patchwork-bot+netdevbpf
  2 siblings, 0 replies; 4+ messages in thread
From: Simon Horman @ 2025-10-13 12:11 UTC (permalink / raw)
  To: Alok Tiwari; +Cc: idosch, razor, davem, edumazet, kuba, pabeni, bridge, netdev

On Mon, Oct 13, 2025 at 03:01:16AM -0700, Alok Tiwari wrote:
> The debug message in br_fill_ifinfo() incorrectly refers to br_fill_info
> instead of the actual function name. Update it for clarity in debugging
> output.
> 
> Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>

Reviewed-by: Simon Horman <horms@kernel.org>


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

* Re: [PATCH net-next] net: bridge: correct debug message function name in br_fill_ifinfo
  2025-10-13 10:01 [PATCH net-next] net: bridge: correct debug message function name in br_fill_ifinfo Alok Tiwari
  2025-10-13 12:11 ` Simon Horman
@ 2025-10-13 12:22 ` Nikolay Aleksandrov
  2025-10-14 20:40 ` patchwork-bot+netdevbpf
  2 siblings, 0 replies; 4+ messages in thread
From: Nikolay Aleksandrov @ 2025-10-13 12:22 UTC (permalink / raw)
  To: Alok Tiwari, idosch, davem, edumazet, kuba, pabeni, horms, bridge,
	netdev
  Cc: alok.a.tiwari

On October 13, 2025 12:01:16 PM GMT+02:00, Alok Tiwari <alok.a.tiwari@oracle.com> wrote:
>The debug message in br_fill_ifinfo() incorrectly refers to br_fill_info
>instead of the actual function name. Update it for clarity in debugging
>output.
>
>Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
>---
> net/bridge/br_netlink.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/net/bridge/br_netlink.c b/net/bridge/br_netlink.c
>index 4e2d53b27221..0264730938f4 100644
>--- a/net/bridge/br_netlink.c
>+++ b/net/bridge/br_netlink.c
>@@ -467,7 +467,7 @@ static int br_fill_ifinfo(struct sk_buff *skb,
> 	else
> 		br = netdev_priv(dev);
> 
>-	br_debug(br, "br_fill_info event %d port %s master %s\n",
>+	br_debug(br, "br_fill_ifinfo event %d port %s master %s\n",
> 		     event, dev->name, br->dev->name);
> 
> 	nlh = nlmsg_put(skb, pid, seq, event, sizeof(*hdr), flags);


Acked-by: Nikolay Aleksandrov <razor@blackwall.org>

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

* Re: [PATCH net-next] net: bridge: correct debug message function name in br_fill_ifinfo
  2025-10-13 10:01 [PATCH net-next] net: bridge: correct debug message function name in br_fill_ifinfo Alok Tiwari
  2025-10-13 12:11 ` Simon Horman
  2025-10-13 12:22 ` Nikolay Aleksandrov
@ 2025-10-14 20:40 ` patchwork-bot+netdevbpf
  2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+netdevbpf @ 2025-10-14 20:40 UTC (permalink / raw)
  To: ALOK TIWARI
  Cc: idosch, razor, davem, edumazet, kuba, pabeni, horms, bridge,
	netdev

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Mon, 13 Oct 2025 03:01:16 -0700 you wrote:
> The debug message in br_fill_ifinfo() incorrectly refers to br_fill_info
> instead of the actual function name. Update it for clarity in debugging
> output.
> 
> Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
> ---
>  net/bridge/br_netlink.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Here is the summary with links:
  - [net-next] net: bridge: correct debug message function name in br_fill_ifinfo
    https://git.kernel.org/netdev/net-next/c/0513a3f97b96

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



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

end of thread, other threads:[~2025-10-14 20:40 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-13 10:01 [PATCH net-next] net: bridge: correct debug message function name in br_fill_ifinfo Alok Tiwari
2025-10-13 12:11 ` Simon Horman
2025-10-13 12:22 ` Nikolay Aleksandrov
2025-10-14 20:40 ` patchwork-bot+netdevbpf

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).