2008-01-29 12:44:50

by Yoav Artzi

[permalink] [raw]
Subject: Sending IOCTLs from 32-bit userland to 64-bit Kernel module

Hi,


I have a 32-bit user land application which sends an IOCTL to a 64-bit
Kernel module. I have a few different cmd codes that I can send through
the IOCTL. For some reason I seem to always get the same IOCTL cmd from
user land, no matter what the ioctl() call is given. This cmd code that
I get has some bytes (W/R and the module code) that are OK, but the rest
is just garbage or zeros. This was originally a 32-bit system, and we
are no converting the Kernel module to 64-bit, so maybe there's
something special for 32-64 communication that miss.


I am working on Linux Kernel v2.6.18.


Thanks


2008-01-29 13:06:00

by Michael Tokarev

[permalink] [raw]
Subject: Re: Sending IOCTLs from 32-bit userland to 64-bit Kernel module

Yoav Artzi wrote:
> Hi,
>
>
> I have a 32-bit user land application which sends an IOCTL to a 64-bit
> Kernel module. I have a few different cmd codes that I can send through
> the IOCTL. For some reason I seem to always get the same IOCTL cmd from
> user land, no matter what the ioctl() call is given. This cmd code that
> I get has some bytes (W/R and the module code) that are OK, but the rest
> is just garbage or zeros. This was originally a 32-bit system, and we
> are no converting the Kernel module to 64-bit, so maybe there's
> something special for 32-64 communication that miss.

Please see numerous examples in kernel source, in many files named
compat_ioctl.c. If your ioctls uses structures with fields that
have different sizes in 32- and 64-bit worlds (most notable int,
various enums etc), there should be corresponding translation
layer as in those examples. If it's your kernel code, that is.
(And try to avoid such types there, use u32 or u64 and the like
that explicitly specify size).

Another possible problem is different alignment of fields in
64- vs 32-bits worlds.

> I am working on Linux Kernel v2.6.18.

If the kernel side isn't your code, the chances are quite high
that this problem has long been fixed in more recent kernels.

/mjt

2008-01-29 13:24:32

by Jiri Slaby

[permalink] [raw]
Subject: Re: Sending IOCTLs from 32-bit userland to 64-bit Kernel module

On 01/29/2008 02:05 PM, Michael Tokarev wrote:
> (And try to avoid such types there, use u32 or u64 and the like
> that explicitly specify size).

... and don't pass pointers to _IOC macros as well.

2008-01-29 13:23:27

by Matti Aarnio

[permalink] [raw]
Subject: Re: Sending IOCTLs from 32-bit userland to 64-bit Kernel module

On Tue, Jan 29, 2008 at 02:13:26PM +0200, Yoav Artzi wrote:
> Hi,
>
>
> I have a 32-bit user land application which sends an IOCTL to a 64-bit
> Kernel module. I have a few different cmd codes that I can send through the
> IOCTL. For some reason I seem to always get the same IOCTL cmd from user
> land, no matter what the ioctl() call is given. This cmd code that I get
> has some bytes (W/R and the module code) that are OK, but the rest is just
> garbage or zeros. This was originally a 32-bit system, and we are no
> converting the Kernel module to 64-bit, so maybe there's something special
> for 32-64 communication that miss.

In x86_64 kernel there is an adaptation layer that translates 32-bit ioctl:s
to kernel internal 64-bit ones, when absolutely necessary.

If you do not need to pass pointers in your ioctl(), then typedef:in passed
parameters with explicite size may save you from lots of trouble:

struct cpioctl {
long var; /* definite trouble 32-bit vs. 64-bit */
uint32_t var2; /* no troubles... */
uint64_t var3;
};

After that, the only issue is that 32-bit user space ioctl() passes
a pointer that is valid in 32-bit space only, bit that is handled by
the common layer without need to make specific adaptation routine.

.. however if the structure has pointer to elsewere in memory, then
things get complicated really fast.


> I am working on Linux Kernel v2.6.18.
> Thanks