From 29be88fab6bb026df27c93ee89c61965b3c43133 Mon Sep 17 00:00:00 2001 From: Jim Harris Date: Wed, 24 Jan 2018 20:56:38 -0700 Subject: [PATCH] test/blob: always use SPDK_BS_PAGE_SIZE instead of PAGE_SIZE PAGE_SIZE just happens to resolve to same value as SPDK_BS_PAGE_SIZE on x86-64, but at least some ARM systems do not define PAGE_SIZE in headers included by blob unit tests, which is only reason this discrepency was not identified until now. Signed-off-by: Jim Harris Change-Id: Ie4afbc35263d6c17eafa1abcbf3d342942c80ab0 Reviewed-on: https://review.gerrithub.io/396320 Tested-by: SPDK Automated Test System Reviewed-by: Ben Walker Reviewed-by: Paul Luse Reviewed-by: Daniel Verkamp --- test/unit/lib/blob/blob.c/blob_ut.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/unit/lib/blob/blob.c/blob_ut.c b/test/unit/lib/blob/blob.c/blob_ut.c index 118fa0cf6..75b824283 100644 --- a/test/unit/lib/blob/blob.c/blob_ut.c +++ b/test/unit/lib/blob/blob.c/blob_ut.c @@ -2420,8 +2420,8 @@ bs_version(void) * region on "disk". We will use this to ensure blob IDs are * correctly reconstructed. */ - memset(&g_dev_buffer[super->used_blobid_mask_start * PAGE_SIZE], 0, - super->used_blobid_mask_len * PAGE_SIZE); + memset(&g_dev_buffer[super->used_blobid_mask_start * SPDK_BS_PAGE_SIZE], 0, + super->used_blobid_mask_len * SPDK_BS_PAGE_SIZE); super->used_blobid_mask_start = 0; super->used_blobid_mask_len = 0; super->crc = _spdk_blob_md_page_calc_crc(super);