* Re: [PATCH] fix drm ioctl ABI default
2002-09-28 1:27 [PATCH] fix drm ioctl ABI default Christoph Hellwig
@ 2002-09-27 19:07 ` Alan Cox
2002-09-27 19:13 ` Ben Collins
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Alan Cox @ 2002-09-27 19:07 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Marcelo Tosatti, linux-kernel
On Sat, 2002-09-28 at 02:27, Christoph Hellwig wrote:
> Add a config option to make the i810 drm ioctl ABI XFree4.1 compatible
> by default (currently that's a module parameter). The XFree folks fucked
> this up by adding members in the middle of a struct and we have to work
> around it now. At least we should have the pre-2.4.20 behaviour as default.
> (And I'd suggest you add that option as y to the defconfig)
With all the vendors now shipping 4.2 this seems a bad thing to default
to the 4,1 interface - especially as the 4.1 server is
o Buggy
o Has security holes that are fixed in 4.2.1 only
Alan
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fix drm ioctl ABI default
2002-09-27 19:07 ` Alan Cox
@ 2002-09-27 19:13 ` Ben Collins
2002-09-27 20:06 ` Alan Cox
2002-09-28 2:21 ` Christoph Hellwig
2002-09-29 20:33 ` Adrian Bunk
2 siblings, 1 reply; 8+ messages in thread
From: Ben Collins @ 2002-09-27 19:13 UTC (permalink / raw)
To: Alan Cox; +Cc: Christoph Hellwig, Marcelo Tosatti, linux-kernel
On Fri, Sep 27, 2002 at 08:07:54PM +0100, Alan Cox wrote:
> On Sat, 2002-09-28 at 02:27, Christoph Hellwig wrote:
> > Add a config option to make the i810 drm ioctl ABI XFree4.1 compatible
> > by default (currently that's a module parameter). The XFree folks fucked
> > this up by adding members in the middle of a struct and we have to work
> > around it now. At least we should have the pre-2.4.20 behaviour as default.
> > (And I'd suggest you add that option as y to the defconfig)
>
> With all the vendors now shipping 4.2 this seems a bad thing to default
> to the 4,1 interface - especially as the 4.1 server is
No, not all vendors are shipping 4.2 yet.
> o Buggy
> o Has security holes that are fixed in 4.2.1 only
...or patched onto 4.1.x from 4.2 source.
4.1 is still out in the wild for most ppl.
--
Debian - http://www.debian.org/
Linux 1394 - http://www.linux1394.org/
Subversion - http://subversion.tigris.org/
Deqo - http://www.deqo.com/
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fix drm ioctl ABI default
2002-09-28 2:21 ` Christoph Hellwig
@ 2002-09-27 20:04 ` Alan Cox
0 siblings, 0 replies; 8+ messages in thread
From: Alan Cox @ 2002-09-27 20:04 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Marcelo Tosatti, linux-kernel
On Sat, 2002-09-28 at 03:21, Christoph Hellwig wrote:
> I don't think that's the proper argument. 2.4.20 silently breaking
> systems that worked fine with 2.4.19 is not the way new linux releases
> work. Vendors can of course feel free to have the new ABI by default.
Linus did the same during earlier 2.4 when we went from 4.0 to 4.1 DRM.
There is a clear precedent for it from the boss 8)
In the general case it will autodetect the problem. There are some
additional bitds (see the dri-devel archive) to investigate that may
make it even more reliable
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fix drm ioctl ABI default
2002-09-27 19:13 ` Ben Collins
@ 2002-09-27 20:06 ` Alan Cox
0 siblings, 0 replies; 8+ messages in thread
From: Alan Cox @ 2002-09-27 20:06 UTC (permalink / raw)
To: Ben Collins; +Cc: Christoph Hellwig, Marcelo Tosatti, linux-kernel
On Fri, 2002-09-27 at 20:13, Ben Collins wrote:
> > o Has security holes that are fixed in 4.2.1 only
>
> ...or patched onto 4.1.x from 4.2 source.
I'd hope from 4.2.1 source plus the later errata fix but yes this is a
good point. Its not an assumption we can reasonably make
I think Cristoph wins
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] fix drm ioctl ABI default
@ 2002-09-28 1:27 Christoph Hellwig
2002-09-27 19:07 ` Alan Cox
0 siblings, 1 reply; 8+ messages in thread
From: Christoph Hellwig @ 2002-09-28 1:27 UTC (permalink / raw)
To: marcelo; +Cc: linux-kernel
Add a config option to make the i810 drm ioctl ABI XFree4.1 compatible
by default (currently that's a module parameter). The XFree folks fucked
this up by adding members in the middle of a struct and we have to work
around it now. At least we should have the pre-2.4.20 behaviour as default.
(And I'd suggest you add that option as y to the defconfig)
--- linux-2.4.20-pre5/drivers/char/drm/Config.in Thu Aug 29 02:58:43 2002
+++ linux/drivers/char/drm/Config.in Fri Sep 6 21:19:07 2002
@@ -10,6 +10,7 @@ tristate ' 3dfx Banshee/Voodoo3+' CONFI
tristate ' ATI Rage 128' CONFIG_DRM_R128
dep_tristate ' ATI Radeon' CONFIG_DRM_RADEON $CONFIG_AGP
dep_tristate ' Intel I810' CONFIG_DRM_I810 $CONFIG_AGP
+dep_mbool ' Enabled XFree 4.1 ioctl interface by default' CONFIG_DRM_I810_XFREE_41 $CONFIG_DRM_I810
dep_tristate ' Intel 830M' CONFIG_DRM_I830 $CONFIG_AGP
dep_tristate ' Matrox g200/g400' CONFIG_DRM_MGA $CONFIG_AGP
dep_tristate ' SiS' CONFIG_DRM_SIS $CONFIG_AGP
--- linux-2.4.20-pre5/drivers/char/drm/i810_dma.c Thu Aug 29 02:58:43 2002
+++ linux/drivers/char/drm/i810_dma.c Fri Sep 6 21:18:07 2002
@@ -30,6 +30,7 @@
*
*/
+#include <linux/config.h>
#include "i810.h"
#include "drmP.h"
#include "i810_drv.h"
@@ -466,8 +467,11 @@ static int i810_dma_initialize(drm_devic
return 0;
}
-
+#ifdef CONFIG_DRM_I810_XFREE_41
+int xfreeversion = 41;
+#else
int xfreeversion = -1;
+#endif
MODULE_PARM(xfreeversion, "i");
MODULE_PARM_DESC(xfreeversion, "The version of XFree86 that needs to be supported");
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fix drm ioctl ABI default
2002-09-27 19:07 ` Alan Cox
2002-09-27 19:13 ` Ben Collins
@ 2002-09-28 2:21 ` Christoph Hellwig
2002-09-27 20:04 ` Alan Cox
2002-09-29 20:33 ` Adrian Bunk
2 siblings, 1 reply; 8+ messages in thread
From: Christoph Hellwig @ 2002-09-28 2:21 UTC (permalink / raw)
To: Alan Cox; +Cc: Marcelo Tosatti, linux-kernel
On Fri, Sep 27, 2002 at 08:07:54PM +0100, Alan Cox wrote:
> With all the vendors now shipping 4.2 this seems a bad thing to default
> to the 4,1 interface - especially as the 4.1 server is
>
> o Buggy
> o Has security holes that are fixed in 4.2.1 only
I don't think that's the proper argument. 2.4.20 silently breaking
systems that worked fine with 2.4.19 is not the way new linux releases
work. Vendors can of course feel free to have the new ABI by default.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fix drm ioctl ABI default
2002-09-27 19:07 ` Alan Cox
2002-09-27 19:13 ` Ben Collins
2002-09-28 2:21 ` Christoph Hellwig
@ 2002-09-29 20:33 ` Adrian Bunk
2002-09-29 20:43 ` Alan Cox
2 siblings, 1 reply; 8+ messages in thread
From: Adrian Bunk @ 2002-09-29 20:33 UTC (permalink / raw)
To: Alan Cox; +Cc: Christoph Hellwig, Marcelo Tosatti, linux-kernel
On 27 Sep 2002, Alan Cox wrote:
> With all the vendors now shipping 4.2 this seems a bad thing to default
> to the 4,1 interface - especially as the 4.1 server is
>...
Debian 3.0 ships with 4.1 and my personal estimation is that the next
release of Debian will be in 2004 (I'm happy if it will be earlier...).
> o Has security holes that are fixed in 4.2.1 only
The Debian maintainer of XFree86 claims that at least the Xlib problem
doesn't affect 4.1 [1].
> Alan
cu
Adrian
[1] http://people.debian.org/~branden/
--
You only think this is a free country. Like the US the UK spends a lot of
time explaining its a free country because its a police state.
Alan Cox
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] fix drm ioctl ABI default
2002-09-29 20:33 ` Adrian Bunk
@ 2002-09-29 20:43 ` Alan Cox
0 siblings, 0 replies; 8+ messages in thread
From: Alan Cox @ 2002-09-29 20:43 UTC (permalink / raw)
To: Adrian Bunk; +Cc: Christoph Hellwig, Marcelo Tosatti, linux-kernel
On Sun, 2002-09-29 at 21:33, Adrian Bunk wrote:
> > o Has security holes that are fixed in 4.2.1 only
>
> The Debian maintainer of XFree86 claims that at least the Xlib problem
> doesn't affect 4.1 [1].
Thats the trivial one. The Shm stuff fixed going 4.2 to 4.2.1 is the bad
stuff, but I sure Brandon will have ported that over too
We'll all need 4.3 for newer boxes though - stuff like PCI domains and
the newer intel chipsets (eg 845G onboard video) simply dont backport 8(
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2002-09-29 20:31 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-09-28 1:27 [PATCH] fix drm ioctl ABI default Christoph Hellwig
2002-09-27 19:07 ` Alan Cox
2002-09-27 19:13 ` Ben Collins
2002-09-27 20:06 ` Alan Cox
2002-09-28 2:21 ` Christoph Hellwig
2002-09-27 20:04 ` Alan Cox
2002-09-29 20:33 ` Adrian Bunk
2002-09-29 20:43 ` Alan Cox
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).