test/rwlock: fix spelling and missing whitespace

Message ID 20210114164837.29217-1-stephen@networkplumber.org (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series test/rwlock: fix spelling and missing whitespace |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-broadcom-Functional success Functional Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-mellanox-Functional success Functional Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-testing success Testing PASS

Commit Message

Stephen Hemminger Jan. 14, 2021, 4:48 p.m. UTC
  Trivial fix to for spelling errors and incorrect spacing.
No change to any built code.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 app/test/test_rwlock.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
  

Comments

Thomas Monjalon Jan. 17, 2021, 5:33 p.m. UTC | #1
14/01/2021 17:48, Stephen Hemminger:
> Trivial fix to for spelling errors and incorrect spacing.
> No change to any built code.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
> -	/* restore by bytes in reverese order */
> +	/* restore by bytes in reveres order */

This comment may require one more fix :)

Applied with above fixed, thanks
  

Patch

diff --git a/app/test/test_rwlock.c b/app/test/test_rwlock.c
index 701187f39884..c4876018ce35 100644
--- a/app/test/test_rwlock.c
+++ b/app/test/test_rwlock.c
@@ -46,6 +46,7 @@  enum {
 static struct {
 	rte_rwlock_t lock;
 	uint64_t tick;
+
 	volatile union {
 		uint8_t u8[RTE_CACHE_LINE_SIZE];
 		uint64_t u64[RTE_CACHE_LINE_SIZE / sizeof(uint64_t)];
@@ -182,7 +183,7 @@  rwlock_test1(void)
 	int i;
 
 	rte_rwlock_init(&sl);
-	for (i=0; i<RTE_MAX_LCORE; i++)
+	for (i = 0; i < RTE_MAX_LCORE; i++)
 		rte_rwlock_init(&sl_tab[i]);
 
 	rte_rwlock_write_lock(&sl);
@@ -252,7 +253,7 @@  try_write(uint32_t lc)
 	if (rc != 0)
 		return rc;
 
-	/* update by bytes in reverese order */
+	/* update by bytes in reverse order */
 	for (i = RTE_DIM(try_rwlock_data.data.u8); i-- != 0; ) {
 
 		/* race condition occurred, lock doesn't work properly */
@@ -269,7 +270,7 @@  try_write(uint32_t lc)
 		try_rwlock_data.data.u8[i] = v;
 	}
 
-	/* restore by bytes in reverese order */
+	/* restore by bytes in reveres order */
 	for (i = RTE_DIM(try_rwlock_data.data.u8); i-- != 0; ) {
 
 		/* race condition occurred, lock doesn't work properly */
@@ -461,7 +462,7 @@  try_rwlock_test_rda(void)
 {
 	try_test_reset();
 
-	/* start read test on all avaialble lcores */
+	/* start read test on all available lcores */
 	rte_eal_mp_remote_launch(try_read_lcore, NULL, CALL_MAIN);
 	rte_eal_mp_wait_lcore();