Re: [PATCH libXau] Improve the handling of normal locking collisions.

2011-10-06 Thread Alan Coopersmith
On 09/20/11 05:06 PM, Alan Coopersmith wrote: On 09/05/11 08:37, Matthew D. Fuller wrote: I've long gotten semi-regular locking failures from xauth where there really shouldn't be. Tracked down to error checking in libXau being too restrictive on the failure mechanisms it expects. -- Matthew

Re: [PATCH v3] dix: add utility functions for double to/fro FP1616/FP3232 conversion

2011-10-06 Thread Michel Dänzer
On Mit, 2011-10-05 at 15:02 -0700, Jeremy Huddleston wrote: Co-authored-by: Jeremy Huddleston jerem...@apple.com Signed-off-by: Peter Hutterer peter.hutte...@who-t.net [...] +frac = ldexp((double)(in 0x), -16); The compiler can't optimize away these ldexp() calls. +tmp = (in

Why are there keyboards and pointers? Why not just an input device?

2011-10-06 Thread Michal Suchanek
Hello, maybe this is a stupid question but why do we have keyboard and mice attached as different kinds of devices? Since mouse events can be mapped even to plain keyboards and many mice have keys or buttons that would like to pretend to be keys what is the benefit? The obvious drawback is it

[PATCH xserver] check for elevated privileges rather than just euid=0

2011-10-06 Thread Antoine Martin
This allows us to run the server as a normal user whilst still being able to use the -modulepath, -logfile and -config switches We define a xf86PrivsElevated which will do the checks and cache the result in case it is called more than once. Also renamed the paths #defines to match their new

Re: [PATCH xserver] check for elevated privileges rather than just euid=0

2011-10-06 Thread Michal Suchanek
Hello, I would like to check this out but how do I tell this actually works? Xephyr and Xvfb already run as non-root, Xorg needs device access to run which will likely fail as non-root. Thanks Michal ___ xorg-devel@lists.x.org: X.Org development

Re: [PATCH xserver] check for elevated privileges rather than just euid=0

2011-10-06 Thread Jamey Sharp
On 10/6/11, Michal Suchanek hramr...@centrum.cz wrote: I would like to check this out but how do I tell this actually works? Xephyr and Xvfb already run as non-root, Xorg needs device access to run which will likely fail as non-root. The motivation for Antoine's patch is the xf86-video-dummy

PATCH document -background none option

2011-10-06 Thread Michal Suchanek
Hello, this patch documents the option introduced in commit 8976e97. Thanks to alanc for pointing out the commit number. Michal From fd9747e3b4f57502407d64716af96e3a26472b42 Mon Sep 17 00:00:00 2001 From: Michal Suchanek hramr...@centrum.cz Date: Thu, 6 Oct 2011 16:40:59 +0200 Subject: [PATCH]

Re: [PATCH-V4] xserver: Optional backtrace handler

2011-10-06 Thread Jeremy Huddleston
I thought I already said this, but: Reviewed-by: Jeremy Huddleston jerem...@apple.com Just send the final version with the accumulated tags to keithp for merging and CC the list. On Oct 6, 2011, at 5:19 AM, Barry Scott wrote: Is there anything else I need to do to make this patch acceptable

Re: [PATCH xserver] check for elevated privileges rather than just euid=0

2011-10-06 Thread Antoine Martin
On 06/10/11 20:39, Michal Suchanek wrote: Hello, I would like to check this out but how do I tell this actually works? I use this patch for using Xorg as an Xdummy server, like so: /usr/local/bin/Xorg +extension GLX +extension RandR +extension Render -logfile $HOME/log -config $HOME/xorg.conf'

[PATCH-V4 resend] xserver: Optional backtrace handler

2011-10-06 Thread Barry Scott
It is useful to be able to run an external program to analyse a crashed server process. The server will run a user supplied program in a subprocess of the server. The subprocess is created when the server starts up so that all resources needed to create the subprocess are available. It is not

Re: [PATCH xserver] check for elevated privileges rather than just euid=0

2011-10-06 Thread Jeremy Huddleston
I don't know if xf86PrivsElevated is the right name for this API. Users might have access even without *elevated* privs. What we really want to know is if the user is privileged, and I can see us eventually updating the implementation of this call to reflect changes in access controls.

Re: PATCH document -background none option

2011-10-06 Thread Jeremy Huddleston
Reviewed-by: Jeremy Huddleston jerem...@apple.com On Oct 6, 2011, at 7:54 AM, Michal Suchanek wrote: Hello, this patch documents the option introduced in commit 8976e97. Thanks to alanc for pointing out the commit number. Michal

Re: [PATCH xserver] check for elevated privileges rather than just euid=0

2011-10-06 Thread Tormod Volden
On Thu, Oct 6, 2011 at 3:05 PM, Antoine Martin wrote: This allows us to run the server as a normal user whilst still being able to use the -modulepath, -logfile and -config switches We define a xf86PrivsElevated which will do the checks and cache the result in case it is called more than once.

