From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id DCFE0427EF for ; Wed, 22 Mar 2023 11:32:26 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D8B5240E09; Wed, 22 Mar 2023 11:32:26 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mails.dpdk.org (Postfix) with ESMTP id ED06540A84 for ; Wed, 22 Mar 2023 11:32:24 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1679481144; 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: content-transfer-encoding:content-transfer-encoding; bh=RhnRrNaqOJi7lFgglu0mXrLJBsSYFB+8YT0liFBkDYo=; b=Tkaa1P1enVjYWx4p851yUV4bvhGhqiqZFDBXpof/Eg4UeBVdffTgVrR5zvKfnwS31NzY20 xTPVJXwLQE5aWXUoqPvYYHPQCJcXxjSvH0CX4BRJJKrFAADbYJCz1fMTIOlRqg0WtgXolf mVjzDSoDDsyDwb3zlwbWYeGC8gbnzF8= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-18-fiWO_wauPe6ad1hQgP1sGQ-1; Wed, 22 Mar 2023 06:32:19 -0400 X-MC-Unique: fiWO_wauPe6ad1hQgP1sGQ-1 Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 2BB1885A5A3; Wed, 22 Mar 2023 10:32:19 +0000 (UTC) Received: from rh.redhat.com (unknown [10.39.192.69]) by smtp.corp.redhat.com (Postfix) with ESMTP id E91A9492C13; Wed, 22 Mar 2023 10:32:17 +0000 (UTC) From: Kevin Traynor To: David Marchand Cc: Yu Jiang , Hemant Agrawal , dpdk stable Subject: patch 'raw/skeleton: fix selftest' has been queued to stable release 21.11.4 Date: Wed, 22 Mar 2023 10:31:48 +0000 Message-Id: <20230322103209.456098-1-ktraynor@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.9 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII"; x-default=true X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 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 Hi, FYI, your patch has been queued to stable release 21.11.4 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 03/24/23. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/kevintraynor/dpdk-stable This queued commit can be viewed at: https://github.com/kevintraynor/dpdk-stable/commit/7dd9b2a9fad4cc70fff8078ed433d2a863dd8c9d Thanks. Kevin --- >From 7dd9b2a9fad4cc70fff8078ed433d2a863dd8c9d Mon Sep 17 00:00:00 2001 From: David Marchand Date: Fri, 10 Mar 2023 15:26:03 +0100 Subject: [PATCH] raw/skeleton: fix selftest [ upstream commit 365ec3c4fe599048c67cc79817ae9dfa090753cc ] ASan reported issues in this driver. rte_rawdev_obj_t context object points at a uint16_t. skeleton_rawdev_enqueue_bufs() and skeleton_rawdev_dequeue_bufs() were incorrectly casting to an int. The enqueue/dequeue selftest had a leak on the enqueued string and was wrong in passing a rte_rawdev_buf pointer array. Fix this by allocating buffers on the stack and check that returned string is the expected one. Bugzilla ID: 999 Fixes: 61c592a8d035 ("raw/skeleton: introduce skeleton rawdev driver") Fixes: 55ca1b0f2151 ("raw/skeleton: add test cases") Signed-off-by: David Marchand Tested-by: Yu Jiang Acked-by: Hemant Agrawal --- drivers/raw/skeleton/skeleton_rawdev.c | 4 +- drivers/raw/skeleton/skeleton_rawdev_test.c | 46 +++++++++------------ 2 files changed, 21 insertions(+), 29 deletions(-) diff --git a/drivers/raw/skeleton/skeleton_rawdev.c b/drivers/raw/skeleton/skeleton_rawdev.c index a49e000146..03b2019e99 100644 --- a/drivers/raw/skeleton/skeleton_rawdev.c +++ b/drivers/raw/skeleton/skeleton_rawdev.c @@ -422,5 +422,5 @@ static int skeleton_rawdev_enqueue_bufs(struct rte_rawdev *dev, * just an integer - for example, a queue-pair. */ - q_id = *((int *)context); + q_id = *((uint16_t *)context); for (i = 0; i < count; i++) @@ -444,5 +444,5 @@ static int skeleton_rawdev_dequeue_bufs(struct rte_rawdev *dev, * just an integer - for example, a queue-pair. */ - q_id = *((int *)context); + q_id = *((uint16_t *)context); for (i = 0; i < count; i++) diff --git a/drivers/raw/skeleton/skeleton_rawdev_test.c b/drivers/raw/skeleton/skeleton_rawdev_test.c index 484468eeb4..cad05ed60f 100644 --- a/drivers/raw/skeleton/skeleton_rawdev_test.c +++ b/drivers/raw/skeleton/skeleton_rawdev_test.c @@ -369,40 +369,32 @@ test_rawdev_enqdeq(void) { int ret; - unsigned int count = 1; uint16_t queue_id = 0; - struct rte_rawdev_buf buffers[1]; - struct rte_rawdev_buf *deq_buffers = NULL; + struct rte_rawdev_buf buffer; + struct rte_rawdev_buf *buffers[1]; + struct rte_rawdev_buf deq_buffer; + struct rte_rawdev_buf *deq_buffers[1]; - buffers[0].buf_addr = malloc(strlen(TEST_DEV_NAME) + 3); - if (!buffers[0].buf_addr) - goto cleanup; - snprintf(buffers[0].buf_addr, strlen(TEST_DEV_NAME) + 2, "%s%d", + buffers[0] = &buffer; + buffer.buf_addr = malloc(strlen(TEST_DEV_NAME) + 3); + if (!buffer.buf_addr) + return TEST_FAILED; + snprintf(buffer.buf_addr, strlen(TEST_DEV_NAME) + 2, "%s%d", TEST_DEV_NAME, 0); - ret = rte_rawdev_enqueue_buffers(test_dev_id, - (struct rte_rawdev_buf **)&buffers, - count, &queue_id); - RTE_TEST_ASSERT_EQUAL((unsigned int)ret, count, + ret = rte_rawdev_enqueue_buffers(test_dev_id, buffers, + RTE_DIM(buffers), &queue_id); + RTE_TEST_ASSERT_EQUAL((unsigned int)ret, RTE_DIM(buffers), "Unable to enqueue buffers"); - deq_buffers = malloc(sizeof(struct rte_rawdev_buf) * count); - if (!deq_buffers) - goto cleanup; - - ret = rte_rawdev_dequeue_buffers(test_dev_id, - (struct rte_rawdev_buf **)&deq_buffers, - count, &queue_id); - RTE_TEST_ASSERT_EQUAL((unsigned int)ret, count, + deq_buffers[0] = &deq_buffer; + ret = rte_rawdev_dequeue_buffers(test_dev_id, deq_buffers, + RTE_DIM(deq_buffers), &queue_id); + RTE_TEST_ASSERT_EQUAL((unsigned int)ret, RTE_DIM(buffers), "Unable to dequeue buffers"); + RTE_TEST_ASSERT_EQUAL(deq_buffers[0]->buf_addr, buffers[0]->buf_addr, + "Did not retrieve expected object"); - if (deq_buffers) - free(deq_buffers); - + free(buffer.buf_addr); return TEST_SUCCESS; -cleanup: - if (buffers[0].buf_addr) - free(buffers[0].buf_addr); - - return TEST_FAILED; } -- 2.39.2 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2023-03-22 10:30:07.955179303 +0000 +++ 0001-raw-skeleton-fix-selftest.patch 2023-03-22 10:30:07.867866508 +0000 @@ -1 +1 @@ -From 365ec3c4fe599048c67cc79817ae9dfa090753cc Mon Sep 17 00:00:00 2001 +From 7dd9b2a9fad4cc70fff8078ed433d2a863dd8c9d Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 365ec3c4fe599048c67cc79817ae9dfa090753cc ] + @@ -20 +21,0 @@ -Cc: stable@dpdk.org @@ -27,2 +28,2 @@ - drivers/raw/skeleton/skeleton_rawdev_test.c | 44 +++++++++------------ - 2 files changed, 21 insertions(+), 27 deletions(-) + drivers/raw/skeleton/skeleton_rawdev_test.c | 46 +++++++++------------ + 2 files changed, 21 insertions(+), 29 deletions(-) @@ -31 +32 @@ -index 53fe49f936..6e99d35536 100644 +index a49e000146..03b2019e99 100644 @@ -34 +35 @@ -@@ -429,5 +429,5 @@ static int skeleton_rawdev_enqueue_bufs(struct rte_rawdev *dev, +@@ -422,5 +422,5 @@ static int skeleton_rawdev_enqueue_bufs(struct rte_rawdev *dev, @@ -41 +42 @@ -@@ -451,5 +451,5 @@ static int skeleton_rawdev_dequeue_bufs(struct rte_rawdev *dev, +@@ -444,5 +444,5 @@ static int skeleton_rawdev_dequeue_bufs(struct rte_rawdev *dev, @@ -49 +50 @@ -index ca15c49990..b7a7f623aa 100644 +index 484468eeb4..cad05ed60f 100644 @@ -52 +53 @@ -@@ -371,38 +371,32 @@ test_rawdev_enqdeq(void) +@@ -369,40 +369,32 @@ test_rawdev_enqdeq(void) @@ -100 +101,2 @@ -- free(deq_buffers); +- if (deq_buffers) +- free(deq_buffers); @@ -105 +107,2 @@ -- free(buffers[0].buf_addr); +- if (buffers[0].buf_addr) +- free(buffers[0].buf_addr);