]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
rwsem-support-optimistic-spinning-checkpatch-fixes
authorAndrew Morton <akpm@linux-foundation.org>
Sat, 17 May 2014 13:19:24 +0000 (23:19 +1000)
committerStephen Rothwell <sfr@canb.auug.org.au>
Wed, 21 May 2014 07:11:31 +0000 (17:11 +1000)
WARNING: line over 80 characters
#205: FILE: kernel/locking/rwsem-xadd.c:275:
+ old = cmpxchg(&sem->count, count, count + RWSEM_ACTIVE_WRITE_BIAS);

WARNING: line over 80 characters
#376: FILE: kernel/locking/rwsem-xadd.c:434:
+  * If there were already threads queued before us and there are no

WARNING: line over 80 characters
#377: FILE: kernel/locking/rwsem-xadd.c:435:
+  * active writers, the lock must be read owned; so we try to wake

total: 0 errors, 3 warnings, 417 lines checked

./patches/rwsem-support-optimistic-spinning.patch has style problems, please review.

If any of these errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.

Please run checkpatch prior to sending patches

Cc: Davidlohr Bueso <davidlohr@hp.com>
Cc: Tim Chen <tim.c.chen@linux.intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
kernel/locking/rwsem-xadd.c

index b562aca654cdcf0e80fed18297ec4da4a3bebfb6..1f99664b7f87f9fb522f94431a12b1ac5d964385 100644 (file)
@@ -431,9 +431,9 @@ struct rw_semaphore __sched *rwsem_down_write_failed(struct rw_semaphore *sem)
                count = ACCESS_ONCE(sem->count);
 
                /*
-                * If there were already threads queued before us and there are no
-                * active writers, the lock must be read owned; so we try to wake
-                * any read locks that were queued ahead of us.
+                * If there were already threads queued before us and there are
+                * no active writers, the lock must be read owned; so we try to
+                * wake any read locks that were queued ahead of us.
                 */
                if (count > RWSEM_WAITING_BIAS)
                        sem = __rwsem_do_wake(sem, RWSEM_WAKE_READERS);