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 8FBA445C9B for ; Tue, 12 Nov 2024 23:10:18 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 88CEF40DCD; Tue, 12 Nov 2024 23:10:18 +0100 (CET) Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) by mails.dpdk.org (Postfix) with ESMTP id 82EC040DCD for ; Tue, 12 Nov 2024 23:10:17 +0100 (CET) Received: by mail-wm1-f43.google.com with SMTP id 5b1f17b1804b1-43163667f0eso53357705e9.0 for ; Tue, 12 Nov 2024 14:10:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1731449417; x=1732054217; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=kx/AtJONhyV08JN4Cps9RuLtU6e22RFJayvXdBbyTu4=; b=hrct9IpcQSn8qKCTOUCHWARSld2OrYLcZ+AK7MkWuo8SuZm0WIpwZkYBnWNNk5uF0b IPfE22hCo1b08sdu0Egj5dLE+a0y4rHCEAL5Jjk9SllYIPW3vjPAnzMNIzpOyFUCG2m6 ntXraLXWXyIh3GKJfGlfBn1ZIKHHa5+3D/2r4ohXqzkdzVFj1Olmi5sKv+DRV9bMLq7W UrYzQRIItfoDulN2zg4LB2IjcKBZcJZecXZy56A6LpRLvDgCvg26wAjGNDWVk3QFWzri 7D+yCXZVzriJz71HqiOjtFrlAyxNarQ1swvjbBNy9Oq8wrJeQyPVWB6Fq+2yc7MT9uWe 7dwg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1731449417; x=1732054217; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=kx/AtJONhyV08JN4Cps9RuLtU6e22RFJayvXdBbyTu4=; b=FQIIGwRJhMUJHBWVYu2GZ5nwmuCzwsPakYDKnkQq+9y9uXGEymW/sFk6wLgmOeCLN9 i3aFNEeuu1Y6k2Idg9F4aNrjVF8sUBhBh/ASrHtHcvQthJ1oC9Z04gsTMyJvMmiHx60d sCpZnbtwi9iGIHBUgbG+6HyosxBIq0+pwZ9KFg+ULygbhklIkstDxdI9vr+ti1M5l7vS GXjJ/+S6Dwc8eDJ4eUlGe9K+sR9VgGUhfwP5UWvNJodbDWh/Wg2uV/zI4M8v+PdKK028 0ZCkmbH8vfTaqg0HXivo9By60DEN5YLlc0s0obH++N+0V15re+GbKOEHa5eDL7q01m7v Q4tw== X-Gm-Message-State: AOJu0Yzo99FEOPQDS85Mdp0Q7FfOOS3bg18Adiaawse/AafB/M3a96KJ OEaRgGtuD5WXQzfp1nxxXZdeIMGTC2pfwkk26fqhR44A3I4NFqV2tT9DHg== X-Google-Smtp-Source: AGHT+IHZpeQNfUgf3lYI+KMTj2ifY+7TrnYXezR3BLKPB2n49DEs0tu+h7KdZpaOxWq2JZbAQhBT6g== X-Received: by 2002:a05:600c:4587:b0:431:52a3:d9ea with SMTP id 5b1f17b1804b1-432b74a0c04mr166072595e9.0.1731449416924; Tue, 12 Nov 2024 14:10:16 -0800 (PST) Received: from localhost ([2a01:4b00:d036:ae00:e89d:e9a1:da72:2f9d]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-432d550c505sm951525e9.33.2024.11.12.14.10.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Nov 2024 14:10:16 -0800 (PST) From: luca.boccassi@gmail.com To: Brian Dooley Cc: dpdk stable Subject: patch 'test/crypto: fix synchronous API calls' has been queued to stable release 22.11.7 Date: Tue, 12 Nov 2024 22:07:49 +0000 Message-ID: <20241112220754.666489-39-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.45.2 In-Reply-To: <20241112220754.666489-1-luca.boccassi@gmail.com> References: <20241023211704.1216956-79-luca.boccassi@gmail.com> <20241112220754.666489-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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 22.11.7 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 11/14/24. 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/bluca/dpdk-stable This queued commit can be viewed at: https://github.com/bluca/dpdk-stable/commit/613e1f7f9edc2f715a217e830b03d009b66c995c Thanks. Luca Boccassi --- >From 613e1f7f9edc2f715a217e830b03d009b66c995c Mon Sep 17 00:00:00 2001 From: Brian Dooley Date: Fri, 25 Oct 2024 15:22:51 +0100 Subject: [PATCH] test/crypto: fix synchronous API calls [ upstream commit 251fdc592da5eddc4d84a95d0c151b0134504a32 ] For the synchronous API the enqueue/dequeue burst functions are not called. Skip these tests when calling the synchronous API. Fixes: 4ad17a1c8fb3 ("test/crypto: fix enqueue/dequeue callback case") Signed-off-by: Brian Dooley --- app/test/test_cryptodev.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c index 2069ccad3b..b093fd4ae9 100644 --- a/app/test/test_cryptodev.c +++ b/app/test/test_cryptodev.c @@ -12692,6 +12692,10 @@ test_enq_callback_setup(void) uint16_t qp_id = 0; int j = 0; + /* Skip test if synchronous API is used */ + if (gbl_action_type == RTE_SECURITY_ACTION_TYPE_CPU_CRYPTO) + return TEST_SKIPPED; + /* Verify the crypto capabilities for which enqueue/dequeue is done. */ cap_idx.type = RTE_CRYPTO_SYM_XFORM_AUTH; cap_idx.algo.auth = RTE_CRYPTO_AUTH_NULL; @@ -12813,6 +12817,10 @@ test_deq_callback_setup(void) uint16_t qp_id = 0; int j = 0; + /* Skip test if synchronous API is used */ + if (gbl_action_type == RTE_SECURITY_ACTION_TYPE_CPU_CRYPTO) + return TEST_SKIPPED; + /* Verify the crypto capabilities for which enqueue/dequeue is done. */ cap_idx.type = RTE_CRYPTO_SYM_XFORM_AUTH; cap_idx.algo.auth = RTE_CRYPTO_AUTH_NULL; -- 2.45.2 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2024-11-12 22:06:59.903326567 +0000 +++ 0039-test-crypto-fix-synchronous-API-calls.patch 2024-11-12 22:06:58.695307642 +0000 @@ -1 +1 @@ -From 251fdc592da5eddc4d84a95d0c151b0134504a32 Mon Sep 17 00:00:00 2001 +From 613e1f7f9edc2f715a217e830b03d009b66c995c Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 251fdc592da5eddc4d84a95d0c151b0134504a32 ] + @@ -10 +11,0 @@ -Cc: stable@dpdk.org @@ -14,2 +15,2 @@ - app/test/test_cryptodev.c | 11 ++++++++++- - 1 file changed, 10 insertions(+), 1 deletion(-) + app/test/test_cryptodev.c | 8 ++++++++ + 1 file changed, 8 insertions(+) @@ -18 +19 @@ -index 25eef342b0..c647baeee1 100644 +index 2069ccad3b..b093fd4ae9 100644 @@ -21,11 +22 @@ -@@ -2496,7 +2496,8 @@ test_queue_pair_descriptor_count(void) - int qp_depth = 0; - int i; - -- RTE_VERIFY(gbl_action_type != RTE_SECURITY_ACTION_TYPE_CPU_CRYPTO); -+ if (gbl_action_type == RTE_SECURITY_ACTION_TYPE_CPU_CRYPTO) -+ return TEST_SKIPPED; - - /* Verify if the queue pair depth API is supported by driver */ - qp_depth = rte_cryptodev_qp_depth_used(ts_params->valid_devs[0], 0); -@@ -15135,6 +15136,10 @@ test_enq_callback_setup(void) +@@ -12692,6 +12692,10 @@ test_enq_callback_setup(void) @@ -42 +33 @@ -@@ -15256,6 +15261,10 @@ test_deq_callback_setup(void) +@@ -12813,6 +12817,10 @@ test_deq_callback_setup(void)