From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by dpdk.org (Postfix, from userid 1017) id 25E5F1B19D; Thu, 21 Sep 2017 10:37:44 +0200 (CEST) In-Reply-To: References: To: test-report@dpdk.org Cc: Gowrishankar Message-Id: <20170921083744.25E5F1B19D@dpdk.org> Date: Thu, 21 Sep 2017 10:37:44 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw29052 [PATCH] examples/l3fwd: optimised packet processing on powerpc 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: , X-List-Received-Date: Thu, 21 Sep 2017 08:37:44 -0000 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/29052 _coding style issues_ ERROR:SPACING: space required after that ',' (ctx:VxV) #108: FILE: examples/l3fwd/l3fwd_altivec.h:74: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #108: FILE: examples/l3fwd/l3fwd_altivec.h:74: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #108: FILE: examples/l3fwd/l3fwd_altivec.h:74: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #108: FILE: examples/l3fwd/l3fwd_altivec.h:74: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #108: FILE: examples/l3fwd/l3fwd_altivec.h:74: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #108: FILE: examples/l3fwd/l3fwd_altivec.h:74: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #108: FILE: examples/l3fwd/l3fwd_altivec.h:74: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required before the open brace '{' #108: FILE: examples/l3fwd/l3fwd_altivec.h:74: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ERROR:SPACING: space required after that ',' (ctx:VxV) #113: FILE: examples/l3fwd/l3fwd_altivec.h:79: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #113: FILE: examples/l3fwd/l3fwd_altivec.h:79: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #113: FILE: examples/l3fwd/l3fwd_altivec.h:79: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #113: FILE: examples/l3fwd/l3fwd_altivec.h:79: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #113: FILE: examples/l3fwd/l3fwd_altivec.h:79: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #113: FILE: examples/l3fwd/l3fwd_altivec.h:79: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #113: FILE: examples/l3fwd/l3fwd_altivec.h:79: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required before the open brace '{' #113: FILE: examples/l3fwd/l3fwd_altivec.h:79: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ERROR:SPACING: space required after that ',' (ctx:VxV) #118: FILE: examples/l3fwd/l3fwd_altivec.h:84: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #118: FILE: examples/l3fwd/l3fwd_altivec.h:84: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #118: FILE: examples/l3fwd/l3fwd_altivec.h:84: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #118: FILE: examples/l3fwd/l3fwd_altivec.h:84: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #118: FILE: examples/l3fwd/l3fwd_altivec.h:84: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #118: FILE: examples/l3fwd/l3fwd_altivec.h:84: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #118: FILE: examples/l3fwd/l3fwd_altivec.h:84: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required before the open brace '{' #118: FILE: examples/l3fwd/l3fwd_altivec.h:84: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ERROR:SPACING: space required after that ',' (ctx:VxV) #123: FILE: examples/l3fwd/l3fwd_altivec.h:89: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #123: FILE: examples/l3fwd/l3fwd_altivec.h:89: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #123: FILE: examples/l3fwd/l3fwd_altivec.h:89: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #123: FILE: examples/l3fwd/l3fwd_altivec.h:89: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #123: FILE: examples/l3fwd/l3fwd_altivec.h:89: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #123: FILE: examples/l3fwd/l3fwd_altivec.h:89: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #123: FILE: examples/l3fwd/l3fwd_altivec.h:89: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required before the open brace '{' #123: FILE: examples/l3fwd/l3fwd_altivec.h:89: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); WARNING:LONG_LINE: line over 90 characters #149: FILE: examples/l3fwd/l3fwd_altivec.h:115: +port_groupx4(uint16_t pn[FWDSTEP + 1], uint16_t *lp, vector unsigned short dp1, vector unsigned short dp2) ERROR:SPACING: space required after that ',' (ctx:VxV) #197: FILE: examples/l3fwd/l3fwd_altivec.h:163: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #197: FILE: examples/l3fwd/l3fwd_altivec.h:163: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #197: FILE: examples/l3fwd/l3fwd_altivec.h:163: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #197: FILE: examples/l3fwd/l3fwd_altivec.h:163: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #197: FILE: examples/l3fwd/l3fwd_altivec.h:163: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #197: FILE: examples/l3fwd/l3fwd_altivec.h:163: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #197: FILE: examples/l3fwd/l3fwd_altivec.h:163: + (vector unsigned short){0,0,0,0,0,0,0xffff,0xffff}); ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #247: FILE: examples/l3fwd/l3fwd_altivec.h:213: + 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, ^ WARNING:LONG_LINE: line over 90 characters #256: FILE: examples/l3fwd/l3fwd_altivec.h:222: + dp2 = vec_perm(dp1, (vector unsigned short){}, (vector unsigned char){0xf9}); WARNING:LONG_LINE: line over 90 characters #433: FILE: examples/l3fwd/l3fwd_lpm_altivec.h:92: + const vector unsigned char bswap_mask = (vector unsigned char){3, 2, 1, 0, 7, 6, 5, 4, WARNING:LONG_LINE: line over 90 characters #437: FILE: examples/l3fwd/l3fwd_lpm_altivec.h:96: + dip = (vector unsigned int)vec_perm(*(vector unsigned char *)&dip, (vector unsigned char){}, bswap_mask); WARNING:LONG_LINE: line over 90 characters #475: FILE: examples/l3fwd/l3fwd_lpm_altivec.h:134: + ipv4_flag[j / FWDSTEP], portid, &pkts_burst[j], &dst_port[j]); total: 53 errors, 5 warnings, 450 lines checked