From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 7B1BEA0579 for ; Wed, 18 Mar 2020 20:04:54 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id E091C1C06D; Wed, 18 Mar 2020 20:04:51 +0100 (CET) Received: by dpdk.org (Postfix, from userid 1017) id 5E06C1C0CA; Wed, 18 Mar 2020 20:04:47 +0100 (CET) In-Reply-To: <20200318190241.3150971-13-jerinj@marvell.com> References: <20200318190241.3150971-13-jerinj@marvell.com> To: test-report@dpdk.org Cc: Message-Id: <20200318190447.5E06C1C0CA@dpdk.org> Date: Wed, 18 Mar 2020 20:04:47 +0100 (CET) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw66861 [PATCH v1 12/32] eal/trace: implement registration payload X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/66861 _coding style issues_ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #178: FILE: lib/librte_eal/common/include/rte_trace_register.h:21: +#define __rte_trace_emit_header_generic(t)\ + RTE_PER_LCORE(trace_point_sz) = __RTE_TRACE_EVENT_HEADER_SZ ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #183: FILE: lib/librte_eal/common/include/rte_trace_register.h:26: +#define rte_trace_ctf_u64(in)\ + RTE_BUILD_BUG_ON(sizeof(uint64_t) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(uint64_t), RTE_STR(in), "uint64_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #186: FILE: lib/librte_eal/common/include/rte_trace_register.h:29: +#define rte_trace_ctf_i64(in)\ + RTE_BUILD_BUG_ON(sizeof(int64_t) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(int64_t), RTE_STR(in), "int64_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #189: FILE: lib/librte_eal/common/include/rte_trace_register.h:32: +#define rte_trace_ctf_u32(in)\ + RTE_BUILD_BUG_ON(sizeof(uint32_t) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(uint32_t), RTE_STR(in), "uint32_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #192: FILE: lib/librte_eal/common/include/rte_trace_register.h:35: +#define rte_trace_ctf_i32(in)\ + RTE_BUILD_BUG_ON(sizeof(int32_t) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(int32_t), RTE_STR(in), "int32_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #195: FILE: lib/librte_eal/common/include/rte_trace_register.h:38: +#define rte_trace_ctf_u16(in)\ + RTE_BUILD_BUG_ON(sizeof(uint16_t) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(uint16_t), RTE_STR(in), "uint16_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #198: FILE: lib/librte_eal/common/include/rte_trace_register.h:41: +#define rte_trace_ctf_i16(in)\ + RTE_BUILD_BUG_ON(sizeof(int16_t) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(int16_t), RTE_STR(in), "int16_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #201: FILE: lib/librte_eal/common/include/rte_trace_register.h:44: +#define rte_trace_ctf_u8(in)\ + RTE_BUILD_BUG_ON(sizeof(uint8_t) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(uint8_t), RTE_STR(in), "uint8_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #204: FILE: lib/librte_eal/common/include/rte_trace_register.h:47: +#define rte_trace_ctf_i8(in)\ + RTE_BUILD_BUG_ON(sizeof(int8_t) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(int8_t), RTE_STR(in), "int8_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #207: FILE: lib/librte_eal/common/include/rte_trace_register.h:50: +#define rte_trace_ctf_int(in)\ + RTE_BUILD_BUG_ON(sizeof(int) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(int), RTE_STR(in), "int32_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #210: FILE: lib/librte_eal/common/include/rte_trace_register.h:53: +#define rte_trace_ctf_long(in)\ + RTE_BUILD_BUG_ON(sizeof(long) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(long), RTE_STR(in), "long") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #213: FILE: lib/librte_eal/common/include/rte_trace_register.h:56: +#define rte_trace_ctf_float(in)\ + RTE_BUILD_BUG_ON(sizeof(float) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(float), RTE_STR(in), "float") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #216: FILE: lib/librte_eal/common/include/rte_trace_register.h:59: +#define rte_trace_ctf_double(in)\ + RTE_BUILD_BUG_ON(sizeof(double) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(double), RTE_STR(in), "double") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #219: FILE: lib/librte_eal/common/include/rte_trace_register.h:62: +#define rte_trace_ctf_ptr(in)\ + RTE_BUILD_BUG_ON(sizeof(void *) != sizeof(typeof(in)));\ + __rte_trace_emit_ctf_field(sizeof(void *), RTE_STR(in), "uintptr_t") ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #222: FILE: lib/librte_eal/common/include/rte_trace_register.h:65: +#define rte_trace_ctf_string(in)\ + RTE_SET_USED(in);\ + __rte_trace_emit_ctf_field(__RTE_TRACE_EMIT_STRING_LEN_MAX,\ + RTE_STR(in)"[32]", "string_bounded_t") total: 15 errors, 0 warnings, 126 lines checked