From patchwork Sat Oct 24 16:11:05 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Honnappa Nagarahalli X-Patchwork-Id: 318943 Delivered-To: patch@linaro.org Received: by 2002:a92:d1d1:0:0:0:0:0 with SMTP id u17csp1343009ilg; Sat, 24 Oct 2020 09:11:50 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy9nRwOOWt10l6UzamLSGpf7Rt+LQfqgk2+92HTqkE9V9R+Vxmm8RyRyrghm7dUawzrhApi X-Received: by 2002:a05:6402:c12:: with SMTP id co18mr7561571edb.162.1603555910395; Sat, 24 Oct 2020 09:11:50 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1603555910; cv=none; d=google.com; s=arc-20160816; b=ZARj1c+vZURYqqRu9oFpNW6efSOYguHtLsvbbWLtqHRr0jYaRTc4E9ycsU/+1gLFiv CbP4ID0ZapX08B6RnhS9PjqK8KSilFDsGHSaOlm7nINnJBrVAcxUxfxMML1IvYTTAA4y H38oo74dhrtU5TQhlcRaqQwEScB187V9p3lrvrBkB2Y5DqO30wvj04dkeKqzHVGe0w7u T1LzQuCQ+7ktTsyS7gBi+LzJb88dQzBnJxGD+YKAOC17f67Zp/lknqt1Vm7Ui4uYceQG FuNuwg3jnyxOw5FxLRmJBjCnZqC5f+S4Y8NACExSX3+LgycCs4Z3JOo7k2OGdAwnPE7M dO5w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:cc:to:from; bh=K6ApnEIpfzcgDkuLCNpsQvowBJ4WT7kAtO6FsOQ9s58=; b=xHbVWd55T7D1VBA5nLKyu6lSTmzmBfpaJSbP3MvuW/ZDCEqRZf8pSezDHDNB+T09Sc mjsrcs0IrKLeVy4ItctXj38W6S/4CGrlENaIeq8UXcJzroxy2Pepx4k/78yTTGfJixU7 rw5ysMKLuGTq/UNypx1RFx2ClJeSDblWV9g3NCsc5gBG0DYndWL+z+2mr+rzpdCTOJMx V3uG4VW+fN/+zCksbKIvREbnpkVKlkLJkDN+uXd8Lt6Tzdmiskcn365mN+BTGWlOzITO +CuJLFtk81q+c8ybJVqdfpER0DCNIQuImh8Dg8hZ05pfq4Ub6O4EMdOWeTW66kk8Ifky xzfQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from dpdk.org (dpdk.org. [92.243.14.124]) by mx.google.com with ESMTP id sa7si3920627ejb.618.2020.10.24.09.11.50; Sat, 24 Oct 2020 09:11:50 -0700 (PDT) Received-SPF: pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) client-ip=92.243.14.124; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 872512A62; Sat, 24 Oct 2020 18:11:27 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id 94C901288 for ; Sat, 24 Oct 2020 18:11:23 +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 03F3412FC; Sat, 24 Oct 2020 09:11:22 -0700 (PDT) Received: from qc2400f-1.austin.arm.com (qc2400f-1.austin.arm.com [10.118.12.27]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id DFC273F66B; Sat, 24 Oct 2020 09:11:21 -0700 (PDT) From: Honnappa Nagarahalli To: dev@dpdk.org, honnappa.nagarahalli@arm.com, konstantin.ananyev@intel.com, stephen@networkplumber.org Cc: dharmik.thakkar@arm.com, ruifeng.wang@arm.com, olivier.matz@6wind.com, david.marchand@redhat.com, nd@arm.com Date: Sat, 24 Oct 2020 11:11:05 -0500 Message-Id: <20201024161112.13730-2-honnappa.nagarahalli@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20201024161112.13730-1-honnappa.nagarahalli@arm.com> References: <20200224203931.21256-1-honnappa.nagarahalli@arm.com> <20201024161112.13730-1-honnappa.nagarahalli@arm.com> Subject: [dpdk-dev] [PATCH v4 1/8] lib/ring: add zero copy APIs 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" Add zero-copy APIs. These APIs provide the capability to copy the data to/from the ring memory directly, without having a temporary copy (for ex: an array of mbufs on the stack). Use cases that involve copying large amount of data to/from the ring can benefit from these APIs. Signed-off-by: Honnappa Nagarahalli Reviewed-by: Dharmik Thakkar --- lib/librte_ring/meson.build | 1 + lib/librte_ring/rte_ring_elem.h | 1 + lib/librte_ring/rte_ring_peek_zc.h | 546 +++++++++++++++++++++++++++++ 3 files changed, 548 insertions(+) create mode 100644 lib/librte_ring/rte_ring_peek_zc.h -- 2.17.1 diff --git a/lib/librte_ring/meson.build b/lib/librte_ring/meson.build index 31c0b4649..36fdcb6a5 100644 --- a/lib/librte_ring/meson.build +++ b/lib/librte_ring/meson.build @@ -11,5 +11,6 @@ headers = files('rte_ring.h', 'rte_ring_hts_c11_mem.h', 'rte_ring_peek.h', 'rte_ring_peek_c11_mem.h', + 'rte_ring_peek_zc.h', 'rte_ring_rts.h', 'rte_ring_rts_c11_mem.h') diff --git a/lib/librte_ring/rte_ring_elem.h b/lib/librte_ring/rte_ring_elem.h index 938b398fc..7034d29c0 100644 --- a/lib/librte_ring/rte_ring_elem.h +++ b/lib/librte_ring/rte_ring_elem.h @@ -1079,6 +1079,7 @@ rte_ring_dequeue_burst_elem(struct rte_ring *r, void *obj_table, #ifdef ALLOW_EXPERIMENTAL_API #include +#include #endif #include diff --git a/lib/librte_ring/rte_ring_peek_zc.h b/lib/librte_ring/rte_ring_peek_zc.h new file mode 100644 index 000000000..482c716ab --- /dev/null +++ b/lib/librte_ring/rte_ring_peek_zc.h @@ -0,0 +1,546 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * + * Copyright (c) 2020 Arm Limited + * Copyright (c) 2007-2009 Kip Macy kmacy@freebsd.org + * All rights reserved. + * Derived from FreeBSD's bufring.h + * Used as BSD-3 Licensed with permission from Kip Macy. + */ + +#ifndef _RTE_RING_PEEK_ZC_H_ +#define _RTE_RING_PEEK_ZC_H_ + +/** + * @file + * @b EXPERIMENTAL: this API may change without prior notice + * It is not recommended to include this file directly. + * Please include instead. + * + * Ring Peek Zero Copy APIs + * These APIs make it possible to split public enqueue/dequeue API + * into 3 parts: + * - enqueue/dequeue start + * - copy data to/from the ring + * - enqueue/dequeue finish + * Along with the advantages of the peek APIs, these APIs provide the ability + * to avoid copying of the data to temporary area (for ex: array of mbufs + * on the stack). + * + * Note that currently these APIs are available only for two sync modes: + * 1) Single Producer/Single Consumer (RTE_RING_SYNC_ST) + * 2) Serialized Producer/Serialized Consumer (RTE_RING_SYNC_MT_HTS). + * It is user's responsibility to create/init ring with appropriate sync + * modes selected. + * + * Following are some examples showing the API usage. + * 1) + * struct elem_obj {uint64_t a; uint32_t b, c;}; + * struct elem_obj *obj; + * + * // Create ring with sync type RTE_RING_SYNC_ST or RTE_RING_SYNC_MT_HTS + * // Reserve space on the ring + * n = rte_ring_enqueue_zc_bulk_elem_start(r, sizeof(elem_obj), 1, &zcd, NULL); + * + * // Produce the data directly on the ring memory + * obj = (struct elem_obj *)zcd->ptr1; + * obj->a = rte_get_a(); + * obj->b = rte_get_b(); + * obj->c = rte_get_c(); + * rte_ring_enqueue_zc_elem_finish(ring, n); + * + * 2) + * // Create ring with sync type RTE_RING_SYNC_ST or RTE_RING_SYNC_MT_HTS + * // Reserve space on the ring + * n = rte_ring_enqueue_zc_burst_start(r, 32, &zcd, NULL); + * + * // Pkt I/O core polls packets from the NIC + * if (n != 0) { + * nb_rx = rte_eth_rx_burst(portid, queueid, zcd->ptr1, zcd->n1); + * if (nb_rx == zcd->n1 && n != zcd->n1) + * nb_rx = rte_eth_rx_burst(portid, queueid, + * zcd->ptr2, n - zcd->n1); + * + * // Provide packets to the packet processing cores + * rte_ring_enqueue_zc_finish(r, nb_rx); + * } + * + * Note that between _start_ and _finish_ none other thread can proceed + * with enqueue/dequeue operation till _finish_ completes. + */ + +#ifdef __cplusplus +extern "C" { +#endif + +#include + +/** + * Ring zero-copy information structure. + * + * This structure contains the pointers and length of the space + * reserved on the ring storage. + */ +struct rte_ring_zc_data { + /* Pointer to the first space in the ring */ + void *ptr1; + /* Pointer to the second space in the ring if there is wrap-around. + * It contains valid value only if wrap-around happens. + */ + void *ptr2; + /* Number of elements in the first pointer. If this is equal to + * the number of elements requested, then ptr2 is NULL. + * Otherwise, subtracting n1 from number of elements requested + * will give the number of elements available at ptr2. + */ + unsigned int n1; +} __rte_cache_aligned; + +static __rte_always_inline void +__rte_ring_get_elem_addr(struct rte_ring *r, uint32_t head, + uint32_t esize, uint32_t num, void **dst1, uint32_t *n1, void **dst2) +{ + uint32_t idx, scale, nr_idx; + uint32_t *ring = (uint32_t *)&r[1]; + + /* Normalize to uint32_t */ + scale = esize / sizeof(uint32_t); + idx = head & r->mask; + nr_idx = idx * scale; + + *dst1 = ring + nr_idx; + *n1 = num; + + if (idx + num > r->size) { + *n1 = r->size - idx; + *dst2 = ring; + } +} + +/** + * @internal This function moves prod head value. + */ +static __rte_always_inline unsigned int +__rte_ring_do_enqueue_zc_elem_start(struct rte_ring *r, unsigned int esize, + uint32_t n, enum rte_ring_queue_behavior behavior, + struct rte_ring_zc_data *zcd, unsigned int *free_space) +{ + uint32_t free, head, next; + + switch (r->prod.sync_type) { + case RTE_RING_SYNC_ST: + n = __rte_ring_move_prod_head(r, RTE_RING_SYNC_ST, n, + behavior, &head, &next, &free); + break; + case RTE_RING_SYNC_MT_HTS: + n = __rte_ring_hts_move_prod_head(r, n, behavior, &head, &free); + break; + case RTE_RING_SYNC_MT: + case RTE_RING_SYNC_MT_RTS: + default: + /* unsupported mode, shouldn't be here */ + RTE_ASSERT(0); + n = 0; + free = 0; + return n; + } + + __rte_ring_get_elem_addr(r, head, esize, n, &zcd->ptr1, + &zcd->n1, &zcd->ptr2); + + if (free_space != NULL) + *free_space = free - n; + return n; +} + +/** + * Start to enqueue several objects on the ring. + * Note that no actual objects are put in the queue by this function, + * it just reserves space for the user on the ring. + * User has to copy objects into the queue using the returned pointers. + * User should call rte_ring_enqueue_zc_elem_finish to complete the + * enqueue operation. + * + * @param r + * A pointer to the ring structure. + * @param esize + * The size of ring element, in bytes. It must be a multiple of 4. + * @param n + * The number of objects to add in the ring. + * @param zcd + * Structure containing the pointers and length of the space + * reserved on the ring storage. + * @param free_space + * If non-NULL, returns the amount of space in the ring after the + * reservation operation has finished. + * @return + * The number of objects that can be enqueued, either 0 or n + */ +__rte_experimental +static __rte_always_inline unsigned int +rte_ring_enqueue_zc_bulk_elem_start(struct rte_ring *r, unsigned int esize, + unsigned int n, struct rte_ring_zc_data *zcd, unsigned int *free_space) +{ + return __rte_ring_do_enqueue_zc_elem_start(r, esize, n, + RTE_RING_QUEUE_FIXED, zcd, free_space); +} + +/** + * Start to enqueue several pointers to objects on the ring. + * Note that no actual pointers are put in the queue by this function, + * it just reserves space for the user on the ring. + * User has to copy pointers to objects into the queue using the + * returned pointers. + * User should call rte_ring_enqueue_zc_finish to complete the + * enqueue operation. + * + * @param r + * A pointer to the ring structure. + * @param n + * The number of objects to add in the ring. + * @param zcd + * Structure containing the pointers and length of the space + * reserved on the ring storage. + * @param free_space + * If non-NULL, returns the amount of space in the ring after the + * reservation operation has finished. + * @return + * The number of objects that can be enqueued, either 0 or n + */ +__rte_experimental +static __rte_always_inline unsigned int +rte_ring_enqueue_zc_bulk_start(struct rte_ring *r, unsigned int n, + struct rte_ring_zc_data *zcd, unsigned int *free_space) +{ + return rte_ring_enqueue_zc_bulk_elem_start(r, sizeof(uintptr_t), n, + zcd, free_space); +} +/** + * Start to enqueue several objects on the ring. + * Note that no actual objects are put in the queue by this function, + * it just reserves space for the user on the ring. + * User has to copy objects into the queue using the returned pointers. + * User should call rte_ring_enqueue_zc_elem_finish to complete the + * enqueue operation. + * + * @param r + * A pointer to the ring structure. + * @param esize + * The size of ring element, in bytes. It must be a multiple of 4. + * @param n + * The number of objects to add in the ring. + * @param zcd + * Structure containing the pointers and length of the space + * reserved on the ring storage. + * @param free_space + * If non-NULL, returns the amount of space in the ring after the + * reservation operation has finished. + * @return + * The number of objects that can be enqueued, either 0 or n + */ +__rte_experimental +static __rte_always_inline unsigned int +rte_ring_enqueue_zc_burst_elem_start(struct rte_ring *r, unsigned int esize, + unsigned int n, struct rte_ring_zc_data *zcd, unsigned int *free_space) +{ + return __rte_ring_do_enqueue_zc_elem_start(r, esize, n, + RTE_RING_QUEUE_VARIABLE, zcd, free_space); +} + +/** + * Start to enqueue several pointers to objects on the ring. + * Note that no actual pointers are put in the queue by this function, + * it just reserves space for the user on the ring. + * User has to copy pointers to objects into the queue using the + * returned pointers. + * User should call rte_ring_enqueue_zc_finish to complete the + * enqueue operation. + * + * @param r + * A pointer to the ring structure. + * @param n + * The number of objects to add in the ring. + * @param zcd + * Structure containing the pointers and length of the space + * reserved on the ring storage. + * @param free_space + * If non-NULL, returns the amount of space in the ring after the + * reservation operation has finished. + * @return + * The number of objects that can be enqueued, either 0 or n + */ +__rte_experimental +static __rte_always_inline unsigned int +rte_ring_enqueue_zc_burst_start(struct rte_ring *r, unsigned int n, + struct rte_ring_zc_data *zcd, unsigned int *free_space) +{ + return rte_ring_enqueue_zc_burst_elem_start(r, sizeof(uintptr_t), n, + zcd, free_space); +} + +/** + * Complete enqueuing several objects on the ring. + * Note that number of objects to enqueue should not exceed previous + * enqueue_start return value. + * + * @param r + * A pointer to the ring structure. + * @param n + * The number of objects to add to the ring. + */ +__rte_experimental +static __rte_always_inline void +rte_ring_enqueue_zc_elem_finish(struct rte_ring *r, unsigned int n) +{ + uint32_t tail; + + switch (r->prod.sync_type) { + case RTE_RING_SYNC_ST: + n = __rte_ring_st_get_tail(&r->prod, &tail, n); + __rte_ring_st_set_head_tail(&r->prod, tail, n, 1); + break; + case RTE_RING_SYNC_MT_HTS: + n = __rte_ring_hts_get_tail(&r->hts_prod, &tail, n); + __rte_ring_hts_set_head_tail(&r->hts_prod, tail, n, 1); + break; + case RTE_RING_SYNC_MT: + case RTE_RING_SYNC_MT_RTS: + default: + /* unsupported mode, shouldn't be here */ + RTE_ASSERT(0); + } +} + +/** + * Complete enqueuing several pointers to objects on the ring. + * Note that number of objects to enqueue should not exceed previous + * enqueue_start return value. + * + * @param r + * A pointer to the ring structure. + * @param n + * The number of pointers to objects to add to the ring. + */ +__rte_experimental +static __rte_always_inline void +rte_ring_enqueue_zc_finish(struct rte_ring *r, unsigned int n) +{ + rte_ring_enqueue_zc_elem_finish(r, n); +} + +/** + * @internal This function moves cons head value and copies up to *n* + * objects from the ring to the user provided obj_table. + */ +static __rte_always_inline unsigned int +__rte_ring_do_dequeue_zc_elem_start(struct rte_ring *r, + uint32_t esize, uint32_t n, enum rte_ring_queue_behavior behavior, + struct rte_ring_zc_data *zcd, unsigned int *available) +{ + uint32_t avail, head, next; + + switch (r->cons.sync_type) { + case RTE_RING_SYNC_ST: + n = __rte_ring_move_cons_head(r, RTE_RING_SYNC_ST, n, + behavior, &head, &next, &avail); + break; + case RTE_RING_SYNC_MT_HTS: + n = __rte_ring_hts_move_cons_head(r, n, behavior, + &head, &avail); + break; + case RTE_RING_SYNC_MT: + case RTE_RING_SYNC_MT_RTS: + default: + /* unsupported mode, shouldn't be here */ + RTE_ASSERT(0); + n = 0; + avail = 0; + return n; + } + + __rte_ring_get_elem_addr(r, head, esize, n, &zcd->ptr1, + &zcd->n1, &zcd->ptr2); + + if (available != NULL) + *available = avail - n; + return n; +} + +/** + * Start to dequeue several objects from the ring. + * Note that no actual objects are copied from the queue by this function. + * User has to copy objects from the queue using the returned pointers. + * User should call rte_ring_dequeue_zc_elem_finish to complete the + * dequeue operation. + * + * @param r + * A pointer to the ring structure. + * @param esize + * The size of ring element, in bytes. It must be a multiple of 4. + * @param n + * The number of objects to remove from the ring. + * @param zcd + * Structure containing the pointers and length of the space + * reserved on the ring storage. + * @param available + * If non-NULL, returns the number of remaining ring entries after the + * dequeue has finished. + * @return + * The number of objects that can be dequeued, either 0 or n + */ +__rte_experimental +static __rte_always_inline unsigned int +rte_ring_dequeue_zc_bulk_elem_start(struct rte_ring *r, unsigned int esize, + unsigned int n, struct rte_ring_zc_data *zcd, unsigned int *available) +{ + return __rte_ring_do_dequeue_zc_elem_start(r, esize, n, + RTE_RING_QUEUE_FIXED, zcd, available); +} + +/** + * Start to dequeue several pointers to objects from the ring. + * Note that no actual pointers are removed from the queue by this function. + * User has to copy pointers to objects from the queue using the + * returned pointers. + * User should call rte_ring_dequeue_zc_finish to complete the + * dequeue operation. + * + * @param r + * A pointer to the ring structure. + * @param n + * The number of objects to remove from the ring. + * @param zcd + * Structure containing the pointers and length of the space + * reserved on the ring storage. + * @param available + * If non-NULL, returns the number of remaining ring entries after the + * dequeue has finished. + * @return + * The number of objects that can be dequeued, either 0 or n + */ +__rte_experimental +static __rte_always_inline unsigned int +rte_ring_dequeue_zc_bulk_start(struct rte_ring *r, unsigned int n, + struct rte_ring_zc_data *zcd, unsigned int *available) +{ + return rte_ring_dequeue_zc_bulk_elem_start(r, sizeof(uintptr_t), + n, zcd, available); +} + +/** + * Start to dequeue several objects from the ring. + * Note that no actual objects are copied from the queue by this function. + * User has to copy objects from the queue using the returned pointers. + * User should call rte_ring_dequeue_zc_elem_finish to complete the + * dequeue operation. + * + * @param r + * A pointer to the ring structure. + * @param esize + * The size of ring element, in bytes. It must be a multiple of 4. + * This must be the same value used while creating the ring. Otherwise + * the results are undefined. + * @param n + * The number of objects to dequeue from the ring. + * @param zcd + * Structure containing the pointers and length of the space + * reserved on the ring storage. + * @param available + * If non-NULL, returns the number of remaining ring entries after the + * dequeue has finished. + * @return + * The number of objects that can be dequeued, either 0 or n + */ +__rte_experimental +static __rte_always_inline unsigned int +rte_ring_dequeue_zc_burst_elem_start(struct rte_ring *r, unsigned int esize, + unsigned int n, struct rte_ring_zc_data *zcd, unsigned int *available) +{ + return __rte_ring_do_dequeue_zc_elem_start(r, esize, n, + RTE_RING_QUEUE_VARIABLE, zcd, available); +} + +/** + * Start to dequeue several pointers to objects from the ring. + * Note that no actual pointers are removed from the queue by this function. + * User has to copy pointers to objects from the queue using the + * returned pointers. + * User should call rte_ring_dequeue_zc_finish to complete the + * dequeue operation. + * + * @param r + * A pointer to the ring structure. + * @param n + * The number of objects to remove from the ring. + * @param zcd + * Structure containing the pointers and length of the space + * reserved on the ring storage. + * @param available + * If non-NULL, returns the number of remaining ring entries after the + * dequeue has finished. + * @return + * The number of objects that can be dequeued, either 0 or n + */ +__rte_experimental +static __rte_always_inline unsigned int +rte_ring_dequeue_zc_burst_start(struct rte_ring *r, unsigned int n, + struct rte_ring_zc_data *zcd, unsigned int *available) +{ + return rte_ring_dequeue_zc_burst_elem_start(r, sizeof(uintptr_t), n, + zcd, available); +} + +/** + * Complete dequeuing several objects from the ring. + * Note that number of objects to dequeued should not exceed previous + * dequeue_start return value. + * + * @param r + * A pointer to the ring structure. + * @param n + * The number of objects to remove from the ring. + */ +__rte_experimental +static __rte_always_inline void +rte_ring_dequeue_zc_elem_finish(struct rte_ring *r, unsigned int n) +{ + uint32_t tail; + + switch (r->cons.sync_type) { + case RTE_RING_SYNC_ST: + n = __rte_ring_st_get_tail(&r->cons, &tail, n); + __rte_ring_st_set_head_tail(&r->cons, tail, n, 0); + break; + case RTE_RING_SYNC_MT_HTS: + n = __rte_ring_hts_get_tail(&r->hts_cons, &tail, n); + __rte_ring_hts_set_head_tail(&r->hts_cons, tail, n, 0); + break; + case RTE_RING_SYNC_MT: + case RTE_RING_SYNC_MT_RTS: + default: + /* unsupported mode, shouldn't be here */ + RTE_ASSERT(0); + } +} + +/** + * Complete dequeuing several objects from the ring. + * Note that number of objects to dequeued should not exceed previous + * dequeue_start return value. + * + * @param r + * A pointer to the ring structure. + * @param n + * The number of objects to remove from the ring. + */ +__rte_experimental +static __rte_always_inline void +rte_ring_dequeue_zc_finish(struct rte_ring *r, unsigned int n) +{ + rte_ring_dequeue_elem_finish(r, n); +} + +#ifdef __cplusplus +} +#endif + +#endif /* _RTE_RING_PEEK_ZC_H_ */ From patchwork Sat Oct 24 16:11:06 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Honnappa Nagarahalli X-Patchwork-Id: 318944 Delivered-To: patch@linaro.org Received: by 2002:a92:d1d1:0:0:0:0:0 with SMTP id u17csp1343226ilg; Sat, 24 Oct 2020 09:12:12 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxo23JUalVhSG4yOZrWNpRxwHb3zSXI2fZmKk0ZnGDJwE5vSb+iquDGzTttWeCm62ihLY+X X-Received: by 2002:aa7:c792:: with SMTP id n18mr7796361eds.209.1603555932197; Sat, 24 Oct 2020 09:12:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1603555932; cv=none; d=google.com; s=arc-20160816; b=1BAkZU+/9q4q39FnW9Wc+drPhIGCBxND6RsgvQPqPGsLvRtsn0dRCUDzm5RmPIrtQP bKYeVrJbkn0N6wcFg3W0XV4NSxknNLutw+3rAOcPDvrwvBs34RZVspXGmhDcVs/fKoCD hykInG4xYA2k8Ez7GfGxkK8PJIvd7iyvcajzie0KTzfwmZRNKDSTTiOdgHbjxEIFsr8E 18zXek/4y3LCaNKNsyjC/GuDK6K5HjetA1MkXm7pbMlV57ydLyWw3mzKn+AO9Gwy13Jy 1uS9QI0+pZBtiO2BDkGgqz8grq2bD4HxjRK50NltOaJPeolSR8ddfBUkm6C5rCKOP+QZ sMHw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:cc:to:from; bh=469df2CbOJfoXDTuqr3CcF0i3jtt4a1CqgiPG2XfR7U=; b=1CQzsouaqy2Xmsw1nbjkAAT+JHxij7HeNTtXoclbYR57vvjFd1ITf/d/bRSRDhP82D GmZ7k0PX6AaKkgrWQsDBbq8FMseVldQpbFs8bYbyAUozx1pSnLhX8fpR62gtVLGZf506 g5JVNf8LRndqXHTTSCHaCPSHVkU3vIHsNt9ppauqb+QqUilZ9GZBv5jq7JJX9gsA853X EIHpyfPYI6uF7zC0AKccri1EpCNB63cvt8B6nME+gP2f+VrM/G1ZkXJgIEIfGUNDAoeD YBHZSwJbOih0LMHWEvRpCJHERWp+BIjf8cZ2Ay5BrdeAfYS1ruBSOIZ1hi5+idzTfCyF aokw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from dpdk.org (dpdk.org. [92.243.14.124]) by mx.google.com with ESMTP id f3si3337426ejz.386.2020.10.24.09.12.12; Sat, 24 Oct 2020 09:12:12 -0700 (PDT) Received-SPF: pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) client-ip=92.243.14.124; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 344EB2BEA; Sat, 24 Oct 2020 18:11:29 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id B63C825D9 for ; Sat, 24 Oct 2020 18:11:24 +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 24131101E; Sat, 24 Oct 2020 09:11:23 -0700 (PDT) Received: from qc2400f-1.austin.arm.com (qc2400f-1.austin.arm.com [10.118.12.27]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 130F83F66B; Sat, 24 Oct 2020 09:11:23 -0700 (PDT) From: Honnappa Nagarahalli To: dev@dpdk.org, honnappa.nagarahalli@arm.com, konstantin.ananyev@intel.com, stephen@networkplumber.org Cc: dharmik.thakkar@arm.com, ruifeng.wang@arm.com, olivier.matz@6wind.com, david.marchand@redhat.com, nd@arm.com Date: Sat, 24 Oct 2020 11:11:06 -0500 Message-Id: <20201024161112.13730-3-honnappa.nagarahalli@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20201024161112.13730-1-honnappa.nagarahalli@arm.com> References: <20200224203931.21256-1-honnappa.nagarahalli@arm.com> <20201024161112.13730-1-honnappa.nagarahalli@arm.com> Subject: [dpdk-dev] [PATCH v4 2/8] test/ring: move common function to header file 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" Move test_ring_inc_ptr to header file so that it can be used by functions in other files. Signed-off-by: Honnappa Nagarahalli Reviewed-by: Dharmik Thakkar --- app/test/test_ring.c | 11 ----------- app/test/test_ring.h | 13 +++++++++++++ 2 files changed, 13 insertions(+), 11 deletions(-) -- 2.17.1 diff --git a/app/test/test_ring.c b/app/test/test_ring.c index a62cb263b..329d538a9 100644 --- a/app/test/test_ring.c +++ b/app/test/test_ring.c @@ -243,17 +243,6 @@ test_ring_deq_impl(struct rte_ring *r, void **obj, int esize, unsigned int n, NULL); } -static void** -test_ring_inc_ptr(void **obj, int esize, unsigned int n) -{ - /* Legacy queue APIs? */ - if ((esize) == -1) - return ((void **)obj) + n; - else - return (void **)(((uint32_t *)obj) + - (n * esize / sizeof(uint32_t))); -} - static void test_ring_mem_init(void *obj, unsigned int count, int esize) { diff --git a/app/test/test_ring.h b/app/test/test_ring.h index d4b15af7c..b44711398 100644 --- a/app/test/test_ring.h +++ b/app/test/test_ring.h @@ -42,6 +42,19 @@ test_ring_create(const char *name, int esize, unsigned int count, (socket_id), (flags)); } +static inline void* +test_ring_inc_ptr(void *obj, int esize, unsigned int n) +{ + size_t sz; + + sz = sizeof(void *); + /* Legacy queue APIs? */ + if (esize != -1) + sz = esize; + + return (void *)((uint32_t *)obj + (n * sz / sizeof(uint32_t))); +} + static __rte_always_inline unsigned int test_ring_enqueue(struct rte_ring *r, void **obj, int esize, unsigned int n, unsigned int api_type) From patchwork Sat Oct 24 16:11:07 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Honnappa Nagarahalli X-Patchwork-Id: 318945 Delivered-To: patch@linaro.org Received: by 2002:a92:d1d1:0:0:0:0:0 with SMTP id u17csp1343403ilg; Sat, 24 Oct 2020 09:12:30 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxQKqcTl0idtmtx8CC9rtgC8+ek04dJsbCoSVejWJUI/KXRmHnP7VzCldR0yUVeKTOai8p3 X-Received: by 2002:a17:906:7016:: with SMTP id n22mr7963053ejj.402.1603555950388; Sat, 24 Oct 2020 09:12:30 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1603555950; cv=none; d=google.com; s=arc-20160816; b=tl/Zlb1ssJGSoDHC+PL65IirQaovKpAjPgFAZTWhR99jt9xawf2C6VTFaFMezdsvvO kbhtTwIuGFqqaiA4WhpCEmv2SIkZAbndYFcLrVA+kb+s3b4hsAIunYnrC3rnVb/IXysg LSgMSv3y/q7ZsVWJlI6g5mHLkoRDlL1zkcy8NG/QGoiLhcdn34OMbRdHai7WhNC7Mlt8 aamALAbZRSN7daFRt3zijhTLxcE+nxmhBqMQQELRaR8UAaS7Xhf28BySjbQXZfxyPGYf M8pTDabNlHqDS0CSQOVwgyFvNs9LXznTifh3lrxbjdkvdp0wpmdvTlcsC3bmF90hfu+L eqSQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:cc:to:from; bh=lXCWzIBXyCkRNtdyxPwZ7xjOhOHW2WTIMZsUKISEODk=; b=OI4m/rY7m2R0esbi1eHD+Af5IC/orr5yT1OtnNE0FLYghy08gJqbc5EatNF1gmYn/1 KmKz8O76iV/se+TY7zafKJIfiTvZ5Z7Gf24VtufivWDChPebfe6Dnfq1h0EW6xRVOW/P zDtYM2czRu9D7lCtZNb7R1OO4VLPCDkzOxQ4Bi9FqTg5p4BQiwzTImroMPPpWRx0zWbl N1B+2FtUO3gGLB809469eH+Ga7cRc7G7x6Uv+l23PDBNMPBZ+lIZzcDPY6ij15FcrRCJ rZEeMwTsJA1dWq752aDUsGImg1qSN73tlTLpTIWRBO1NsgoV0PKXB6ZQk99aaPlAgX8l TvQg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from dpdk.org (dpdk.org. [92.243.14.124]) by mx.google.com with ESMTP id b8si3107833edn.129.2020.10.24.09.12.30; Sat, 24 Oct 2020 09:12:30 -0700 (PDT) Received-SPF: pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) client-ip=92.243.14.124; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 11F354C6B; Sat, 24 Oct 2020 18:11:31 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id D324526A8 for ; Sat, 24 Oct 2020 18:11:25 +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 5FB2B12FC; Sat, 24 Oct 2020 09:11:24 -0700 (PDT) Received: from qc2400f-1.austin.arm.com (qc2400f-1.austin.arm.com [10.118.12.27]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 4B28E3F66B; Sat, 24 Oct 2020 09:11:24 -0700 (PDT) From: Honnappa Nagarahalli To: dev@dpdk.org, honnappa.nagarahalli@arm.com, konstantin.ananyev@intel.com, stephen@networkplumber.org Cc: dharmik.thakkar@arm.com, ruifeng.wang@arm.com, olivier.matz@6wind.com, david.marchand@redhat.com, nd@arm.com Date: Sat, 24 Oct 2020 11:11:07 -0500 Message-Id: <20201024161112.13730-4-honnappa.nagarahalli@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20201024161112.13730-1-honnappa.nagarahalli@arm.com> References: <20200224203931.21256-1-honnappa.nagarahalli@arm.com> <20201024161112.13730-1-honnappa.nagarahalli@arm.com> Subject: [dpdk-dev] [PATCH v4 3/8] test/ring: add functional tests for zero copy APIs 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" Add functional tests for zero copy APIs. Test enqueue/dequeue functions are created using the zero copy APIs to fit into the existing testing method. Signed-off-by: Honnappa Nagarahalli Reviewed-by: Dharmik Thakkar Acked-by: Konstantin Ananyev --- app/test/test_ring.c | 196 +++++++++++++++++++++++++++++++++++++++++++ app/test/test_ring.h | 42 ++++++++++ 2 files changed, 238 insertions(+) -- 2.17.1 diff --git a/app/test/test_ring.c b/app/test/test_ring.c index 329d538a9..99fe4b46f 100644 --- a/app/test/test_ring.c +++ b/app/test/test_ring.c @@ -1,5 +1,6 @@ /* SPDX-License-Identifier: BSD-3-Clause * Copyright(c) 2010-2014 Intel Corporation + * Copyright(c) 2020 Arm Limited */ #include @@ -68,6 +69,149 @@ static const int esize[] = {-1, 4, 8, 16, 20}; +/* Wrappers around the zero-copy APIs. The wrappers match + * the normal enqueue/dequeue API declarations. + */ +static unsigned int +test_ring_enqueue_zc_bulk(struct rte_ring *r, void * const *obj_table, + unsigned int n, unsigned int *free_space) +{ + unsigned int ret; + struct rte_ring_zc_data zcd; + + ret = rte_ring_enqueue_zc_bulk_start(r, n, &zcd, free_space); + if (ret > 0) { + /* Copy the data to the ring */ + test_ring_copy_to(&zcd, obj_table, sizeof(void *), ret); + rte_ring_enqueue_zc_finish(r, ret); + } + + return ret; +} + +static unsigned int +test_ring_enqueue_zc_bulk_elem(struct rte_ring *r, const void *obj_table, + unsigned int esize, unsigned int n, unsigned int *free_space) +{ + unsigned int ret; + struct rte_ring_zc_data zcd; + + ret = rte_ring_enqueue_zc_bulk_elem_start(r, esize, n, + &zcd, free_space); + if (ret > 0) { + /* Copy the data to the ring */ + test_ring_copy_to(&zcd, obj_table, esize, ret); + rte_ring_enqueue_zc_finish(r, ret); + } + + return ret; +} + +static unsigned int +test_ring_enqueue_zc_burst(struct rte_ring *r, void * const *obj_table, + unsigned int n, unsigned int *free_space) +{ + unsigned int ret; + struct rte_ring_zc_data zcd; + + ret = rte_ring_enqueue_zc_burst_start(r, n, &zcd, free_space); + if (ret > 0) { + /* Copy the data to the ring */ + test_ring_copy_to(&zcd, obj_table, sizeof(void *), ret); + rte_ring_enqueue_zc_finish(r, ret); + } + + return ret; +} + +static unsigned int +test_ring_enqueue_zc_burst_elem(struct rte_ring *r, const void *obj_table, + unsigned int esize, unsigned int n, unsigned int *free_space) +{ + unsigned int ret; + struct rte_ring_zc_data zcd; + + ret = rte_ring_enqueue_zc_burst_elem_start(r, esize, n, + &zcd, free_space); + if (ret > 0) { + /* Copy the data to the ring */ + test_ring_copy_to(&zcd, obj_table, esize, ret); + rte_ring_enqueue_zc_finish(r, ret); + } + + return ret; +} + +static unsigned int +test_ring_dequeue_zc_bulk(struct rte_ring *r, void **obj_table, + unsigned int n, unsigned int *available) +{ + unsigned int ret; + struct rte_ring_zc_data zcd; + + ret = rte_ring_dequeue_zc_bulk_start(r, n, &zcd, available); + if (ret > 0) { + /* Copy the data from the ring */ + test_ring_copy_from(&zcd, obj_table, sizeof(void *), ret); + rte_ring_dequeue_zc_finish(r, ret); + } + + return ret; +} + +static unsigned int +test_ring_dequeue_zc_bulk_elem(struct rte_ring *r, void *obj_table, + unsigned int esize, unsigned int n, unsigned int *available) +{ + unsigned int ret; + struct rte_ring_zc_data zcd; + + ret = rte_ring_dequeue_zc_bulk_elem_start(r, esize, n, + &zcd, available); + if (ret > 0) { + /* Copy the data from the ring */ + test_ring_copy_from(&zcd, obj_table, esize, ret); + rte_ring_dequeue_zc_finish(r, ret); + } + + return ret; +} + +static unsigned int +test_ring_dequeue_zc_burst(struct rte_ring *r, void **obj_table, + unsigned int n, unsigned int *available) +{ + unsigned int ret; + struct rte_ring_zc_data zcd; + + ret = rte_ring_dequeue_zc_burst_start(r, n, &zcd, available); + if (ret > 0) { + /* Copy the data from the ring */ + test_ring_copy_from(&zcd, obj_table, sizeof(void *), ret); + rte_ring_dequeue_zc_finish(r, ret); + } + + return ret; +} + +static unsigned int +test_ring_dequeue_zc_burst_elem(struct rte_ring *r, void *obj_table, + unsigned int esize, unsigned int n, unsigned int *available) +{ + unsigned int ret; + struct rte_ring_zc_data zcd; + + ret = rte_ring_dequeue_zc_burst_elem_start(r, esize, n, + &zcd, available); + if (ret > 0) { + /* Copy the data from the ring */ + test_ring_copy_from(&zcd, obj_table, esize, ret); + rte_ring_dequeue_zc_finish(r, ret); + } + + return ret; +} + static const struct { const char *desc; uint32_t api_type; @@ -219,6 +363,58 @@ static const struct { .felem = rte_ring_dequeue_burst_elem, }, }, + { + .desc = "SP/SC sync mode (ZC)", + .api_type = TEST_RING_ELEM_BULK | TEST_RING_THREAD_SPSC, + .create_flags = RING_F_SP_ENQ | RING_F_SC_DEQ, + .enq = { + .flegacy = test_ring_enqueue_zc_bulk, + .felem = test_ring_enqueue_zc_bulk_elem, + }, + .deq = { + .flegacy = test_ring_dequeue_zc_bulk, + .felem = test_ring_dequeue_zc_bulk_elem, + }, + }, + { + .desc = "MP_HTS/MC_HTS sync mode (ZC)", + .api_type = TEST_RING_ELEM_BULK | TEST_RING_THREAD_DEF, + .create_flags = RING_F_MP_HTS_ENQ | RING_F_MC_HTS_DEQ, + .enq = { + .flegacy = test_ring_enqueue_zc_bulk, + .felem = test_ring_enqueue_zc_bulk_elem, + }, + .deq = { + .flegacy = test_ring_dequeue_zc_bulk, + .felem = test_ring_dequeue_zc_bulk_elem, + }, + }, + { + .desc = "SP/SC sync mode (ZC)", + .api_type = TEST_RING_ELEM_BURST | TEST_RING_THREAD_SPSC, + .create_flags = RING_F_SP_ENQ | RING_F_SC_DEQ, + .enq = { + .flegacy = test_ring_enqueue_zc_burst, + .felem = test_ring_enqueue_zc_burst_elem, + }, + .deq = { + .flegacy = test_ring_dequeue_zc_burst, + .felem = test_ring_dequeue_zc_burst_elem, + }, + }, + { + .desc = "MP_HTS/MC_HTS sync mode (ZC)", + .api_type = TEST_RING_ELEM_BURST | TEST_RING_THREAD_DEF, + .create_flags = RING_F_MP_HTS_ENQ | RING_F_MC_HTS_DEQ, + .enq = { + .flegacy = test_ring_enqueue_zc_burst, + .felem = test_ring_enqueue_zc_burst_elem, + }, + .deq = { + .flegacy = test_ring_dequeue_zc_burst, + .felem = test_ring_dequeue_zc_burst_elem, + }, + } }; static unsigned int diff --git a/app/test/test_ring.h b/app/test/test_ring.h index b44711398..b525abb79 100644 --- a/app/test/test_ring.h +++ b/app/test/test_ring.h @@ -55,6 +55,48 @@ test_ring_inc_ptr(void *obj, int esize, unsigned int n) return (void *)((uint32_t *)obj + (n * sz / sizeof(uint32_t))); } +static inline void +test_ring_mem_copy(void *dst, void * const *src, int esize, unsigned int num) +{ + size_t sz; + + sz = num * sizeof(void *); + if (esize != -1) + sz = esize * num; + + memcpy(dst, src, sz); +} + +/* Copy to the ring memory */ +static inline void +test_ring_copy_to(struct rte_ring_zc_data *zcd, void * const *src, int esize, + unsigned int num) +{ + test_ring_mem_copy(zcd->ptr1, src, esize, zcd->n1); + if (zcd->n1 != num) { + if (esize == -1) + src = src + zcd->n1; + else + src = (void * const *)((const uint32_t *)src + + (zcd->n1 * esize / sizeof(uint32_t))); + test_ring_mem_copy(zcd->ptr2, src, + esize, num - zcd->n1); + } +} + +/* Copy from the ring memory */ +static inline void +test_ring_copy_from(struct rte_ring_zc_data *zcd, void *dst, int esize, + unsigned int num) +{ + test_ring_mem_copy(dst, zcd->ptr1, esize, zcd->n1); + + if (zcd->n1 != num) { + dst = test_ring_inc_ptr(dst, esize, zcd->n1); + test_ring_mem_copy(dst, zcd->ptr2, esize, num - zcd->n1); + } +} + static __rte_always_inline unsigned int test_ring_enqueue(struct rte_ring *r, void **obj, int esize, unsigned int n, unsigned int api_type) From patchwork Sat Oct 24 16:11:08 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Honnappa Nagarahalli X-Patchwork-Id: 318946 Delivered-To: patch@linaro.org Received: by 2002:a92:d1d1:0:0:0:0:0 with SMTP id u17csp1343628ilg; Sat, 24 Oct 2020 09:12:52 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxZPBrrmZhuZDfoHyO82X12Nhw8/Iu/eFdDFyT+KjI06UtG0FnNxlnbD/PEQD/lqMyw5OFI X-Received: by 2002:a17:906:6d99:: with SMTP id h25mr7517834ejt.281.1603555972113; Sat, 24 Oct 2020 09:12:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1603555972; cv=none; d=google.com; s=arc-20160816; b=TUveCCj4Oz06BKI8nVWYmXCRgYMArVdkPUhzRvMVL9VyHrWKUbuiQX66os4HQPEOtr Z5ATM2JFz8EjjiCQgAD8q54poNvfCt+0ahcgmClnEW1a6hRPk1KbJkvCSq40A52pKVOs EO7Yj/O5qfk4EOT6o3asP9wg026MEMLtpT2ASWMuGlVCm1Ul/D+MR+64ROR2+oOrvLxx jScR/okhDIGobB8yCdqpXgCYpGUd/xDRalRUzpGx2+wXlx5kLbTZ2zlK3vd/QsRes1tZ E+j7FOoe7MJNcL03IMRWvH0kcil6TW1Rt2vgs+1D2h8pwHtuLxRCfzdS1TJondCuaTs3 waWQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:cc:to:from; bh=RyMkA5i+b8nSd4W9XgRpTXJToPzr968io/FX0ORHig4=; b=L5CzZK0KIyajiC7D3Aw/ypzEyDXkqZty02uWbTy5ix2FApgLtFuVg7A8b5xWGgwUWo rLWJ6bE8hHvTiFRRVkPBRBKfLdRfO+QqQSkT3tXH49W2yRxP8xdJ5TrewiQoU7Hz9SLU H9Kr0PDK3A4ETonSedVxapBTkhvfliVh7lOL//oMldl043Gj7mSf0kTIV57PWa2gp+xW Nfo/cfj2j+cxxR2NmHjg2SlzaeqbPZAg0ye5R4wVWrDJYZsRwNcZKLHDY7o06e1b1NNm qjeifMQzLSxekCPFRUvUK9s40iYcGXokJltsxgk9Y2joxtnBIChpsqzSHTC1X3R00eAI iBPg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from dpdk.org (dpdk.org. [92.243.14.124]) by mx.google.com with ESMTP id gb8si3245022ejc.394.2020.10.24.09.12.51; Sat, 24 Oct 2020 09:12:52 -0700 (PDT) Received-SPF: pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) client-ip=92.243.14.124; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 9BEC84C89; Sat, 24 Oct 2020 18:11:34 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id AE51F26A8 for ; Sat, 24 Oct 2020 18:11:25 +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 6D5201396; Sat, 24 Oct 2020 09:11:25 -0700 (PDT) Received: from qc2400f-1.austin.arm.com (qc2400f-1.austin.arm.com [10.118.12.27]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 596903F66B; Sat, 24 Oct 2020 09:11:25 -0700 (PDT) From: Honnappa Nagarahalli To: dev@dpdk.org, honnappa.nagarahalli@arm.com, konstantin.ananyev@intel.com, stephen@networkplumber.org Cc: dharmik.thakkar@arm.com, ruifeng.wang@arm.com, olivier.matz@6wind.com, david.marchand@redhat.com, nd@arm.com Date: Sat, 24 Oct 2020 11:11:08 -0500 Message-Id: <20201024161112.13730-5-honnappa.nagarahalli@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20201024161112.13730-1-honnappa.nagarahalli@arm.com> References: <20200224203931.21256-1-honnappa.nagarahalli@arm.com> <20201024161112.13730-1-honnappa.nagarahalli@arm.com> Subject: [dpdk-dev] [PATCH v4 4/8] test/ring: add stress tests for zero copy APIs 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" Add stress tests for zero copy API. Signed-off-by: Honnappa Nagarahalli Reviewed-by: Dharmik Thakkar Acked-by: Konstantin Ananyev --- app/test/meson.build | 2 + app/test/test_ring_mt_peek_stress_zc.c | 56 ++++++++++++++++++++++ app/test/test_ring_st_peek_stress_zc.c | 65 ++++++++++++++++++++++++++ app/test/test_ring_stress.c | 6 +++ app/test/test_ring_stress.h | 2 + 5 files changed, 131 insertions(+) create mode 100644 app/test/test_ring_mt_peek_stress_zc.c create mode 100644 app/test/test_ring_st_peek_stress_zc.c -- 2.17.1 diff --git a/app/test/meson.build b/app/test/meson.build index 8bfb02890..88c831a92 100644 --- a/app/test/meson.build +++ b/app/test/meson.build @@ -108,9 +108,11 @@ test_sources = files('commands.c', 'test_ring_mpmc_stress.c', 'test_ring_hts_stress.c', 'test_ring_mt_peek_stress.c', + 'test_ring_mt_peek_stress_zc.c', 'test_ring_perf.c', 'test_ring_rts_stress.c', 'test_ring_st_peek_stress.c', + 'test_ring_st_peek_stress_zc.c', 'test_ring_stress.c', 'test_rwlock.c', 'test_sched.c', diff --git a/app/test/test_ring_mt_peek_stress_zc.c b/app/test/test_ring_mt_peek_stress_zc.c new file mode 100644 index 000000000..7e0bd511a --- /dev/null +++ b/app/test/test_ring_mt_peek_stress_zc.c @@ -0,0 +1,56 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright(c) 2020 Arm Limited + */ + +#include "test_ring.h" +#include "test_ring_stress_impl.h" +#include + +static inline uint32_t +_st_ring_dequeue_bulk(struct rte_ring *r, void **obj, uint32_t n, + uint32_t *avail) +{ + uint32_t m; + struct rte_ring_zc_data zcd; + + m = rte_ring_dequeue_zc_bulk_start(r, n, &zcd, avail); + n = (m == n) ? n : 0; + if (n != 0) { + /* Copy the data from the ring */ + test_ring_copy_from(&zcd, obj, -1, n); + rte_ring_dequeue_zc_finish(r, n); + } + + return n; +} + +static inline uint32_t +_st_ring_enqueue_bulk(struct rte_ring *r, void * const *obj, uint32_t n, + uint32_t *free) +{ + uint32_t m; + struct rte_ring_zc_data zcd; + + m = rte_ring_enqueue_zc_bulk_start(r, n, &zcd, free); + n = (m == n) ? n : 0; + if (n != 0) { + /* Copy the data from the ring */ + test_ring_copy_to(&zcd, obj, -1, n); + rte_ring_enqueue_zc_finish(r, n); + } + + return n; +} + +static int +_st_ring_init(struct rte_ring *r, const char *name, uint32_t num) +{ + return rte_ring_init(r, name, num, + RING_F_MP_HTS_ENQ | RING_F_MC_HTS_DEQ); +} + +const struct test test_ring_mt_peek_stress_zc = { + .name = "MT_PEEK_ZC", + .nb_case = RTE_DIM(tests), + .cases = tests, +}; diff --git a/app/test/test_ring_st_peek_stress_zc.c b/app/test/test_ring_st_peek_stress_zc.c new file mode 100644 index 000000000..2933e30bf --- /dev/null +++ b/app/test/test_ring_st_peek_stress_zc.c @@ -0,0 +1,65 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright(c) 2020 Arm Limited + */ + +#include "test_ring.h" +#include "test_ring_stress_impl.h" +#include + +static inline uint32_t +_st_ring_dequeue_bulk(struct rte_ring *r, void **obj, uint32_t n, + uint32_t *avail) +{ + uint32_t m; + struct rte_ring_zc_data zcd; + + static rte_spinlock_t lck = RTE_SPINLOCK_INITIALIZER; + + rte_spinlock_lock(&lck); + + m = rte_ring_dequeue_zc_bulk_start(r, n, &zcd, avail); + n = (m == n) ? n : 0; + if (n != 0) { + /* Copy the data from the ring */ + test_ring_copy_from(&zcd, obj, -1, n); + rte_ring_dequeue_zc_finish(r, n); + } + + rte_spinlock_unlock(&lck); + return n; +} + +static inline uint32_t +_st_ring_enqueue_bulk(struct rte_ring *r, void * const *obj, uint32_t n, + uint32_t *free) +{ + uint32_t m; + struct rte_ring_zc_data zcd; + + static rte_spinlock_t lck = RTE_SPINLOCK_INITIALIZER; + + rte_spinlock_lock(&lck); + + m = rte_ring_enqueue_zc_bulk_start(r, n, &zcd, free); + n = (m == n) ? n : 0; + if (n != 0) { + /* Copy the data from the ring */ + test_ring_copy_to(&zcd, obj, -1, n); + rte_ring_enqueue_zc_finish(r, n); + } + + rte_spinlock_unlock(&lck); + return n; +} + +static int +_st_ring_init(struct rte_ring *r, const char *name, uint32_t num) +{ + return rte_ring_init(r, name, num, RING_F_SP_ENQ | RING_F_SC_DEQ); +} + +const struct test test_ring_st_peek_stress_zc = { + .name = "ST_PEEK_ZC", + .nb_case = RTE_DIM(tests), + .cases = tests, +}; diff --git a/app/test/test_ring_stress.c b/app/test/test_ring_stress.c index c4f82ea56..1af45e0fc 100644 --- a/app/test/test_ring_stress.c +++ b/app/test/test_ring_stress.c @@ -49,9 +49,15 @@ test_ring_stress(void) n += test_ring_mt_peek_stress.nb_case; k += run_test(&test_ring_mt_peek_stress); + n += test_ring_mt_peek_stress_zc.nb_case; + k += run_test(&test_ring_mt_peek_stress_zc); + n += test_ring_st_peek_stress.nb_case; k += run_test(&test_ring_st_peek_stress); + n += test_ring_st_peek_stress_zc.nb_case; + k += run_test(&test_ring_st_peek_stress_zc); + printf("Number of tests:\t%u\nSuccess:\t%u\nFailed:\t%u\n", n, k, n - k); return (k != n); diff --git a/app/test/test_ring_stress.h b/app/test/test_ring_stress.h index c85d6fa92..416d68c9a 100644 --- a/app/test/test_ring_stress.h +++ b/app/test/test_ring_stress.h @@ -36,4 +36,6 @@ extern const struct test test_ring_mpmc_stress; extern const struct test test_ring_rts_stress; extern const struct test test_ring_hts_stress; extern const struct test test_ring_mt_peek_stress; +extern const struct test test_ring_mt_peek_stress_zc; extern const struct test test_ring_st_peek_stress; +extern const struct test test_ring_st_peek_stress_zc; From patchwork Sat Oct 24 16:11:09 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Honnappa Nagarahalli X-Patchwork-Id: 318948 Delivered-To: patch@linaro.org Received: by 2002:a92:d1d1:0:0:0:0:0 with SMTP id u17csp1344018ilg; Sat, 24 Oct 2020 09:13:35 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz5KDeDQI3o2qH5u2rcXJGqAzRZKQy+3yomJ+ee/tGKTOzXtAitEKIdxPfsmyXg0pgAnFQc X-Received: by 2002:aa7:cacd:: with SMTP id l13mr7617408edt.177.1603556015390; Sat, 24 Oct 2020 09:13:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1603556015; cv=none; d=google.com; s=arc-20160816; b=V9PFRwEPJLoiNnHm5YLMvghc3jA54rA5SLRUqGLE82zEXQFk+Mttc9NC2svH47euWR 2K3/Am1Z+bIkAnvQrCNwqlOFIBNy005Z1VfLf0+7zBCVstpVdIpKz0tB0NmPTe5VKe9W CG6oDTTqRTxl9X/lZ0ASrgN9Qtwgupv/5eOGMJMokiTF0naWKKQMzZss7MPgVdPgYZhO Ez4t7H1PDkBHDoY3mMOeh1Lr2sVzj5lIalnPdVv40UurEDa1PAzF5MzMOkhh3iACUpoR VnTd6NlQf79kUSJi2Zvmk9cx0TwRZekBszdvbg176TfSm8QKewymuWUpAvq9IKvGLySh FQKA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:cc:to:from; bh=4fK7FgJNF+UIso6tVDxvzeZQ7QcuxXimK5+svhNpr3U=; b=l2heutZddgbHFzUzOUpr/NN2ogV7NGsXBnpXuVZv4xn3XyTqgd0Ge/+C1EEPjZUr4Z VGhV/zvDT9vAxWhMfWC7elK0T0bh66xw2bWEgREE5836lONOVLMoEeVqvc12vQ0ntExQ MXINFEh3fkalU7J4sZAcrANVvrQvePdaet2FOFTWdejx1XztjvkLMEqmRIbZnyd7hS8B hgUUza/Nzq9fJQG+3KZgo2lDpbJ7nh/VSKTxhcWAIS4ntaTW2WvMProzKSYdmPx/7T2V 75PSQ3aseH3m0IdXH1LNwHnHP6IrLVOFOnC2eBMQf3MVfMJf5wWTE5P9EPpJuAb+RgwA icpA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from dpdk.org (dpdk.org. [92.243.14.124]) by mx.google.com with ESMTP id d12si4025635ejj.237.2020.10.24.09.13.33; Sat, 24 Oct 2020 09:13:35 -0700 (PDT) Received-SPF: pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) client-ip=92.243.14.124; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 9FB9554AE; Sat, 24 Oct 2020 18:11:37 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id 85EE729C8 for ; Sat, 24 Oct 2020 18:11:26 +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 483F5142F; Sat, 24 Oct 2020 09:11:26 -0700 (PDT) Received: from qc2400f-1.austin.arm.com (qc2400f-1.austin.arm.com [10.118.12.27]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 369BB3F66B; Sat, 24 Oct 2020 09:11:26 -0700 (PDT) From: Honnappa Nagarahalli To: dev@dpdk.org, honnappa.nagarahalli@arm.com, konstantin.ananyev@intel.com, stephen@networkplumber.org Cc: dharmik.thakkar@arm.com, ruifeng.wang@arm.com, olivier.matz@6wind.com, david.marchand@redhat.com, nd@arm.com Date: Sat, 24 Oct 2020 11:11:09 -0500 Message-Id: <20201024161112.13730-6-honnappa.nagarahalli@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20201024161112.13730-1-honnappa.nagarahalli@arm.com> References: <20200224203931.21256-1-honnappa.nagarahalli@arm.com> <20201024161112.13730-1-honnappa.nagarahalli@arm.com> Subject: [dpdk-dev] [PATCH v4 5/8] doc/ring: add zero copy peek APIs 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" Add zero copy peek API documentation. Signed-off-by: Honnappa Nagarahalli --- doc/guides/prog_guide/ring_lib.rst | 41 ++++++++++++++++++++++++++ doc/guides/rel_notes/release_20_11.rst | 9 ++++++ 2 files changed, 50 insertions(+) -- 2.17.1 diff --git a/doc/guides/prog_guide/ring_lib.rst b/doc/guides/prog_guide/ring_lib.rst index 895484d95..247646d38 100644 --- a/doc/guides/prog_guide/ring_lib.rst +++ b/doc/guides/prog_guide/ring_lib.rst @@ -452,6 +452,47 @@ selected. As an example of usage: Note that between ``_start_`` and ``_finish_`` none other thread can proceed with enqueue(/dequeue) operation till ``_finish_`` completes. +Ring Peek Zero Copy API +----------------------- + +Along with the advantages of the peek APIs, zero copy APIs provide the ability +to copy the data to the ring memory directly without the need for temporary +storage (for ex: array of mbufs on the stack). + +These APIs make it possible to split public enqueue/dequeue API into 3 phases: + +* enqueue/dequeue start + +* copy data to/from the ring + +* enqueue/dequeue finish + +Note that this API is available only for two sync modes: + +* Single Producer/Single Consumer (SP/SC) + +* Multi-producer/Multi-consumer with Head/Tail Sync (HTS) + +It is a user responsibility to create/init ring with appropriate sync modes. +Following is an example of usage: + +.. code-block:: c + + /* Reserve space on the ring */ + n = rte_ring_enqueue_zc_burst_start(r, 32, &zcd, NULL); + /* Pkt I/O core polls packets from the NIC */ + if (n != 0) { + nb_rx = rte_eth_rx_burst(portid, queueid, zcd->ptr1, zcd->n1); + if (nb_rx == zcd->n1 && n != zcd->n1) + nb_rx = rte_eth_rx_burst(portid, queueid, zcd->ptr2, + n - zcd->n1); + /* Provide packets to the packet processing cores */ + rte_ring_enqueue_zc_finish(r, nb_rx); + } + +Note that between ``_start_`` and ``_finish_`` no other thread can proceed +with enqueue(/dequeue) operation till ``_finish_`` completes. + References ---------- diff --git a/doc/guides/rel_notes/release_20_11.rst b/doc/guides/rel_notes/release_20_11.rst index d8ac359e5..fdc78b3da 100644 --- a/doc/guides/rel_notes/release_20_11.rst +++ b/doc/guides/rel_notes/release_20_11.rst @@ -55,6 +55,15 @@ New Features Also, make sure to start the actual text at the margin. ======================================================= +* **Added zero copy APIs for rte_ring.** + + For rings with producer/consumer in ``RTE_RING_SYNC_ST``, ``RTE_RING_SYNC_MT_HTS`` + modes, these APIs split enqueue/dequeue operation into three phases + (enqueue/dequeue start, copy data to/from ring, enqueue/dequeue finish). + Along with the advantages of the peek APIs, these provide the ability to + copy the data to the ring memory directly without the need for temporary + storage. + * **Added write combining store APIs.** Added ``rte_write32_wc`` and ``rte_write32_wc_relaxed`` APIs From patchwork Sat Oct 24 16:11:10 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Honnappa Nagarahalli X-Patchwork-Id: 318947 Delivered-To: patch@linaro.org Received: by 2002:a92:d1d1:0:0:0:0:0 with SMTP id u17csp1343837ilg; Sat, 24 Oct 2020 09:13:15 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzjw0zaOs0Hbvmv+DfyS8PqANtW17TZ+p0hJrbY3pG4FBsCu1UJH2e15U7dMblq7Ot2X4Xr X-Received: by 2002:a17:907:435b:: with SMTP id oc19mr7136635ejb.311.1603555995603; Sat, 24 Oct 2020 09:13:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1603555995; cv=none; d=google.com; s=arc-20160816; b=nWX0EgQhbMLamkBGMIOGmpp8nMjqENsoWExRVrhPzm1A59oWFYklrkK3bpSfgVqA8W Su0p65TZaT9NqwwckvaI8nAk+JOgi2xXJ5GRGQpYPNgdmPy/0VZLlctW9bnq0kEwfmB4 spYp2zJyTAKqhwoBCBDOldaD+fOJw7UkwJE8vDxvyouF9AS+CaTYb93/tqsgjarCl/H/ K2EHMFkDEXwMo7bVA64kwmxH5UhCczTh4x2ylnQmZM45kMeXQ0Fqyd9iblgBhHI1+JTa RptBVNuZ4NYEk35pq6puxRvJ7rYSZ/mNQTLRdK/NPLvsUREO1Nkmb3M/WRWZXA6Zr3KQ HR0Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:cc:to:from; bh=G2TfokirCDL4Iwlew9p1Jd9imSeONUbauO+haNUF4HA=; b=Xx0UvAcbhOsZjcqCAfmV4CNHdKntzzASWJQUabHnfYDBgvzabhEx9U+mrNNlPQrWhC uYyMY8QS4znStlEOz+xULyxCeaK5zp2p+egcIdNIRQD9P4gw/PzQjDfll31oFLO/w734 L9dulLGZWWIBzkHf1bSbJtqKQ4qUsiZciobIZSuRtcw2zq+wJ4SRE7WnTkNWg5kcWHvM j9an21/yxvFVkZmiNlfeNqKnsR4K0uifxS/rJOejwcVzZ3NyjWFA+3VAqNdj7p+kgAKc juvusa6OjIRKDpEhJkpnbWjzO9436dz1dC5kcS+iJp77Q2XuiNHRsqMGzID3f/apPptm L74A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from dpdk.org (dpdk.org. [92.243.14.124]) by mx.google.com with ESMTP id g24si3413209ejw.434.2020.10.24.09.13.13; Sat, 24 Oct 2020 09:13:15 -0700 (PDT) Received-SPF: pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) client-ip=92.243.14.124; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 3CE964CA6; Sat, 24 Oct 2020 18:11:36 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id 67DCD26A8; Sat, 24 Oct 2020 18:11:27 +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 1E2C812FC; Sat, 24 Oct 2020 09:11:27 -0700 (PDT) Received: from qc2400f-1.austin.arm.com (qc2400f-1.austin.arm.com [10.118.12.27]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 164363F66B; Sat, 24 Oct 2020 09:11:27 -0700 (PDT) From: Honnappa Nagarahalli To: dev@dpdk.org, honnappa.nagarahalli@arm.com, konstantin.ananyev@intel.com, stephen@networkplumber.org Cc: dharmik.thakkar@arm.com, ruifeng.wang@arm.com, olivier.matz@6wind.com, david.marchand@redhat.com, nd@arm.com, stable@dpdk.org Date: Sat, 24 Oct 2020 11:11:10 -0500 Message-Id: <20201024161112.13730-7-honnappa.nagarahalli@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20201024161112.13730-1-honnappa.nagarahalli@arm.com> References: <20200224203931.21256-1-honnappa.nagarahalli@arm.com> <20201024161112.13730-1-honnappa.nagarahalli@arm.com> Subject: [dpdk-dev] [PATCH v4 6/8] test/ring: fix the memory dump size 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" Pass the correct number of bytes to dump the memory. Fixes: bf28df24e915 ("test/ring: add contention stress test" Cc: konstantin.ananyev@intel.com Cc: stable@dpdk.org Signed-off-by: Honnappa Nagarahalli Reviewed-by: Dharmik Thakkar Acked-by: Konstantin Ananyev --- app/test/test_ring_stress_impl.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.17.1 diff --git a/app/test/test_ring_stress_impl.h b/app/test/test_ring_stress_impl.h index 3b9a480eb..f9ca63b90 100644 --- a/app/test/test_ring_stress_impl.h +++ b/app/test/test_ring_stress_impl.h @@ -159,7 +159,7 @@ check_updt_elem(struct ring_elem *elm[], uint32_t num, "offending object: %p\n", __func__, rte_lcore_id(), num, i, elm[i]); rte_memdump(stdout, "expected", check, sizeof(*check)); - rte_memdump(stdout, "result", elm[i], sizeof(elm[i])); + rte_memdump(stdout, "result", elm[i], sizeof(*elm[i])); rte_spinlock_unlock(&dump_lock); return -EINVAL; } From patchwork Sat Oct 24 16:11:11 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Honnappa Nagarahalli X-Patchwork-Id: 318949 Delivered-To: patch@linaro.org Received: by 2002:a92:d1d1:0:0:0:0:0 with SMTP id u17csp1344207ilg; Sat, 24 Oct 2020 09:13:55 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyRxgzT5JkUTTbDoQE7j5jRE2cTdLc5d8UzKd7NBfSOse9cMlj9xdSYGkGwWSjxoFQ19qpi X-Received: by 2002:a17:906:a1d8:: with SMTP id bx24mr7238832ejb.161.1603556035645; Sat, 24 Oct 2020 09:13:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1603556035; cv=none; d=google.com; s=arc-20160816; b=r+WPROalpGwk3bDIuSjuVU6rXhdF+AnfibTyd9TiF/M52fWKfvK8gqnqRBEmNBkz2U skte5RaQwCC16+u0UJPE5lcPROvQgqlStcsKhEogbgfGI/Ft4jHsQamgmuCU9waw4/2g Vljh6RyZhQi92kvbpzgKe8qp173gJf1AAEYW6BwK1LCveg/xW/5T5MUQnLnIT2EIawx8 6F7BKFZqlLpgX2PkJu4SIczAIJfztQLnebDBh1ADdG7Ip93m+/eRNdHx8J9W+RPhTFWn Hcgg8gLWfqZialNpOt8b0YnlTgVGnhaabvaCt3ggQ4yOlDTM40GCe/W4nW5uqwwT4OTX WnCw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:cc:to:from; bh=TPbivzCUEjLa5iLw/D6gbOFDY47NUnGhUiLms7lWHF8=; b=gpMoOseVMVLYoLCKPE5R7egRtL8IILGUlXgRbaishZBNhlTRKpZiRA1rKuKSmSCimL m0x/ubGNr+bfrxV6W/mKZY9XxTk/oAclVVTUK5w8mPU9ZOnHGMAPadjZO3jj6mTEnnmZ mz8FWawDJfgMg9OEGe8FfF+tDMcQ9lV6DR5dA4j41vslK2pjf1fbK9hG2gGIEBklrtIk y8M13DKpIoMOAkbFNDhCYFogaELWDn+LmVskJFTWJE6rQ381ytDGJnnRJf5gN9LHLXse 3KUcjC/CKJJgOffkG7DJdxKuQd/5bX10sBd00WhNN0tWcR940vcRcVUTZaPfL0qM1bMR AWCQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from dpdk.org (dpdk.org. [92.243.14.124]) by mx.google.com with ESMTP id w10si4168408ejv.268.2020.10.24.09.13.54; Sat, 24 Oct 2020 09:13:55 -0700 (PDT) Received-SPF: pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) client-ip=92.243.14.124; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1508C569B; Sat, 24 Oct 2020 18:11:39 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id B68DC2DCC for ; Sat, 24 Oct 2020 18:11:29 +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 2ED20142F; Sat, 24 Oct 2020 09:11:28 -0700 (PDT) Received: from qc2400f-1.austin.arm.com (qc2400f-1.austin.arm.com [10.118.12.27]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 1D3D43F66B; Sat, 24 Oct 2020 09:11:28 -0700 (PDT) From: Honnappa Nagarahalli To: dev@dpdk.org, honnappa.nagarahalli@arm.com, konstantin.ananyev@intel.com, stephen@networkplumber.org Cc: dharmik.thakkar@arm.com, ruifeng.wang@arm.com, olivier.matz@6wind.com, david.marchand@redhat.com, nd@arm.com Date: Sat, 24 Oct 2020 11:11:11 -0500 Message-Id: <20201024161112.13730-8-honnappa.nagarahalli@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20201024161112.13730-1-honnappa.nagarahalli@arm.com> References: <20200224203931.21256-1-honnappa.nagarahalli@arm.com> <20201024161112.13730-1-honnappa.nagarahalli@arm.com> Subject: [dpdk-dev] [PATCH v4 7/8] test/ring: remove unnecessary braces 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" Remove unnecessary braces to improve readability. Signed-off-by: Honnappa Nagarahalli --- app/test/test_ring.h | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) -- 2.17.1 diff --git a/app/test/test_ring.h b/app/test/test_ring.h index b525abb79..c8bfec839 100644 --- a/app/test/test_ring.h +++ b/app/test/test_ring.h @@ -35,11 +35,11 @@ test_ring_create(const char *name, int esize, unsigned int count, int socket_id, unsigned int flags) { /* Legacy queue APIs? */ - if ((esize) == -1) - return rte_ring_create((name), (count), (socket_id), (flags)); + if (esize == -1) + return rte_ring_create(name, count, socket_id, flags); else - return rte_ring_create_elem((name), (esize), (count), - (socket_id), (flags)); + return rte_ring_create_elem(name, esize, count, + socket_id, flags); } static inline void* @@ -102,7 +102,7 @@ test_ring_enqueue(struct rte_ring *r, void **obj, int esize, unsigned int n, unsigned int api_type) { /* Legacy queue APIs? */ - if ((esize) == -1) + if (esize == -1) switch (api_type) { case (TEST_RING_THREAD_DEF | TEST_RING_ELEM_SINGLE): return rte_ring_enqueue(r, *obj); @@ -163,7 +163,7 @@ test_ring_dequeue(struct rte_ring *r, void **obj, int esize, unsigned int n, unsigned int api_type) { /* Legacy queue APIs? */ - if ((esize) == -1) + if (esize == -1) switch (api_type) { case (TEST_RING_THREAD_DEF | TEST_RING_ELEM_SINGLE): return rte_ring_dequeue(r, obj); From patchwork Sat Oct 24 16:11:12 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Honnappa Nagarahalli X-Patchwork-Id: 318950 Delivered-To: patch@linaro.org Received: by 2002:a92:d1d1:0:0:0:0:0 with SMTP id u17csp1344337ilg; Sat, 24 Oct 2020 09:14:09 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy7WrfgCivRNwduiykMd3LGbMGyr8SwY4HEUHch9ovWPag4VNLRSmF8fd1e7FZBCcFN6R0p X-Received: by 2002:a17:906:b043:: with SMTP id bj3mr7344362ejb.543.1603556049488; Sat, 24 Oct 2020 09:14:09 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1603556049; cv=none; d=google.com; s=arc-20160816; b=hhBT02pqp5ZgTY9JjDDsDv6JIFnJpEaZFMutTP5UYDr/xveNNI4GDH0ZUSOe7r4XZz RmC4nLC38lAvpbD0TURNNWac8bF+U6zTRxlsJcrvtmrC18MRWiLq4Y9+VmztBCC+p3rj S4+6qRkdrF0F+V2G15OwcicfrvTwfg0Q2+QSRqZqDLaSN0cb6y/tDmr99JQhIi8HHqyn TF/BcRa+kJmsQPFkxxtuHmgPuC9a99UBMsvuGZ7CBZKdAfcqgBR9eiNaaj3yvlgc1QwT HqTCfrKdjFO1n4fDd8f+nobJDtflrGJFtGD64zdPoeCwL737KXYw367n1R8+QiFg8Sz3 eofQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:cc:to:from; bh=vCXnwduCuuwbGhU0q+GJjT36y3IXjF9zwAb3ILwcWXo=; b=MMrWMuOu3yg3nv7y8DdUTGUKzZGEB9Ukr1oCBVijna8qlUc5XetkjsskGwXkJUoU5Q ZhU/tVAnH91gSPWl3DTCP+Nlz/MaMbCcAHkJgU4loWJHybH4mC6sPSmqT4nalXVmHGWy WKmnNmHDpeFTh7TuY++k6EGzZoWYVJB+O4VYqLBl0aQD60w1oRaMK+2PEjWtXpHvO26u uIxLa+2NhvIO77Yux/om5syMiMFmhsOuDsUvgsxaLHT2lwbGNIOotFbz/XcfEH+QJ44A GXJA9WUlwr5sJKEE2uEVBAfOa8N2/4R5WK/RMQHLbzp+fcmcvqbk3DC4bz2YuZ83KL5M xRZA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from dpdk.org (dpdk.org. [92.243.14.124]) by mx.google.com with ESMTP id i19si3482608edv.19.2020.10.24.09.14.09; Sat, 24 Oct 2020 09:14:09 -0700 (PDT) Received-SPF: pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) client-ip=92.243.14.124; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 808AA58F9; Sat, 24 Oct 2020 18:11:40 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id 8BCE12DCC for ; Sat, 24 Oct 2020 18:11:29 +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 1090B1435; Sat, 24 Oct 2020 09:11:29 -0700 (PDT) Received: from qc2400f-1.austin.arm.com (qc2400f-1.austin.arm.com [10.118.12.27]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 08F893F66B; Sat, 24 Oct 2020 09:11:29 -0700 (PDT) From: Honnappa Nagarahalli To: dev@dpdk.org, honnappa.nagarahalli@arm.com, konstantin.ananyev@intel.com, stephen@networkplumber.org Cc: dharmik.thakkar@arm.com, ruifeng.wang@arm.com, olivier.matz@6wind.com, david.marchand@redhat.com, nd@arm.com Date: Sat, 24 Oct 2020 11:11:12 -0500 Message-Id: <20201024161112.13730-9-honnappa.nagarahalli@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20201024161112.13730-1-honnappa.nagarahalli@arm.com> References: <20200224203931.21256-1-honnappa.nagarahalli@arm.com> <20201024161112.13730-1-honnappa.nagarahalli@arm.com> Subject: [dpdk-dev] [PATCH v4 8/8] test/ring: user uintptr_t instead of unsigned long 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" Use uintptr_t instead of unsigned long while initializing the array of pointers. Signed-off-by: Honnappa Nagarahalli --- app/test/test_ring.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.17.1 diff --git a/app/test/test_ring.c b/app/test/test_ring.c index 99fe4b46f..51c05cabb 100644 --- a/app/test/test_ring.c +++ b/app/test/test_ring.c @@ -447,7 +447,7 @@ test_ring_mem_init(void *obj, unsigned int count, int esize) /* Legacy queue APIs? */ if (esize == -1) for (i = 0; i < count; i++) - ((void **)obj)[i] = (void *)(unsigned long)i; + ((void **)obj)[i] = (void *)(uintptr_t)i; else for (i = 0; i < (count * esize / sizeof(uint32_t)); i++) ((uint32_t *)obj)[i] = i;