Re: [REPOST PATCH xauth] Look for FamilyLocal if inet or inet6 address is loopback

2013-09-30 Thread Jeremy Huddleston Sequoia
You had incorrect casting in the IN6_IS_ADDR_* macros. Does Linux (or whatever platform you tested on) cast the arguments to those macros for you (thus correcting / undoing the casting that you provided)? That seems quite dangerous. I initially reverted this but then noticed it was a trivial

Re: [PATCH:libXdmcp 3/4] Ensure ARRAYofARRAY8 pointers are initialized to NULL

2013-09-30 Thread Mouse
[...] Has X.org given up on portability to systems where all-bits-0 is not a nil pointer? [...] Yes. [...lots of other such cases...] Do you know of any such systems X was ever ported to? No - not that I've gone looking, mind you. But that's hardly surprising; X making such assumptions

Re: [PATCH:libXdmcp 3/4] Ensure ARRAYofARRAY8 pointers are initialized to NULL

2013-09-30 Thread Alan Coopersmith
On 09/30/13 07:07 AM, Mouse wrote: [...] Has X.org given up on portability to systems where all-bits-0 is not a nil pointer? [...] Yes. [...lots of other such cases...] Do you know of any such systems X was ever ported to? No - not that I've gone looking, mind you. But that's hardly

WaitForSomething(): select: Invalid argument, part 2

2013-09-30 Thread Felipe Contreras
Hi, I've been investigating the bug regarding WaitForSomething() invalid arguments[1], and I've made some progress, but could use some help. This has happened sporadically in my machine, it was happening reliably with Chromium but the issue went away before I could track it down, and now it's

[PATCH 0/3] glx: A few fairly important bugfixes

2013-09-30 Thread Adam Jackson
Patch 1 removes the current context cache, which is something of a prerequisite for making glamor more sane, since glx really shouldn't assume it's the only GL user in the server. Patch 2 makes it possible to run piglit to completion on indirect contexts without getting oomkilled. Patch 3 fixes

[PATCH 1/3] glx: Remove current context cache

2013-09-30 Thread Adam Jackson
This is something of a pessimization, since now we'll call the provider's MakeCurrent hook more often. But that's correct, since the provider might have a current GL context itself (think glamor), so while this cache might be useful it should be done down in the provider itself. Signed-off-by:

[PATCH 2/3] glx: Fix memory leak in context garbage collection

2013-09-30 Thread Adam Jackson
I broke this, back in: commit a48dadc98a28c969741979b70b7a639f24f4cbbd Author: Adam Jackson a...@redhat.com Date: Mon Mar 21 11:59:29 2011 -0400 glx: Reimplement context tags In that, I changed the glx client state to not explicitly track the list of current contexts for

[PATCH 3/3] glx: Fix a corner case in DrawableGone

2013-09-30 Thread Adam Jackson
Losing the drawable does not change our notion of current client. Since the GL under us doesn't understand having a current context without current drawables (sigh), we do still need to loseCurrent so that we re-bind the context on the next request. Bugzilla:

[PATCH v2 xserver] miext/shadow: missing c2p_core.h breaks make distcheck target.

2013-09-30 Thread Gaetan Nadon
Reviewed-by: Julien Cristau jcris...@debian.org Signed-off-by: Gaetan Nadon mems...@videotron.ca --- miext/shadow/Makefile.am |1 + 1 file changed, 1 insertion(+) diff --git a/miext/shadow/Makefile.am b/miext/shadow/Makefile.am index 1db8a26..27cf414 100644 --- a/miext/shadow/Makefile.am +++

[PATCH input-vmmouse] tools: make install fails when user has no write permission in /lib

2013-09-30 Thread Gaetan Nadon
The location of the udevdir is obtained from pkg-config. This is generally /lib/udev. Most people run their build scripts as non-root and do not want to overwrite or add files on their workstation system. This was not the behaviour in release 12.8.0. The code in configure.ac set udevdir based on

xkb, us keyboard intl variant, sharp s. ß s