Re: [PATCH libXaw3d 0/5] Warning fixes

2011-10-06 Thread Matt Turner
On Mon, Oct 3, 2011 at 11:34 PM, Guillem Jover guil...@hadrons.org wrote: Hi! Here's some changes I had laying around. regards, guillem Guillem Jover (5):  Disable unused input and unput lex functions  Remove unused label  Only use variable t in MoveThumb() on XAW_ARROW_SCROLLBARS  

[PATCH libXaw3d] Remove worthless isupper/islower checks

2011-10-06 Thread Matt Turner
Was toupper() *really* expensive when this was written? :) Also, initialize action_type with toupper(), since the check immediately following assumes that its value is uppercase. Signed-off-by: Matt Turner matts...@gmail.com --- src/Paned.c |9 ++--- 1 files changed, 2 insertions(+), 7

[PATCH modular] Publish.sh: batch release and autotagging of modules

2011-10-06 Thread Gaetan Nadon
Features: - Navigate the build tree from a given list of modules to release - Accepts a list in the format produced by build.sh -L - Tag and version names picked-up from Makefile - Section name picked-up from module list - It only release tarballs it has created from the module - Timely feedback,

Re: [PATCH xserver] check for elevated privileges rather than just euid=0

2011-10-06 Thread Michal Suchanek
On 6 October 2011 17:30, Antoine Martin anto...@nagafix.co.uk wrote: On 06/10/11 20:39, Michal Suchanek wrote: Hello, I would like to check this out but how do I tell this actually works? I use this patch for using Xorg as an Xdummy server, like so: /usr/local/bin/Xorg +extension GLX

Re: PATCH document -background none option

2011-10-06 Thread Alan Coopersmith
On 10/ 6/11 07:54 AM, Michal Suchanek wrote: this patch documents the option introduced in commit 8976e97. +++ b/hw/xfree86/man/Xorg.man Since the option is in the core server argument processing (os/utils.c, not under hw/xfree86/common), it should probably go in man/Xserver.man which is

Re: [PATCH evdev, v2] Support smooth scrolling on wheel emulation

2011-10-06 Thread Max Schwarz
Hi Peter, you could make the support compiled in, but not compiled _out_. so even if you HAVE_SMOOTH_SCROLLING, the old bits are ready to go when enabled. if no smooth scrolling axis is otherwise present on the device, just post button events as previously. Well, the proper thing to do would

Re: Why are there keyboards and pointers? Why not just an input device?

2011-10-06 Thread Peter Hutterer
On Thu, Oct 06, 2011 at 01:41:07PM +0200, Michal Suchanek wrote: maybe this is a stupid question but why do we have keyboard and mice attached as different kinds of devices? Since mouse events can be mapped even to plain keyboards and many mice have keys or buttons that would like to pretend

[PATCH] x86emu: improve single-step debugging

2011-10-06 Thread Kees Cook
This allows for other consumers to do single-step decoding/emulation when using x86emu. Additionally adds a stand-alone Makefile for building out of tree, which is very handy for doing emulation debugging. Signed-off-by: Kees Cook k...@outflux.net --- forwarded from

Re: [PATCH] x86emu: improve single-step debugging

2011-10-06 Thread Alan Coopersmith
On 10/ 6/11 03:36 PM, Kees Cook wrote: This allows for other consumers to do single-step decoding/emulation when using x86emu. Additionally adds a stand-alone Makefile for building out of tree, which is very handy for doing emulation debugging. What ever happened to the plan to move it out to

Re: [PATCH] x86emu: improve single-step debugging

2011-10-06 Thread Kees Cook
On Thu, Oct 06, 2011 at 03:49:33PM -0700, Alan Coopersmith wrote: On 10/ 6/11 03:36 PM, Kees Cook wrote: This allows for other consumers to do single-step decoding/emulation when using x86emu. Additionally adds a stand-alone Makefile for building out of tree, which is very handy for doing

[PATCH xserver 1/2] xorg.conf.man: fix 382: warning: missing `)' (got `R') #35054

2011-10-06 Thread Gaetan Nadon
http://www.gnu.org/software/groff/manual/html_node/Man-usage.html Signed-off-by: Gaetan Nadon mems...@videotron.ca --- hw/xfree86/man/xorg.conf.man |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/hw/xfree86/man/xorg.conf.man b/hw/xfree86/man/xorg.conf.man index

[PATCH xserver 2/2] XWinrc.man: fix warning: tab character in unquoted macro argument #35054

2011-10-06 Thread Gaetan Nadon
Spaces or tabs do not affect the text output layout. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- hw/xwin/man/XWinrc.man | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/hw/xwin/man/XWinrc.man b/hw/xwin/man/XWinrc.man index e4c454f..71d8dad 100644 ---

