From: Mark Charlebois <charl...@gmail.com> This code makes a compile time type check that is optimized away. Clang complains that it generates an unused function.
I believe GCC won't complain for a static inline fuction but would if it was just a static function. Adding the unused attribute to the function declaration removes the warning. This code works for both gcc and clang. Signed-off-by: Mark Charlebois <charl...@gmail.com> Signed-off-by: Behan Webster <beh...@converseincode.com> --- include/linux/moduleparam.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/moduleparam.h b/include/linux/moduleparam.h index c3eb102..5ce1f67 100644 --- a/include/linux/moduleparam.h +++ b/include/linux/moduleparam.h @@ -346,6 +346,7 @@ static inline void destroy_params(const struct kernel_param *params, /* The macros to do compile-time type checking stolen from Jakub Jelinek, who IIRC came up with this idea for the 2.4 module init code. */ #define __param_check(name, p, type) \ + static inline type *__check_##name(void) __attribute__ ((unused)); \ static inline type *__check_##name(void) { return(p); } extern struct kernel_param_ops param_ops_byte; -- 1.8.3.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/