mbox series

[v4,0/3] Bug fixes in event crypto adapter test

Message ID 20201008125234.22408-1-adwivedi@marvell.com (mailing list archive)
Headers
Series Bug fixes in event crypto adapter test |

Message

Ankur Dwivedi Oct. 8, 2020, 12:52 p.m. UTC 
  This patch series resolves bugs in the event crypto adapter test
application.

v4:
The following check-git-log.sh errors were resolved:
* Resolved wrong acked by tag in all the 3 patches.
* In the "test/event_crypto_adapter: return error for unsupported mode"
  patch, resolved "Headline too long" error.
* In the "test/event_crypto_adapter: fix function arguments" patch,
  added Fixes tag.

v3:
* In the "test/event_crypto_adapter: fix function arguments" patch,
  fixed arguments of rte_event_crypto_adapter_caps_get().
* In the "test/event_crypto_adapter: free resources during exit" patch,
  the following changes were done:
  - Added rte_event_crypto_adapter_stop() in crypto_adapter_teardown().
  - Checking the return value of rte_event_crypto_adapter_free().
  - Moved crypto_adapter_setup_done flag to the end.

v2:
* In the "test/event_crypto_adapter: return error with unsupported mode"
  patch the following changes were done:
  - Skip mismatch of mode and capability for software eventdev.
  - In test_crypto_adapter_conf_op_new_mode() and
    test_crypto_adapter_conf_op_forward_mode(),the local variable
    for holding return of test_crypto_adapter_conf() is removed.
  - Minor change in commit message.

Ankur Dwivedi (3):
  test/event_crypto_adapter: return error for unsupported mode
  test/event_crypto_adapter: fix function arguments
  test/event_crypto_adapter: free resources during exit

 app/test/test_event_crypto_adapter.c | 75 ++++++++++++++++++++--------
 1 file changed, 55 insertions(+), 20 deletions(-)

Comments

Jerin Jacob Oct. 8, 2020, 3 p.m. UTC | #1 
On Thu, Oct 8, 2020 at 6:22 PM Ankur Dwivedi <adwivedi@marvell.com> wrote:
>
> This patch series resolves bugs in the event crypto adapter test
> application.
>
> v4:
> The following check-git-log.sh errors were resolved:
> * Resolved wrong acked by tag in all the 3 patches.
> * In the "test/event_crypto_adapter: return error for unsupported mode"
>   patch, resolved "Headline too long" error.
> * In the "test/event_crypto_adapter: fix function arguments" patch,
>   added Fixes tag.
>
> v3:
> * In the "test/event_crypto_adapter: fix function arguments" patch,
>   fixed arguments of rte_event_crypto_adapter_caps_get().
> * In the "test/event_crypto_adapter: free resources during exit" patch,
>   the following changes were done:
>   - Added rte_event_crypto_adapter_stop() in crypto_adapter_teardown().
>   - Checking the return value of rte_event_crypto_adapter_free().
>   - Moved crypto_adapter_setup_done flag to the end.
>
> v2:
> * In the "test/event_crypto_adapter: return error with unsupported mode"
>   patch the following changes were done:
>   - Skip mismatch of mode and capability for software eventdev.
>   - In test_crypto_adapter_conf_op_new_mode() and
>     test_crypto_adapter_conf_op_forward_mode(),the local variable
>     for holding return of test_crypto_adapter_conf() is removed.
>   - Minor change in commit message.
>
> Ankur Dwivedi (3):
>   test/event_crypto_adapter: return error for unsupported mode
>   test/event_crypto_adapter: fix function arguments
>   test/event_crypto_adapter: free resources during exit


Series applied to dpdk-next-eventdev/for-main. Thanks.




>
>  app/test/test_event_crypto_adapter.c | 75 ++++++++++++++++++++--------
>  1 file changed, 55 insertions(+), 20 deletions(-)
>
> --
> 2.28.0
>