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 181A2A04B7; Sun, 20 Sep 2020 13:49:08 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 76ACA1D9B2; Sun, 20 Sep 2020 13:49:07 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id 788D91C201 for ; Sun, 20 Sep 2020 13:49:06 +0200 (CEST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id C03DDD6E; Sun, 20 Sep 2020 04:49:05 -0700 (PDT) Received: from net-arm-n1sdp.shanghai.arm.com (net-arm-n1sdp.shanghai.arm.com [10.169.208.213]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 66FE53F68F; Sun, 20 Sep 2020 04:49:04 -0700 (PDT) From: Feifei Wang To: Cc: dev@dpdk.org, nd@arm.com, Feifei Wang Date: Sun, 20 Sep 2020 06:48:49 -0500 Message-Id: <20200920114856.20697-1-feifei.wang2@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20200729063105.11299-1-feifei.wang2@arm.com> References: <20200729063105.11299-1-feifei.wang2@arm.com> Subject: [dpdk-dev] [PATCH v6 0/7] X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Several bugs in ring unit tests were identified and fixed. Added data validation on objects after enqueue/dequeue operations for data integration check. Unit tests were enhanced by checking return value of enqueue/dequeue operations and validating data integrity of objects. v2: 1. add check to validate the dequeued objects in test_ring.c and fix some bugs of it. (David/Honnappa) 2. remove the patch to change the description for the param of rte_ring_[sp/mp]_enqueue APIs. (David/Konstantin/Honnappa) v3: 1. Adjust the 'check dequeued objects' patch from the last to the first in the series. (Honnappa) 2. Add the new function 'test_ring_mem_cmp to replace 'memset' and print information of enqueue/dequeue elements if validation fails. (Honnappa) 3. Add test to validate the whether the enqueue and dequeue operation is successful for each time. (Honnappa) 4. Expand the scope of application for the macro TEST_RING_VERIF, and replace all ring APIs check with this macro. (Honnappa) v4: 1. Split the 'memcmp' bug fix patch into two patches, one is memcmp issue patch, the other is to add new function 'test_ring_mem_cmp'. (Honnappa) v5: 1. Adjust bug fixed patches to the first of the series, to ensure that the back porting will go smooth. (Honnappa) v6: 1. Add extra parameter to the macro. (Konstantin) Feifei Wang (7): test/ring: fix wrong parameter passed to the enqueue APIs test/ring: fix wrong number of enq/deq elements test/ring: fix wrong size used in memcmp test/ring: add check to validate dequeued objects test/ring: validate the return value of enq/deq elements test/ring: add new function to validate dequeue data test/ring: improve the application of macro app/test/test_ring.c | 362 ++++++++++++++++++++++--------------------- app/test/test_ring.h | 6 +- 2 files changed, 192 insertions(+), 176 deletions(-) -- 2.17.1