1
0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-21 13:34:40 +03:00

writecache: add settings cleaner and max_age

available in dm-writecache 1.2
This commit is contained in:
David Teigland 2020-03-31 14:14:50 -05:00
parent d15c466f95
commit a7b2fc8f57
7 changed files with 98 additions and 1 deletions

View File

@ -951,6 +951,8 @@ struct writecache_settings {
uint64_t autocommit_time; /* in milliseconds */
uint32_t fua;
uint32_t nofua;
uint32_t cleaner;
uint32_t max_age;
/*
* Allow an unrecognized key and its val to be passed to the kernel for
@ -970,6 +972,8 @@ struct writecache_settings {
unsigned autocommit_time_set:1;
unsigned fua_set:1;
unsigned nofua_set:1;
unsigned cleaner_set:1;
unsigned max_age_set:1;
};
int dm_tree_node_add_writecache_target(struct dm_tree_node *node,

View File

@ -2670,6 +2670,10 @@ static int _writecache_emit_segment_line(struct dm_task *dmt,
count += 1;
if (seg->writecache_settings.nofua_set)
count += 1;
if (seg->writecache_settings.cleaner_set && seg->writecache_settings.cleaner)
count += 1;
if (seg->writecache_settings.max_age_set)
count += 2;
if (seg->writecache_settings.new_key)
count += 2;
@ -2713,6 +2717,14 @@ static int _writecache_emit_segment_line(struct dm_task *dmt,
EMIT_PARAMS(pos, " nofua");
}
if (seg->writecache_settings.cleaner_set && seg->writecache_settings.cleaner) {
EMIT_PARAMS(pos, " cleaner");
}
if (seg->writecache_settings.max_age_set) {
EMIT_PARAMS(pos, " max_age %u", seg->writecache_settings.max_age);
}
if (seg->writecache_settings.new_key) {
EMIT_PARAMS(pos, " %s %s",
seg->writecache_settings.new_key,

View File

@ -454,6 +454,14 @@ int writecache_settings_to_str_list(struct writecache_settings *settings, struct
if (!_writecache_setting_str_list_add("nofua", (uint64_t)settings->nofua, NULL, result, mem))
errors++;
if (settings->cleaner_set && settings->cleaner)
if (!_writecache_setting_str_list_add("cleaner", (uint64_t)settings->cleaner, NULL, result, mem))
errors++;
if (settings->max_age_set)
if (!_writecache_setting_str_list_add("max_age", (uint64_t)settings->max_age, NULL, result, mem))
errors++;
if (settings->new_key && settings->new_val)
if (!_writecache_setting_str_list_add(settings->new_key, 0, settings->new_val, result, mem))
errors++;

View File

@ -1434,7 +1434,7 @@ static int _cache_settings_disp(struct dm_report *rh, struct dm_pool *mem,
if (!(result = str_list_create(mem)))
return_0;
if (!writecache_settings_to_str_list(&seg->writecache_settings, result, mem))
if (!writecache_settings_to_str_list((struct writecache_settings *)&seg->writecache_settings, result, mem))
return_0;
return _field_set_string_list(rh, field, result, private, 0, NULL);

View File

@ -26,6 +26,9 @@
#include "lib/metadata/lv_alloc.h"
#include "lib/config/defaults.h"
static int _writecache_cleaner_supported;
static int _writecache_max_age_supported;
#define SEG_LOG_ERROR(t, p...) \
log_error(t " segment %s of logical volume %s.", ## p, \
dm_config_parent_name(sn), seg->lv->name), 0;
@ -120,6 +123,18 @@ static int _writecache_text_import(struct lv_segment *seg,
seg->writecache_settings.nofua_set = 1;
}
if (dm_config_has_node(sn, "cleaner")) {
if (!dm_config_get_uint32(sn, "cleaner", &seg->writecache_settings.cleaner))
return SEG_LOG_ERROR("Unknown writecache_setting in");
seg->writecache_settings.cleaner_set = 1;
}
if (dm_config_has_node(sn, "max_age")) {
if (!dm_config_get_uint32(sn, "max_age", &seg->writecache_settings.max_age))
return SEG_LOG_ERROR("Unknown writecache_setting in");
seg->writecache_settings.max_age_set = 1;
}
if (dm_config_has_node(sn, "writecache_setting_key")) {
const char *key;
const char *val;
@ -184,6 +199,14 @@ static int _writecache_text_export(const struct lv_segment *seg,
outf(f, "nofua = %u", seg->writecache_settings.nofua);
}
if (seg->writecache_settings.cleaner_set && seg->writecache_settings.cleaner) {
outf(f, "cleaner = %u", seg->writecache_settings.cleaner);
}
if (seg->writecache_settings.max_age_set) {
outf(f, "max_age = %u", seg->writecache_settings.max_age);
}
if (seg->writecache_settings.new_key && seg->writecache_settings.new_val) {
outf(f, "writecache_setting_key = \"%s\"",
seg->writecache_settings.new_key);
@ -208,6 +231,7 @@ static int _target_present(struct cmd_context *cmd,
{
static int _writecache_checked = 0;
static int _writecache_present = 0;
uint32_t maj, min, patchlevel;
if (!activation())
return 0;
@ -215,6 +239,19 @@ static int _target_present(struct cmd_context *cmd,
if (!_writecache_checked) {
_writecache_checked = 1;
_writecache_present = target_present(cmd, TARGET_NAME_WRITECACHE, 1);
if (!target_version(TARGET_NAME_WRITECACHE, &maj, &min, &patchlevel))
return_0;
if (maj < 1) {
log_error("writecache target version older than minimum 1.0.0");
return 0;
}
if (min >= 2) {
_writecache_cleaner_supported = 1;
_writecache_max_age_supported = 1;
}
}
return _writecache_present;
@ -257,6 +294,18 @@ static int _writecache_add_target_line(struct dev_manager *dm,
return 0;
}
if (!_writecache_cleaner_supported && seg->writecache_settings.cleaner_set && seg->writecache_settings.cleaner) {
log_warn("WARNING: ignoring writecache setting \"cleaner\" which is not supported by kernel for LV %s.", seg->lv->name);
seg->writecache_settings.cleaner = 0;
seg->writecache_settings.cleaner_set = 0;
}
if (!_writecache_max_age_supported && seg->writecache_settings.max_age_set) {
log_warn("WARNING: ignoring writecache setting \"max_age\" which is not supported by kernel for LV %s.", seg->lv->name);
seg->writecache_settings.max_age = 0;
seg->writecache_settings.max_age_set = 0;
}
if ((pmem = lv_on_pmem(seg->writecache)) < 0)
return_0;

View File

@ -658,6 +658,16 @@ static int _lvchange_writecache(struct cmd_context *cmd,
seg->writecache_settings.nofua = settings.nofua;
set_count++;
}
if (settings.cleaner_set) {
seg->writecache_settings.cleaner_set = settings.cleaner_set;
seg->writecache_settings.cleaner = settings.cleaner;
set_count++;
}
if (settings.max_age_set) {
seg->writecache_settings.max_age_set = settings.max_age_set;
seg->writecache_settings.max_age = settings.max_age;
set_count++;
}
if (!set_count) {
/*

View File

@ -1262,6 +1262,20 @@ static int _get_one_writecache_setting(struct cmd_context *cmd, struct writecach
return 1;
}
if (!strncmp(key, "cleaner", strlen("cleaner"))) {
if (sscanf(val, "%u", &settings->cleaner) != 1)
goto_bad;
settings->cleaner_set = 1;
return 1;
}
if (!strncmp(key, "max_age", strlen("max_age"))) {
if (sscanf(val, "%u", &settings->max_age) != 1)
goto_bad;
settings->max_age_set = 1;
return 1;
}
if (settings->new_key) {
log_error("Setting %s is not recognized. Only one unrecognized setting is allowed.", key);
return 0;