From fe0ef947e1b70ea8fae1f5dfe53949afa65c0eac Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Sat, 14 May 2011 18:56:39 -0700 Subject: _LibGraphNode: re-use the _key attribute This allows us to avoid repeating any previous stat calls, which helps to avoid potential race conditions due to inconsistent stat results when the file system is being modified concurrently. --- pym/portage/dbapi/vartree.py | 6 +++--- pym/portage/util/_dyn_libs/LinkageMapELF.py | 13 ++++++++++--- 2 files changed, 13 insertions(+), 6 deletions(-) (limited to 'pym') diff --git a/pym/portage/dbapi/vartree.py b/pym/portage/dbapi/vartree.py index 4e19b1065..8de794573 100644 --- a/pym/portage/dbapi/vartree.py +++ b/pym/portage/dbapi/vartree.py @@ -2309,7 +2309,7 @@ class dblink(object): def path_to_node(path): node = path_node_map.get(path) if node is None: - node = LinkageMap._LibGraphNode(path, root) + node = LinkageMap._LibGraphNode(linkmap._obj_key(path)) alt_path_node = lib_graph.get(node) if alt_path_node is not None: node = alt_path_node @@ -2464,6 +2464,7 @@ class dblink(object): # Since preserved libraries can be consumers of other preserved # libraries, use a graph to track consumer relationships. plib_dict = self.vartree.dbapi._plib_registry.getPreservedLibs() + linkmap = self.vartree.dbapi._linkmap lib_graph = digraph() preserved_nodes = set() preserved_paths = set() @@ -2474,7 +2475,7 @@ class dblink(object): def path_to_node(path): node = path_node_map.get(path) if node is None: - node = LinkageMap._LibGraphNode(path, root) + node = LinkageMap._LibGraphNode(linkmap._obj_key(path)) alt_path_node = lib_graph.get(node) if alt_path_node is not None: node = alt_path_node @@ -2482,7 +2483,6 @@ class dblink(object): path_node_map[path] = node return node - linkmap = self.vartree.dbapi._linkmap for cpv, plibs in plib_dict.items(): for f in plibs: path_cpv_map[f] = cpv diff --git a/pym/portage/util/_dyn_libs/LinkageMapELF.py b/pym/portage/util/_dyn_libs/LinkageMapELF.py index 3305aca49..2c40a346e 100644 --- a/pym/portage/util/_dyn_libs/LinkageMapELF.py +++ b/pym/portage/util/_dyn_libs/LinkageMapELF.py @@ -60,7 +60,7 @@ class LinkageMapELF(object): """Helper class used as _obj_properties keys for objects.""" - __slots__ = ("__weakref__", "_key") + __slots__ = ("_key",) def __init__(self, obj, root): """ @@ -135,8 +135,15 @@ class LinkageMapELF(object): class _LibGraphNode(_ObjectKey): __slots__ = ("alt_paths",) - def __init__(self, obj, root): - LinkageMapELF._ObjectKey.__init__(self, obj, root) + def __init__(self, key): + """ + Create a _LibGraphNode from an existing _ObjectKey. + This re-uses the _key attribute in order to avoid repeating + any previous stat calls, which helps to avoid potential race + conditions due to inconsistent stat results when the + file system is being modified concurrently. + """ + self._key = key._key self.alt_paths = set() def __str__(self): -- cgit v1.2.3-1-g7c22