mbox series

[API-NEXT,v3,0/2] event subtype implementation

Message ID 20170616114829.3261-1-dmitry.ereminsolenikov@linaro.org
Headers show
Series event subtype implementation | expand

Message

Dmitry Eremin-Solenikov June 16, 2017, 11:48 a.m. UTC
Applies on top of [API-NEXT PATCH v2 0/3] IPSEC packet event

These patches provide event subtype implementation and tests. Ideally they
should be merged together with event subtypes API definitions.

Changes since v2:
 - Validate odp_event_types() implementation
 - Validate CRYPTO_COMPL event type and subtype

Changes since v1:
 - Implement odp_event_types()
 - Update event subtype properly in odp_crypto.c


Dmitry Eremin-Solenikov (2):
  linux-generic: events subtype implementation
  validation: test correctness of events subtype implementation

 .../linux-generic/include/odp_buffer_inlines.h     |  2 +
 .../linux-generic/include/odp_buffer_internal.h    |  3 ++
 platform/linux-generic/odp_crypto.c                | 15 ++++---
 platform/linux-generic/odp_event.c                 | 15 +++++++
 platform/linux-generic/odp_packet.c                |  1 +
 platform/linux-generic/odp_pool.c                  | 11 +++++
 test/common_plat/validation/api/buffer/buffer.c    | 50 ++++++++++++++++------
 .../validation/api/crypto/odp_crypto_test_inp.c    | 14 ++++++
 test/common_plat/validation/api/packet/packet.c    | 21 ++++++++-
 test/common_plat/validation/api/timer/timer.c      | 32 ++++++++++++--
 10 files changed, 140 insertions(+), 24 deletions(-)

-- 
2.11.0

Comments

Dmitry Eremin-Solenikov June 28, 2017, 2:52 p.m. UTC | #1
On 16.06.2017 14:48, Dmitry Eremin-Solenikov wrote:
> Applies on top of [API-NEXT PATCH v2 0/3] IPSEC packet event

> 

> These patches provide event subtype implementation and tests. Ideally they

> should be merged together with event subtypes API definitions.

> 

> Changes since v2:

>  - Validate odp_event_types() implementation

>  - Validate CRYPTO_COMPL event type and subtype

> 

> Changes since v1:

>  - Implement odp_event_types()

>  - Update event subtype properly in odp_crypto.c


Any updates/review comments/etc?

> 

> 

> Dmitry Eremin-Solenikov (2):

>   linux-generic: events subtype implementation

>   validation: test correctness of events subtype implementation

> 

>  .../linux-generic/include/odp_buffer_inlines.h     |  2 +

>  .../linux-generic/include/odp_buffer_internal.h    |  3 ++

>  platform/linux-generic/odp_crypto.c                | 15 ++++---

>  platform/linux-generic/odp_event.c                 | 15 +++++++

>  platform/linux-generic/odp_packet.c                |  1 +

>  platform/linux-generic/odp_pool.c                  | 11 +++++

>  test/common_plat/validation/api/buffer/buffer.c    | 50 ++++++++++++++++------

>  .../validation/api/crypto/odp_crypto_test_inp.c    | 14 ++++++

>  test/common_plat/validation/api/packet/packet.c    | 21 ++++++++-

>  test/common_plat/validation/api/timer/timer.c      | 32 ++++++++++++--

>  10 files changed, 140 insertions(+), 24 deletions(-)

> 



-- 
With best wishes
Dmitry
Savolainen, Petri (Nokia - FI/Espoo) June 29, 2017, 7:31 a.m. UTC | #2
Reviewed-by: Petri Savolainen <petri.savolainen@linaro.org>



> -----Original Message-----

> From: lng-odp [mailto:lng-odp-bounces@lists.linaro.org] On Behalf Of

> Dmitry Eremin-Solenikov

> Sent: Friday, June 16, 2017 2:48 PM

> To: lng-odp@lists.linaro.org

> Subject: [lng-odp] [PATCH API-NEXT v3 0/2] event subtype implementation

> 

> Applies on top of [API-NEXT PATCH v2 0/3] IPSEC packet event

> 

> These patches provide event subtype implementation and tests. Ideally they

> should be merged together with event subtypes API definitions.

> 

> Changes since v2:

>  - Validate odp_event_types() implementation

>  - Validate CRYPTO_COMPL event type and subtype

> 

> Changes since v1:

>  - Implement odp_event_types()

>  - Update event subtype properly in odp_crypto.c

> 

> 

> Dmitry Eremin-Solenikov (2):

>   linux-generic: events subtype implementation

>   validation: test correctness of events subtype implementation

> 

>  .../linux-generic/include/odp_buffer_inlines.h     |  2 +

>  .../linux-generic/include/odp_buffer_internal.h    |  3 ++

>  platform/linux-generic/odp_crypto.c                | 15 ++++---

>  platform/linux-generic/odp_event.c                 | 15 +++++++

>  platform/linux-generic/odp_packet.c                |  1 +

>  platform/linux-generic/odp_pool.c                  | 11 +++++

>  test/common_plat/validation/api/buffer/buffer.c    | 50 ++++++++++++++++-

> -----

>  .../validation/api/crypto/odp_crypto_test_inp.c    | 14 ++++++

>  test/common_plat/validation/api/packet/packet.c    | 21 ++++++++-

>  test/common_plat/validation/api/timer/timer.c      | 32 ++++++++++++--

>  10 files changed, 140 insertions(+), 24 deletions(-)

> 

> --

> 2.11.0