]> sourceware.org Git - lvm2.git/commit
Revert initial solution to bug 733114 - I/O error message during splitmirror
authorJonathan Earl Brassow <jbrassow@redhat.com>
Thu, 6 Oct 2011 14:49:16 +0000 (14:49 +0000)
committerJonathan Earl Brassow <jbrassow@redhat.com>
Thu, 6 Oct 2011 14:49:16 +0000 (14:49 +0000)
commit6c0b0e5d9a79349fb3acd03780c184f72da62e55
tree0cecd9fe1c01322858d72a480d97b9e55997491a
parent83c606ae3018e161dd0d4730ddb8c03c217638f9
Revert initial solution to bug 733114 - I/O error message during splitmirror

The original commit comments can be located via this git commit ID:
7d8e615c0b30fc2ef300c90378a51f01c328128c

There were three possible solutions to the original problem proposed in the
initial check-in.  The one chosen was as follows:
    2) Do like _remove_mirror_images does and suspend the original, then suspend
    the sub-lv (the error target), then resume the sub-lv, and finally resume the
    original LV.  This seems like extra pointless operations to me, but it doesn't
    produce the error message (although, I'm not sure why) and it allows us to
    leave the visible flag in place.
Turns out, the cluster also views the extra suspend/resume operations as
pointless too and ignores them.  So, this solution doesn't work in a cluster.
Further, I've noticed that in addition to the remote cluster nodes still getting
I/O errors from scanning the error target, they also have a different LVM and
DM views of the same LV.  IOW, while the LVM level (gotten from the LVM metadata)
sees the correct name for the newly split LV, device-mapper still maintains the
old names.

Because the original fix failed to completely fix the problem (or work-around it)
and because a better solution must be found to address the additional cluster
issue of device renaming, I am reverting the above mentioned commit.
WHATS_NEW
lib/metadata/mirror.c
This page took 0.029301 seconds and 5 git commands to generate.