On Tue, Apr 18, 2023 at 08:26:11PM +0800, Zhu Yanjun wrote: > From: Zhu Yanjun <yanjun.zhu@xxxxxxxxx> > > Add the function names to the pr_ logs. As such, if some bugs occur, > with function names, it is easy to locate the bugs. > > Signed-off-by: Zhu Yanjun <yanjun.zhu@xxxxxxxxx> > --- > drivers/infiniband/sw/rxe/rxe.h | 2 +- > drivers/infiniband/sw/rxe/rxe_queue.h | 16 ++++------------ > 2 files changed, 5 insertions(+), 13 deletions(-) After you delete the warn_on's there is very little left: rivers/infiniband/sw/rxe/rxe.c: pr_err("rxe creation allowed on top of a real device only\n"); drivers/infiniband/sw/rxe/rxe.c: pr_info("loaded\n"); drivers/infiniband/sw/rxe/rxe.c: pr_info("unloaded\n"); drivers/infiniband/sw/rxe/rxe_net.c: pr_err("Failed to create IPv4 UDP tunnel\n"); drivers/infiniband/sw/rxe/rxe_net.c: pr_warn("IPv6 is not supported, can not create a UDPv6 socket\n"); drivers/infiniband/sw/rxe/rxe_net.c: pr_err("Failed to create IPv6 UDP tunnel\n"); drivers/infiniband/sw/rxe/rxe_net.c: pr_err("Failed to register netdev notifier\n"); It is not exactly mysterious where these come from? Why do we need the function name? Jason