From 4402f725136dd8ac8a8d7c643e934797714b08b5 Mon Sep 17 00:00:00 2001 From: Andy Grover Date: Mon, 11 Feb 2013 16:55:11 -0800 Subject: [PATCH] Make kmod.modprobe() raise an error if no modules found Add 'quiet' option to override. Add docstring. Signed-off-by: Andy Grover --- kmod/kmod.pyx | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/kmod/kmod.pyx b/kmod/kmod.pyx index 556b0f2..3e73a1c 100644 --- a/kmod/kmod.pyx +++ b/kmod/kmod.pyx @@ -102,8 +102,18 @@ cdef class Kmod (object): for mod in self.loaded(): yield (mod.name, mod.size) - def modprobe(self, alias_name, *args, **kwargs): - for mod in self.lookup(alias_name=alias_name): + def modprobe(self, name, quiet=False, *args, **kwargs): + """ + Load a module (or alias) and all modules on which it depends. + The 'quiet' option defaults to False; set to True to mimic the behavior + of the '--quiet' commandline option. + """ + mods = list(self.lookup(alias_name=name)) + + if not mods and not quiet: + raise _KmodError('Could not modprobe %s' % name) + + for mod in mods: mod.insert(*args, **kwargs) def rmmod(self, module_name, *args, **kwargs): -- 2.26.2