Merge master.kernel.org:/pub/scm/linux/kernel/git/gregkh/driver-2.6

* master.kernel.org:/pub/scm/linux/kernel/git/gregkh/driver-2.6:
  Revert "Driver core: let request_module() send a /sys/modules/kmod/-uevent"
  Driver core: fix error by cleanup up symlinks properly
  make kernel/kmod.c:kmod_mk static
  power management: fix struct layout and docs
  power management: no valid states w/o pm_ops
  Driver core: more fallout from class_device changes for pcmcia
  sysfs: move struct sysfs_dirent to private header
  driver core: refcounting fix
  Driver core: remove class_device_rename

4 files changed, 20 insertions, 138 deletions

diff --git a/kernel/kmod.c b/kernel/kmod.c
index 9f923f8ce6a0..796276141e51 100644
--- a/kernel/kmod.c
+++ b/kernel/kmod.c
@@ -36,8 +36,6 @@
 #include <linux/resource.h>
 #include <asm/uaccess.h>
 
-extern int delete_module(const char *name, unsigned int flags);
-
 extern int max_threads;
 
 static struct workqueue_struct *khelper_wq;
@@ -48,7 +46,6 @@ static struct workqueue_struct *khelper_wq;
         modprobe_path is set via /proc/sys.
 */
 char modprobe_path[KMOD_PATH_LEN] = "/sbin/modprobe";
-struct module_kobject kmod_mk;
 
 /**
  * request_module - try to load a kernel module
@@ -78,11 +75,6 @@ int request_module(const char *fmt, ...)
         static atomic_t kmod_concurrent = ATOMIC_INIT(0);
 #define MAX_KMOD_CONCURRENT 50  /* Completely arbitrary value - KAO */
         static int kmod_loop_msg;
-        char modalias[16 + MODULE_NAME_LEN] = "MODALIAS=";
-        char *uevent_envp[2] = {
-                modalias,
-                NULL
-        };
 
         va_start(args, fmt);
         ret = vsnprintf(module_name, MODULE_NAME_LEN, fmt, args);
@@ -90,12 +82,6 @@ int request_module(const char *fmt, ...)
         if (ret >= MODULE_NAME_LEN)
                 return -ENAMETOOLONG;
 
-        strcpy(&modalias[strlen("MODALIAS=")], module_name);
-        kobject_uevent_env(&kmod_mk.kobj, KOBJ_CHANGE, uevent_envp);
-
-        if (modprobe_path[0] == '\0')
-                goto out;
-
         /* If modprobe needs a service that is in a module, we get a recursive
          * loop.  Limit the number of running kmod threads to max_threads/2 or
          * MAX_KMOD_CONCURRENT, whichever is the smaller.  A cleaner method
@@ -122,115 +108,9 @@ int request_module(const char *fmt, ...)
 
         ret = call_usermodehelper(modprobe_path, argv, envp, 1);
         atomic_dec(&kmod_concurrent);
-out:
         return ret;
 }
 EXPORT_SYMBOL(request_module);
-
-static ssize_t store_mod_request(struct module_attribute *mattr,
-                                 struct module *mod,
-                              const char *buffer, size_t count)
-{
-        char name[MODULE_NAME_LEN];
-        int ret;
-
-        if (count < 1 || count+1 > MODULE_NAME_LEN)
-                return -EINVAL;
-        memcpy(name, buffer, count);
-        name[count] = '\0';
-        if (name[count-1] == '\n')
-                name[count-1] = '\0';
-
-        ret = request_module(name);
-        if (ret < 0)
-                return ret;
-        return count;
-}
-
-static struct module_attribute mod_request = {
-        .attr = { .name = "mod_request", .mode = S_IWUSR, .owner = THIS_MODULE },
-        .store = store_mod_request,
-};
-
-#ifdef CONFIG_MODULE_UNLOAD
-static ssize_t store_mod_unload(struct module_attribute *mattr,
-                            struct module *mod,
-                            const char *buffer, size_t count)
-{
-        char name[MODULE_NAME_LEN];
-        int ret;
-
-        if (count < 1 || count+1 > MODULE_NAME_LEN)
-                return -EINVAL;
-        memcpy(name, buffer, count);
-        name[count] = '\0';
-        if (name[count-1] == '\n')
-                name[count-1] = '\0';
-
-        ret = delete_module(name, O_NONBLOCK);
-        if (ret < 0)
-                return ret;
-        return count;
-}
-
-static struct module_attribute mod_unload = {
-        .attr = { .name = "mod_unload", .mode = S_IWUSR, .owner = THIS_MODULE },
-        .store = store_mod_unload,
-};
-#endif
-
-static ssize_t show_mod_request_helper(struct module_attribute *mattr,
-                                       struct module *mod,
-                                       char *buffer)
-{
-        return sprintf(buffer, "%s\n", modprobe_path);
-}
-
-static ssize_t store_mod_request_helper(struct module_attribute *mattr,
-                                        struct module *mod,
-                                        const char *buffer, size_t count)
-{
-        if (count < 1 || count+1 > KMOD_PATH_LEN)
-                return -EINVAL;
-        memcpy(modprobe_path, buffer, count);
-        modprobe_path[count] = '\0';
-        if (modprobe_path[count-1] == '\n')
-                modprobe_path[count-1] = '\0';
-        return count;
-}
-
-static struct module_attribute mod_request_helper = {
-        .attr = {
-                .name = "mod_request_helper",
-                .mode = S_IWUSR | S_IRUGO,
-                .owner = THIS_MODULE
-        },
-        .show = show_mod_request_helper,
-        .store = store_mod_request_helper,
-};
-
-void __init kmod_sysfs_init(void)
-{
-        int ret;
-
-        kmod_mk.mod = THIS_MODULE;
-        kobj_set_kset_s(&kmod_mk, module_subsys);
-        kobject_set_name(&kmod_mk.kobj, "kmod");
-        kobject_init(&kmod_mk.kobj);
-        ret = kobject_add(&kmod_mk.kobj);
-        if (ret < 0)
-                goto out;
-
-        ret = sysfs_create_file(&kmod_mk.kobj, &mod_request_helper.attr);
-        ret = sysfs_create_file(&kmod_mk.kobj, &mod_request.attr);
-#ifdef CONFIG_MODULE_UNLOAD
-        ret = sysfs_create_file(&kmod_mk.kobj, &mod_unload.attr);
-#endif
-
-        kobject_uevent(&kmod_mk.kobj, KOBJ_ADD);
-out:
-        return;
-}
 #endif /* CONFIG_KMOD */
 
 struct subprocess_info {
diff --git a/kernel/module.c b/kernel/module.c
index 8c25b1a04fa6..f77e893e4620 100644
--- a/kernel/module.c
+++ b/kernel/module.c
@@ -653,11 +653,20 @@ static void wait_for_zero_refcount(struct module *mod)
         mutex_lock(&module_mutex);
 }
 
