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

2011-10-13 Thread Mark Kettenis
> From: Jeremy Huddleston > Date: Wed, 12 Oct 2011 19:28:00 -0700 > > On Oct 12, 2011, at 6:12 PM, Alan Coopersmith wrote: > > > On 10/12/11 10:50 AM, Keith Packard wrote: > >> 2. I'd love to figure out how to fork at the time of the error; this > >> would encourage people to actually use t

[PATCH] Warning fix for shm.c

2011-10-13 Thread Jon TURNEY
shm.c: In function 'CheckForShmSyscall': shm.c:182:5: warning: function declaration isn't a prototype [-Wstrict-prototypes] Signed-off-by: Jon TURNEY --- Xext/shm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Xext/shm.c b/Xext/shm.c index 9c8beb2..f2d84be 100644 --

Re: [PATCH v2 0/3] screen crossing changes again

2011-10-13 Thread Michal Suchanek
On 13 October 2011 03:39, Peter Hutterer wrote: > > Same thing as before, this time with the option of non-zero desktop origin. > untested for non-zero origins, I'm failing to actually configure this > anywhere. > Hello, Non-zero origin works for me with positive origin (and a mouse). Outputs t

Re: PATCH document -background none option

2011-10-13 Thread Michal Suchanek
On 12 October 2011 02:40, Alan Coopersmith wrote: > On 10/ 7/11 05:41 AM, Michal Suchanek wrote: >> >> On 6 October 2011 23:39, Alan Coopersmith >>  wrote: >>> >>> On 10/ 6/11 07:54 AM, Michal Suchanek wrote: this patch documents the option introduced in commit 8976e97. >>> +++ b/hw

Re: [Patch] Let luit use posix_openpt()

2011-10-13 Thread Guillem Jover
On Wed, 2011-10-12 at 20:59:54 +0200, Ed Schouten wrote: > From e599ea2515e9eda2d35b72431df5569f2b99f318 Mon Sep 17 00:00:00 2001 > From: Ed Schouten > Date: Wed, 12 Oct 2011 20:53:21 +0200 > Subject: [PATCH] Add support for posix_openpt(). > > Instead of opening /dev/ptmx directly, we should use

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

2011-10-13 Thread Barry Scott
On Wednesday 12 October 2011 18:50:32 Keith Packard wrote: > On Thu, 6 Oct 2011 13:19:35 +0100, Barry Scott > wrote: > > > Is there anything else I need to do to make this patch acceptable > > for inclusion in Xorg? > > Two things: > > 1. This option needs to be restricted to root, much like

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

2011-10-13 Thread Simon Farnsworth
On Wednesday 12 October 2011, Keith Packard wrote: > 2. I'd love to figure out how to fork at the time of the error; this > would encourage people to actually use this option regularly. > A question - what is it about preforking a backtrace handler that you think will put people off using it?

Re: PATCH document -background none option

2011-10-13 Thread Matt Turner
On Thu, Oct 13, 2011 at 8:06 AM, Michal Suchanek wrote: > On 12 October 2011 02:40, Alan Coopersmith > wrote: >> I'd change that to: >>  For security reasons this is not the default, as the screen contents >>  might show a previous user session. > > Are you sure there's supposed to be a comma? >

PATCH v3 Document -background none option

2011-10-13 Thread Michal Suchanek
Hello, resending updated patch. v2: place the option in Xserver man page rather than Xorg man page v3: reword based in input from Alan Coopersmith Thanks Michal ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-d

Re: PATCH v4 Document -background none option

2011-10-13 Thread Michal Suchanek
Hello, resending updated patch. v2: place the option in Xserver man page rather than Xorg man page v3: reword based in input from Alan Coopersmith v4: remove extra space causing formatting error Thanks Michal ___ xorg-devel@lists.x.org: X.Org developm

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

2011-10-13 Thread Keith Packard
On Thu, 13 Oct 2011 14:49:42 +0100, Simon Farnsworth wrote: > A question - what is it about preforking a backtrace handler that you think > will put people off using it? It's an ugly hack to work around a bug in glibc. A non-prefork version wouldn't consume any resources until the server actual

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

2011-10-13 Thread Daniel Stone
Hi, On 13 October 2011 17:53, Keith Packard wrote: > On Thu, 13 Oct 2011 14:49:42 +0100, Simon Farnsworth > wrote: >> A question - what is it about preforking a backtrace handler that you think >> will put people off using it? > > It's an ugly hack to work around a bug in glibc. A non-prefork v

Re: [PATCH xserver] check for elevated privileges not uid=0 (V2)

2011-10-13 Thread Julien Cristau
On Mon, Oct 10, 2011 at 16:40:25 +0200, Michal Suchanek wrote: > Hello, > > On 10 October 2011 16:07, 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 > > Thanks for the patch. > > Work

