diff options
author | Tony Asleson <tasleson@redhat.com> | 2022-08-17 12:09:30 -0500 |
---|---|---|
committer | Tony Asleson <tasleson@redhat.com> | 2022-09-16 10:49:37 -0500 |
commit | 60e4ba36e0843c9536278ff33ce6dfda9cfd8079 (patch) | |
tree | 5269cecfa263bab97f168cfe656e5cdbb5140872 | |
parent | cfc87157a4fdec39c2b570bd35ce812cda3003f6 (diff) | |
download | lvm2-60e4ba36e0843c9536278ff33ce6dfda9cfd8079.tar.gz |
lvmdbusd: Remove unused locking functionality
I don't think we have ever utilized this, remove.
-rw-r--r-- | daemons/lvmdbusd/objectmanager.py | 35 |
1 files changed, 0 insertions, 35 deletions
diff --git a/daemons/lvmdbusd/objectmanager.py b/daemons/lvmdbusd/objectmanager.py index 95d33db19..a0c4a50ef 100644 --- a/daemons/lvmdbusd/objectmanager.py +++ b/daemons/lvmdbusd/objectmanager.py @@ -53,15 +53,6 @@ class ObjectManager(AutomatedProperties): (self, ), cb, cbe, False) cfg.worker_q.put(r) - def locked(self): - """ - If some external code need to run across a number of different - calls into ObjectManager while blocking others they can use this method - to lock others out. - :return: - """ - return ObjectManagerLock(self.rlock) - @dbus.service.signal( dbus_interface="org.freedesktop.DBus.ObjectManager", signature='oa{sa{sv}}') @@ -337,29 +328,3 @@ class ObjectManager(AutomatedProperties): # (uuid, lvm_id, str(path_create), path)) return path - - -class ObjectManagerLock(object): - """ - The sole purpose of this class is to allow other code the ability to - lock the object manager using a `with` statement, eg. - - with cfg.om.locked(): - # Do stuff with object manager - - This will ensure that the lock is always released (assuming this is done - correctly) - """ - - def __init__(self, recursive_lock): - self._lock = recursive_lock - - def __enter__(self): - # Acquire lock - self._lock.acquire() - - # noinspection PyUnusedLocal - def __exit__(self, e_type, e_value, e_traceback): - # Release lock - self._lock.release() - self._lock = None |