shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2025-01-09 01:18:39 +03:00
Code Releases Activity

Refactor _has_matching_pv_tag to provide a fn that takes PV structs.

Browse Source
This commit is contained in:
Alasdair Kergon 2012-05-11 15:26:30 +00:00
parent dcc18799ec
commit b63eb871fd
1 changed files with 12 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
lib/metadata/lv_manip.c
View File

@ -1248,13 +1248,13 @@ static int _is_same_pv(struct pv_match *pvmatch __attribute((unused)), struct pv
* Does PV area have a tag listed in allocation/cling_tag_list that * Does PV area have a tag listed in allocation/cling_tag_list that
* matches a tag of the PV of the existing segment? * matches a tag of the PV of the existing segment?
*/ */
static int _has_matching_pv_tag(struct pv_match *pvmatch, struct pv_segment *pvseg, struct pv_area *pva) static int _pvs_have_matching_tag(const struct dm_config_node *cling_tag_list_cn, struct physical_volume *pv1, struct physical_volume *pv2)
{ {
const struct dm_config_value *cv; const struct dm_config_value *cv;
const char *str; const char *str;
const char *tag_matched; const char *tag_matched;
for (cv = pvmatch->cling_tag_list_cn->v; cv; cv = cv->next) { for (cv = cling_tag_list_cn->v; cv; cv = cv->next) {
if (cv->type != DM_CFG_STRING) { if (cv->type != DM_CFG_STRING) {
log_error("Ignoring invalid string in config file entry " log_error("Ignoring invalid string in config file entry "
"allocation/cling_tag_list"); "allocation/cling_tag_list");
@ -1283,21 +1283,21 @@ static int _has_matching_pv_tag(struct pv_match *pvmatch, struct pv_segment *pvs
/* Wildcard matches any tag against any tag. */ /* Wildcard matches any tag against any tag. */
if (!strcmp(str, "*")) { if (!strcmp(str, "*")) {
if (!str_list_match_list(&pvseg->pv->tags, &pva->map->pv->tags, &tag_matched)) if (!str_list_match_list(&pv1->tags, &pv2->tags, &tag_matched))
continue; continue;
else { else {
log_debug("Matched allocation PV tag %s on existing %s with free space on %s.", log_debug("Matched allocation PV tag %s on existing %s with free space on %s.",
tag_matched, pv_dev_name(pvseg->pv), pv_dev_name(pva->map->pv)); tag_matched, pv_dev_name(pv1), pv_dev_name(pv2));
return 1; return 1;
} }
} }
if (!str_list_match_item(&pvseg->pv->tags, str) || if (!str_list_match_item(&pv1->tags, str) ||
!str_list_match_item(&pva->map->pv->tags, str)) !str_list_match_item(&pv2->tags, str))
continue; continue;
else { else {
log_debug("Matched allocation PV tag %s on existing %s with free space on %s.", log_debug("Matched allocation PV tag %s on existing %s with free space on %s.",
str, pv_dev_name(pvseg->pv), pv_dev_name(pva->map->pv)); str, pv_dev_name(pv1), pv_dev_name(pv2));
return 1; return 1;
} }
} }
@ -1305,6 +1305,11 @@ static int _has_matching_pv_tag(struct pv_match *pvmatch, struct pv_segment *pvs
return 0; return 0;
} }
static int _has_matching_pv_tag(struct pv_match *pvmatch, struct pv_segment *pvseg, struct pv_area *pva)
{
return _pvs_have_matching_tag(pvmatch->cling_tag_list_cn, pvseg->pv, pva->map->pv);
}
/* /*
* Is PV area contiguous to PV segment? * Is PV area contiguous to PV segment?
*/ */