[PATCH] crtc: match full preferred modes if possible when choosing an initial config

2011-10-13 Thread Jesse Barnes
It's fairly common to have multiple, identical monitors plugged in. In that case, it's preferable to run the monitor's preferred mode on each output, rather than just matching the width & height and end up with different timings or refresh rates. Signed-off-by: Jesse Barnes --- hw/xfree86/modes

Re: [PATCH] crtc: match full preferred modes if possible when choosing an initial config

2011-10-13 Thread Keith Packard
On Thu, 13 Oct 2011 12:01:10 -0700, Jesse Barnes wrote: > It's fairly common to have multiple, identical monitors plugged in. In > that case, it's preferable to run the monitor's preferred mode on each > output, rather than just matching the width & height and end up with > different timings or

Re: [PATCH] crtc: match full preferred modes if possible when choosing an initial config

2011-10-13 Thread Alex Deucher
On Thu, Oct 13, 2011 at 3:01 PM, Jesse Barnes wrote: > It's fairly common to have multiple, identical monitors plugged in.  In > that case, it's preferable to run the monitor's preferred mode on each > output, rather than just matching the width & height and end up with > different timings or refr

Re: PATCH v4 Document -background none option

2011-10-13 Thread Alan Coopersmith
On 10/13/11 08:34 AM, Michal Suchanek wrote: Hello, resending updated patch. v2: place the option in Xserver man page rather than Xorg man page v3: reword based in input from Alan Coopersmith v4: remove extra space causing formatting error I didn't see a patch attached to either the v3 or v4

Re: PATCH v4 Document -background none option

2011-10-13 Thread Michal Suchanek
On 13 October 2011 22:26, Alan Coopersmith wrote: > On 10/13/11 08:34 AM, Michal Suchanek wrote: >> >> Hello, >> >> resending updated patch. >> >> v2: place the option in Xserver man page rather than Xorg man page >> v3: reword based in input from Alan Coopersmith >> v4: remove extra space causing

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

2011-10-13 Thread Michal Suchanek
On 13 October 2011 20:10, Daniel Stone wrote: > Hi, > > On 13 October 2011 17:53, Keith Packard wrote: >> On Thu, 13 Oct 2011 14:49:42 +0100, Simon Farnsworth >> wrote: >>> A question - what is it about preforking a backtrace handler that you think >>> will put people off using it? >> >> It's a

Re: [PATCH xserver] check for elevated privileges not uid=0 (V2)

2011-10-13 Thread Michal Suchanek
On 13 October 2011 20:13, Julien Cristau wrote: > On Mon, Oct 10, 2011 at 16:40:25 +0200, Michal Suchanek wrote: > >> Hello, >> >> On 10 October 2011 16:07, Antoine Martin wrote: >> > This allows us to run the server as a normal user whilst still >> > being able to use the -modulepath, -logfile a

Re: [PATCH xserver] check for elevated privileges not uid=0 (V2)

2011-10-13 Thread Julien Cristau
On Thu, Oct 13, 2011 at 23:25:07 +0200, Michal Suchanek wrote: > On 13 October 2011 20:13, Julien Cristau wrote: > > On Mon, Oct 10, 2011 at 16:40:25 +0200, Michal Suchanek wrote: > > > >>  $ X +extension GLX +extension RANDR +extension RENDER -logfile > >> /scratch/xdummy.log -config ~hramrach/x

Re: transset

2011-10-13 Thread Daniel Forchheimer
Hi Arnaud and Alan, I would be more than happy if the transset-df fork was merge into xorg. That was my hope from the beginning and people have approach me about it before but it just never seemed to happen completely. I don't feel I have the time or even knowledge to help out much but do f

Re: [PATCH] dix: mark motion events as emulated if we're scrolling from button press

2011-10-13 Thread Peter Hutterer
ping? On Wed, Oct 05, 2011 at 12:14:47PM +1000, Peter Hutterer wrote: > The protocol requires that the emulated event is marked as such. So if a > driver with smooth scrolling axis sends legacy button events, the motion > event must be marked as emulated. > > Pass the real type to emulate_scroll_

Re: PATCH v4 Document -background none option

2011-10-13 Thread Tormod Volden
On Thu, Oct 13, 2011 at 10:55 PM, Michal Suchanek wrote: >>> v3: reword based in input from Alan Coopersmith To nitpick on the nitpicking (at the risk of bikeshedding): +Asks the driver not to clear the background on startup, if the driver supports that. The other option descriptions start with l

Re: PATCH Check module version each time it is loaded

2011-10-13 Thread Peter Hutterer
On Mon, Oct 10, 2011 at 10:10:23AM +0200, Michal Suchanek wrote: > On 10 October 2011 05:38, Peter Hutterer wrote: > > On Fri, Oct 07, 2011 at 02:58:35PM +0200, Michal Suchanek wrote: > >> Hello, > >> > >> The X server "duplicates" modules by loading them again from the disk. > >> > >> Since this