-int delete_module(const char *name, unsigned int flags)
+asmlinkage long
+sys_delete_module(const char __user *name_user, unsigned int flags)
 {
         struct module *mod;
+        char name[MODULE_NAME_LEN];
         int ret, forced = 0;
 
+        if (!capable(CAP_SYS_MODULE))
+                return -EPERM;
+
+        if (strncpy_from_user(name, name_user, MODULE_NAME_LEN-1) < 0)
+                return -EFAULT;
+        name[MODULE_NAME_LEN-1] = '\0';
+
         if (mutex_lock_interruptible(&module_mutex) != 0)
                 return -EINTR;
 
@@ -718,21 +727,6 @@ int delete_module(const char *name, unsigned int flags)
         return ret;
 }
 
-asmlinkage long
-sys_delete_module(const char __user *name_user, unsigned int flags)
-{
-        char name[MODULE_NAME_LEN];
-
-        if (!capable(CAP_SYS_MODULE))
-                return -EPERM;
-
-        if (strncpy_from_user(name, name_user, MODULE_NAME_LEN-1) < 0)
-                return -EFAULT;
-        name[MODULE_NAME_LEN-1] = '\0';
-
-        return delete_module(name, flags);
-}
-
 static void print_unload_info(struct seq_file *m, struct module *mod)
 {
         struct module_use *use;
@@ -2425,6 +2419,12 @@ void module_remove_driver(struct device_driver *drv)
                         kfree(driver_name);
                 }
         }
+        /*
+         * Undo the additional reference we added in module_add_driver()
+         * via kset_find_obj()
+         */
+        if (drv->mod_name)
+                kobject_put(&drv->kobj);
 }
 EXPORT_SYMBOL(module_remove_driver);
 #endif
diff --git a/kernel/params.c b/kernel/params.c
index 7a751570b56d..e265b13195b1 100644
--- a/kernel/params.c
+++ b/kernel/params.c
@@ -707,7 +707,6 @@ static int __init param_sysfs_init(void)
         }
 
         param_sysfs_builtin();
-        kmod_sysfs_init();
 
         return 0;
 }
diff --git a/kernel/power/main.c b/kernel/power/main.c
index e1c413120469..a064dfd8877a 100644
--- a/kernel/power/main.c
+++ b/kernel/power/main.c
@@ -167,7 +167,10 @@ static inline int valid_state(suspend_state_t state)
         if (state == PM_SUSPEND_DISK)
                 return 1;
 
-        if (pm_ops && pm_ops->valid && !pm_ops->valid(state))
+        /* all other states need lowlevel support and need to be
+         * valid to the lowlevel implementation, no valid callback
+         * implies that all are valid. */
+        if (!pm_ops || (pm_ops->valid && !pm_ops->valid(state)))
                 return 0;
         return 1;
 }