summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Teigland <teigland@redhat.com>2021-02-02 15:14:05 -0600
committerDavid Teigland <teigland@redhat.com>2021-02-02 15:14:05 -0600
commit834cba000a68675909844865f56e77ab62c1943b (patch)
treedd7c4dcae9da2d117a53397389f6fc998e2102d7
parentcb54d0801d776205788f8f46b31dd9e487833343 (diff)
downloadlvm2-834cba000a68675909844865f56e77ab62c1943b.tar.gz
hints: clean up comment and init settings
no behavior change
-rw-r--r--tools/lvmcmdline.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/tools/lvmcmdline.c b/tools/lvmcmdline.c
index b84a9a014..5e0edcade 100644
--- a/tools/lvmcmdline.c
+++ b/tools/lvmcmdline.c
@@ -2427,13 +2427,15 @@ static int _get_current_settings(struct cmd_context *cmd)
/*
* enable_hints is set to 1 if any commands are using hints.
- * use_hints is set to 1 if this command doesn't use the hints.
+ * use_hints is set to 1 if this command should use the hints.
* enable_hints=1 and use_hints=0 means that this command won't
* use the hints, but it may invalidate the hints that are used
* by other commands.
*
* enable_hints=0 means no commands are using hints, so this
* command would not need to invalidate hints for other cmds.
+ *
+ * Code should check !enable_hints before checking use_hints.
*/
cmd->enable_hints = 1;
@@ -2444,8 +2446,10 @@ static int _get_current_settings(struct cmd_context *cmd)
cmd->use_hints = 0;
if ((hint_mode = find_config_tree_str(cmd, devices_hints_CFG, NULL))) {
- if (!strcmp(hint_mode, "none"))
+ if (!strcmp(hint_mode, "none")) {
cmd->enable_hints = 0;
+ cmd->use_hints = 0;
+ }
}
cmd->partial_activation = 0;