Sorry Taras I haven't. I'll try to get to it today.
-----Original Message-----
From: Taras Kondratiuk [mailto:[email protected]]
Sent: Tuesday, March 03, 2015 11:24 AM
To: Robbie King (robking)
Cc: [email protected]; [email protected]
Subject: Re: [RFC 0/2] validation: crypto: remove code duplication
On 02/27/2015 01:19 AM, Taras Kondratiuk wrote:
> Current crypto tests have a few issues:
> - Asynchronous tests assume that operation is always executed asynchronously,
> but specification does not demand it (I hope I read it correctly). So tests
> have to handle synchronous operations also.
> - If the first issue is solved, then Sync test is a subset of Async ->
> they can't be merged.
>
> The same approach should allow to easily add tests for 'out of place'
> operations.
>
> Taras Kondratiuk (2):
> validation: crypto: handle synchronous operations in async test
> validation: crypto: combine sync and async tests
>
> test/validation/Makefile.am | 3 +-
> ...ypto_test_async_inp.c => odp_crypto_test_inp.c} | 68 ++++--
> ...ypto_test_async_inp.h => odp_crypto_test_inp.h} | 8 +-
> test/validation/crypto/odp_crypto_test_sync_inp.c | 271
> ---------------------
> test/validation/crypto/odp_crypto_test_sync_inp.h | 17 --
> test/validation/odp_crypto.c | 9 +-
> 6 files changed, 60 insertions(+), 316 deletions(-)
> rename test/validation/crypto/{odp_crypto_test_async_inp.c =>
> odp_crypto_test_inp.c} (86%)
> rename test/validation/crypto/{odp_crypto_test_async_inp.h =>
> odp_crypto_test_inp.h} (62%)
> delete mode 100644 test/validation/crypto/odp_crypto_test_sync_inp.c
> delete mode 100644 test/validation/crypto/odp_crypto_test_sync_inp.h
>
Robbie, have you had a chance to look on this?
_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp
