From: Andrew Morton WARNING: line over 80 characters #25: FILE: include/linux/sem.h:96: + struct list_head sem_pending; /* pending operations to be processed */ ERROR: space required before the open parenthesis '(' #105: FILE: ipc/sem.c:409: + while(&q->list != &sma->sem_pending) { WARNING: line over 80 characters #135: FILE: ipc/sem.c:436: + n = list_entry(sma->sem_pending.next, struct sem_queue, list); WARNING: line over 80 characters #137: FILE: ipc/sem.c:438: + n = list_entry(q->list.next, struct sem_queue, list); total: 1 errors, 3 warnings, 206 lines checked ./patches/ipc-semc-convert-sem_arraysem_pending-to-struct-list_head.patch has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Please run checkpatch prior to sending patches Cc: Manfred Spraul Cc: Nadia Derbey Cc: Pierre Peiffer Signed-off-by: Andrew Morton --- ipc/sem.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff -puN ipc/sem.c~ipc-semc-convert-sem_arraysem_pending-to-struct-list_head-checkpatch-fixes ipc/sem.c --- a/ipc/sem.c~ipc-semc-convert-sem_arraysem_pending-to-struct-list_head-checkpatch-fixes +++ a/ipc/sem.c @@ -406,7 +406,7 @@ static void update_queue (struct sem_arr struct sem_queue * q; q = list_entry(sma->sem_pending.next, struct sem_queue, list); - while(&q->list != &sma->sem_pending) { + while (&q->list != &sma->sem_pending) { error = try_atomic_semop(sma, q->sops, q->nsops, q->undo, q->pid); @@ -433,9 +433,11 @@ static void update_queue (struct sem_arr */ if (q->alter) { list_del(&q->list); - n = list_entry(sma->sem_pending.next, struct sem_queue, list); + n = list_entry(sma->sem_pending.next, + struct sem_queue, list); } else { - n = list_entry(q->list.next, struct sem_queue, list); + n = list_entry(q->list.next, struct sem_queue, + list); list_del(&q->list); } _