Categorize a few more options

Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/options.c b/options.c
index 574916179f04a968b9877df234ebd3a599f84728..8681c218bf51e4c1068216af33a9c4a972119e0c 100644 (file)
--- a/options.c
+++ b/options.c
@@ -930,6 +930,18 @@ static struct opt_group fio_opt_cat_groups[] = {
                .name   = "I/O Flow",
                .mask   = FIO_OPT_G_IO_FLOW,
        },
                .name   = "I/O Flow",
                .mask   = FIO_OPT_G_IO_FLOW,
        },

+       {
+               .name   = "Description",
+               .mask   = FIO_OPT_G_DESC,
+       },
+       {
+               .name   = "Filename",
+               .mask   = FIO_OPT_G_FILENAME,
+       },
+       {
+               .name   = "General I/O",
+               .mask   = FIO_OPT_G_IO_BASIC,
+       },

        {
                .name   = NULL,
        }
        {
                .name   = NULL,
        }


@@ -951,7 +963,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .off1   = td_var_offset(description),
                .help   = "Text job description",
                .category = FIO_OPT_C_GENERAL,
                .off1   = td_var_offset(description),
                .help   = "Text job description",
                .category = FIO_OPT_C_GENERAL,

-               .group  = FIO_OPT_G_INVALID,
+               .group  = FIO_OPT_G_DESC,

        },
        {
                .name   = "name",
        },
        {
                .name   = "name",


@@ -960,7 +972,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .off1   = td_var_offset(name),
                .help   = "Name of this job",
                .category = FIO_OPT_C_GENERAL,
                .off1   = td_var_offset(name),
                .help   = "Name of this job",
                .category = FIO_OPT_C_GENERAL,

-               .group  = FIO_OPT_G_INVALID,
+               .group  = FIO_OPT_G_DESC,

        },
        {
                .name   = "kb_base",
        },
        {
                .name   = "kb_base",


@@ -983,7 +995,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .prio   = -1, /* must come after "directory" */
                .help   = "File(s) to use for the workload",
                .category = FIO_OPT_C_FILE,
                .prio   = -1, /* must come after "directory" */
                .help   = "File(s) to use for the workload",
                .category = FIO_OPT_C_FILE,

-               .group  = FIO_OPT_G_INVALID,
+               .group  = FIO_OPT_G_FILENAME,

        },
        {
                .name   = "directory",
        },
        {
                .name   = "directory",


@@ -993,7 +1005,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .cb     = str_directory_cb,
                .help   = "Directory to store files in",
                .category = FIO_OPT_C_FILE,
                .cb     = str_directory_cb,
                .help   = "Directory to store files in",
                .category = FIO_OPT_C_FILE,

-               .group  = FIO_OPT_G_INVALID,
+               .group  = FIO_OPT_G_FILENAME,

        },
        {
                .name   = "lockfile",
        },
        {
                .name   = "lockfile",


@@ -1006,7 +1018,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .hide   = 0,
                .def    = "none",
                .category = FIO_OPT_C_FILE,
                .hide   = 0,
                .def    = "none",
                .category = FIO_OPT_C_FILE,

-               .group  = FIO_OPT_G_INVALID,
+               .group  = FIO_OPT_G_FILENAME,

                .posval = {
                          { .ival = "none",
                            .oval = FILE_LOCK_NONE,
                .posval = {
                          { .ival = "none",
                            .oval = FILE_LOCK_NONE,


@@ -1031,7 +1043,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .cb     = str_opendir_cb,
                .help   = "Recursively add files from this directory and down",
                .category = FIO_OPT_C_FILE,
                .cb     = str_opendir_cb,
                .help   = "Recursively add files from this directory and down",
                .category = FIO_OPT_C_FILE,

-               .group  = FIO_OPT_G_INVALID,
+               .group  = FIO_OPT_G_FILENAME,

        },
        {
                .name   = "rw",
        },
        {
                .name   = "rw",


@@ -1044,7 +1056,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .def    = "read",
                .verify = rw_verify,
                .category = FIO_OPT_C_IO,
                .def    = "read",
                .verify = rw_verify,
                .category = FIO_OPT_C_IO,

-               .group  = FIO_OPT_G_INVALID,
+               .group  = FIO_OPT_G_IO_BASIC,

                .posval = {
                          { .ival = "read",
                            .oval = TD_DDIR_READ,
                .posval = {
                          { .ival = "read",
                            .oval = TD_DDIR_READ,


@@ -1080,7 +1092,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .help   = "IO offset generator modifier",
                .def    = "sequential",
                .category = FIO_OPT_C_IO,
                .help   = "IO offset generator modifier",
                .def    = "sequential",
                .category = FIO_OPT_C_IO,

-               .group  = FIO_OPT_G_INVALID,
+               .group  = FIO_OPT_G_IO_BASIC,

                .posval = {
                          { .ival = "sequential",
                            .oval = RW_SEQ_SEQ,
                .posval = {
                          { .ival = "sequential",
                            .oval = RW_SEQ_SEQ,


@@ -1101,7 +1113,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .help   = "IO engine to use",
                .def    = FIO_PREFERRED_ENGINE,
                .category = FIO_OPT_C_IO,
                .help   = "IO engine to use",
                .def    = FIO_PREFERRED_ENGINE,
                .category = FIO_OPT_C_IO,

-               .group  = FIO_OPT_G_INVALID,
+               .group  = FIO_OPT_G_IO_BASIC,

                .posval = {
                          { .ival = "sync",
                            .help = "Use read/write",
                .posval = {
                          { .ival = "sync",
                            .help = "Use read/write",


@@ -1192,7 +1204,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .interval = 1,
                .def    = "1",
                .category = FIO_OPT_C_IO,
                .interval = 1,
                .def    = "1",
                .category = FIO_OPT_C_IO,

-               .group  = FIO_OPT_G_IO_DEPTH,
+               .group  = FIO_OPT_G_IO_BASIC,

        },
        {
                .name   = "iodepth_batch",
        },
        {
                .name   = "iodepth_batch",


@@ -1207,7 +1219,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .interval = 1,
                .def    = "1",
                .category = FIO_OPT_C_IO,
                .interval = 1,
                .def    = "1",
                .category = FIO_OPT_C_IO,

-               .group  = FIO_OPT_G_IO_DEPTH,
+               .group  = FIO_OPT_G_IO_BASIC,

        },
        {
                .name   = "iodepth_batch_complete",
        },
        {
                .name   = "iodepth_batch_complete",


@@ -1221,7 +1233,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .interval = 1,
                .def    = "1",
                .category = FIO_OPT_C_IO,
                .interval = 1,
                .def    = "1",
                .category = FIO_OPT_C_IO,

-               .group  = FIO_OPT_G_IO_DEPTH,
+               .group  = FIO_OPT_G_IO_BASIC,

        },
        {
                .name   = "iodepth_low",
        },
        {
                .name   = "iodepth_low",


@@ -1233,7 +1245,7 @@ struct fio_option fio_options[FIO_MAX_OPTS] = {
                .hide   = 1,
                .interval = 1,
                .category = FIO_OPT_C_IO,
                .hide   = 1,
                .interval = 1,
                .category = FIO_OPT_C_IO,

-               .group  = FIO_OPT_G_IO_DEPTH,
+               .group  = FIO_OPT_G_IO_BASIC,

        },
        {
                .name   = "size",
        },
        {
                .name   = "size",

diff --git a/options.h b/options.h
index 256bf60e6fe6f50129551faa1c6cdcc0017e41a8..d49be46735ca802131f5986370448d851e2039ee 100644 (file)
--- a/options.h
+++ b/options.h
@@ -72,6 +72,9 @@ enum opt_category_group {
        __FIO_OPT_G_IOLOG,
        __FIO_OPT_G_IO_DEPTH,
        __FIO_OPT_G_IO_FLOW,
        __FIO_OPT_G_IOLOG,
        __FIO_OPT_G_IO_DEPTH,
        __FIO_OPT_G_IO_FLOW,

+       __FIO_OPT_G_DESC,
+       __FIO_OPT_G_FILENAME,
+       __FIO_OPT_G_IO_BASIC,

        __FIO_OPT_G_NR,
 
        FIO_OPT_G_RATE          = (1U << __FIO_OPT_G_RATE),
        __FIO_OPT_G_NR,
 
        FIO_OPT_G_RATE          = (1U << __FIO_OPT_G_RATE),


@@ -82,6 +85,9 @@ enum opt_category_group {
        FIO_OPT_G_IOLOG         = (1U << __FIO_OPT_G_IOLOG),
        FIO_OPT_G_IO_DEPTH      = (1U << __FIO_OPT_G_IO_DEPTH),
        FIO_OPT_G_IO_FLOW       = (1U << __FIO_OPT_G_IO_FLOW),
        FIO_OPT_G_IOLOG         = (1U << __FIO_OPT_G_IOLOG),
        FIO_OPT_G_IO_DEPTH      = (1U << __FIO_OPT_G_IO_DEPTH),
        FIO_OPT_G_IO_FLOW       = (1U << __FIO_OPT_G_IO_FLOW),

+       FIO_OPT_G_DESC          = (1U << __FIO_OPT_G_DESC),
+       FIO_OPT_G_FILENAME      = (1U << __FIO_OPT_G_FILENAME),
+       FIO_OPT_G_IO_BASIC      = (1U << __FIO_OPT_G_IO_BASIC),

        FIO_OPT_G_INVALID       = (1U << __FIO_OPT_G_NR),
 };
 
        FIO_OPT_G_INVALID       = (1U << __FIO_OPT_G_NR),
 };