Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread Samuel Thibault
Daniel Stone, le Tue 24 Aug 2010 14:46:24 +1000, a écrit : Do we need to consider the above scenario? If the above scenario doesn't need to be cared, I will update the patch to assure that the CLOCK_MONOTONIC_COARSE posix timer will be tried only when there exists the corresponding

Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread ykzhao
On Tue, 2010-08-24 at 09:59 +0800, Daniel Stone wrote: On Tue, Aug 24, 2010 at 08:32:48AM +0800, ykzhao wrote: On Mon, 2010-08-23 at 23:25 +0800, Adam Jackson wrote: On Mon, 2010-08-23 at 10:23 +0200, Mark Kettenis wrote: diff --git a/os/utils.c b/os/utils.c index 51455cc..a08d591

Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread Daniel Stone
On Tue, Aug 24, 2010 at 03:55:41PM +0800, ykzhao wrote: On Tue, 2010-08-24 at 09:59 +0800, Daniel Stone wrote: That doesn't change anything - if a system is using ID 6 for something else, then using 6 is wholly incorrect, no matter whether you use the constant directly, define some other

Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread ykzhao
On Tue, 2010-08-24 at 12:46 +0800, Daniel Stone wrote: Hi, On Tue, Aug 24, 2010 at 11:05:23AM +0800, ykzhao wrote: Maybe there is no definition of CLOCK_MONOTONIC_COARSE in /usr/include/bits/time.h when compiling the xorg. But the xorg is executed on the linux kernel that supports the

Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread Samuel Thibault
ykzhao, le Tue 24 Aug 2010 15:55:41 +0800, a écrit : b. CLOCK_MONOTONIC_COARSE is already defined. But the corresponding id is not 6. That won't happen on Linux, the value is now cast into stone. That's why it's safe to use #ifdef __linux__ # ifndef CLOCK_MONOTONIC_COARSE # define

Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread ykzhao
On Tue, 2010-08-24 at 16:13 +0800, Daniel Stone wrote: On Tue, Aug 24, 2010 at 03:55:41PM +0800, ykzhao wrote: On Tue, 2010-08-24 at 09:59 +0800, Daniel Stone wrote: That doesn't change anything - if a system is using ID 6 for something else, then using 6 is wholly incorrect, no matter

Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread Mikhail Gusarov
Twas brillig at 16:45:34 24.08.2010 UTC+08 when yakui.z...@intel.com did gyre and gimble: y What side effect will it bring if we define it explicitly and use it y when it is not defined in system header? Bad side-effect is X.org becoming OS again :) It isn't hard to require newer

Re: [PATCH xserver] config: remove --with-dri-driver-path option, use dri.pc #29740

2010-08-24 Thread Michel Dänzer
On Mon, 2010-08-23 at 10:53 -0400, Gaetan Nadon wrote: The trusted location of the dri driver directory can be obtained from the mesa dri.pc pkg-config file. Could just use dri.pc for the default value of the option, rather than removing it altogether? -- Earthling Michel Dänzer |

Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread Julien Cristau
On Tue, Aug 24, 2010 at 16:15:44 +0800, ykzhao wrote: I also agree that it is very pretty to get the time by using mentioned order. But the configure script already helps us to test whether the MONOTONIC_CLOCK is supported(It uses the CLOCK_MONOTONIC as the argument of clock_gettime). If

Re: [PATCH xserver] config: remove --with-dri-driver-path option, use dri.pc #29740

2010-08-24 Thread Gaetan Nadon
On Tue, 2010-08-24 at 12:56 +0200, Michel Dänzer wrote: Could just use dri.pc for the default value of the option, rather than removing it altogether? Yes. I think the author of the bug report wishes to also remove the option as it is no longer useful. This is an opportunity for me to get

Re: libXt-1.0.8

2010-08-24 Thread Gaetan Nadon
On Mon, 2010-08-23 at 17:59 -0500, Pedro Parra wrote: It seems as if the pkgconfig packages is not installed but I believe it is installed, since I can type pkg-config --help and I get the help menu. I can see all the .pc files from different libraries under /usr/lib/pkgconfig Perhaps you

Re: libXt-1.0.8

