lkml.org 
[lkml]   [2004]   [Oct]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH x86_64]: Add TCSBRKP ioctl translation for arch/x86_64/ia32
Date
On Sünnavend 09 Oktober 2004 11:47, Jin, Gordon wrote:
> --- linux-2.6.8/arch/x86_64/ia32/ia32_ioctl.c.orig      2004-09-23 09:21:20.000000000 -0700
> +++ linux-2.6.8/arch/x86_64/ia32/ia32_ioctl.c   2004-09-23 09:22:31.000000000 -0700
> @@ -189,6 +189,7 @@ COMPATIBLE_IOCTL(RTC_SET_TIME)
>  COMPATIBLE_IOCTL(RTC_WKALM_SET)
>  COMPATIBLE_IOCTL(RTC_WKALM_RD)
>  COMPATIBLE_IOCTL(FIOQSIZE)
> +COMPATIBLE_IOCTL(TCSBRKP)
>  
>  /* And these ioctls need translation */
>  HANDLE_IOCTL(TIOCGDEV, tiocgdev)

This is not the optimal solution, for two reasons:

- If it is needed on all architectures, you should instead remove it
from the ones that currently define it and put it in
include/linux/compat_ioctl.h.

- It is not actually COMPATIBLE_IOCTL, but rather ULONG_IOCTL,
as far as I can tell. Yes, I made the same mistake in
arch/s390/kernel/compat_ioctl.c, but you can fix it now
that you are touching it anyway.

Arnd <><
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2005-03-22 14:06    [W:0.082 / U:0.112 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site