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

blobfs: remove this deprecated library 



Note: there are some blobstore tests that depend on SPDK_TEST_BLOBFS,
so we will keep this test flag around for now. It should be cleaned up
though at some point.

Signed-off-by: default avatarJim Harris <jim.harris@nvidia.com>
Change-Id: I8259c5eeb8de044cd5374fd613998333c19b9947
Reviewed-on: https://review.spdk.io/c/spdk/spdk/+/26450


Tested-by: default avatarSPDK Automated Test System <spdkbot@gmail.com>
Reviewed-by: default avatarBoris Glimcher <Boris.Glimcher@emc.com>
Reviewed-by: default avatarChangpeng Liu <changpeliu@tencent.com>
Reviewed-by: default avatarKonrad Sztyber <ksztyber@nvidia.com>
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarTomasz Zawadzki <tomasz@tzawadzki.com>
parent fe181797

CONFIG

+0 −3
Original line number Diff line number Diff line
@@ -180,9 +180,6 @@ CONFIG_URING_PATH= 
# Path to custom built OPENSSL library

CONFIG_OPENSSL_PATH=



# Build with FUSE support

CONFIG_FUSE=n



# Build with RAID5f support

CONFIG_RAID5F=n

MAINTAINERS

+0 −5
Original line number Diff line number Diff line
@@ -66,11 +66,6 @@ S: Supported 
BDEV FIO PLUGIN

F:	app/fio/bdev/



BLOB FILESYSTEM

F:	lib/blobfs/

F:	module/blobfs/

S:	Obsolete



BLOBSTORE LIBRARY

M:	Tomasz Zawadzki <tomasz.zawadzki@nutanix.com>

F:	lib/blob/

autotest.sh

+0 −1
Original line number Diff line number Diff line
@@ -268,7 +268,6 @@ if [ $SPDK_RUN_FUNCTIONAL_TEST -eq 1 ]; then 
	if [ $SPDK_TEST_BLOBFS -eq 1 ]; then

		run_test "blobstore" $rootdir/test/blobstore/blobstore.sh

		run_test "blobstore_grow" $rootdir/test/blobstore/blobstore_grow/blobstore_grow.sh

		run_test "blobfs" $rootdir/test/blobfs/blobfs.sh

		run_test "hello_blob" $SPDK_EXAMPLE_DIR/hello_blob \

			examples/blob/hello_world/hello_blob.json

	fi

configure

+0 −16
Original line number Diff line number Diff line
@@ -112,8 +112,6 @@ function usage() { 
	echo " --without-uring-zns       Build I/O uring module without ZNS (zoned namespaces) support."

	echo " --with-openssl[=DIR]      Build OPENSSL with custom path. Otherwise the regular system paths will"

	echo "                           be searched."

	echo " --with-fuse               Build FUSE components for mounting a blobfs filesystem."

	echo " --without-fuse            No path required."

	echo " --with-fsdev              Build fsdev and fuse_dispatcher library components"

	echo " --without-fsdev           No path required"

	echo " --with-nvme-cuse          Build NVMe driver with support for CUSE-based character devices."
@@ -619,12 +617,6 @@ for i in "$@"; do 
			check_dir "$i"

			CONFIG[OPENSSL_PATH]=$(readlink -f ${i#*=})

			;;

		--with-fuse)

			CONFIG[FUSE]=y

			;;

		--without-fuse)

			CONFIG[FUSE]=n

			;;

		--with-fsdev)

			CONFIG[FSDEV]=y

			;;
@@ -1192,14 +1184,6 @@ if [[ "${CONFIG[URING]}" = "y" || "${CONFIG[XNVME]}" = "y" ]]; then 
	fi

fi



if [[ "${CONFIG[FUSE]}" = "y" ]]; then

	if [[ ! -d /usr/include/fuse3 ]] && [[ ! -d /usr/local/include/fuse3 ]]; then

		echo "--with-fuse requires libfuse3."

		echo "Please install then re-run this script."

		exit 1

	fi

fi



if [ "${CONFIG[CET]}" = "y" ]; then

	if ! echo -e 'int main(void) { return 0; }\n' | "${BUILD_CMD[@]}" -fcf-protection - 2> /dev/null; then

		echo "--enable-cet requires compiler/linker that supports CET."

deprecation.md

+0 −8
Original line number Diff line number Diff line
@@ -54,11 +54,3 @@ See GitHub issue [2801](https://github.com/spdk/spdk/issues/2801) for additional 


New SPDK partition types should use GUID `6527994e-2c5a-4eec-9613-8f5944074e8b` which will create

a bdev of the correct size.
 


### blobfs



#### 'spdk_fs_init', 'spdk_fs_load', 'blobfs_\*' RPCs



This entire blobfs library is deprecated and will be removed in 25.09 release.

All functions in this library are effectively deprecated, but only these two

are officially marked as such to ensure the library's deprecation is noticed.