[LTP] [PATCH v3 02/16] lib: Add interface to list supported filesystems
Sandeep Patil
sspatil@google.com
Thu Nov 9 19:38:58 CET 2017
On Wed, Oct 11, 2017 at 04:41:16PM +0200, Cyril Hrubis wrote:
> A filesystem is supported if kernel can mount it (we do not get ENODEV
> when we attempt to mount it) and if there is mkfs installed so that we
> can format a test device.
>
> The function starts with a whitelist of filesystems to use and loops
> over them filtering out unsupported ones, then finally returns a list
> of filesystem that could be used for the testing.
>
> Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
> ---
> include/tst_fs.h | 4 ++
> lib/tst_supported_fs_types.c | 118 +++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 122 insertions(+)
> create mode 100644 lib/tst_supported_fs_types.c
>
> diff --git a/include/tst_fs.h b/include/tst_fs.h
> index 52befbec9..1f6d2bab5 100644
> --- a/include/tst_fs.h
> +++ b/include/tst_fs.h
> @@ -146,6 +146,10 @@ int tst_get_path(const char *prog_name, char *buf, size_t buf_len);
> */
> int tst_fill_file(const char *path, char pattern, size_t bs, size_t bcount);
>
> +/*
> + * Returns NULL-terminated array of kernel-supported filesystems.
> + */
> +const char **tst_get_supported_fs_types(void);
>
> #ifdef TST_TEST_H__
> static inline long tst_fs_type(const char *path)
> diff --git a/lib/tst_supported_fs_types.c b/lib/tst_supported_fs_types.c
> new file mode 100644
> index 000000000..a23b1ed52
> --- /dev/null
> +++ b/lib/tst_supported_fs_types.c
> @@ -0,0 +1,118 @@
> +/*
> + * Copyright (c) 2017 Cyril Hrubis <chrubis@suse.cz>
> + *
> + * This program is free software: you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation, either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program. If not, see <http://www.gnu.org/licenses/>.
> + */
> +
> +#include <stdio.h>
> +#include <errno.h>
> +#include <stdlib.h>
> +#include <sys/mount.h>
> +#include <sys/wait.h>
> +
> +#define TST_NO_DEFAULT_MAIN
> +#include "tst_test.h"
> +#include "tst_fs.h"
> +
> +static const char *const fs_type_whitelist[] = {
> + "ext2",
> + "ext3",
> + "ext4",
> + "xfs",
> + "btrfs",
> + "vfat",
> + "exfat",
> + "ntfs",
> + NULL
> +};
> +
> +static const char *fs_types[ARRAY_SIZE(fs_type_whitelist)];
> +
> +static int has_mkfs(const char *fs_type)
> +{
> + char buf[128];
> + int ret;
> +
> + sprintf(buf, "mkfs.%s >/dev/null 2>&1", fs_type);
> +
> + ret = tst_system(buf);
> +
> + if (WEXITSTATUS(ret) == 127) {
> + tst_res(TINFO, "mkfs.%s does not exist", fs_type);
> + return 0;
> + }
> +
> + tst_res(TINFO, "mkfs.%s does exist", fs_type);
> + return 1;
> +}
> +
> +static int has_kernel_support(const char *fs_type)
> +{
> + static int fuse_supported = -1;
> + const char *tmpdir = getenv("TMPDIR");
> + char buf[128];
> + int ret;
> +
> + if (!tmpdir)
> + tmpdir = "/tmp";
> +
> + mount("/dev/zero", tmpdir, fs_type, 0, NULL);
> + if (errno != ENODEV) {
> + tst_res(TINFO, "Kernel supports %s", fs_type);
> + return 1;
> + }
> +
> + /* Is FUSE supported by kernel? */
> + if (fuse_supported == -1) {
> + ret = open("/dev/fuse", O_RDWR);
> + if (ret < 0) {
> + fuse_supported = 0;
> + } else {
> + fuse_supported = 1;
> + SAFE_CLOSE(ret);
> + }
> + }
> +
> + if (!fuse_supported)
> + return 0;
> +
> + /* Is FUSE implementation installed? */
> + sprintf(buf, "mount.%s >/dev/null 2>&1", fs_type);
> +
> + ret = tst_system(buf);
> + if (WEXITSTATUS(ret) == 127) {
> + tst_res(TINFO, "Filesystem %s is not supported", fs_type);
> + return 0;
> + }
> +
> + tst_res(TINFO, "FUSE does support %s", fs_type);
> + return 1;
> +}
> +
> +static int is_supported(const char *fs_type)
> +{
> + return has_kernel_support(fs_type) && has_mkfs(fs_type);
> +}
It will be nice if this was an exported API as well.
Other than that, this is very useful for a bunch of tests
that rely on formatting + mounting devices for error checks.
Acked-by: Sandeep Patil <sspatil@google.com>
More information about the ltp
mailing list