From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <checkpatch@dpdk.org>
Received: by dpdk.org (Postfix, from userid 1017)
 id 0ADAE5687; Thu, 24 Nov 2016 09:58:00 +0100 (CET)
In-Reply-To: <1479977798-13417-1-git-send-email-olivier.matz@6wind.com>
References: <1479977798-13417-1-git-send-email-olivier.matz@6wind.com>
To: test-report@dpdk.org
Cc: Olivier Matz <olivier.matz@6wind.com>
Message-Id: <20161124085800.0ADAE5687@dpdk.org>
Date: Thu, 24 Nov 2016 09:58:00 +0100 (CET)
From: checkpatch@dpdk.org
Subject: [dpdk-test-report] |WARNING| [PATCH 0/5] virtio/mbuf: fix virtio
	tso with shared mbufs
X-BeenThere: test-report@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: automatic test reports <test-report.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/test-report>,
 <mailto:test-report-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/test-report/>
List-Post: <mailto:test-report@dpdk.org>
List-Help: <mailto:test-report-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/test-report>,
 <mailto:test-report-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Thu, 24 Nov 2016 08:58:00 -0000

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

_coding style issues_


ERROR:DIFF_IN_COMMIT_MSG: Avoid using diff content in the commit message - patch(1) might not work
#61: 
--- a/lib/librte_mbuf/rte_mbuf.c

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#68: FILE: lib/librte_mbuf/rte_mbuf.c:282:
+       if (0 && off + len <= rte_pktmbuf_data_len(seg))$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#68: FILE: lib/librte_mbuf/rte_mbuf.c:282:
+       if (0 && off + len <= rte_pktmbuf_data_len(seg))
                return rte_pktmbuf_mtod_offset(seg, char *, off);

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#81: FILE: lib/librte_mbuf/rte_mbuf.h:1674:
+       if (likely(0 && off + len <= rte_pktmbuf_data_len(m)))$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#81: FILE: lib/librte_mbuf/rte_mbuf.h:1674:
+       if (likely(0 && off + len <= rte_pktmbuf_data_len(m)))
                return rte_pktmbuf_mtod_offset(m, char *, off);

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#95: FILE: drivers/net/virtio/virtio_rxtx.c:228:
+       if (unlikely(1 || rte_pktmbuf_data_is_shared(m, 0, hdrlen))) {$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#95: FILE: drivers/net/virtio/virtio_rxtx.c:228:
+       if (unlikely(1 || rte_pktmbuf_data_is_shared(m, 0, hdrlen))) {
                shared = 1;

ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 2 errors, 6 warnings, 0 checks, 24 lines checked