Skip to content

updates to stacking and lvconvert

Jun'ichi Nomura lvm-devel 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest11 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/11lvconvert takes '--wait' instead of '--background'+0 -02 replies
1/11lvconvert takes '--wait' instead of '--background'+0 -00 replies
2/11Add upward link from underlying LV to stacked seg+0 -03 replies
2/11Add upward link from underlying LV to stacked seg+0 -01 reply
2/11Add upward link from underlying LV to stacked seg+0 -00 replies
2/11Add upward link from underlying LV to stacked seg+0 -00 replies
3/11Move mirror log initialization functions+0 -00 replies
4/11Move _find_tmp_mirror()+0 -00 replies
5/11Move removable_pvs checking+0 -00 replies
6/11Export find_tmp_mirror()+0 -00 replies
7/11lvconvert adds/removes mirror log to/from the original LV+0 -02 replies
8/11Allow "mirror" segment with only 1 area+0 -00 replies
9/11_init_mirror_log() to cope with activation status+0 -00 replies
10/11Fix _remove_mirror_images() to cope with stacked mirror+0 -00 replies
11/11Fix _remove_mirror_images() to skip AREA_LV+0 -00 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/11] updates to stacking and lvconvert

Hi,

These patches do followings:

- Change default behavior of lvconvert
[ 1/11] lvconvert takes '--wait' instead of '--background'

- Add generalized upward link from LV to seg
[ 2/11] Add upward link from underlying LV to stacked seg

- Cosmetic changes for later bugfixes
[ 3/11] Move mirror log initialization functions [ 4/11] Move _find_tmp_mirror()
[ 5/11] Move removable_pvs checking

- Fix mirror log addition/removal for LV under conversion [ 6/11] Export find_tmp_mirror()
[ 7/11] lvconvert adds/removes mirror log to/from the original LV

- Fix remove_mirror_images()
[ 8/11] Allow "mirror" segment with only 1 area
[ 9/11] _init_mirror_log() to cope with activation status [10/11] Fix _remove_mirror_images() to cope with stacked mirror [11/11] Fix _remove_mirror_images() to skip AREA_LV

- Appendix: 2 test scripts for mirroring

Since the last week's patchset,

Quoted 3 lines

Other things I would like to work on next are:
- Generalization of seg->mirrored_seg as upward link from LV to seg(s) which use the LV.

Done with the patch "2/11" of this series.

Quoted 2 lines

- Review and fix remove_mirror_images() so that 'removable_pvs' filtering works on stacked mirror.

remove_mirror_images() didn't work nicely with non-empty removable_pvs.
Fixed by patches "8/11" - "11/11".

Quoted 3 lines

- Review the suspend code path. I think 'noflush' should be avoided for lower-level mirrors. Otherwise the upper-level mirror can't suspend.

This was not a problem.

Quoted 6 lines

- Remaining patches from the last post:
https://www.redhat.com/archives/lvm-devel/2007-December/msg00045.html
[11/13] Add 'lv_mirrors' field to reporting functions [12/13] Remove hardcoded "_mlog" name from deptree construction [13/13] Allow disk logs for temporary resync mirror
- Creating a script for 'make check'

Signature · 1 line
Jun'ichi Nomura, NEC Corporation of America