From 1f3c37468c04295fb4a27b8a0ab72861cc0eb741 Mon Sep 17 00:00:00 2001 From: Konrad Sztyber Date: Fri, 13 Jan 2023 10:50:17 +0100 Subject: [PATCH] ut/accel: don't stub isa-l crypto functions Unit tests are already linked with isa-l-crypto if CONFIG_ISAL_CRYPTO is set, so there's no need to stub them. And by not stubbing them, we can do tests involving actual encryption/decryption. Signed-off-by: Konrad Sztyber Change-Id: I162a2cd26112cc5adb8eeed7336f4280aa4bdb6b Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16291 Tested-by: SPDK CI Jenkins Reviewed-by: Ben Walker Reviewed-by: Aleksey Marchuk --- test/unit/lib/accel/accel.c/accel_ut.c | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/test/unit/lib/accel/accel.c/accel_ut.c b/test/unit/lib/accel/accel.c/accel_ut.c index 4adfbe334..4b550dc80 100644 --- a/test/unit/lib/accel/accel.c/accel_ut.c +++ b/test/unit/lib/accel/accel.c/accel_ut.c @@ -22,16 +22,6 @@ DEFINE_STUB(pmem_memset_persist, void *, (void *pmemdest, int c, size_t len), NU DEFINE_STUB_V(spdk_memory_domain_destroy, (struct spdk_memory_domain *domain)); DEFINE_STUB(spdk_memory_domain_get_dma_device_id, const char *, (struct spdk_memory_domain *domain), "UT_DMA"); -#ifdef SPDK_CONFIG_ISAL -DEFINE_STUB_V(XTS_AES_128_enc, (uint8_t *k2, uint8_t *k1, uint8_t *tweak, uint64_t lba_size, - const uint8_t *src, uint8_t *dst)); -DEFINE_STUB_V(XTS_AES_128_dec, (uint8_t *k2, uint8_t *k1, uint8_t *tweak, uint64_t lba_size, - const uint8_t *src, uint8_t *dst)); -DEFINE_STUB_V(XTS_AES_256_enc, (uint8_t *k2, uint8_t *k1, uint8_t *tweak, uint64_t lba_size, - const uint8_t *src, uint8_t *dst)); -DEFINE_STUB_V(XTS_AES_256_dec, (uint8_t *k2, uint8_t *k1, uint8_t *tweak, uint64_t lba_size, - const uint8_t *src, uint8_t *dst)); -#endif int spdk_memory_domain_create(struct spdk_memory_domain **domain, enum spdk_dma_device_type type,