shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2025-08-30 05:49:28 +03:00
Code Releases Activity

Remove false "failed to find tree node for <lv>" error from _cached_info().

Browse Source 
When activating a merging origin it is valid, and expected, to not have
a node in the deptree for both the origin and its merging snapshot.  The
_cached_info() caller is only concerned with whether a device is open.
If there isn't a node in the tree the associated device is definitely
not open.
This commit is contained in:
Mike Snitzer
2010-02-08 23:28:06 +00:00
parent 57f5ec5d6c
commit d6bf26af8e
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
WHATS_NEW
View File

@ -1,5 +1,6 @@
Version 2.02.61 - Version 2.02.61 -
=================================== ===================================
Remove false "failed to find tree node for" error when activating merging origin.
Make lvconvert --repair --use-policies exit with success when no action is needed. Make lvconvert --repair --use-policies exit with success when no action is needed.
Add multiple snapshot lv 'lvconvert --merge @tag' support via process_each_lv(). Add multiple snapshot lv 'lvconvert --merge @tag' support via process_each_lv().
Fix inappropriate second resync when adding mimage to core-logged mirror. Fix inappropriate second resync when adding mimage to core-logged mirror.

5
lib/activate/dev_manager.c
View File

@ -264,10 +264,9 @@ static const struct dm_info *_cached_info(struct dm_pool *mem,
return NULL; return NULL;
} }
if (!(dnode = dm_tree_find_node_by_uuid(dtree, dlid))) { /* An activating merging origin won't have a node in the tree yet */
log_error("failed to find tree node for %s", lv->name); if (!(dnode = dm_tree_find_node_by_uuid(dtree, dlid)))
return NULL; return NULL;
}
if (!(dinfo = dm_tree_node_get_info(dnode))) { if (!(dinfo = dm_tree_node_get_info(dnode))) {
log_error("failed to get info from tree node for %s", lv->name); log_error("failed to get info from tree node for %s", lv->name);