Commit aa485663 authored by Jim Harris's avatar Jim Harris Committed by Tomasz Zawadzki
Browse files

bdev: remove _spdk_bdev function prefix 



The _ was supposed to indicate an internal function,
but really leaving off the spdk in the function
is the standard way to denote an internal function.

Signed-off-by: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: I74facafb67b793502838b9c1b5f90aec2c88c69b

Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/475033


Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarAlexey Marchuk <alexeymar@mellanox.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Broadcom SPDK FC-NVMe CI <spdk-ci.pdl@broadcom.com>
parent 0e58d154

lib/bdev/bdev.c

+230 −235

File changed.

Preview size limit exceeded, changes collapsed.

test/unit/lib/bdev/bdev.c/bdev_ut.c

+4 −4
Original line number Diff line number Diff line
@@ -972,25 +972,25 @@ bdev_io_spans_boundary_test(void) 
	bdev_io.bdev = &bdev;



	/* bdev has no optimal_io_boundary set - so this should return false. */

	CU_ASSERT(_spdk_bdev_io_should_split(&bdev_io) == false);

	CU_ASSERT(bdev_io_should_split(&bdev_io) == false);



	bdev.optimal_io_boundary = 32;

	bdev_io.type = SPDK_BDEV_IO_TYPE_RESET;



	/* RESETs are not based on LBAs - so this should return false. */

	CU_ASSERT(_spdk_bdev_io_should_split(&bdev_io) == false);

	CU_ASSERT(bdev_io_should_split(&bdev_io) == false);



	bdev_io.type = SPDK_BDEV_IO_TYPE_READ;

	bdev_io.u.bdev.offset_blocks = 0;

	bdev_io.u.bdev.num_blocks = 32;



	/* This I/O run right up to, but does not cross, the boundary - so this should return false. */

	CU_ASSERT(_spdk_bdev_io_should_split(&bdev_io) == false);

	CU_ASSERT(bdev_io_should_split(&bdev_io) == false);



	bdev_io.u.bdev.num_blocks = 33;



	/* This I/O spans a boundary. */

	CU_ASSERT(_spdk_bdev_io_should_split(&bdev_io) == true);

	CU_ASSERT(bdev_io_should_split(&bdev_io) == true);

}



static void