XDC2012: Joystick Input thoughts

2012-09-20 Thread Nathan Schulte
I apologize if I've used the wrong list, but I figure as XDC2012 is only mentioned on #xorg-devel (not on #xorg) on freenode (IRC) that this is the right place. I had a comment regarding the feedback/output to input devices discussed in the session. My immediate thought was regarding the LEDs on

[PULL] XQuartz issues exposed by OS X 10.7.5 and 10.8.2

2012-09-20 Thread Jeremy Huddleston Sequoia
The following changes since commit d01921ec18c21f21d377b60626cc2d3418b84a7c: Merge remote-tracking branch 'ajax/ioperm' (2012-09-20 17:26:48 +0200) are available in the git repository at: git://people.freedesktop.org/~jeremyhu/xserver master for you to fetch changes up to a55ccb7e8f80992e

Re: [PATCH] miext/damage: Only wrap into the GC ops chain if there's a listener (v3)

2012-09-20 Thread Aaron Plattner
On 09/20/2012 02:57 PM, Adam Jackson wrote: before after Operation -- 1148346.9 1191807.5 ( 1.04) PutImage 10x10 square 2091666.1 2180983.0 ( 1.04) ShmPutImage 10x10 square v3: In miDamage{R,Unr}egister, bump the seri

[PATCH] Use new pixman_glyph_cache_t API that will be in pixman 0.28.0

2012-09-20 Thread Søren Sandmann
From: Søren Sandmann Pedersen This new API allows glyphs to be cached in a data structure in pixman, and entire glyph strings to be composited in one go. Also bump pixman dependency to 0.27.2. Results from the cairo peformance test suite running against Xvfb with a screen size of 1680x1050@32bp

Re: [PATCH 0/5] dix: Remove more backing store leftovers

2012-09-20 Thread Adam Jackson
On 8/25/12 7:43 PM, Daniel Martin wrote: These patches remove some more backing store leftovers. Basicly dead snippets since Nov 2010 (commit: c4c4676). For the series: Reviewed-by: Adam Jackson The necessary driver changes are trivial, I'll happily run through them once this is merged. -

[PATCH] miext/damage: Only wrap into the GC ops chain if there's a listener (v3)

