From d60bee6b920a33b808e2d25f63a892ad03f9be20 Mon Sep 17 00:00:00 2001 From: Andrew Morton Date: Tue, 13 Sep 2022 14:45:19 -0700 Subject: [PATCH] ipc-msg-mitigate-the-lock-contention-with-percpu-counter-checkpatch-fixes WARNING: labels should not be indented #158: FILE: ipc/msg.c:1319: + fail_msg_hdrs: WARNING: labels should not be indented #160: FILE: ipc/msg.c:1321: + fail_msg_bytes: ERROR: space required after that ';' (ctx:VxV) #203: FILE: ipc/util.h:75: +static inline int msg_init_ns(struct ipc_namespace *ns) { return 0;} ^ total: 1 errors, 2 warnings, 144 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. ./patches/ipc-msg-mitigate-the-lock-contention-with-percpu-counter.patch has style problems, please review. NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. Please run checkpatch prior to sending patches Cc: Alexander Mikhalitsyn Cc: Alexey Gladkov Cc: Christoph Lameter Cc: Davidlohr Bueso Cc: Dennis Zhou Cc: "Eric W . Biederman" Cc: Jiebin Sun Cc: Manfred Spraul Cc: Shakeel Butt Cc: Tejun Heo Cc: Tim Chen Cc: Vasily Averin Signed-off-by: Andrew Morton --- ipc/msg.c | 8 ++++---- ipc/util.h | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/ipc/msg.c b/ipc/msg.c index f2bb4c193ecf33..9fb58de557fba5 100644 --- a/ipc/msg.c +++ b/ipc/msg.c @@ -1316,10 +1316,10 @@ int msg_init_ns(struct ipc_namespace *ns) ipc_init_ids(&ns->ids[IPC_MSG_IDS]); return 0; - fail_msg_hdrs: - percpu_counter_destroy(&ns->percpu_msg_bytes); - fail_msg_bytes: - return ret; +fail_msg_hdrs: + percpu_counter_destroy(&ns->percpu_msg_bytes); +fail_msg_bytes: + return ret; } #ifdef CONFIG_IPC_NS diff --git a/ipc/util.h b/ipc/util.h index 1b0086c6346f26..b2906e36653942 100644 --- a/ipc/util.h +++ b/ipc/util.h @@ -72,7 +72,7 @@ void msg_exit_ns(struct ipc_namespace *ns); void shm_exit_ns(struct ipc_namespace *ns); #else static inline void sem_init_ns(struct ipc_namespace *ns) { } -static inline int msg_init_ns(struct ipc_namespace *ns) { return 0;} +static inline int msg_init_ns(struct ipc_namespace *ns) { return 0; } static inline void shm_init_ns(struct ipc_namespace *ns) { } static inline void sem_exit_ns(struct ipc_namespace *ns) { }