2010-08-24 Thread Pedro Parra
Hi thanks for you help. I have version 0.23 for pkg-config When I run the pkg-config --print-errors --cflags xt I get: Package xt was not found in the pkg-config search path. Perhaps you should add the directory containing 'xt.pc' to the PKG_CONFIG_PATH environment variable No package 'xt'

Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread Alan Coopersmith
Daniel Stone wrote: Hi, On Tue, Aug 24, 2010 at 08:55:22AM +0800, ykzhao wrote: What Mark mentioned is that the CLOCK_MONOTONIC_COARSE posix timer is not supported while the corresponding ID is used for other posix timer. Right? 6 has no meaning to clock_gettime().

Re: [PATCH] use CLOCK_MONOTONIC_COARSE posix timer instead of CLOCK_MONOTONIC in Xorg

2010-08-24 Thread Alan Coopersmith
ykzhao wrote: On Mon, 2010-08-23 at 23:25 +0800, Adam Jackson wrote: On Mon, 2010-08-23 at 10:23 +0200, Mark Kettenis wrote: From: yakui.z...@intel.com Date: Mon, 23 Aug 2010 15:20:05 +0800 From: Zhao Yakui yakui.z...@intel.com --- os/utils.c | 14 +- 1 files changed, 13

Re: [PATCH 1/2] Xext: Fix edge case with {Positive, Negative}Transition triggers.

2010-08-24 Thread Adam Jackson
On Tue, 2010-08-24 at 13:48 +1000, Christopher James Halse Rogers wrote: The {Positive,Negative}Transition triggers only fire when the counter goes from strictly {below,above} the threshold. If SyncComputeBracketValues gets called exactly at this threshold we may update the bracket values so

Re: [PATCH 2/2] IDLETIME: Fix edge-case in IdleTimeBlockHandler

2010-08-24 Thread Adam Jackson
On Tue, 2010-08-24 at 13:48 +1000, Christopher James Halse Rogers wrote: Ensure that if we're called exactly on the threshold of a NegativeTransition trigger that we reshedule to pick up an idle time over the threshold. Signed-off-by: Christopher James Halse Rogers

Re: [PATCH] xfree86: use a thread for the generation of input events

2010-08-24 Thread Vignatti Tiago (Nokia-MS/Helsinki)
On Mon, Aug 23, 2010 at 08:04:26PM +0200, ext Adam Jackson wrote: On Mon, 2010-08-23 at 11:17 +0300, Tiago Vignatti wrote: Thanks for posting this, I'm eager to see this land for 1.10. @@ -260,6 +266,9 @@ int main(int argc, char *argv[], char *envp[])

XDS 2010 approaching

2010-08-24 Thread Matthieu Herrb
Hi, This message in intended for people attending XDS 2010. Normally, you should have subscribed to the events mailing list http://lists.x.org/mailman/listinfo/events and have received the message I just posted there: http://lists.x.org/archives/events/2010-August/10.html If you are not,

Re: [PATCH] xfree86: use a thread for the generation of input events

2010-08-24 Thread Alan Coopersmith
Vignatti Tiago (Nokia-MS/Helsinki) wrote: Yep, makes sense. However Peter mentioned a good point: that drivers will want to know if the server is assuming thread or SIGIO, mostly to do malloc tricks I guess. So we may want to just drop off the SIGIO code for once and enable always the threaded

Re: [PATCH] xfree86: use a thread for the generation of input events

2010-08-24 Thread Mark Kettenis
Date: Tue, 24 Aug 2010 22:26:42 +0300 From: Vignatti Tiago (Nokia-MS/Helsinki) tiago.vigna...@nokia.com And then you'd build the server such that either or both of those two functions is #define'd to 0. Which we kind of already do for SIGIO except it's a stub function instead of a

[PATCH 2/2] xfree86: Remove unused xf86PixmapKeyRec

2010-08-24 Thread Adam Jackson
Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/common/xf86.h|3 --- hw/xfree86/common/xf86Globals.c |1 - hw/xfree86/common/xf86Init.c|3 +-- 3 files changed, 1 insertions(+), 6 deletions(-) diff --git a/hw/xfree86/common/xf86.h b/hw/xfree86/common/xf86.h

[PATCH 1/6] linux: Remove redundant variable

2010-08-24 Thread Adam Jackson
Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/os-support/linux/lnx_init.c |8 +++- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git a/hw/xfree86/os-support/linux/lnx_init.c b/hw/xfree86/os-support/linux/lnx_init.c index c8cec2e..a5bd4bb 100644 ---

[PATCH 2/6] xfree86: Remove os-support/sysv

2010-08-24 Thread Adam Jackson
Nothing's using it, the SysV derivatives we support have their own custom versions. Signed-off-by: Adam Jackson a...@redhat.com --- configure.ac|1 - hw/xfree86/os-support/Makefile.am |2 +- hw/xfree86/os-support/sysv/Makefile.am |1 -

