[net-next PATCH 1/1] tipc: add back tipc prefix to log messages

Subsystems: networking [general], the rest, tipc network layer

STALE3148d REVIEWED: 1 (0M)

1 review trailer.

3 messages, 2 authors, 2018-01-15 · open the first message on its own page

[net-next PATCH 1/1] tipc: add back tipc prefix to log messages

From: John Thompson <hidden>
Date: 2018-01-09 19:50:24

The tipc prefix to log messages generated by tipc was
removed in commit 07f6c4bc0 ("tipc: convert tipc reference
table to use generic rhashtable").

This is still a useful prefix and so add it back.

Acked-by: Jon Maloy <redacted>
Signed-off-by: John Thompson <redacted>
---
 net/tipc/core.c | 2 --
 net/tipc/core.h | 2 ++
 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/tipc/core.c b/net/tipc/core.c
index 0b982d048fb9..4cd9e57446ec 100644
--- a/net/tipc/core.c
+++ b/net/tipc/core.c
@@ -34,8 +34,6 @@
  * POSSIBILITY OF SUCH DAMAGE.
  */

-#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
-
 #include "core.h"
 #include "name_table.h"
 #include "subscr.h"
diff --git a/net/tipc/core.h b/net/tipc/core.h
index 964342689f2c..f89f9a3c18c2 100644
--- a/net/tipc/core.h
+++ b/net/tipc/core.h
@@ -37,6 +37,8 @@
 #ifndef _TIPC_CORE_H
 #define _TIPC_CORE_H

+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+
 #include <linux/tipc.h>
 #include <linux/tipc_config.h>
 #include <linux/tipc_netlink.h>
-- 
2.15.0
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot

Re: [net-next PATCH 1/1] tipc: add back tipc prefix to log messages

From: David Miller <davem@davemloft.net>
Date: 2018-01-15 18:42:47

From: John Thompson <redacted>
Date: Wed, 10 Jan 2018 08:50:24 +1300
The tipc prefix to log messages generated by tipc was
removed in commit 07f6c4bc0 ("tipc: convert tipc reference
table to use generic rhashtable").

This is still a useful prefix and so add it back.

Acked-by: Jon Maloy <redacted>
Signed-off-by: John Thompson <redacted>
Applied, thank you.

Re: [net-next PATCH 1/1] tipc: add back tipc prefix to log messages

From: David Miller <davem@davemloft.net>
Date: 2018-01-15 18:43:41

From: David Miller <davem@davemloft.net>
Date: Mon, 15 Jan 2018 13:42:45 -0500 (EST)
From: John Thompson <redacted>
Date: Wed, 10 Jan 2018 08:50:24 +1300
quoted
The tipc prefix to log messages generated by tipc was
removed in commit 07f6c4bc0 ("tipc: convert tipc reference
table to use generic rhashtable").

This is still a useful prefix and so add it back.

Acked-by: Jon Maloy <redacted>
Signed-off-by: John Thompson <redacted>
Applied, thank you.
Actually, this breaks the build, reverted:

In file included from net/tipc/addr.h:44:0,
                 from net/tipc/addr.c:38:
net/tipc/core.h:40:0: warning: "pr_fmt" redefined
 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
In file included from ./include/linux/kernel.h:14:0,
                 from net/tipc/addr.c:37:
./include/linux/printk.h:285:0: note: this is the location of the previous definition
 #define pr_fmt(fmt) fmt
 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help