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

hints: fix non-empty hints list when not using hints

Browse Source 
When hints are invalid and ignored, the list of hints
could be non-empty (from additions before an invalid
hint was found).  This confused the calling code which
was checking for an empty list to see if hints were used.
Ensure the list is empty when hints are not used.
This commit is contained in:
David Teigland 2019-04-11 11:49:18 -05:00
parent 0cc80ccfd5
commit d0b869e46a
2 changed files with 19 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
lib/label/hints.c
View File

@ -1147,12 +1147,15 @@ check:
* Returns 1: use hints that are returned in hints list.
*/
int get_hints(struct cmd_context *cmd, struct dm_list *hints, int *newhints,
int get_hints(struct cmd_context *cmd, struct dm_list *hints_out, int *newhints,
struct dm_list *devs_in, struct dm_list *devs_out)
{
struct dm_list hints_list;
int needs_refresh = 0;
char *vgname = NULL;
dm_list_init(&hints_list);
/* Decide below if the caller should create new hints. */
*newhints = NEWHINTS_NONE;
@ -1230,7 +1233,7 @@ int get_hints(struct cmd_context *cmd, struct dm_list *hints, int *newhints,
/*
* couln't read file for some reason, not normal, just skip using hints
*/
if (!_read_hint_file(cmd, hints, &needs_refresh)) {
if (!_read_hint_file(cmd, &hints_list, &needs_refresh)) {
log_debug("get_hints: read fail");
_unlock_hints();
return 0;
@ -1259,7 +1262,7 @@ int get_hints(struct cmd_context *cmd, struct dm_list *hints, int *newhints,
* of the hints file so it will be recreated, and we must
* be following that since we found no hints.
*/
if (dm_list_empty(hints)) {
if (dm_list_empty(&hints_list)) {
log_debug("get_hints: no entries");
if (!_lock_hints(LOCK_EX, NONBLOCK))
@ -1279,12 +1282,14 @@ int get_hints(struct cmd_context *cmd, struct dm_list *hints, int *newhints,
* us which devs are PVs. We might want to enable this optimization
* separately.)
*/
_get_single_vgname_cmd_arg(cmd, hints, &vgname);
_get_single_vgname_cmd_arg(cmd, &hints_list, &vgname);
_apply_hints(cmd, hints, vgname, devs_in, devs_out);
_apply_hints(cmd, &hints_list, vgname, devs_in, devs_out);
log_debug("get_hints: applied using %d other %d",
dm_list_size(devs_out), dm_list_size(devs_in));
dm_list_splice(hints_out, &hints_list);
return 1;
}

16
lib/label/label.c
View File

@ -855,7 +855,7 @@ int label_scan(struct cmd_context *cmd)
{
struct dm_list all_devs;
struct dm_list scan_devs;
struct dm_list hints;
struct dm_list hints_list;
struct dev_iter *iter;
struct device_list *devl, *devl2;
struct device *dev;
@ -866,7 +866,7 @@ int label_scan(struct cmd_context *cmd)
dm_list_init(&all_devs);
dm_list_init(&scan_devs);
dm_list_init(&hints);
dm_list_init(&hints_list);
/*
* Iterate through all the devices in dev-cache (block devs that appear
@ -930,8 +930,10 @@ int label_scan(struct cmd_context *cmd)
* able to avoid rescan in vg_read, but locking early would
* apply to more cases.)
*/
if (!get_hints(cmd, &hints, &newhints, &all_devs, &scan_devs))
if (!get_hints(cmd, &hints_list, &newhints, &all_devs, &scan_devs)) {
dm_list_splice(&scan_devs, &all_devs);
dm_list_init(&hints_list);
}
log_debug("Will scan %d devices skip %d", dm_list_size(&scan_devs), dm_list_size(&all_devs));
@ -977,8 +979,8 @@ int label_scan(struct cmd_context *cmd)
dm_list_init(&cmd->hints);
if (!dm_list_empty(&hints)) {
if (!validate_hints(cmd, &hints)) {
if (!dm_list_empty(&hints_list)) {
if (!validate_hints(cmd, &hints_list)) {
/*
* We scanned a subset of all devices based on hints.
* With the results from the scan we may decide that
@ -986,11 +988,11 @@ int label_scan(struct cmd_context *cmd)
*/
log_debug("Will scan %d remaining devices", dm_list_size(&all_devs));
_scan_list(cmd, cmd->filter, &all_devs, NULL);
_free_hints(&hints);
_free_hints(&hints_list);
newhints = 0;
} else {
/* The hints may be used by another device iteration. */
dm_list_splice(&cmd->hints, &hints);
dm_list_splice(&cmd->hints, &hints_list);
}
}