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 0F59FA0487 for ; Thu, 4 Jul 2019 04:29:08 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 0392A374C; Thu, 4 Jul 2019 04:29:08 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id A83DC4C93; Thu, 4 Jul 2019 04:29:06 +0200 (CEST) In-Reply-To: <1562239487-34431-1-git-send-email-xiao.zhang@intel.com> References: <1562239487-34431-1-git-send-email-xiao.zhang@intel.com> To: test-report@dpdk.org Cc: Xiao Zhang Message-Id: <20190704022906.A83DC4C93@dpdk.org> Date: Thu, 4 Jul 2019 04:29:06 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw56060 [DPDK] net/e1000: Intel i219 reset hang issue fix. 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/56060 _coding style issues_ WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #16: Unit hang may occur if multiple descriptors are available in the rings during ERROR:GERRIT_CHANGE_ID: Remove Gerrit Change-Id's before submitting upstream. #19: Change-Id: Ia5e8dd06ad890d19f5093f1696faf68f7f52b491 WARNING:BRACES: braces {} are not necessary for single statement blocks #60: FILE: drivers/net/e1000/igb_ethdev.c:1593: + if (hw->mac.type >= e1000_pch_spt) { + igb_flush_desc_rings(dev); + } ERROR:SPACING: space required before the open parenthesis '(' #91: FILE: drivers/net/e1000/igb_rxtx.c:2981: + if(tdt != txq->tx_tail) ERROR:SPACING: spaces required around that '==' (ctx:VxW) #100: FILE: drivers/net/e1000/igb_rxtx.c:2990: + if (txq->tx_tail== txq->nb_tx_desc) ^ ERROR:SPACING: spaces required around that '=' (ctx:VxW) #101: FILE: drivers/net/e1000/igb_rxtx.c:2991: + txq->tx_tail= 0; ^ WARNING:BRACES: braces {} are not necessary for single statement blocks #159: FILE: drivers/net/e1000/igb_rxtx.c:3049: + if (fextnvm7 & E1000_FEXTNVM7_NEED_DESCRING_FLUSH) { + e1000_flush_rx_ring(dev); + } total: 4 errors, 3 warnings, 0 checks, 116 lines checked