Commit 29be88fa authored by Jim Harris's avatar Jim Harris Committed by Daniel Verkamp
Browse files

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: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: Ie4afbc35263d6c17eafa1abcbf3d342942c80ab0

Reviewed-on: https://review.gerrithub.io/396320


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarPaul Luse <paul.e.luse@intel.com>
Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
parent 33e88d79

test/unit/lib/blob/blob.c/blob_ut.c

+2 −2
Original line number Diff line number Diff line
@@ -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);