shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-22 17:35:59 +03:00
Code Releases Activity

libdm: track implicit dependecies

Browse Source 
When the node enters dtree with implicit dependency, it
automatically has udev flags from parent node
and could not be changed later when the node has been
entered again via i.e lvm's preload tracking.

Resolve this by tracking whether the node has been
created by implicit dependency tracking or has been
entered explicitely. Implicit node could be later
upgraded by an explicit _add_dev() with proper udev_flags.

For implicit devices add special udev flags to avoid
any scan and udev rule processing if we resume such device.

Patch allows easier removing of orphan nodes.
This commit is contained in:
Zdenek Kabelac 2014-04-07 20:21:20 +02:00
parent 05a421960f
commit bd2500e62e
2 changed files with 16 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
WHATS_NEW_DM
View File

@ -1,5 +1,6 @@
Version 1.02.85 - Version 1.02.85 -
=================================== ===================================
Add tracking flag about implicitly added devices into dm_tree.
Stop timeout thread immediately when the last worker thread is finished. Stop timeout thread immediately when the last worker thread is finished.
Fix dmeventd logging with parallel wait event processing. Fix dmeventd logging with parallel wait event processing.
Reuse _node_send_messages() for validation of transaction_id in preload. Reuse _node_send_messages() for validation of transaction_id in preload.

20
libdm/libdm-deptree.c
View File

@ -263,6 +263,7 @@ struct dm_tree_node {
struct dm_list used_by; /* Nodes that use this node */ struct dm_list used_by; /* Nodes that use this node */
int activation_priority; /* 0 gets activated first */ int activation_priority; /* 0 gets activated first */
int implicit_deps; /* 1 device only implicitly referenced */
uint16_t udev_flags; /* Udev control flags */ uint16_t udev_flags; /* Udev control flags */
@ -1171,7 +1172,8 @@ struct dm_tree_node *dm_tree_add_new_dev(struct dm_tree *dtree, const char *name
static struct dm_tree_node *_add_dev(struct dm_tree *dtree, static struct dm_tree_node *_add_dev(struct dm_tree *dtree,
struct dm_tree_node *parent, struct dm_tree_node *parent,
uint32_t major, uint32_t minor, uint32_t major, uint32_t minor,
uint16_t udev_flags) uint16_t udev_flags,
int implicit_deps)
{ {
struct dm_task *dmt = NULL; struct dm_task *dmt = NULL;
struct dm_info info; struct dm_info info;
@ -1191,6 +1193,10 @@ static struct dm_tree_node *_add_dev(struct dm_tree *dtree,
NULL, udev_flags))) NULL, udev_flags)))
goto_out; goto_out;
new = 1; new = 1;
node->implicit_deps = implicit_deps;
} else if (!implicit_deps && node->implicit_deps) {
node->udev_flags = udev_flags;
node->implicit_deps = 0;
} }
if (!_link_tree_nodes(parent, node)) { if (!_link_tree_nodes(parent, node)) {
@ -1213,8 +1219,12 @@ static struct dm_tree_node *_add_dev(struct dm_tree *dtree,
/* Add dependencies to tree */ /* Add dependencies to tree */
for (i = 0; i < deps->count; i++) for (i = 0; i < deps->count; i++)
/* Implicit devices are by default temporary */
if (!_add_dev(dtree, node, MAJOR(deps->device[i]), if (!_add_dev(dtree, node, MAJOR(deps->device[i]),
MINOR(deps->device[i]), udev_flags)) { MINOR(deps->device[i]), udev_flags |
DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG |
DM_UDEV_DISABLE_DISK_RULES_FLAG |
DM_UDEV_DISABLE_OTHER_RULES_FLAG, 1)) {
node = NULL; node = NULL;
goto_out; goto_out;
} }
@ -1228,13 +1238,13 @@ out:
int dm_tree_add_dev(struct dm_tree *dtree, uint32_t major, uint32_t minor) int dm_tree_add_dev(struct dm_tree *dtree, uint32_t major, uint32_t minor)
{ {
return _add_dev(dtree, &dtree->root, major, minor, 0) ? 1 : 0; return _add_dev(dtree, &dtree->root, major, minor, 0, 0) ? 1 : 0;
} }
int dm_tree_add_dev_with_udev_flags(struct dm_tree *dtree, uint32_t major, int dm_tree_add_dev_with_udev_flags(struct dm_tree *dtree, uint32_t major,
uint32_t minor, uint16_t udev_flags) uint32_t minor, uint16_t udev_flags)
{ {
return _add_dev(dtree, &dtree->root, major, minor, udev_flags) ? 1 : 0; return _add_dev(dtree, &dtree->root, major, minor, udev_flags, 0) ? 1 : 0;
} }
static int _rename_node(const char *old_name, const char *new_name, uint32_t major, static int _rename_node(const char *old_name, const char *new_name, uint32_t major,
@ -3939,7 +3949,7 @@ int dm_tree_node_add_target_area(struct dm_tree_node *node,
/* FIXME Check correct macro use */ /* FIXME Check correct macro use */
if (!(dev_node = _add_dev(node->dtree, node, MAJOR(info.st_rdev), if (!(dev_node = _add_dev(node->dtree, node, MAJOR(info.st_rdev),
MINOR(info.st_rdev), 0))) MINOR(info.st_rdev), 0, 0)))
return_0; return_0;
} }