Replace ARGSUSED with proper warnings

Passes (except for one zstd line) Linux on clang 13 and GCC 10.2.1-6 and FreeBSD 13 (clang 11) with -Wall -Wextra -Wno-sign-compare -Wno-missing-field-initializers -Werror Module sources with ARGSUSED also grew a -Werror=unused-parameter Signed-off-by: Ahelenia Ziemiańska <[email protected]> Closes openzfs#12167
nabijaczleweli · Jun 6, 2021 · 3cf54c9 · 3cf54c9
1 parent 0bf2d24
commit 3cf54c9
Show file tree

Hide file tree

Showing 218 changed files with 1,779 additions and 952 deletions.
diff --git a/cmd/raidz_test/raidz_test.c b/cmd/raidz_test/raidz_test.c
@@ -71,12 +71,13 @@ static void print_opts(raidz_test_opts_t *opts, boolean_t force)
 {
 	char *verbose;
 	switch (opts->rto_v) {
-		case 0:
+		case D_ALL:
 			verbose = "no";
 			break;
-		case 1:
+		case D_INFO:
 			verbose = "info";
 			break;
+		case D_DEBUG:
 		default:
 			verbose = "debug";
 			break;
@@ -266,6 +267,7 @@ cmp_data(raidz_test_opts_t *opts, raidz_map_t *rm)
 static int
 init_rand(void *data, size_t size, void *private)
 {
+	(void) private;
 	int i;
 	int *dst = (int *)data;
 

diff --git a/cmd/raidz_test/raidz_test.h b/cmd/raidz_test/raidz_test.h
@@ -92,13 +92,13 @@ static inline size_t ilog2(size_t a)
 
 #define	LOG(lvl, a...)				\
 {						\
-	if (rto_opts.rto_v >= lvl)		\
+	if ((rto_opts.rto_v + 1) >= (lvl + 1))	\
 		(void) fprintf(stdout, a);	\
 }						\
 
 #define	LOG_OPT(lvl, opt, a...)			\
 {						\
-	if (opt->rto_v >= lvl)			\
+	if ((opt->rto_v + 1) >= (lvl + 1))	\
 		(void) fprintf(stdout, a);	\
 }						\