[PATCH] 2.5.54 kill module.h compiler warnings

Mikael Pettersson (mikpe@csd.uu.se)
Thu, 2 Jan 2003 16:57:19 +0100 (MET)


Rusty,

Compiling kernel 2.5.54 with CONFIG_MODULES=n results in
tons and tons of the following warnings:

include/linux/module.h:317: warning: statement with no effect
include/linux/module.h:353: warning: statement with no effect

patch-2.5.54 changed *MOD_INC_USE_COUNT from macros to
__deprecated functions, but also dropped the (void) casts
in front of the try_module_get() calls. Without modules,
try_module_get() is the constant 1, hence the warnings.

The patch below silences the warnings by adding back the
missing (void) casts. Works for me.

/Mikael

--- linux-2.5.54/include/linux/module.h.~1~ 2003-01-02 14:27:56.000000000 +0100
+++ linux-2.5.54/include/linux/module.h 2003-01-02 16:39:49.000000000 +0100
@@ -314,7 +314,7 @@
/*
* Yes, we ignore the retval here, that's why it's deprecated.
*/
- try_module_get(module);
+ (void)try_module_get(module);
}

static inline void __deprecated __MOD_DEC_USE_COUNT(struct module *module)
@@ -350,7 +350,7 @@
local_inc(&module->ref[get_cpu()].count);
put_cpu();
#else
- try_module_get(module);
+ (void)try_module_get(module);
#endif
}
#define MOD_INC_USE_COUNT \
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/