[PATCH] apm: clean up module initalization
authorNeil Horman <nhorman@tuxdriver.com>
Tue, 26 Sep 2006 06:32:21 +0000 (23:32 -0700)
committerLinus Torvalds <torvalds@g5.osdl.org>
Tue, 26 Sep 2006 15:48:55 +0000 (08:48 -0700)
Clean up module initalization for apm.c.  I had started by auditing for
proper return code checks in misc_register, but I found that in the event
of an initalization failure, a proc file and a kernel thread were left
hanging out.  this patch properly cleans up those loose ends on any
initalization failure.

Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Acked-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
arch/i386/kernel/apm.c

index 8591f2fa920cdb77223e5cd53456f9b1e3e0ccb4..24fd577861f1c19bc4ee71d48cc57c0381ea0b54 100644 (file)
@@ -2339,6 +2339,7 @@ static int __init apm_init(void)
        ret = kernel_thread(apm, NULL, CLONE_KERNEL | SIGCHLD);
        if (ret < 0) {
                printk(KERN_ERR "apm: disabled - Unable to start kernel thread.\n");
+               remove_proc_entry("apm", NULL);
                return -ENOMEM;
        }
 
@@ -2348,7 +2349,13 @@ static int __init apm_init(void)
                return 0;
        }
 
-       misc_register(&apm_device);
+       /*
+        * Note we don't actually care if the misc_device cannot be registered.
+        * this driver can do its job without it, even if userspace can't
+        * control it.  just log the error
+        */
+       if (misc_register(&apm_device))
+               printk(KERN_WARNING "apm: Could not register misc device.\n");
 
        if (HZ != 100)
                idle_period = (idle_period * HZ) / 100;