lkml.org 
[lkml]   [2003]   [May]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] make vt_ioctl ix86isms explicit
On Sat, May 24, 2003 at 10:55:41AM -0700, Linus Torvalds wrote:
> They were added because this was how the X server got IO permissions a
> million years ago. It comes from some random old UNIX/386 thing, it
> predates Linux itself as far as I know.
>
> I'm fairly certain that X itself no longer uses it at all, but there may
> be other programs that still do (unlikely). Your patch looks sane,
> although it might be equally sane to just remove the code altogether.

What about this one instead? If no one sees this messages we can just
rip it out in 2.7.<early>


--- 1.23/drivers/char/vt_ioctl.c Mon May 12 16:12:47 2003
+++ edited/drivers/char/vt_ioctl.c Fri May 23 22:13:19 2003
@@ -59,7 +59,7 @@
*/
unsigned char keyboard_type = KB_101;

-#if !defined(__alpha__) && !defined(__ia64__) && !defined(__mips__) && !defined(__arm__) && !defined(__sh__)
+#ifdef CONFIG_X86
asmlinkage long sys_ioperm(unsigned long from, unsigned long num, int on);
#endif

@@ -424,23 +424,29 @@
ucval = keyboard_type;
goto setchar;

-#if !defined(__alpha__) && !defined(__ia64__) && !defined(__mips__) && !defined(__arm__) && !defined(__sh__)
/*
* These cannot be implemented on any machine that implements
- * ioperm() in user level (such as Alpha PCs).
+ * ioperm() in user level (such as Alpha PCs) or not at all.
+ *
+ * XXX: you should never use these, just call ioperm directly..
*/
+#ifdef CONFIG_X86
case KDADDIO:
case KDDELIO:
/*
* KDADDIO and KDDELIO may be able to add ports beyond what
* we reject here, but to be safe...
*/
+ printk(KERN_INFO "[%s]: using obsolete KDADDIO/KDDELIO ioctl\n",
+ current->comm);
if (arg < GPFIRST || arg > GPLAST)
return -EINVAL;
return sys_ioperm(arg, 1, (cmd == KDADDIO)) ? -ENXIO : 0;

case KDENABIO:
case KDDISABIO:
+ printk(KERN_INFO "[%s]: using obsolete KDENABIO/KDDISABIO ioctl\n",
+ current->comm);
return sys_ioperm(GPFIRST, GPNUM,
(cmd == KDENABIO)) ? -ENXIO : 0;
#endif
-
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: 2005-03-22 13:35    [from the cache]
©2003-2011 Jasper Spaans