From d266df35815fc53c237c5cf96ff1a6391e2bbe6f Mon Sep 17 00:00:00 2001 From: Michael Jeanson Date: Wed, 19 Dec 2018 11:44:57 -0500 Subject: [PATCH] Cleanup: missing sign compare fixes Signed-off-by: Michael Jeanson Signed-off-by: Mathieu Desnoyers --- tests/benchmark/test_perthreadlock.c | 2 +- tests/benchmark/test_urcu_gc.c | 4 ++-- tests/benchmark/test_urcu_qsbr_gc.c | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/benchmark/test_perthreadlock.c b/tests/benchmark/test_perthreadlock.c index 512b1fc..f02e00b 100644 --- a/tests/benchmark/test_perthreadlock.c +++ b/tests/benchmark/test_perthreadlock.c @@ -220,7 +220,7 @@ void *thr_writer(void *data) cmm_smp_mb(); for (;;) { - for (tidx = 0; tidx < nr_readers; tidx++) { + for (tidx = 0; tidx < (long)nr_readers; tidx++) { urcu_mutex_lock(&per_thread_lock[tidx].lock); } test_array.a = 0; diff --git a/tests/benchmark/test_urcu_gc.c b/tests/benchmark/test_urcu_gc.c index 1e50e4c..34cf141 100644 --- a/tests/benchmark/test_urcu_gc.c +++ b/tests/benchmark/test_urcu_gc.c @@ -55,7 +55,7 @@ static unsigned long wdelay; static struct test_array *test_rcu_pointer; -static unsigned int reclaim_batch = 1; +static long reclaim_batch = 1; struct reclaim_queue { void **queue; /* Beginning of queue */ @@ -449,7 +449,7 @@ int main(int argc, char **argv) printf("SUMMARY %-25s testdur %4lu nr_readers %3u rdur %6lu wdur %6lu " "nr_writers %3u " "wdelay %6lu nr_reads %12llu nr_writes %12llu nr_ops %12llu " - "batch %u\n", + "batch %ld\n", argv[0], duration, nr_readers, rduration, wduration, nr_writers, wdelay, tot_reads, tot_writes, tot_reads + tot_writes, reclaim_batch); diff --git a/tests/benchmark/test_urcu_qsbr_gc.c b/tests/benchmark/test_urcu_qsbr_gc.c index 11f0359..26bc352 100644 --- a/tests/benchmark/test_urcu_qsbr_gc.c +++ b/tests/benchmark/test_urcu_qsbr_gc.c @@ -57,7 +57,7 @@ static unsigned long duration; /* read-side C.S. duration, in loops */ static unsigned long rduration; -static unsigned int reclaim_batch = 1; +static long reclaim_batch = 1; struct reclaim_queue { void **queue; /* Beginning of queue */ @@ -450,7 +450,7 @@ int main(int argc, char **argv) printf("SUMMARY %-25s testdur %4lu nr_readers %3u rdur %6lu wdur %6lu " "nr_writers %3u " "wdelay %6lu nr_reads %12llu nr_writes %12llu nr_ops %12llu " - "batch %u\n", + "batch %ld\n", argv[0], duration, nr_readers, rduration, wduration, nr_writers, wdelay, tot_reads, tot_writes, tot_reads + tot_writes, reclaim_batch); -- 2.34.1