ipc-msg-avoid-waking-sender-upon-full-queue-checkpatch-fixes

WARNING: braces {} are not necessary for single statement blocks
#134: FILE: ipc/msg.c:679:
+		if (msg_fits_inqueue(msq, msgsz)) {
 			break;
 		}

total: 0 errors, 1 warnings, 118 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-avoid-waking-sender-upon-full-queue.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: Davidlohr Bueso <dave@stgolabs.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

diff --git a/ipc/msg.c b/ipc/msg.c
index 1c455df759387502fc096d0a4d199856f50f619c..1ce3d180c58ffa4f50a078c98f73eb418b81ac6e 100644 (file)
--- a/ipc/msg.c
+++ b/ipc/msg.c
@@ -676,9 +676,8 @@ long do_msgsnd(int msqid, long mtype, void __user *mtext,
                if (err)
                        goto out_unlock0;
 
-               if (msg_fits_inqueue(msq, msgsz)) {
+               if (msg_fits_inqueue(msq, msgsz))
                        break;
-               }
 
                /* queue full, wait: */
                if (msgflg & IPC_NOWAIT) {