automatic DPDK test reports
 help / color / mirror / Atom feed
* [dpdk-test-report] |WARNING| pw56880 [PATCH v5 1/3] eal/arm64: add 128-bit atomic compare exchange
       [not found] <1563800774-2710-1-git-send-email-phil.yang@arm.com>
@ 2019-07-22 13:07 ` checkpatch
  0 siblings, 0 replies; only message in thread
From: checkpatch @ 2019-07-22 13:07 UTC (permalink / raw)
  To: test-report; +Cc: Phil Yang

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/56880

_coding style issues_


WARNING:LINE_CONTINUATIONS: Avoid unnecessary line continuations
#156: FILE: lib/librte_eal/common/include/arch/arm/rte_atomic_64.h:65:
+	 */                                                                     \

WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#179: FILE: lib/librte_eal/common/include/arch/arm/rte_atomic_64.h:88:
+#define __ATOMIC128_LDX_OP(ldx_op_name, op_string)                          \
+static inline rte_int128_t                                                  \
+ldx_op_name(const rte_int128_t *src)                                        \
+{                                                                           \
+	rte_int128_t ret;                                                       \
+	asm volatile(                                                           \
+			op_string " %0, %1, %2"                                         \
+			: "=&r" (ret.val[0]),                                           \
+			  "=&r" (ret.val[1])                                            \
+			: "Q" (src->val[0])                                             \
+			: "memory");                                                    \
+	return ret;                                                             \
+}

WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#196: FILE: lib/librte_eal/common/include/arch/arm/rte_atomic_64.h:105:
+#define __ATOMIC128_STX_OP(stx_op_name, op_string)                          \
+static inline uint32_t                                                      \
+stx_op_name(rte_int128_t *dst, const rte_int128_t src)                      \
+{                                                                           \
+	uint32_t ret;                                                           \
+	asm volatile(                                                           \
+			op_string " %w0, %1, %2, %3"                                    \
+			: "=&r" (ret)                                                   \
+			: "r" (src.val[0]),                                             \
+			  "r" (src.val[1]),                                             \
+			  "Q" (dst->val[0])                                             \
+			: "memory");                                                    \
+	/* Return 0 on success, 1 on failure */                                 \
+	return ret;                                                             \
+}

total: 0 errors, 3 warnings, 261 lines checked
__rte_experimental must appear alone on the line immediately preceding the return type of a function.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2019-07-22 13:07 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1563800774-2710-1-git-send-email-phil.yang@arm.com>
2019-07-22 13:07 ` [dpdk-test-report] |WARNING| pw56880 [PATCH v5 1/3] eal/arm64: add 128-bit atomic compare exchange checkpatch

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).