[PATCH] BLOCK: Move the Ext3 device ioctl compat stuff to the Ext3 driver [try #6]

Move the Ext3 device ioctl compat stuff from fs/compat_ioctl.c to the Ext3
driver so that the Ext3 header file doesn't need to be included.

Signed-Off-By: David Howells <dhowells@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

3 files changed, 60 insertions, 1 deletions

diff --git a/fs/ext3/dir.c b/fs/ext3/dir.c
index 429acbb4e064..d0b54f30b914 100644
--- a/fs/ext3/dir.c
+++ b/fs/ext3/dir.c
@@ -44,6 +44,9 @@ const struct file_operations ext3_dir_operations = {
         .read           = generic_read_dir,
         .readdir        = ext3_readdir,         /* we take BKL. needed?*/
         .ioctl          = ext3_ioctl,           /* BKL held */
+#ifdef CONFIG_COMPAT
+        .compat_ioctl   = ext3_compat_ioctl,
+#endif
         .fsync          = ext3_sync_file,       /* BKL held */
 #ifdef CONFIG_EXT3_INDEX
         .release        = ext3_release_dir,
diff --git a/fs/ext3/file.c b/fs/ext3/file.c
index 994efd189f4e..74ff20f9d09b 100644
--- a/fs/ext3/file.c
+++ b/fs/ext3/file.c
@@ -114,6 +114,9 @@ const struct file_operations ext3_file_operations = {
         .readv          = generic_file_readv,
         .writev         = generic_file_writev,
         .ioctl          = ext3_ioctl,
+#ifdef CONFIG_COMPAT
+        .compat_ioctl   = ext3_compat_ioctl,
+#endif
         .mmap           = generic_file_mmap,
         .open           = generic_file_open,
         .release        = ext3_release_file,
diff --git a/fs/ext3/ioctl.c b/fs/ext3/ioctl.c
index 3a6b012d120c..12daa6869572 100644
--- a/fs/ext3/ioctl.c
+++ b/fs/ext3/ioctl.c
@@ -13,9 +13,10 @@
 #include <linux/ext3_fs.h>
 #include <linux/ext3_jbd.h>
 #include <linux/time.h>
+#include <linux/compat.h>
+#include <linux/smp_lock.h>
 #include <asm/uaccess.h>
 
-
 int ext3_ioctl (struct inode * inode, struct file * filp, unsigned int cmd,
                 unsigned long arg)
 {
@@ -252,3 +253,55 @@ flags_err:
                 return -ENOTTY;
         }
 }
+
+#ifdef CONFIG_COMPAT
+long ext3_compat_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
+{
+        struct inode *inode = file->f_dentry->d_inode;
+        int ret;
+
+        /* These are just misnamed, they actually get/put from/to user an int */
+        switch (cmd) {
+        case EXT3_IOC32_GETFLAGS:
+                cmd = EXT3_IOC_GETFLAGS;
+                break;
+        case EXT3_IOC32_SETFLAGS:
+                cmd = EXT3_IOC_SETFLAGS;
+                break;
+        case EXT3_IOC32_GETVERSION:
+                cmd = EXT3_IOC_GETVERSION;
+                break;
+        case EXT3_IOC32_SETVERSION:
+                cmd = EXT3_IOC_SETVERSION;
+                break;
+        case EXT3_IOC32_GROUP_EXTEND:
+                cmd = EXT3_IOC_GROUP_EXTEND;
+                break;
+        case EXT3_IOC32_GETVERSION_OLD:
+                cmd = EXT3_IOC_GETVERSION_OLD;
+                break;
+        case EXT3_IOC32_SETVERSION_OLD:
+                cmd = EXT3_IOC_SETVERSION_OLD;
+                break;
+#ifdef CONFIG_JBD_DEBUG
+        case EXT3_IOC32_WAIT_FOR_READONLY:
+                cmd = EXT3_IOC_WAIT_FOR_READONLY;
+                break;
+#endif
+        case EXT3_IOC32_GETRSVSZ:
+                cmd = EXT3_IOC_GETRSVSZ;
+                break;
+        case EXT3_IOC32_SETRSVSZ:
+                cmd = EXT3_IOC_SETRSVSZ;
+                break;
+        case EXT3_IOC_GROUP_ADD:
+                break;
+        default:
+                return -ENOIOCTLCMD;
+        }
+        lock_kernel();
+        ret = ext3_ioctl(inode, file, cmd, (unsigned long) compat_ptr(arg));
+        unlock_kernel();
+        return ret;
+}
+#endif