2012-09-20 Thread Adam Jackson
before after Operation -- 1148346.9 1191807.5 ( 1.04) PutImage 10x10 square 2091666.1 2180983.0 ( 1.04) ShmPutImage 10x10 square v3: In miDamage{R,Unr}egister, bump the serial number of the affected drawable (and all chi

[PATCH 7/7] dix: Fix types in WindowOptRec

2012-09-20 Thread Adam Jackson
No reason for these to be 64 bits on LP64. Reviewed-by: Daniel Stone Signed-off-by: Adam Jackson --- include/windowstr.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/windowstr.h b/include/windowstr.h index 9caafba..a1e608f 100644 --- a/include/windowstr.h

[PATCH 6/7] os: Hide the Connection{In, Out}put implementation details

2012-09-20 Thread Adam Jackson
Reviewed-by: Daniel Stone Signed-off-by: Adam Jackson --- os/connection.c |4 ++-- os/io.c | 21 +++-- os/osdep.h | 18 ++ 3 files changed, 23 insertions(+), 20 deletions(-) diff --git a/os/connection.c b/os/connection.c index 4561caa..6cd8bc

[PATCH 5/7] dix: Extend initial connection handshake for forwarding proxies

2012-09-20 Thread Adam Jackson
Forwarding proxies like sshd will appear to be local, even though they aren't really. This leads to weird behaviour for extensions that truly require running under the same OS services as the client, like MIT-SHM and DRI2. Add two new legal values for the initial connection's byteOrder field, 'r'

[PATCH 4/7] dix: Repack ClientRec

2012-09-20 Thread Adam Jackson
Pick smaller types where possible, including bitfielding some Bools and small enums, then shuffle the result to be hole-free. 192 -> 128 bytes on LP64, 144 -> 96 bytes on ILP32. Signed-off-by: Adam Jackson --- include/dixstruct.h | 38 ++ 1 files changed, 1

[PATCH 3/7] dix: Pull client-is-local flag up to the ClientRec

2012-09-20 Thread Adam Jackson
Reviewed-by: Daniel Stone Signed-off-by: Adam Jackson --- Xext/xf86bigfont.c|4 ++-- hw/kdrive/ephyr/ephyrdriext.c |4 ++-- hw/xfree86/common/xf86DGA.c |2 +- hw/xfree86/common/xf86vmode.c |6 +++--- hw/xfree86/dri/xf86dri.c |4 ++-- hw/xfree86/dri2/dri2ext

[PATCH 2/7] os: Repack ConnectionOutput for LP64

2012-09-20 Thread Adam Jackson
Reviewed-by: Daniel Stone Signed-off-by: Adam Jackson --- os/osdep.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/os/osdep.h b/os/osdep.h index fff0887..587bd03 100644 --- a/os/osdep.h +++ b/os/osdep.h @@ -119,8 +119,8 @@ typedef struct _connectionInput { typedef

[PATCH 1/7] xfree86: Bump video ABI to 14

2012-09-20 Thread Adam Jackson
Signed-off-by: Adam Jackson --- hw/xfree86/common/xf86Module.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/xfree86/common/xf86Module.h b/hw/xfree86/common/xf86Module.h index 83f9790..fd90aac 100644 --- a/hw/xfree86/common/xf86Module.h +++ b/hw/xfree86/common/xf86M

[PATCH 0/7] Resume breaking ABI

2012-09-20 Thread Adam Jackson
Mostly a resend of a now familiar series, but with two minor changes. One is just bumping the ABI version. #4 in the series is a slightly more aggressive packing than I'd sent before, but it gets ClientRec inside of two cachelines. Which is nice. This series is also available as the 'server-1.4-

xcursor-themes - why no index.theme files?

2012-09-20 Thread Thomas Klausner
Subject basically says it, but: Why are no index.theme file installed for the cursor themes from xcursor-themes? Thomas ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listin

Re: [PATCH 5/5] dix: Remove MapUnmapEventsEnabled and friends

2012-09-20 Thread Ville Syrjälä
On Thu, Sep 20, 2012 at 11:16:26AM -0400, Adam Jackson wrote: > This hack was added to suppress events generated by Composite's internal > unmap/map cycle on redirection state change. Since that cycle was > removed in 193ecc8b4, these can go. I sent an identical patch a long time ago, but I suppo

Re: [PATCH] dix: set the device transformation matrix

2012-09-20 Thread walter harms
Am 20.09.2012 17:59, schrieb Chase Douglas: > On Wed, Sep 19, 2012 at 12:56 PM, Peter Hutterer > wrote: >> The property handler is registered after setting the property, so >> dev->transform remains as all-zeros. That causes pixman_f_transform_invert() >> to fail (in transformAbsolute()) and inv

Re: [PATCH] dix: set the device transformation matrix

2012-09-20 Thread Chase Douglas
On Wed, Sep 19, 2012 at 12:56 PM, Peter Hutterer wrote: > The property handler is registered after setting the property, so > dev->transform remains as all-zeros. That causes pixman_f_transform_invert() > to fail (in transformAbsolute()) and invert remains as garbage. This > may then cause a curso

Re: [PATCH 5/5] dix: Remove MapUnmapEventsEnabled and friends

2012-09-20 Thread Keith Packard
Adam Jackson writes: > This hack was added to suppress events generated by Composite's internal > unmap/map cycle on redirection state change. Since that cycle was > removed in 193ecc8b4, these can go. Reviewed-by: Keith Packard -- keith.pack...@intel.com pgpuSJFyd7MIl.pgp Description: PGP

Re: [PATCH 4/5] dix: Factor out DeliverUnmapNotify

2012-09-20 Thread Keith Packard
Adam Jackson writes: > Signed-off-by: Adam Jackson Reviewed-by: Keith Packard -- keith.pack...@intel.com pgpeZGw73pMAx.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel I

Re: [PATCH 3/5] dix: Factor out DeliverMapNotify

2012-09-20 Thread Keith Packard
Adam Jackson writes: > Signed-off-by: Adam Jackson Reviewed-by: Keith Packard -- keith.pack...@intel.com pgpjhWqNYSsnI.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel I

Re: [PATCH 2/5] dix: Factor out MaybeDeliverMapRequest

2012-09-20 Thread Keith Packard
Adam Jackson writes: > Signed-off-by: Adam Jackson Seems identical to me. Reviewed-by: Keith Packard -- keith.pack...@intel.com pgpMU6pFpArkc.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.or

Re: [PATCH 1/5] dix: Fix some indentation

2012-09-20 Thread Keith Packard
Adam Jackson writes: > Signed-off-by: Adam Jackson Nice formatting cleanup. Reviewed-by: Keith Packard -- keith.pack...@intel.com pgppNZcDaDrSn.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.

Re: [PATCH 0/3] Fix initialization when iopl is forbidden

2012-09-20 Thread Keith Packard
Adam Jackson writes: > Why make the comments speak better if you can make the code speak > better, after all. A fine plan. Merged. 70e5766..d01921e master -> master -- keith.pack...@intel.com pgpgQ8av65rku.pgp Description: PGP signature ___ xo

[PATCH 5/5] dix: Remove MapUnmapEventsEnabled and friends

2012-09-20 Thread Adam Jackson
This hack was added to suppress events generated by Composite's internal unmap/map cycle on redirection state change. Since that cycle was removed in 193ecc8b4, these can go. Signed-off-by: Adam Jackson --- dix/window.c | 33 +++-- include/window.h |3 ---

[PATCH 4/5] dix: Factor out DeliverUnmapNotify

2012-09-20 Thread Adam Jackson
Signed-off-by: Adam Jackson --- dix/window.c | 31 +++ 1 files changed, 15 insertions(+), 16 deletions(-) diff --git a/dix/window.c b/dix/window.c index 127f4c1..40f2516 100644 --- a/dix/window.c +++ b/dix/window.c @@ -2737,6 +2737,17 @@ UnrealizeTree(WindowPtr pWin

[PATCH 3/5] dix: Factor out DeliverMapNotify

2012-09-20 Thread Adam Jackson
Signed-off-by: Adam Jackson --- dix/window.c | 31 +++ 1 files changed, 15 insertions(+), 16 deletions(-) diff --git a/dix/window.c b/dix/window.c index 804ac0e..127f4c1 100644 --- a/dix/window.c +++ b/dix/window.c @@ -2543,6 +2543,17 @@ MaybeDeliverMapRequest(Windo

[PATCH 2/5] dix: Factor out MaybeDeliverMapRequest

2012-09-20 Thread Adam Jackson
Signed-off-by: Adam Jackson --- dix/window.c | 40 ++-- 1 files changed, 18 insertions(+), 22 deletions(-) diff --git a/dix/window.c b/dix/window.c index 6fb420d..804ac0e 100644 --- a/dix/window.c +++ b/dix/window.c @@ -2529,6 +2529,20 @@ MapUnmapEventsEnabl

[PATCH 1/5] dix: Fix some indentation

2012-09-20 Thread Adam Jackson
Signed-off-by: Adam Jackson --- dix/window.c | 14 +- 1 files changed, 5 insertions(+), 9 deletions(-) diff --git a/dix/window.c b/dix/window.c index cdd12dc..6fb420d 100644 --- a/dix/window.c +++ b/dix/window.c @@ -1886,8 +1886,7 @@ AnyWindowOverlapsMe(WindowPtr pWin, WindowPtr pH

[PATCH 0/5]: Various dix window cleanups

2012-09-20 Thread Adam Jackson
Was staring at this code recently trying to figure out a bug. Didn't succeed yet, but I may as well leave it nicer than I found it. 2-4 are mostly to make the logic of the callers more readable, 5 removes a hack that Composite is no longer using. Small code size win: textdata bss

Re: [PATCH 0/3] Fix initialization when iopl is forbidden

2012-09-20 Thread Adam Jackson
On 9/20/12 9:42 AM, Keith Packard wrote: Adam Jackson writes: Will resend with better commentary, thanks. Do you have this series with new commentary added? Otherwise, I'll just merge it as-is. I do, for a loose definition of "added" and/or "commentary": http://cgit.freedesktop.org/~ajax/x

Re: X Integration test suite

2012-09-20 Thread Peter Hutterer
On Thu, Sep 20, 2012 at 04:16:15PM +0200, Matt Dew wrote: > > On 08/30/2012 06:13 AM, Peter Hutterer wrote: > >On Wed, Aug 29, 2012 at 04:22:33PM -0700, Chase Douglas wrote: > >>On 08/29/2012 03:36 PM, Peter Hutterer wrote: > >>>On Wed, Aug 29, 2012 at 01:14:35PM -0700, Chase Douglas wrote: >

Re: X Integration test suite

2012-09-20 Thread Matt Dew
On 08/30/2012 06:13 AM, Peter Hutterer wrote: On Wed, Aug 29, 2012 at 04:22:33PM -0700, Chase Douglas wrote: On 08/29/2012 03:36 PM, Peter Hutterer wrote: On Wed, Aug 29, 2012 at 01:14:35PM -0700, Chase Douglas wrote: On 08/28/2012 03:57 AM, Peter Hutterer wrote: One of the things I've spent

Re: [PATCH 0/3] Fix initialization when iopl is forbidden

2012-09-20 Thread Keith Packard
Adam Jackson writes: > Will resend with better commentary, thanks. Do you have this series with new commentary added? Otherwise, I'll just merge it as-is. -- keith.pack...@intel.com pgptFzNNX2VBo.pgp Description: PGP signature ___ xorg-devel@lists.

Re: [PATCH 2/3] linux: Make failure to iopl non-fatal

2012-09-20 Thread Keith Packard
Adam Jackson writes: > We load the driver list, then enable I/O, then call driver probe based > on whether I/O enable succeeded. That's bad, because the loaded > security policy might forbid port access. We happen to treat that as > fatal for some reason, which means even drivers that don't nee

Re: [PATCH 1/3] linux: Refactor xf86{En,Dis}ableIO

2012-09-20 Thread Keith Packard
Adam Jackson writes: > Pull platform methods into their own sections for legibility, and > rewrite the ifdefs to be more concise. Yeah, nice cleanup. Reviewed-by: Keith Packard -- keith.pack...@intel.com pgpn244iGPcPy.pgp Description: PGP signature _