[PATCH 3/6] linux: Don't muck about with tty permissions

2010-08-24 Thread Adam Jackson
This is not X's job. Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/os-support/linux/lnx_init.c | 62 1 files changed, 0 insertions(+), 62 deletions(-) diff --git a/hw/xfree86/os-support/linux/lnx_init.c b/hw/xfree86/os-support/linux/lnx_init.c

[PATCH 4/6] xfree86: Remove dead ifdefs from VT switching

2010-08-24 Thread Adam Jackson
Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/os-support/shared/VTsw_usl.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/hw/xfree86/os-support/shared/VTsw_usl.c b/hw/xfree86/os-support/shared/VTsw_usl.c index d4ef0f7..818de17 100644 ---

[PATCH 5/6] linux: Refactor VT switch on {re, }generation and shutdown

2010-08-24 Thread Adam Jackson
This makes more things fatal than were fatal before, but that's correct; if you need the VT, then failing to get it on regeneration means things are about to go very very badly. Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/os-support/linux/lnx_init.c | 38

[PATCH 6/6] xfree86: Remove -keeptty

2010-08-24 Thread Adam Jackson
This debugging option has outlived its usefulness. If you need to send signals to the server, use two machines and ssh. Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/doc/man/Xorg.man.pre |5 hw/xfree86/os-support/bsd/bsd_init.c | 37

Re: [PATCH] xfree86: use a thread for the generation of input events

2010-08-24 Thread Matthew Garrett
On Tue, Aug 24, 2010 at 10:51:42PM +0200, Mark Kettenis wrote: I'm sceptical about this. For one thing OpenBSD has a userland threads library. This mostly works, but it has some nasty side-effects because the library switches all file descriptors in non-blocking mode. This is especially

Re: [PATCH xserver] config: remove --with-dri-driver-path option, use dri.pc #29740

2010-08-24 Thread Kristian Høgsberg
2010/8/24 Gaetan Nadon mems...@videotron.ca: On Tue, 2010-08-24 at 12:56 +0200, Michel Dänzer wrote: Could just use dri.pc for the default value of the option, rather than removing it altogether? Yes. I think the author of the bug report wishes to also remove the option as it is no longer

Re: [PATCH 2/6] xfree86: Remove os-support/sysv

2010-08-24 Thread Alan Coopersmith
Adam Jackson wrote: Nothing's using it, the SysV derivatives we support have their own custom versions. Signed-off-by: Adam Jackson a...@redhat.com --- configure.ac|1 - hw/xfree86/os-support/Makefile.am |2 +-

[PATCH] xfree86: Document mapping terminate to Ctrl+Alt+Backspace. (#25083)

2010-08-24 Thread Jesse Adkins
Explain how to map terminate in xorg.conf, and with setxkbmap. Signed-off-by: Jesse Adkins jesserayadk...@gmail.com --- hw/xfree86/doc/man/Xorg.man.pre | 15 +++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/hw/xfree86/doc/man/Xorg.man.pre

Re: [PATCH] xfree86: Document mapping terminate to Ctrl+Alt+Backspace. (#25083)

2010-08-24 Thread Peter Hutterer
On Tue, Aug 24, 2010 at 02:32:51PM -0700, Jesse Adkins wrote: Explain how to map terminate in xorg.conf, and with setxkbmap. Signed-off-by: Jesse Adkins jesserayadk...@gmail.com --- hw/xfree86/doc/man/Xorg.man.pre | 15 +++ 1 files changed, 15 insertions(+), 0 deletions(-)

1.10 release manager

2010-08-24 Thread Peter Hutterer
Do we have any volunteers for 1.10? Quite frankly (and egoistically), Keith as RM worked well for me for the last two cycles, so if he wants to continue for another round he'd have my vote. If anyone else wants to step forward though, please do so. Keith, are you volunteering this time again?

Re: [PATCH xserver] config: remove --with-dri-driver-path option, use dri.pc #29740

2010-08-24 Thread Dan Nicholson
On Tue, Aug 24, 2010 at 2:12 PM, Kristian Høgsberg k...@bitplanet.net wrote: 2010/8/24 Gaetan Nadon mems...@videotron.ca: On Tue, 2010-08-24 at 12:56 +0200, Michel Dänzer wrote: Could just use dri.pc for the default value of the option, rather than removing it altogether? Yes. I think the