From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by dpdk.org (Postfix, from userid 1017) id AAFA01B18C; Thu, 21 Sep 2017 10:24:08 +0200 (CEST) In-Reply-To: <8B0A0B0226AB0F47A8D0EBBBBA1F005102BC7F7D@DGGEML502-MBX.china.huawei.com> References: <8B0A0B0226AB0F47A8D0EBBBBA1F005102BC7F7D@DGGEML502-MBX.china.huawei.com> To: test-report@dpdk.org Cc: "liucheng \(J\)" Message-Id: <20170921082408.AAFA01B18C@dpdk.org> Date: Thu, 21 Sep 2017 10:24:08 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw29041 [PATCH] librte_eal:add heap lock in malloc_heap_get_stats 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:24:08 -0000 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/29041 _coding style issues_ WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #13: There is no lock in function malloc_heap_get_stats. When we call this function, the elem may free by other thread. WARNING:LEADING_SPACE: please, no spaces at the start of a line #38: FILE: lib/librte_eal/common/malloc_heap.c:191: + rte_spinlock_lock(&heap->lock);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #46: FILE: lib/librte_eal/common/malloc_heap.c:204: + rte_spinlock_unlock(&heap->lock);$ ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s) total: 1 errors, 3 warnings, 21 lines checked