Re: [PATCH] Do not trap access to timer and keyboard

2010-11-10 Thread Keith Packard
On Sun, 24 Oct 2010 15:18:28 +0200, Samuel Thibault wrote: > Disable timer/keyboard trapping on GNU/Hurd for now Merged. acba00c..a6c64d9 master -> master -- keith.pack...@intel.com pgpBDDQkMJrcZ.pgp Description: PGP signature ___ xorg-dev

Re: [PATCH] Do not trap access to timer and keyboard

2010-10-28 Thread Adam Jackson
On Sun, 2010-10-24 at 15:18 +0200, Samuel Thibault wrote: > Adam Jackson, le Fri 19 Mar 2010 14:29:55 -0400, a écrit : > > Well in that case, the ioperm() is definitely bogus on all platforms, > > since all it can do is make us crash. But it indicates that the int10 > > wrapper needs to do a bette

Re: [PATCH] Do not trap access to timer and keyboard

2010-10-24 Thread Samuel Thibault
Adam Jackson, le Fri 19 Mar 2010 14:29:55 -0400, a écrit : > Well in that case, the ioperm() is definitely bogus on all platforms, > since all it can do is make us crash. But it indicates that the int10 > wrapper needs to do a better job of emulating those ports, since the > kernel is definitely g

Re: [PATCH] Do not trap access to timer and keyboard

2010-03-19 Thread Samuel Thibault
Adam Jackson, le Fri 19 Mar 2010 14:29:55 -0400, a écrit : > (Gedankenexperiment: Kernel and X attempt a config space cycle at the > same time. Kernel writes to 0xCF8. X writes to 0xCF8. Kernel reads or > writes to 0xCFC. Sure, I don't doubt such races can exist. > What garbage does the kernel

Re: [PATCH] Do not trap access to timer and keyboard

2010-03-19 Thread Adam Jackson
On Wed, 2010-03-17 at 23:26 +0100, Samuel Thibault wrote: > Adam Jackson, le Wed 17 Mar 2010 17:44:34 -0400, a écrit : > > On Sat, 2010-03-13 at 02:26 +0100, Samuel Thibault wrote: > > > -ioperm(0x40,4,0); /* trap access to the timer chip */ > > > -ioperm(0x60,4,0); /* trap access to the ke

Re: [PATCH] Do not trap access to timer and keyboard

2010-03-18 Thread olafBuddenhagen
Hi, On Sat, Mar 13, 2010 at 02:26:26AM +0100, Samuel Thibault wrote: > diff --git a/hw/xfree86/os-support/hurd/hurd_video.c > b/hw/xfree86/os-support/hurd/hurd_video.c > index 4a99db3..e049ceb 100644 > --- a/hw/xfree86/os-support/hurd/hurd_video.c > +++ b/hw/xfree86/os-support/hurd/hurd_video.c

Re: [PATCH] Do not trap access to timer and keyboard

2010-03-17 Thread Samuel Thibault
Adam Jackson, le Wed 17 Mar 2010 17:44:34 -0400, a écrit : > On Sat, 2010-03-13 at 02:26 +0100, Samuel Thibault wrote: > > -ioperm(0x40,4,0); /* trap access to the timer chip */ > > -ioperm(0x60,4,0); /* trap access to the keyboard controller */ > > I'm not sold on this. You really do not

Re: [PATCH] Do not trap access to timer and keyboard

2010-03-17 Thread Adam Jackson
On Sat, 2010-03-13 at 02:26 +0100, Samuel Thibault wrote: > Some VESA BIOSes need to access to them. > > Signed-off-by: Samuel Thibault > --- > hw/xfree86/os-support/hurd/hurd_video.c |2 -- > 1 files changed, 0 insertions(+), 2 deletions(-) > > diff --git a/hw/xfree86/os-support/hurd/hurd_

[PATCH] Do not trap access to timer and keyboard

2010-03-12 Thread Samuel Thibault
Some VESA BIOSes need to access to them. Signed-off-by: Samuel Thibault --- hw/xfree86/os-support/hurd/hurd_video.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/hw/xfree86/os-support/hurd/hurd_video.c b/hw/xfree86/os-support/hurd/hurd_video.c index 4a99db3..e049ceb