revert "vanishing ioctl handler debugging"
authorAndrew Morton <akpm@linux-foundation.org>
Mon, 16 Jul 2007 06:41:02 +0000 (23:41 -0700)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Mon, 16 Jul 2007 16:05:47 +0000 (09:05 -0700)
Revert my do_ioctl() debugging patch: Paul fixed the bug.

Cc: Paul Fulghum <paulkf@microgate.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ioctl.c

index 8c90cbc..479c103 100644 (file)
@@ -12,7 +12,6 @@
 #include <linux/fs.h>
 #include <linux/security.h>
 #include <linux/module.h>
-#include <linux/kallsyms.h>
 
 #include <asm/uaccess.h>
 #include <asm/ioctls.h>
@@ -21,7 +20,6 @@ static long do_ioctl(struct file *filp, unsigned int cmd,
                unsigned long arg)
 {
        int error = -ENOTTY;
-       void *f;
 
        if (!filp->f_op)
                goto out;
@@ -31,16 +29,10 @@ static long do_ioctl(struct file *filp, unsigned int cmd,
                if (error == -ENOIOCTLCMD)
                        error = -EINVAL;
                goto out;
-       } else if ((f = filp->f_op->ioctl)) {
+       } else if (filp->f_op->ioctl) {
                lock_kernel();
-               if (!filp->f_op->ioctl) {
-                       printk("%s: ioctl %p disappeared\n", __FUNCTION__, f);
-                       print_symbol("symbol: %s\n", (unsigned long)f);
-                       dump_stack();
-               } else {
-                       error = filp->f_op->ioctl(filp->f_path.dentry->d_inode,
-                                                 filp, cmd, arg);
-               }
+               error = filp->f_op->ioctl(filp->f_path.dentry->d_inode,
+                                         filp, cmd, arg);
                unlock_kernel();
        }