summaryrefslogtreecommitdiff
path: root/libdm/libdm-report.c
diff options
context:
space:
mode:
authorDavid Teigland <teigland@redhat.com>2016-08-12 15:52:18 -0500
committerDavid Teigland <teigland@redhat.com>2016-10-21 16:16:11 -0500
commite65c3727d89e2c87938b11473273910a7bdac8da (patch)
treed4dad5e602e49b2773f2f7f778263e5be40dba65 /libdm/libdm-report.c
parent5c55c4ac18f60822da6bf61dde9dec3467a9e79a (diff)
downloadlvm2-dev-dct-cmd-defs30.tar.gz
commands: new method for defining commandsdev-dct-cmd-defs30
. Define a prototype for every lvm command. . Match every user command with one definition. . Generate help text and man pages from them. The new file command-lines.in defines a prototype for every unique lvm command. A unique lvm command is a unique combination of: command name + required option args + required positional args. Each of these prototypes also includes the optional option args and optional positional args that the command will accept, a description, and a unique string ID for the definition. Any valid command will match one of the prototypes. Here's an example of the lvresize command definitions from command-lines.in, there are three unique lvresize commands: lvresize --size SizeMB LV OO: --alloc Alloc, --autobackup Bool, --force, --nofsck, --nosync, --noudevsync, --reportformat String, --resizefs, --stripes Number, --stripesize SizeKB, --test, --poolmetadatasize SizeMB OP: PV ... ID: lvresize_by_size DESC: Resize an LV by a specified size. lvresize LV PV ... OO: --alloc Alloc, --autobackup Bool, --force, --nofsck, --nosync, --noudevsync, --reportformat String, --resizefs, --stripes Number, --stripesize SizeKB, --test ID: lvresize_by_pv DESC: Resize an LV by a specified PV. lvresize --poolmetadatasize SizeMB LV_thinpool OO: --alloc Alloc, --autobackup Bool, --force, --nofsck, --nosync, --noudevsync, --reportformat String, --stripes Number, --stripesize SizeKB, --test OP: PV ... ID: lvresize_pool_metadata_by_size DESC: Resize a pool metadata SubLV by a specified size. The three commands have separate definitions because they have different required parameters. Required parameters are specified on the first line of the definition. Optional options are listed after OO, and optional positional args are listed after OP. This data is used to generate corresponding command definition structures for lvm in command-lines.h. usage/help output is also auto generated, so it is always in sync with the definitions. Example of the corresponding generated structure in command-lines.h for the first lvresize prototype (these structures are never edited directly): commands[85].name = "lvresize"; commands[85].command_line_id = "lvresize_by_size"; commands[85].command_line_enum = lvresize_by_size_CMD; commands[85].fn = lvresize; commands[85].ro_count = 1; commands[85].rp_count = 1; commands[85].oo_count = 22; commands[85].op_count = 1; commands[85].desc = "DESC: Resize an LV by a specified size."; commands[85].usage = "lvresize --size Number[m|unit] LV" " [ --resizefs, --poolmetadatasize Number[m|unit], COMMON_OPTIONS ]" " [ PV ... ]"; commands[85].usage_common = " [ --alloc contiguous|cling|cling_by_tags|normal|anywhere|inherit, --commandprofile String, --config String, --driverloaded y|n, --nosync, --noudevsync, --profile String, --reportformat String, --version, --autobackup y|n, --debug, --force, --help, --stripes Number, --stripesize Number[k|unit], --nofsck, --quiet, --test, --verbose, --yes, --commandprofile String, --config String, --debug, --driverloaded y|n, --help, --profile String, --quiet, --verbose, --version, --yes, --force, --test, --noudevsync ]"; commands[85].required_opt_args[0].opt = size_ARG; commands[85].required_opt_args[0].def.val_bits = val_enum_to_bit(sizemb_VAL); commands[85].required_pos_args[0].pos = 1; commands[85].required_pos_args[0].def.val_bits = val_enum_to_bit(lv_VAL); commands[85].optional_opt_args[0].opt = commandprofile_ARG; commands[85].optional_opt_args[0].def.val_bits = val_enum_to_bit(string_VAL); commands[85].optional_opt_args[1].opt = config_ARG; commands[85].optional_opt_args[1].def.val_bits = val_enum_to_bit(string_VAL); commands[85].optional_opt_args[2].opt = debug_ARG; commands[85].optional_opt_args[3].opt = driverloaded_ARG; commands[85].optional_opt_args[3].def.val_bits = val_enum_to_bit(bool_VAL); commands[85].optional_opt_args[4].opt = help_ARG; commands[85].optional_opt_args[5].opt = profile_ARG; commands[85].optional_opt_args[5].def.val_bits = val_enum_to_bit(string_VAL); commands[85].optional_opt_args[6].opt = quiet_ARG; commands[85].optional_opt_args[7].opt = verbose_ARG; commands[85].optional_opt_args[8].opt = version_ARG; commands[85].optional_opt_args[9].opt = yes_ARG; commands[85].optional_opt_args[10].opt = alloc_ARG; commands[85].optional_opt_args[10].def.val_bits = val_enum_to_bit(alloc_VAL); commands[85].optional_opt_args[11].opt = autobackup_ARG; commands[85].optional_opt_args[11].def.val_bits = val_enum_to_bit(bool_VAL); commands[85].optional_opt_args[12].opt = force_ARG; commands[85].optional_opt_args[13].opt = nofsck_ARG; commands[85].optional_opt_args[14].opt = nosync_ARG; commands[85].optional_opt_args[15].opt = noudevsync_ARG; commands[85].optional_opt_args[16].opt = reportformat_ARG; commands[85].optional_opt_args[16].def.val_bits = val_enum_to_bit(string_VAL); commands[85].optional_opt_args[17].opt = resizefs_ARG; commands[85].optional_opt_args[18].opt = stripes_ARG; commands[85].optional_opt_args[18].def.val_bits = val_enum_to_bit(number_VAL); commands[85].optional_opt_args[19].opt = stripesize_ARG; commands[85].optional_opt_args[19].def.val_bits = val_enum_to_bit(sizekb_VAL); commands[85].optional_opt_args[20].opt = test_ARG; commands[85].optional_opt_args[21].opt = poolmetadatasize_ARG; commands[85].optional_opt_args[21].def.val_bits = val_enum_to_bit(sizemb_VAL); commands[85].optional_pos_args[0].pos = 2; commands[85].optional_pos_args[0].def.val_bits = val_enum_to_bit(pv_VAL); commands[85].optional_pos_args[0].def.flags = ARG_DEF_FLAG_MAY_REPEAT; Every user-entered command is compared against the set of command structures, and matched with one. An error is reported if an entered command does not have the required parameters for any definition. The closest match is printed as a suggestion, and running lvresize --help will display the usage for each possible lvresize command. The prototype syntax used for help/man output includes required --option and positional args on the first line, and optional --option and positional args enclosed in [ ] on subsequent lines. command_name <required_opt_args> <required_pos_args> [ <optional_opt_args> ] [ <optional_pos_args> ] $ lvresize --help lvresize - Resize a logical volume Resize an LV by a specified size. lvresize --size Number[m|unit] LV [ --resizefs, --poolmetadatasize Number[m|unit], COMMON_OPTIONS ] [ PV ... ] Resize an LV by a specified PV. lvresize LV PV ... [ --resizefs, COMMON_OPTIONS ] Resize a pool metadata SubLV by a specified size. lvresize --poolmetadatasize Number[m|unit] LV_thinpool [ COMMON_OPTIONS ] [ PV ... ] Common options: [ --alloc contiguous|cling|cling_by_tags|normal|anywhere|inherit, --commandprofile String, --config String, --driverloaded y|n, --nosync, --noudevsync, --profile String, --reportformat String, --version, --autobackup y|n, --debug, --force, --help, --stripes Number, --stripesize Number[k|unit], --nofsck, --quiet, --test, --verbose, --yes, --commandprofile String, --config String, --debug, --driverloaded y|n, --help, --profile String, --quiet, --verbose, --version, --yes, --force, --test, --noudevsync ] (Use --help --help for usage notes.) $ lvresize --poolmetadatasize 4 Failed to find a matching command definition. Closest command usage is: lvresize --poolmetadatasize Number[m|unit] LV_thinpool Man page prototypes are also generated from the same original command definitions, and are always in sync with the code and help text. Very early in command execution, a matching command definition is found. lvm then knows the operation being done, and that the provided args conform to the definition. This will allow lots of ad hoc checking/validation to be removed throughout the code. Each command definition can also be routed to a specific function to implement it. The function is associated with an enum value for the command definition (generated from the ID string.) These per-command-definition implementation functions have not yet been created, so all commands currently fall back to the existing per-command-name implementation functions. Using per-command-definition functions will allow lots of code to be removed which tries to figure out what the command is meant to do. This is currently based on ad hoc and complicated option analysis. When using the new functions, what the command is doing is already known from the associated command definition. So, this first phase validates every user-entered command against the set of command prototypes, then calls the existing implementation. The second phase can associate an implementation function with each definition, and take further advantage of the known operation to avoid the complicated option analysis.
Diffstat (limited to 'libdm/libdm-report.c')
0 files changed, 0 insertions, 0 deletions