[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [PATCH 3/3] Don't activate / de-activate dmraid sets on setup / teardown


On 11/06/2009 03:19 PM, Chris Lumens wrote:
diff --git a/storage/devices.py b/storage/devices.py
index bc8951f..7b4b314 100644
--- a/storage/devices.py
+++ b/storage/devices.py
@@ -2682,34 +2682,19 @@ class DMRaidArrayDevice(DiskDevice):
              self.sysfsPath = ''

-    def teardown(self, recursive=None):
-        """ Close, or tear down, a device. """
+    def deactivate(self):
+        """ Deactivate the raid set. """

Aren't we still going to end up getting DiskDevice.teardown called for
these devices in some loop somewhere?  Same for setup.

Ah, yes I forgot to put that in the commit message, yes this change means
that setup() and teardown() no get replaced with the ones from the super,
which in case of a recursive teardown means the members (parents) of the
set will have there teardown() called, since these parents are of
type StorageDevice, the teardown is a no-op.

I already checked using the super's setup() / teardown() would not
cause any issues, but I forgot to put this in the commit msg :|



- Chris

Anaconda-devel-list mailing list
Anaconda-devel-list redhat com

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]