test/crypto: fix device number
Checks
Commit Message
In testsuite_setup(), ts_params is configured for first valid device.
The same device should be used as valid device in
test_device_configure_invalid_dev_id test case.
Fixes: 202d375c60bc ("app/test: add cryptodev unit and performance tests")
Cc: stable@dpdk.org
Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
---
app/test/test_cryptodev.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Ankur Dwivedi
> Sent: Thursday, July 30, 2020 12:20 PM
> To: dev@dpdk.org
> Cc: Doherty, Declan <declan.doherty@intel.com>; akhil.goyal@nxp.com;
> anoobj@marvell.com; Ankur Dwivedi <adwivedi@marvell.com>;
> stable@dpdk.org
> Subject: [dpdk-dev] [PATCH] test/crypto: fix device number
>
> In testsuite_setup(), ts_params is configured for first valid device.
> The same device should be used as valid device in
> test_device_configure_invalid_dev_id test case.
>
> Fixes: 202d375c60bc ("app/test: add cryptodev unit and performance tests")
> Cc: stable@dpdk.org
>
> Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
> ---
Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> > Subject: [dpdk-dev] [PATCH] test/crypto: fix device number
> >
> > In testsuite_setup(), ts_params is configured for first valid device.
> > The same device should be used as valid device in
> > test_device_configure_invalid_dev_id test case.
> >
> > Fixes: 202d375c60bc ("app/test: add cryptodev unit and performance tests")
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
> > ---
>
> Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
Applied to dpdk-next-crypto
Thanks.
@@ -755,7 +755,7 @@ struct crypto_unittest_params {
"Need at least %d devices for test", 1);
/* valid dev_id values */
- dev_id = ts_params->valid_devs[ts_params->valid_dev_count - 1];
+ dev_id = ts_params->valid_devs[0];
/* Stop the device in case it's started so it can be configured */
rte_cryptodev_stop(dev_id);