* [dpdk-stable] [PATCH v2 5/6] test/crypto: fix return value on test skipped [not found] <20210402142424.1353789-1-ciara.power@intel.com> @ 2021-04-02 14:24 ` Ciara Power 2021-04-13 17:07 ` [dpdk-stable] [EXT] " Akhil Goyal 0 siblings, 1 reply; 2+ messages in thread From: Ciara Power @ 2021-04-02 14:24 UTC (permalink / raw) To: dev Cc: declan.doherty, gakhil, aconole, hemant.agrawal, anoobj, ruifeng.wang, asomalap, ajit.khaparde, g.singh, Ciara Power, roy.fan.zhang, stable The blockcipher testcase return value TEST_SUCCESS was incorrect for one conditional check, it should have been TEST_SKIPPED similar to the other condition checks in this function when the testcase is skipped. Fixes: 4868f6591c6f ("test/crypto: add cases for raw datapath API") Cc: roy.fan.zhang@intel.com Cc: stable@dpdk.org Signed-off-by: Ciara Power <ciara.power@intel.com> --- app/test/test_cryptodev_blockcipher.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test/test_cryptodev_blockcipher.c b/app/test/test_cryptodev_blockcipher.c index 135e57b9fa..8e168724be 100644 --- a/app/test/test_cryptodev_blockcipher.c +++ b/app/test/test_cryptodev_blockcipher.c @@ -160,7 +160,7 @@ test_blockcipher_one_case(const struct blockcipher_test_case *t, printf("Raw Data Path APIs do not support OOP, " "Test Skipped.\n"); snprintf(test_msg, BLOCKCIPHER_TEST_MSG_LEN, "SKIPPED"); - status = TEST_SUCCESS; + status = TEST_SKIPPED; goto error_exit; } } -- 2.25.1 ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [dpdk-stable] [EXT] [PATCH v2 5/6] test/crypto: fix return value on test skipped 2021-04-02 14:24 ` [dpdk-stable] [PATCH v2 5/6] test/crypto: fix return value on test skipped Ciara Power @ 2021-04-13 17:07 ` Akhil Goyal 0 siblings, 0 replies; 2+ messages in thread From: Akhil Goyal @ 2021-04-13 17:07 UTC (permalink / raw) To: Ciara Power, dev Cc: declan.doherty, aconole, hemant.agrawal, Anoob Joseph, ruifeng.wang, asomalap, ajit.khaparde, g.singh, roy.fan.zhang, stable > The blockcipher testcase return value TEST_SUCCESS was incorrect for > one conditional check, it should have been TEST_SKIPPED similar to the > other condition checks in this function when the testcase is skipped. > > Fixes: 4868f6591c6f ("test/crypto: add cases for raw datapath API") > Cc: roy.fan.zhang@intel.com > Cc: stable@dpdk.org > > Signed-off-by: Ciara Power <ciara.power@intel.com> > --- Acked-by: Akhil Goyal <gakhil@marvell.com> ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-04-13 17:08 UTC | newest] Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20210402142424.1353789-1-ciara.power@intel.com> 2021-04-02 14:24 ` [dpdk-stable] [PATCH v2 5/6] test/crypto: fix return value on test skipped Ciara Power 2021-04-13 17:07 ` [dpdk-stable] [EXT] " Akhil Goyal
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).