[dpdk-test-report] |WARNING| pw86376 [RFC] eal: add fair reader writer lock

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Jan 12 07:05:58 CET 2021


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/86376

_coding style issues_


ERROR:SPACING: spaces required around that '=' (ctx:VxV)
#340: FILE: app/test/test_fair_rwlock.c:185:
+	for (i=0; i<RTE_MAX_LCORE; i++)
 	      ^

ERROR:SPACING: spaces required around that '<' (ctx:VxV)
#340: FILE: app/test/test_fair_rwlock.c:185:
+	for (i=0; i<RTE_MAX_LCORE; i++)
 	           ^

WARNING:TYPO_SPELLING: 'reverese' may be misspelled - perhaps 'reverse'?
#410: FILE: app/test/test_fair_rwlock.c:255:
+	/* update by bytes in reverese order */

WARNING:TYPO_SPELLING: 'reverese' may be misspelled - perhaps 'reverse'?
#427: FILE: app/test/test_fair_rwlock.c:272:
+	/* restore by bytes in reverese order */

WARNING:TYPO_SPELLING: 'avaialble' may be misspelled - perhaps 'available'?
#619: FILE: app/test/test_fair_rwlock.c:464:
+	/* start read test on all avaialble lcores */

ERROR:CODE_INDENT: code indent should use tabs where possible
#799: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:31:
+        rte_ticketlock_t writer;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#799: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:31:
+        rte_ticketlock_t writer;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#800: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:32:
+        uint16_t readers;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#800: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:32:
+        uint16_t readers;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#821: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:53:
+        rte_ticketlock_init(&rwl->writer);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#821: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:53:
+        rte_ticketlock_init(&rwl->writer);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#822: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:54:
+        rwl->readers = 0;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#822: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:54:
+        rwl->readers = 0;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#839: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:71:
+        __atomic_fetch_add(&rwl->readers, 1, __ATOMIC_ACQUIRE);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#839: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:71:
+        __atomic_fetch_add(&rwl->readers, 1, __ATOMIC_ACQUIRE);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#841: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:73:
+        /* If a writer raced in and got the lock, we need to back out. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#842: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:74:
+        if (rte_ticketlock_is_locked(&rwl->writer)) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#842: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:74:
+        if (rte_ticketlock_is_locked(&rwl->writer)) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#843: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:75:
+                /* Drop our reference so the Writer can continue */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#844: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:76:
+                __atomic_sub_fetch(&rwl->readers, 1, __ATOMIC_RELEASE);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#844: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:76:
+                __atomic_sub_fetch(&rwl->readers, 1, __ATOMIC_RELEASE);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#846: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:78:
+                /* Wait for the Writer to finish then get our new reference */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#847: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:79:
+                rte_ticketlock_lock(&rwl->writer);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#847: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:79:
+                rte_ticketlock_lock(&rwl->writer);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#848: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:80:
+                __atomic_add_fetch(&rwl->readers, 1, __ATOMIC_ACQUIRE);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#848: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:80:
+                __atomic_add_fetch(&rwl->readers, 1, __ATOMIC_ACQUIRE);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#849: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:81:
+                rte_ticketlock_unlock(&rwl->writer);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#849: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:81:
+                rte_ticketlock_unlock(&rwl->writer);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#850: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:82:
+        }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#850: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:82:
+        }$

ERROR:CODE_INDENT: code indent should use tabs where possible
#871: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:103:
+        /* Get our reference count */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#872: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:104:
+        __atomic_fetch_add(&rwl->readers, 1, __ATOMIC_ACQUIRE);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#872: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:104:
+        __atomic_fetch_add(&rwl->readers, 1, __ATOMIC_ACQUIRE);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#874: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:106:
+        /* If a writer has the lock, we need to back out. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#875: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:107:
+        if (rte_ticketlock_is_locked(&rwl->writer)) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#875: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:107:
+        if (rte_ticketlock_is_locked(&rwl->writer)) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#876: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:108:
+                __atomic_sub_fetch(&rwl->readers, 1, __ATOMIC_RELEASE);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#876: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:108:
+                __atomic_sub_fetch(&rwl->readers, 1, __ATOMIC_RELEASE);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#877: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:109:
+                return -EBUSY;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#877: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:109:
+                return -EBUSY;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#878: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:110:
+        }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#878: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:110:
+        }$

ERROR:CODE_INDENT: code indent should use tabs where possible
#896: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:128:
+        __atomic_fetch_sub(&rwl->readers, 1, __ATOMIC_RELEASE);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#896: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:128:
+        __atomic_fetch_sub(&rwl->readers, 1, __ATOMIC_RELEASE);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#912: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:144:
+        /* Wait for our turn to be writer */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#913: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:145:
+        rte_ticketlock_lock(&rwl->writer);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#913: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:145:
+        rte_ticketlock_lock(&rwl->writer);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#915: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:147:
+        /* Wait for all outstanding readers */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#938: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:170:
+        if (!rte_ticketlock_trylock(&rwl->writer))$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#938: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:170:
+        if (!rte_ticketlock_trylock(&rwl->writer))$

ERROR:CODE_INDENT: code indent should use tabs where possible
#939: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:171:
+                return -EBUSY;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#939: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:171:
+                return -EBUSY;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#943: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:175:
+                rte_ticketlock_unlock(&rwl->writer);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#943: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:175:
+                rte_ticketlock_unlock(&rwl->writer);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#944: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:176:
+                return -EBUSY;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#944: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:176:
+                return -EBUSY;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#945: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:177:
+        }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#945: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:177:
+        }$

ERROR:CODE_INDENT: code indent should use tabs where possible
#963: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:195:
+        /* Unblock readers */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#964: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:196:
+        rte_ticketlock_unlock(&rwl->writer);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#964: FILE: lib/librte_eal/include/generic/rte_fair_rwlock.h:196:
+        rte_ticketlock_unlock(&rwl->writer);$

total: 34 errors, 27 warnings, 880 lines checked
Warning in app/test/test_fair_rwlock.c:
Using rte_atomicNN_xxx


More information about the test-report mailing list