sascha/devlib
1
0
Fork 0
mirror of https://github.com/ARM-software/devlib.git synced 2025-09-08 13:01:54 +01:00
Code Issues Releases Wiki Activity

cgroups: fix pylin bug

Browse Source 
In:

   commit 454b9450 ("pylint fixes")

we added:

```diff
@@ -363,7 +368,7 @@ class CgroupsModule(Module):

         # Get the list of the available controllers
         subsys = self.list_subsystems()
-        if len(subsys) == 0:
+        if subsys:
             self.logger.warning('No CGroups controller available')
             return

```

which changed the invariant condition to enabled the cgroup module:
the module is enabled we we can find a "non empty" list of subsystems.

Let's fix this to bail out on an empyt list.

Signed-off-by: Patrick Bellasi <patrick.bellasi@arm.com>
This commit is contained in:
Patrick Bellasi
2018-07-18 16:21:12 +01:00
committed by Marc Bonnici
parent 68b418dac2
commit 241c7e01bd
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
devlib/module/cgroups.py
View File

@@ -368,7 +368,7 @@ class CgroupsModule(Module):
# Get the list of the available controllers # Get the list of the available controllers
subsys = self.list_subsystems() subsys = self.list_subsystems()
if subsys: if not subsys:
self.logger.warning('No CGroups controller available') self.logger.warning('No CGroups controller available')
return return