[PATCH] BLOCK: Move the msdos device ioctl compat stuff to the msdos driver [try #6]
Move the msdos device ioctl compat stuff from fs/compat_ioctl.c to the msdos driver so that the msdos header file doesn't need to be included. Signed-Off-By: David Howells <dhowells@redhat.com> Signed-off-by: Jens Axboe <axboe@kernel.dk>
This commit is contained in:
parent
52a700c567
commit
188f83dfe0
2 changed files with 56 additions and 49 deletions
|
@ -108,7 +108,6 @@
|
||||||
#include <linux/nbd.h>
|
#include <linux/nbd.h>
|
||||||
#include <linux/random.h>
|
#include <linux/random.h>
|
||||||
#include <linux/filter.h>
|
#include <linux/filter.h>
|
||||||
#include <linux/msdos_fs.h>
|
|
||||||
#include <linux/pktcdvd.h>
|
#include <linux/pktcdvd.h>
|
||||||
|
|
||||||
#include <linux/hiddev.h>
|
#include <linux/hiddev.h>
|
||||||
|
@ -1937,51 +1936,6 @@ static int mtd_rw_oob(unsigned int fd, unsigned int cmd, unsigned long arg)
|
||||||
return err;
|
return err;
|
||||||
}
|
}
|
||||||
|
|
||||||
#define VFAT_IOCTL_READDIR_BOTH32 _IOR('r', 1, struct compat_dirent[2])
|
|
||||||
#define VFAT_IOCTL_READDIR_SHORT32 _IOR('r', 2, struct compat_dirent[2])
|
|
||||||
|
|
||||||
static long
|
|
||||||
put_dirent32 (struct dirent *d, struct compat_dirent __user *d32)
|
|
||||||
{
|
|
||||||
if (!access_ok(VERIFY_WRITE, d32, sizeof(struct compat_dirent)))
|
|
||||||
return -EFAULT;
|
|
||||||
|
|
||||||
__put_user(d->d_ino, &d32->d_ino);
|
|
||||||
__put_user(d->d_off, &d32->d_off);
|
|
||||||
__put_user(d->d_reclen, &d32->d_reclen);
|
|
||||||
if (__copy_to_user(d32->d_name, d->d_name, d->d_reclen))
|
|
||||||
return -EFAULT;
|
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
static int vfat_ioctl32(unsigned fd, unsigned cmd, unsigned long arg)
|
|
||||||
{
|
|
||||||
struct compat_dirent __user *p = compat_ptr(arg);
|
|
||||||
int ret;
|
|
||||||
mm_segment_t oldfs = get_fs();
|
|
||||||
struct dirent d[2];
|
|
||||||
|
|
||||||
switch(cmd)
|
|
||||||
{
|
|
||||||
case VFAT_IOCTL_READDIR_BOTH32:
|
|
||||||
cmd = VFAT_IOCTL_READDIR_BOTH;
|
|
||||||
break;
|
|
||||||
case VFAT_IOCTL_READDIR_SHORT32:
|
|
||||||
cmd = VFAT_IOCTL_READDIR_SHORT;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
|
|
||||||
set_fs(KERNEL_DS);
|
|
||||||
ret = sys_ioctl(fd,cmd,(unsigned long)&d);
|
|
||||||
set_fs(oldfs);
|
|
||||||
if (ret >= 0) {
|
|
||||||
ret |= put_dirent32(&d[0], p);
|
|
||||||
ret |= put_dirent32(&d[1], p + 1);
|
|
||||||
}
|
|
||||||
return ret;
|
|
||||||
}
|
|
||||||
|
|
||||||
struct raw32_config_request
|
struct raw32_config_request
|
||||||
{
|
{
|
||||||
compat_int_t raw_minor;
|
compat_int_t raw_minor;
|
||||||
|
@ -2726,9 +2680,6 @@ HANDLE_IOCTL(SONET_GETFRSENSE, do_atm_ioctl)
|
||||||
HANDLE_IOCTL(BLKBSZGET_32, do_blkbszget)
|
HANDLE_IOCTL(BLKBSZGET_32, do_blkbszget)
|
||||||
HANDLE_IOCTL(BLKBSZSET_32, do_blkbszset)
|
HANDLE_IOCTL(BLKBSZSET_32, do_blkbszset)
|
||||||
HANDLE_IOCTL(BLKGETSIZE64_32, do_blkgetsize64)
|
HANDLE_IOCTL(BLKGETSIZE64_32, do_blkgetsize64)
|
||||||
/* vfat */
|
|
||||||
HANDLE_IOCTL(VFAT_IOCTL_READDIR_BOTH32, vfat_ioctl32)
|
|
||||||
HANDLE_IOCTL(VFAT_IOCTL_READDIR_SHORT32, vfat_ioctl32)
|
|
||||||
/* Raw devices */
|
/* Raw devices */
|
||||||
HANDLE_IOCTL(RAW_SETBIND, raw_ioctl)
|
HANDLE_IOCTL(RAW_SETBIND, raw_ioctl)
|
||||||
HANDLE_IOCTL(RAW_GETBIND, raw_ioctl)
|
HANDLE_IOCTL(RAW_GETBIND, raw_ioctl)
|
||||||
|
|
56
fs/fat/dir.c
56
fs/fat/dir.c
|
@ -20,6 +20,7 @@
|
||||||
#include <linux/dirent.h>
|
#include <linux/dirent.h>
|
||||||
#include <linux/smp_lock.h>
|
#include <linux/smp_lock.h>
|
||||||
#include <linux/buffer_head.h>
|
#include <linux/buffer_head.h>
|
||||||
|
#include <linux/compat.h>
|
||||||
#include <asm/uaccess.h>
|
#include <asm/uaccess.h>
|
||||||
|
|
||||||
static inline loff_t fat_make_i_pos(struct super_block *sb,
|
static inline loff_t fat_make_i_pos(struct super_block *sb,
|
||||||
|
@ -741,10 +742,65 @@ static int fat_dir_ioctl(struct inode * inode, struct file * filp,
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#ifdef CONFIG_COMPAT
|
||||||
|
#define VFAT_IOCTL_READDIR_BOTH32 _IOR('r', 1, struct compat_dirent[2])
|
||||||
|
#define VFAT_IOCTL_READDIR_SHORT32 _IOR('r', 2, struct compat_dirent[2])
|
||||||
|
|
||||||
|
static long fat_compat_put_dirent32(struct dirent *d,
|
||||||
|
struct compat_dirent __user *d32)
|
||||||
|
{
|
||||||
|
if (!access_ok(VERIFY_WRITE, d32, sizeof(struct compat_dirent)))
|
||||||
|
return -EFAULT;
|
||||||
|
|
||||||
|
__put_user(d->d_ino, &d32->d_ino);
|
||||||
|
__put_user(d->d_off, &d32->d_off);
|
||||||
|
__put_user(d->d_reclen, &d32->d_reclen);
|
||||||
|
if (__copy_to_user(d32->d_name, d->d_name, d->d_reclen))
|
||||||
|
return -EFAULT;
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
static long fat_compat_dir_ioctl(struct file *file, unsigned cmd,
|
||||||
|
unsigned long arg)
|
||||||
|
{
|
||||||
|
struct compat_dirent __user *p = compat_ptr(arg);
|
||||||
|
int ret;
|
||||||
|
mm_segment_t oldfs = get_fs();
|
||||||
|
struct dirent d[2];
|
||||||
|
|
||||||
|
switch (cmd) {
|
||||||
|
case VFAT_IOCTL_READDIR_BOTH32:
|
||||||
|
cmd = VFAT_IOCTL_READDIR_BOTH;
|
||||||
|
break;
|
||||||
|
case VFAT_IOCTL_READDIR_SHORT32:
|
||||||
|
cmd = VFAT_IOCTL_READDIR_SHORT;
|
||||||
|
break;
|
||||||
|
default:
|
||||||
|
return -ENOIOCTLCMD;
|
||||||
|
}
|
||||||
|
|
||||||
|
set_fs(KERNEL_DS);
|
||||||
|
lock_kernel();
|
||||||
|
ret = fat_dir_ioctl(file->f_dentry->d_inode, file,
|
||||||
|
cmd, (unsigned long) &d);
|
||||||
|
unlock_kernel();
|
||||||
|
set_fs(oldfs);
|
||||||
|
if (ret >= 0) {
|
||||||
|
ret |= fat_compat_put_dirent32(&d[0], p);
|
||||||
|
ret |= fat_compat_put_dirent32(&d[1], p + 1);
|
||||||
|
}
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
#endif /* CONFIG_COMPAT */
|
||||||
|
|
||||||
const struct file_operations fat_dir_operations = {
|
const struct file_operations fat_dir_operations = {
|
||||||
.read = generic_read_dir,
|
.read = generic_read_dir,
|
||||||
.readdir = fat_readdir,
|
.readdir = fat_readdir,
|
||||||
.ioctl = fat_dir_ioctl,
|
.ioctl = fat_dir_ioctl,
|
||||||
|
#ifdef CONFIG_COMPAT
|
||||||
|
.compat_ioctl = fat_compat_dir_ioctl,
|
||||||
|
#endif
|
||||||
.fsync = file_fsync,
|
.fsync = file_fsync,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue