|WARNING| pw170460 [PATCH v2] uapi: update headers to v6.19
checkpatch at dpdk.org
checkpatch at dpdk.org
Thu Oct 1 11:07:01 CEST 2026
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/170460
_coding style issues_
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#259: FILE: kernel/linux/uapi/linux/stddef.h:40:
+#define __DECLARE_FLEX_ARRAY(T, member) \
+ T member[0]
BUT SEE:
do {} while (0) advice is over-stated in a few situations:
The more obvious case is macros, like MODULE_PARM_DESC, invoked at
file-scope, where C disallows code (it must be in functions). See
$exceptions if you have one to add by name.
More troublesome is declarative macros used at top of new scope,
like DECLARE_PER_CPU. These might just compile with a do-while-0
wrapper, but would be incorrect. Most of these are handled by
detecting struct,union,etc declaration primitives in $exceptions.
Theres also macros called inside an if (block), which "return" an
expression. These cannot do-while, and need a ({}) wrapper.
Enjoy this qualification while we work to improve our heuristics.
WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __aligned(16) over __attribute__((aligned(16)))
#311: FILE: kernel/linux/uapi/linux/types.h:12:
+typedef __signed__ __int128 __s128 __attribute__((aligned(16)));
WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __aligned(16) over __attribute__((aligned(16)))
#312: FILE: kernel/linux/uapi/linux/types.h:13:
+typedef unsigned __int128 __u128 __attribute__((aligned(16)));
WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __aligned(8) over __attribute__((aligned(8)))
#349: FILE: kernel/linux/uapi/linux/types.h:50:
+#define __aligned_u64 __u64 __attribute__((aligned(8)))
WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __aligned(8) over __attribute__((aligned(8)))
#350: FILE: kernel/linux/uapi/linux/types.h:51:
+#define __aligned_s64 __s64 __attribute__((aligned(8)))
WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __aligned(8) over __attribute__((aligned(8)))
#351: FILE: kernel/linux/uapi/linux/types.h:52:
+#define __aligned_be64 __be64 __attribute__((aligned(8)))
WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __aligned(8) over __attribute__((aligned(8)))
#352: FILE: kernel/linux/uapi/linux/types.h:53:
+#define __aligned_le64 __le64 __attribute__((aligned(8)))
total: 1 errors, 6 warnings, 301 lines checked
More information about the test-report
mailing list