[Devel] [PATCH RHEL7 COMMIT] ms/kmod: remove false positive warning "waiting module removal not supported: please upgrade"
Konstantin Khorenko
khorenko at virtuozzo.com
Thu Jun 9 02:17:18 PDT 2016
The commit is pushed to "branch-rh7-3.10.0-327.18.2.vz7.14.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh7-3.10.0-327.18.2.vz7.14.13
------>
commit 78612e86dbbf2114115bb6475c6734bd9f1f60b6
Author: Rusty Russell <rusty at rustcorp.com.au>
Date: Thu Jun 9 13:17:18 2016 +0400
ms/kmod: remove false positive warning "waiting module removal not supported: please upgrade"
commit 79465d2fd48e68940c2bdecddbdecd45bbba06fe
Date: Mon Apr 28 11:05:43 2014 +0930
module: remove warning about waiting module removal.
We remove the waiting module removal in commit 3f2b9c9cdf38 (September
2013), but it turns out that modprobe in kmod (< version 16) was
asking for waiting module removal. No one noticed since modprobe would
check for 0 usage immediately before trying to remove the module, and
the race is unlikely.
However, it means that anyone running old (but not ancient) kmod
versions is hitting the printk designed to see if anyone was running
"rmmod -w". All reports so far have been false positives, so remove
the warning.
Fixes: 3f2b9c9cdf389e303b2273679af08aab5f153517
Reported-by: Valerio Vanni <valerio.vanni at inwind.it>
Cc: Elliott, Robert (Server Storage) <Elliott at hp.com>
Cc: stable at kernel.org
Acked-by: Lucas De Marchi <lucas.de.marchi at gmail.com>
Signed-off-by: Rusty Russell <rusty at rustcorp.com.au>
Backported to vz7 kernel,
fixes mainline patch 3f2b9c9cd ("module: remove rmmod --wait option.")
backported by Kirill Tkhai for PSBM-27580
https://bugs.openvz.org/browse/OVZ-6748
https://jira.sw.ru/browse/PSBM-47877
Signed-off-by: Vasily Averin <vvs at virtuozzo.com>
---
kernel/module.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/kernel/module.c b/kernel/module.c
index 10e1384..762716f 100644
--- a/kernel/module.c
+++ b/kernel/module.c
@@ -892,11 +892,6 @@ SYSCALL_DEFINE2(delete_module, const char __user *, name_user,
return -EFAULT;
name[MODULE_NAME_LEN-1] = '\0';
- if (!(flags & O_NONBLOCK)) {
- printk(KERN_WARNING
- "waiting module removal not supported: please upgrade");
- }
-
if (mutex_lock_interruptible(&module_mutex) != 0)
return -EINTR;
More information about the Devel
mailing list