Re: [PATCH v2 0/3] screen crossing changes again

2011-10-13 Thread Peter Hutterer
On Thu, Oct 13, 2011 at 01:58:01PM +0200, Michal Suchanek wrote: > On 13 October 2011 03:39, Peter Hutterer wrote: > > > > Same thing as before, this time with the option of non-zero desktop origin. > > untested for non-zero origins, I'm failing to actually configure this > > anywhere. > > > > He

Re: [Patch] Let luit use posix_openpt()

2011-10-13 Thread Alan Coopersmith
On 10/13/11 06:17 AM, Guillem Jover wrote: On Wed, 2011-10-12 at 20:59:54 +0200, Ed Schouten wrote: From e599ea2515e9eda2d35b72431df5569f2b99f318 Mon Sep 17 00:00:00 2001 From: Ed Schouten Date: Wed, 12 Oct 2011 20:53:21 +0200 Subject: [PATCH] Add support for posix_openpt(). Instead of opening

Re: PATCH v4 Document -background none option

2011-10-13 Thread Alan Coopersmith
On 10/13/11 01:55 PM, Michal Suchanek wrote: +For security reasons this is not the default as as the screen contents might Replace "as as" with just one "as" and I'm happy. Thanks for going the rounds with us. Reviewed-by: Alan Coopersmith -- -Alan Coopersmith-alan.coopersm

Re: [PATCH libXi multitouch 2/4] Make users aware of the unstable nature of this development branch

2011-10-13 Thread Peter Hutterer
On Thu, Sep 15, 2011 at 12:05:31AM -0700, Chase Douglas wrote: > On Sep 15, 2011, at 12:01 AM, Jamey Sharp wrote: > > > On Wed, Sep 14, 2011 at 10:33:55PM -0700, Chase Douglas wrote: > >> +if ! test "x$UNSTABLE_LIB" = xyes; then > >> +AC_MSG_ERROR([This branch contains elements which have not

Re: [PATCHv2] makekeys: Fix build/target word size mismatch when cross-compiling

2011-10-13 Thread Guillem Jover
On Thu, 2011-09-22 at 18:44:13 -0400, Derek Buitenhuis wrote: > diff --git a/src/util/Makefile.am b/src/util/Makefile.am > index 3b100e0..09c27a2 100644 > --- a/src/util/Makefile.am > +++ b/src/util/Makefile.am > @@ -3,6 +3,7 @@ noinst_PROGRAMS=makekeys > > makekeys_CFLAGS = \ > $(X11_CFLA

[PATCH libX11] include: Add GetReqSized() for request buffers of specific size

2011-10-13 Thread Peter Hutterer
Some XI2 requests change in size over different versions and libXi would need to hack around GetReq and GetReqExtra. Add a new GetReqSized so the library can explicitly specify the size of the request in 4-byte units. Signed-off-by: Peter Hutterer --- Instead of hacking around in libXi with GetRe

[PATCH v2 libXi multitouch 4/4] Implement support for XI 2.2

2011-10-13 Thread Peter Hutterer
From: Chase Douglas XI 2.1 has not been implemented yet, so this code defines version info for it too. These will be dropped when XI 2.1 support is merged. The code is based on what we have in the prototype libXi in Ubuntu. The main differences are: * No touch valuator classes * No TouchUpdateU

Re: transset

2011-10-13 Thread Arnaud Fontaine
Hi, Daniel Forchheimer writes: > I would be more than happy if the transset-df fork was merge into > xorg. That was my hope from the beginning and people have approach me > about it before but it just never seemed to happen completely. That's great, thanks. > I don't feel I have the time

Re: [PATCH] xfree86: Add Loongson MIPS support

2011-10-13 Thread Matt Kraai
On Wed, Oct 12, 2011 at 05:15:57PM +0200, Mark Kettenis wrote: > > Date: Wed, 12 Oct 2011 07:48:00 -0700 > > From: Alan Coopersmith > > > > On 10/11/11 11:58 PM, Matt Kraai wrote: > > > +#elif defined(__mips__) > > > + fd = open("/dev/mem", O_RDWR); > > > + IOPortBase = (volatile unsigned char *)

[PULL 1.11] input fixes for 1.11.2

2011-10-13 Thread Peter Hutterer
Commit bbe6a69da398dba93679c23c3c50f9916cb562ca The following changes since commit 603ad6608a0dbe2fb1bdc4bcd8b4117f2b0bc8b2: XQuartz: pbproxy: Add missing AM_OBJCFLAGS (2011-10-09 01:36:11 -0700) are available in the git repository at: git://people.freedesktop.org/~whot/xserver.git server-1.