Re: [PATCH xserver 1/2] xorg.conf.man: fix 382: warning: missing `)' (got `R') #35054

2011-10-06 Thread Alan Coopersmith
On 10/ 6/11 05:39 PM, Gaetan Nadon wrote: http://www.gnu.org/software/groff/manual/html_node/Man-usage.html Signed-off-by: Gaetan Nadonmems...@videotron.ca --- hw/xfree86/man/xorg.conf.man |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

transset

2011-10-06 Thread Alan Coopersmith
On 10/ 4/11 11:59 PM, bugzilla-dae...@freedesktop.org wrote: https://bugs.freedesktop.org/show_bug.cgi?id=25542 Jeremy Huddlestonjerem...@freedesktop.org changed: What|Removed |Added

[PATCH libXaw3d] Include stdlib.h for abs()

2011-10-06 Thread Matt Turner
Signed-off-by: Matt Turner matts...@gmail.com --- src/List.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/List.c b/src/List.c index b491000..6b9fe13 100644 --- a/src/List.c +++ b/src/List.c @@ -39,6 +39,7 @@ in this Software without prior written authorization from

[PATCH libXaw3d] Remove unused variable 'i'

2011-10-06 Thread Matt Turner
Signed-off-by: Matt Turner matts...@gmail.com --- src/XawIm.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/src/XawIm.c b/src/XawIm.c index 92b92cd..b01f0d0 100644 --- a/src/XawIm.c +++ b/src/XawIm.c @@ -1287,8 +1287,6 @@ CompileResourceList(XtResourceList res,

Re: [PATCH libXaw3d] Include stdlib.h for abs()

2011-10-06 Thread Alan Coopersmith
On 10/ 6/11 07:28 PM, Matt Turner wrote: Signed-off-by: Matt Turnermatts...@gmail.com --- src/List.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/List.c b/src/List.c index b491000..6b9fe13 100644 --- a/src/List.c +++ b/src/List.c @@ -39,6 +39,7 @@ in this

Re: [PATCH libXaw3d] Remove unused variable 'i'

2011-10-06 Thread Alan Coopersmith
On 10/ 6/11 07:31 PM, Matt Turner wrote: Signed-off-by: Matt Turnermatts...@gmail.com --- src/XawIm.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/src/XawIm.c b/src/XawIm.c index 92b92cd..b01f0d0 100644 --- a/src/XawIm.c +++ b/src/XawIm.c @@ -1287,8 +1287,6 @@

[PATCH 1/2 v2] Add map_legacy interface

2011-10-06 Thread Jeremy Huddleston
This allows platforms to hand back mmaps of the low 1M (ISA) address space on a per-domain basis. Signed-off-by: Adam Jackson a...@redhat.com Reviewed-by: Jeremy Huddleston jerem...@apple.com --- Changes from v1: Added pci_device_unmap_legacy Moved to map_flags rather than write_enabled

[PATCH 2/2 v2] linux: Implement map_legacy

2011-10-06 Thread Jeremy Huddleston
Signed-off-by: Adam Jackson a...@redhat.com Reviewed-by: Jeremy Huddleston jerem...@apple.com --- Changes from v1: Added pci_device_unmap_legacy Moved to map_flags rather than write_enabled open with flags rather than O_RDWR src/linux_sysfs.c | 53

Re: [PATCH:libpciaccess] Solaris: Give better error on realloc failure

2011-10-06 Thread Alan Coopersmith
On 10/ 5/11 12:03 AM, Dirk Wallenstein wrote: On Tue, Oct 04, 2011 at 10:53:20PM -0700, Alan Coopersmith wrote: + Error allocating memory for PCI devices: + %s\n discarding additional devices\n, +

Re: [PATCH v2 xserver 2/2] Xinput: use appropriate copyright markup for a multi licensed doc

2011-10-06 Thread Alan Coopersmith
On 09/27/11 06:14 AM, Gaetan Nadon wrote: Removed a duplicate paragraph. Signed-off-by: Gaetan Nadonmems...@videotron.ca --- PDF output = Copyright © 1989, 1990, 1991 Hewlett-Packard Company Permission to use, copy, modify, and distribute this documentation for any purpose and

Re: [PATCH libXaw3d] Remove unused variable 'i'

2011-10-06 Thread Jeremy Huddleston
Both: Reviewed-by: Jeremy Huddleston jerem...@apple.com On Oct 6, 2011, at 19:31, Matt Turner wrote: Signed-off-by: Matt Turner matts...@gmail.com --- src/XawIm.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/src/XawIm.c b/src/XawIm.c index 92b92cd..b01f0d0

[RFC] Bus cleanup, last chance to comment before [PULL]

2011-10-06 Thread Jeremy Huddleston
libpciaccess patches are on xorg-devel. They address all comments sent to the list, and I'll push them in the next few days if nobody shouts. They are required for this patchset. This patchset has been altered slightly since I last sent it out. The changes react to API changes in