diff options
| author | Alasdair Kergon <agk@redhat.com> | 2011-04-28 20:29:59 +0000 |
|---|---|---|
| committer | Alasdair Kergon <agk@redhat.com> | 2011-04-28 20:29:59 +0000 |
| commit | 9cda028a96e1f0f30758e7a69d06576951ca9193 (patch) | |
| tree | c9d1311b85b3036a07dab5a5bd3134b2ee45526d /lib/metadata/mirror.c | |
| parent | 96c4abee626b83d5847ca3c34b2e6fdf6534b56f (diff) | |
| download | lvm2-9cda028a96e1f0f30758e7a69d06576951ca9193.tar.gz lvm2-9cda028a96e1f0f30758e7a69d06576951ca9193.tar.xz lvm2-9cda028a96e1f0f30758e7a69d06576951ca9193.zip | |
clean up critical section patch
Diffstat (limited to 'lib/metadata/mirror.c')
| -rw-r--r-- | lib/metadata/mirror.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/metadata/mirror.c b/lib/metadata/mirror.c index 95d07bda..b02c7bb3 100644 --- a/lib/metadata/mirror.c +++ b/lib/metadata/mirror.c @@ -987,11 +987,11 @@ static int _remove_mirror_images(struct logical_volume *lv, } /* FIXME: second suspend should not be needed - * Explicitly suspend temporary LV - * This balance critical_section_inc() calls with critical_section_dec() in resume - * (both localy and in cluster) and also properly propagates precommited + * Explicitly suspend temporary LV. + * This balances critical_section_inc() calls with critical_section_dec() + * in resume (both local and cluster) and also properly propagates precommitted * metadata into dm table on other nodes. - * (visible flag set causes the suspend is not properly propagated?) + * FIXME: check propagation of suspend with visible flag */ if (temp_layer_lv && !suspend_lv(temp_layer_lv->vg->cmd, temp_layer_lv)) log_error("Problem suspending temporary LV %s", temp_layer_lv->name); |