diff options
| author | Andrew Jeffery <andrew@aj.id.au> | 2018-05-10 02:17:34 +0930 |
|---|---|---|
| committer | Andrew Jeffery <andrew@aj.id.au> | 2018-05-15 09:30:25 +0930 |
| commit | 005e47adf4498040ca457840e01d74d58b918db0 (patch) | |
| tree | 8c7037d73ffdf69b84881760defaed66b44565fc | |
| parent | 4d49f95c2350820266ed4c6b2ae675a3e959c6b9 (diff) | |
| download | phosphor-objmgr-005e47adf4498040ca457840e01d74d58b918db0.tar.gz phosphor-objmgr-005e47adf4498040ca457840e01d74d58b918db0.zip | |
server: Only update_interfaces() if process_old_owner() finds an owner
This avoids a large number of set operations that clearly work on empty
sets, but serve no purpose in the circumstance.
Change-Id: Idda0ddac2e2df5bef44a3ff3a5790f0ba9c5a47a
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
| -rw-r--r-- | obmc/mapper/server.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/obmc/mapper/server.py b/obmc/mapper/server.py index 8d23a4c..97845f1 100644 --- a/obmc/mapper/server.py +++ b/obmc/mapper/server.py @@ -391,9 +391,10 @@ class ObjectMapper(dbus.service.Object): for path, item in self.cache.dataitems(): old = self.interfaces_get(item, owned_name) - # remove all interfaces for this service - self.update_interfaces( - path, owned_name, old=old, new=[]) + if old: + # remove all interfaces for this service + self.update_interfaces( + path, owned_name, old=old, new=[]) def bus_handler(self, owned_name, old, new): if obmc.dbuslib.bindings.is_unique(owned_name) or \ |