2013-09-30 Thread Uwe Brauer
Hello I am on Kubuntu 10.04, and using the keyboards as provided by xkb. However the us keyboard with the intl variant, allows to type a and to obtain ä (I presume this is adiaeresis, but I am not sure) Although I see in the configuration file (/usr/share/X11/xkb/symbols/us the symbol ssharp

Re: [PATCH 2/3] glx: Fix memory leak in context garbage collection

2013-09-30 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: I broke this, back in: commit a48dadc98a28c969741979b70b7a639f24f4cbbd Author: Adam Jackson a...@redhat.com Date: Mon Mar 21 11:59:29 2011 -0400 glx: Reimplement context tags In that, I changed the glx client state to not

Re: xkb, us keyboard intl variant, sharp s. ß s

2013-09-30 Thread Mouse
key AC02 { [ s, S,ssharp, section ]}; I don't know how to type it I'm guessing here, but my guess is, hold down Mode_Switch and press s. (To get section - § - hold down Mode_Switch and Shift and press s.) If you don't have a key with the Mode_Switch KeySym

[PATCH video-vmware] Do not use per-target compilation flags

2013-09-30 Thread Gaetan Nadon
Using vmware_drv_la_CFLAGS triggers the per-target compilation flag feature which requires AM_PROG_CC_C_O automake macro. The implementation of this feature is changing in automake 1.14 such that the macro is obsolete. Given that using this per-target feature is not needed, might as well remove

Re: [PATCH input-vmmouse] tools: make install fails when user has no write permission in /lib

2013-09-30 Thread Jakob Bornecrantz
Looks good! On Mon, Sep 30, 2013 at 7:56 PM, Gaetan Nadon mems...@videotron.ca wrote: The location of the udevdir is obtained from pkg-config. This is generally /lib/udev. Most people run their build scripts as non-root and do not want to overwrite or add files on their workstation system.

Re: [REPOST PATCH xauth] Look for FamilyLocal if inet or inet6 address is loopback

2013-09-30 Thread Julien Cristau
On Aug 14, 2013, at 8:08, Egbert Eich e...@freedesktop.org wrote: libxcb uses FamilyLocal authorization if the host name or IP in the display string is from the loopback device. This patch adds the same behavior to xauth. This fixes a long standing problem that for ssh tunneled

Re: [PATCH 2/3] glx: Fix memory leak in context garbage collection

2013-09-30 Thread Adam Jackson
On Mon, 2013-09-30 at 11:17 -0700, Eric Anholt wrote: But I'm lost on this one. loseCurrent is just -core-unbindContext, which is mesa's dri_util.c dereferencing the drawables and calling the driver UnbindContext, which is just _mesa_make_current(NULL, NULL, NULL) regardless of the current

Re: [PATCH video-vmware] Do not use per-target compilation flags

2013-09-30 Thread Jakob Bornecrantz
Looks good, thanks. On Mon, Sep 30, 2013 at 8:51 PM, Gaetan Nadon mems...@videotron.ca wrote: Using vmware_drv_la_CFLAGS triggers the per-target compilation flag feature which requires AM_PROG_CC_C_O automake macro. The implementation of this feature is changing in automake 1.14 such that

[PATCH] xfree86: Find primary entity when bus types are nominally different

2013-09-30 Thread Connor Behan
As of server 1.13, systems with DRM and Udev will have BUS_PLATFORM as their primary bus type. However, drivers not implementing a platformProbe function will still create entities of type BUS_PCI. We need to account for this when checking for the primary entity. Signed-off-by: Connor Behan

Re: [PATCH input-vmmouse] tools: make install fails when user has no write permission in /lib

2013-09-30 Thread Peter Hutterer
On Mon, Sep 30, 2013 at 01:56:53PM -0400, Gaetan Nadon wrote: The location of the udevdir is obtained from pkg-config. This is generally /lib/udev. Most people run their build scripts as non-root and do not want to overwrite or add files on their workstation system. This was not the

Re: glamor: Changes to 'master'

2013-09-30 Thread Adam Jackson
On Thu, 2013-09-26 at 14:28 -0700, Zhigang Gong wrote: commit 0bf74791b93cd6a96ef0ba95c0e17bcf9aba5834 Author: Zhigang Gong zhigang.g...@linux.intel.com Date: Fri Sep 27 05:25:54 2013 +0800 Shoud return null subpixmap if we fail to get a valid map address. The patch is

[PATCH xauth] Handle v4-mapped inet6 addresses correctly

2013-09-30 Thread Egbert Eich
Handling of v4-mapped inet6 addresses has been introduced with commit 58140dbbd39389ad6af58e201e055f3d4b92d368: Look for FamilyLocal if inet or inet6 address is loopback These adresses should be treated as inet addresses. This patch makes the code consistent with the handling if xcb_auth.c in

Re: [PATCH input-vmmouse] tools: make install fails when user has no write permission in /lib

2013-09-30 Thread Gaetan Nadon
On 13-09-30 06:24 PM, Peter Hutterer wrote: On Mon, Sep 30, 2013 at 01:56:53PM -0400, Gaetan Nadon wrote: The location of the udevdir is obtained from pkg-config. This is generally /lib/udev. Most people run their build scripts as non-root and do not want to overwrite or add files on their

Re: [PATCH input-vmmouse] tools: make install fails when user has no write permission in /lib

2013-09-30 Thread Gaetan Nadon
On 13-09-30 06:24 PM, Peter Hutterer wrote: On Mon, Sep 30, 2013 at 01:56:53PM -0400, Gaetan Nadon wrote: The location of the udevdir is obtained from pkg-config. This is generally /lib/udev. Most people run their build scripts as non-root and do not want to overwrite or add files on their