lkml.org 
[lkml]   [2007]   [Oct]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] Fix blktrace setup 32-bit ioctl on 64-bit kernels
Hi,

The layout of struct blk_user_trace_setup is a bit unfortunate, it gets
padded differently on 32-bit and 64-bit archs. So right now it's not
possible to trace 64-bit kernels with a 32-bit app. This patch fixes
that up by adding a compat ioctl handler for BLKTRACESETUP.

Signed-off-by: Jens Axboe <jens.axboe@oracle.com>

diff --git a/fs/compat_ioctl.c b/fs/compat_ioctl.c
index 5a5b711..b18b9cc 100644
--- a/fs/compat_ioctl.c
+++ b/fs/compat_ioctl.c
@@ -2052,6 +2052,51 @@ static int raw_ioctl(unsigned fd, unsigned cmd, unsigned long arg)
}
return ret;
}
+
+struct blk_user_trace_setup32 {
+ char name[32];
+ u16 act_mask;
+ u16 pad;
+ u32 buf_size;
+ u32 buf_nr;
+ u64 start_lba;
+ u64 end_lba;
+ u32 pid;
+} __attribute__((packed));
+
+#define BLKTRACESETUP32 _IOWR(0x12,115,struct blk_user_trace_setup32)
+
+static int blktrace32_setup(int fd, unsigned cmd, unsigned long arg)
+{
+ struct blk_user_trace_setup __user *buts = compat_alloc_user_space(sizeof(*buts));
+ struct blk_user_trace_setup32 __user *buts32 = compat_ptr(arg);
+ int err;
+
+ if (copy_in_user(&buts->name, &buts32->name, BDEVNAME_SIZE) ||
+ get_user(buts->act_mask, &buts32->act_mask) ||
+ get_user(buts->buf_size, &buts32->buf_size) ||
+ get_user(buts->buf_nr, &buts32->buf_nr) ||
+ get_user(buts->start_lba, &buts32->start_lba) ||
+ get_user(buts->end_lba, &buts32->end_lba) ||
+ get_user(buts->pid, &buts32->pid))
+ return -EFAULT;
+
+ err = sys_ioctl(fd, BLKTRACESETUP, (unsigned long) buts);
+ if (err)
+ return err;
+
+ if (copy_to_user(&buts32->name, &buts->name, BDEVNAME_SIZE) ||
+ put_user(buts32->act_mask, &buts->act_mask) ||
+ put_user(buts32->buf_size, &buts->buf_size) ||
+ put_user(buts32->buf_nr, &buts->buf_nr) ||
+ put_user(buts32->start_lba, &buts->start_lba) ||
+ put_user(buts32->end_lba, &buts->end_lba) ||
+ put_user(buts32->pid, &buts->pid))
+ return -EFAULT;
+
+ return err;
+}
+
#endif /* CONFIG_BLOCK */

struct serial_struct32 {
@@ -2555,7 +2600,7 @@ COMPATIBLE_IOCTL(BLKSECTSET)
COMPATIBLE_IOCTL(BLKSSZGET)
COMPATIBLE_IOCTL(BLKTRACESTART)
COMPATIBLE_IOCTL(BLKTRACESTOP)
-COMPATIBLE_IOCTL(BLKTRACESETUP)
+HANDLE_IOCTL(BLKTRACESETUP32, blktrace32_setup)
COMPATIBLE_IOCTL(BLKTRACETEARDOWN)
ULONG_IOCTL(BLKRASET)
ULONG_IOCTL(BLKFRASET)
--
Jens Axboe

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-10-02 09:41    [W:0.091 / U:0.304 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site