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 0F721A052B for ; Wed, 29 Jul 2020 15:48:46 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id A817EA69; Wed, 29 Jul 2020 15:48:45 +0200 (CEST) Received: from us-smtp-1.mimecast.com (us-smtp-2.mimecast.com [207.211.31.81]) by dpdk.org (Postfix) with ESMTP id 7190010A3 for ; Wed, 29 Jul 2020 15:48:43 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1596030522; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=HCfO71ScLB563IoHuuD1i6DEc0t9BL54hSAA9kiH9mc=; b=UDub2bdf/tXKP1VyiM+lxppH/heoU/emnRqewSJ/sLgqHgCPZurZMOnyN4PJNU5hFjXXjS WXQEp5DyAnqPWqCPG/FquWcT0fHyCSWL0vxFqlJcZqoeLos6LM1s4S+MKl0Xfs7yt7qzD3 3trv/gveHdLVQRkAzdqCsNfHrItyVV8= Received: from mail-ua1-f70.google.com (mail-ua1-f70.google.com [209.85.222.70]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-17-MOzsObUhO7eV1cJvto234A-1; Wed, 29 Jul 2020 09:48:40 -0400 X-MC-Unique: MOzsObUhO7eV1cJvto234A-1 Received: by mail-ua1-f70.google.com with SMTP id 34so7236684uat.3 for ; Wed, 29 Jul 2020 06:48:39 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=HCfO71ScLB563IoHuuD1i6DEc0t9BL54hSAA9kiH9mc=; b=sk3Ei3zQ8SBdK+izPboApMQEEWLtgZz2VBZ2RzWay4TOjv2eQbmGzlqQyl0UVbFhG7 qyng9XtsJjJT6VvefiQBkiRGN2TBJGu1WnVk61d1C1nhuZhjjPYBv7tNCGY3UgThVd0E O9hFg4qOtpeJo3Ouq5APlp/H7aIL1h22u3O+6GtIpXySSUpZL1Q2okgQJC/fmXyxhHiC 5lcV94sEUID+g7tqztd32n/RFqYb9pIeCtoBkEoQ3Des6adu5WJzA/PHgVC2kCeIikHx +agDDulUqYOsvpeYzWdSX6clTNYoMNTyfoHAnaPrXA/b4jMYkv6V9aLATAnlNTy9zBgJ 7NHA== X-Gm-Message-State: AOAM530DJ3juTZOxRqHrtpl8u3+acT5zG+j5a02DefDuRls//tq8hhv6 sro55bC3YO1qdSCJB8FBvUyvYSmxc+bmflI4iolc1uyKSjvSz92/6+IeOe0TMWcu/68sm4Mya7t jq3jYuiAaFtq69a2Sm5w81VA= X-Received: by 2002:ac5:ccaf:: with SMTP id p15mr23618409vkm.12.1596030519487; Wed, 29 Jul 2020 06:48:39 -0700 (PDT) X-Google-Smtp-Source: ABdhPJw8PVYMiVt2Qln9EvZf2acttAsjQGYZOIylLOeFhTImQpRs8k7HrtRB4Hd9Ty2L0yPC55NRXKOaJeuC5y5GKfQ= X-Received: by 2002:ac5:ccaf:: with SMTP id p15mr23618396vkm.12.1596030519232; Wed, 29 Jul 2020 06:48:39 -0700 (PDT) MIME-Version: 1.0 References: <20200729063105.11299-1-feifei.wang2@arm.com> <20200729063105.11299-3-feifei.wang2@arm.com> In-Reply-To: <20200729063105.11299-3-feifei.wang2@arm.com> From: David Marchand Date: Wed, 29 Jul 2020 15:48:28 +0200 Message-ID: To: Feifei Wang Cc: Honnappa Nagarahalli , Konstantin Ananyev , Gavin Hu , Olivier Matz , dev , nd , dpdk stable Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=dmarchan@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH v1 2/2] test/ring: fix wrong param passed to the enqueue APIs X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hello Feifei, On Wed, Jul 29, 2020 at 8:32 AM Feifei Wang wrote: > > When enqueue one element (object of type void*) to ring in the > performance test, a pointer (the object to be enqueued) should be > passed to rte_ring_[sp|mp]enqueue APIs, not the pointer to a table > of void *pointers (objects). Good catch. Are we missing a check in the UT so that dequeued object is what had been enqueued? > > Fixes: a9fe152363e2 ("test/ring: add custom element size functional tests") > Cc: honnappa.nagarahalli@arm.com > Cc: stable@dpdk.org > > Signed-off-by: Feifei Wang > Reviewed-by: Ruifeng Wang > --- > app/test/test_ring.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/app/test/test_ring.h b/app/test/test_ring.h > index aa6ae67ca..d4b15af7c 100644 > --- a/app/test/test_ring.h > +++ b/app/test/test_ring.h > @@ -50,11 +50,11 @@ test_ring_enqueue(struct rte_ring *r, void **obj, int esize, unsigned int n, > if ((esize) == -1) > switch (api_type) { > case (TEST_RING_THREAD_DEF | TEST_RING_ELEM_SINGLE): > - return rte_ring_enqueue(r, obj); > + return rte_ring_enqueue(r, *obj); > case (TEST_RING_THREAD_SPSC | TEST_RING_ELEM_SINGLE): > - return rte_ring_sp_enqueue(r, obj); > + return rte_ring_sp_enqueue(r, *obj); > case (TEST_RING_THREAD_MPMC | TEST_RING_ELEM_SINGLE): > - return rte_ring_mp_enqueue(r, obj); > + return rte_ring_mp_enqueue(r, *obj); > case (TEST_RING_THREAD_DEF | TEST_RING_ELEM_BULK): > return rte_ring_enqueue_bulk(r, obj, n, NULL); > case (TEST_RING_THREAD_SPSC | TEST_RING_ELEM_BULK): > -- > 2.17.1 > -- David Marchand