Re: how to turn source code into debian compatible driver

2014-04-24 Thread Pekka Paalanen
On Wed, 23 Apr 2014 15:23:52 + James Robb ja...@gearedinteractive.ca wrote: Hello, I have download this file: http://cgit.freedesktop.org/xorg/driver/xf86-input-synaptics/snapshot/xf86-input-synaptics-1.7.99.1.tar.gz under the impression it is the packaged code I need to compile the

Re: [PATCH] fonts: Fix Xinerama sleep logic

2014-04-24 Thread Alan Coopersmith
Jon TURNEY wrote: On 08/30/13 07:55 AM, Adam Jackson wrote: I introduced this bug, and I have no idea what I was thinking. The xinerama_sleep label I introduced is actually the closure cleanup path. What you actually want to do is re-queue the work, which is spelled ClientSignal. To reproduce

Re: how to turn source code into debian compatible driver

2014-04-24 Thread Michael Thayer
Hello James, On 23/04/14 17:23, James Robb wrote: I have download this file: http://cgit.freedesktop.org/xorg/driver/xf86-input-synaptics/snapshot/xf86-input-synaptics-1.7.99.1.tar.gz under the impression it is the packaged code I need to compile the latest updates to the

[PATCH xf86-input-mouse] Make absolute input reporting in Solaris aware of resolution changes

2014-04-24 Thread Michael Thayer
Currently on Solaris absolute input reporting only takes resolution changes into account when the video driver is using the pre-RandR 1.2 APIs, and there it uses the physical resolution, not the virtual. This patch fixes those two things. Signed-off-by: Michael Thayer michael.tha...@oracle.com

[PATCH xf86-video-qxl 3/3] Add support for server managed fds

2014-04-24 Thread Hans de Goede
Signed-off-by: Hans de Goede hdego...@redhat.com --- src/qxl_driver.c | 4 src/qxl_kms.c| 32 +++- 2 files changed, 31 insertions(+), 5 deletions(-) diff --git a/src/qxl_driver.c b/src/qxl_driver.c index c81b243..b9aa0e9 100644 --- a/src/qxl_driver.c +++

[PATCH xf86-video-qxl 2/3] Fix qxl_driver_func to adhere to the API

2014-04-24 Thread Hans de Goede
The driverFunc callback MUST check the passed in operand and only return TRUE it if understands it and has handled it. It must NOT blindly assume the op is GET_REQUIRED_HW_INTERFACES. While at also always define driverFunc, and welcome qxl to the 21st century. Signed-off-by: Hans de Goede

[PATCH xf86-video-qxl 1/3] Add support for XSERVER_PLATFORM_BUS

2014-04-24 Thread Hans de Goede
This is a preparation patch for adding support for server managed fds. Signed-off-by: Hans de Goede hdego...@redhat.com --- src/qxl.h| 6 ++ src/qxl_driver.c | 48 ++-- 2 files changed, 52 insertions(+), 2 deletions(-) diff --git

[PATCH] Add a return value to set_cursor_position() to allow it to report failure

2014-04-24 Thread Michael Thayer
set_cursor_position() may need to be able to fail and have the server fall back to a software cursor in at least the situation in which we are running on virtual hardware and using the host cursor as a hardware cursor for the guest but cannot change its position. Rename relevant APIs to force

[PATCH 1/4] present: restore root window pixmap when aborting a flip

2014-04-24 Thread Frank Binns
If a 2D application is started on top of a fullscreen 3D application, which is flipping, then we need to stop flipping and restore the root window to using the screen pixmap. Normally this would be done as part of an unflip. However, in the case that there is a pending flip there is no mechanism

[PATCH 4/4] present: sync flips unless requested not to

2014-04-24 Thread Frank Binns
When the display driver supports async flipping this mode of operation is always used. This means that, when the swap interval is none 0, the present extension relies on vblank events to get synchronised flips. However, by the time present gets the event, requests a flip and the GPU has finished

[PATCH 3/4] present: remove invalid assert

2014-04-24 Thread Frank Binns
In the case that synchronised page flips are being performed, it's possible to hit this assert when killing an application. This is a result of the pending flip being flagged as aborted, as part of window destruction, but the pending flip not being cleared until the corresponding flip event has

[PATCH 2/4] present: don't vblank abort pending flip during window destruction

2014-04-24 Thread Frank Binns
As part of present's window destroy function, any vblanks associated with the destroyed window are aborted and destroyed. However, this can include a pending flip. This means that when the corresponding flip event is finally processed it gets ignored. As a result, any deferred unflip is never

[PATCH 0/4] misc X server fixes

2014-04-24 Thread Frank Binns
The following set of patches address problems that occur at X server reset time. They were discovered by running applications without a window manager. Brendan King (3): fb: fix screen pixmap leak on server reset dix: reference the cursor just once in InitializeSprite() dix: fix pixmap leak

[PATCH 2/4] dix: reference the cursor just once in InitializeSprite()

2014-04-24 Thread Frank Binns
From: Brendan King brendan.k...@imgtec.com The new current cursor was being referenced twice, resulting in a memory leak when the current server generation ended. Signed-off-by: Frank Binns frank.bi...@imgtec.com --- dix/events.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 4/4] xfixes: disable cursor on X server reset

2014-04-24 Thread Frank Binns
The initial state of the cursor is set to disabled but this was never be re-disabled during X server reset. This meant any application run after an X server reset would have the cursor displayed even if it hadn't requested this to be the case. Signed-off-by: Frank Binns frank.bi...@imgtec.com ---

[PATCH 1/4] fb: fix screen pixmap leak on server reset

2014-04-24 Thread Frank Binns
From: Brendan King brendan.k...@imgtec.com Call FreePixmap() instead of free() to destroy the screen pixmap in fbCloseScreen(). Signed-off-by: Frank Binns frank.bi...@imgtec.com --- fb/fbscreen.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fb/fbscreen.c b/fb/fbscreen.c

[PATCH 3/4] dix: fix pixmap leak on server reset

2014-04-24 Thread Frank Binns
From: Brendan King brendan.k...@imgtec.com The server is leaking a pixmap (created by CreateDefaultStipple()) on reset. The leak is caused by some X Server graphics contexts not being freed on reset by the machine independent cursor code in the server, which in turn is caused by the cursor

Re: [PATCH] Add a command line argument for disabling indirect GLX.

2014-04-24 Thread James Cloos
Does preventing indirect glx prevent glx over x over tcp? (Not a criticism of the patch; I don't remember whether direct is possible w/o shared mem.) -JimC -- James Cloos cl...@jhcloos.com OpenPGP: 0x997A9F17ED7DAEA6 ___ xorg-devel@lists.x.org:

[PATCH driver/ast 00/15] Set of cleanup patches

2014-04-24 Thread Egbert Eich
While I was testing the AST 2400 support I cleaned up the code a bit. Basically I - fixed a bunch of compiler warnings - removed unused functions - cleaned up namespace issues (either made functions static or renamed them so that they live in the ast driver name space). This removes potential

[PATCH driver/ast 05/15] Clean up namespace in ast_vgatools.c

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de - Make local functions static This sometimes helps the compiler to optimize. - Make sure exported functions have 'AST' in their name. This avoids name space conflicts with the server or other drivers. - Place declaration of exported functions in a header file.

[PATCH driver/ast 00/15] Set of cleanup patches

2014-04-24 Thread Egbert Eich
[2nd attempt - sorry for the noise] While I was testing the AST 2400 support I cleaned up the code a bit. Basically I - fixed a bunch of compiler warnings - removed unused functions - cleaned up namespace issues (either made functions static or renamed them so that they live in the ast driver

[PATCH driver/ast 09/15] Clean up namespace in ast_accel.c

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de - Made sure exported functions have 'AST' in their name. This avoids name space conflicts with the server or other drivers. - Placed declaration of exported functions in a header file. This ensures that changes to the function type or arguments are followed

[PATCH driver/ast 08/15] Clean up namespace in ast_mode.c

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de - Make local functions static This sometimes helps the compiler to optimize. - Made sure exported functions have 'AST' in their name. This avoids name space conflicts with the server or other drivers. - Placed declaration of exported functions in a header file.

[PATCH driver/ast 12/15] Fix typo

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de Chontel - Chrontel Signed-off-by: Egbert Eich e...@suse.de --- src/ast_mode.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/ast_mode.c b/src/ast_mode.c index 59488c4..a52ee71 100644 --- a/src/ast_mode.c +++ b/src/ast_mode.c @@

[PATCH driver/ast 06/15] Clean up namespace in ast_2dtool.c

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de - Make local functions static This sometimes helps the compiler to optimize. - Make sure exported functions have 'AST' in their name. This avoids name space conflicts with the server or other drivers. - Place declaration of exported functions in a header file.

[PATCH 2/2] selinux: don't checkout for AVCs on select() EINTR

2014-04-24 Thread Adam Jackson
From: Ray Strode rstr...@redhat.com wakeup handlers are called even when select() returns EINTR, and when they're called the passed fd set is undefined. This commit fixes the selinux wakeup handler to avoid checking for AVCs over the netlink socket spuriously. Reviewed-by: Adam Jackson

[PATCH 1/2] dbus: don't dispatch dbus events on select() EINTR

2014-04-24 Thread Adam Jackson
From: Ray Strode rstr...@redhat.com wakeup handlers are called even when select() returns EINTR, and when they're called the passed fd set is undefined. This commit fixes the dbus wakeup handler to avoid dispatching into dbus spuriously. Reviewed-by: Adam Jackson a...@redhat.com Signed-off-by:

[PATCH driver/ast 07/15] Clean up namespace in ast_cursor.c

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de - Make sure exported functions have 'AST' in their name. This avoids name space conflicts with the server or other drivers. - Place declaration of exported functions in a header file. This ensures that changes to the function type or arguments are followed

[PATCH driver/ast 11/15] Include local headers only when needed

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de Move inclusion of local headers from ast.h to the individual files. Include only those headers which are needed. Signed-off-by: Egbert Eich e...@suse.de --- src/ast.h | 5 - src/ast_2dtool.c | 2 ++ src/ast_accel.c | 1 + src/ast_cursor.c | 2 ++

[PATCH driver/ast 01/15] Fix compiler warnings: get rid if xf86PciInfo.h

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de xf86PciInfo.h is not needed in the ASpeed driver at all. Signed-off-by: Egbert Eich e...@suse.de --- src/ast_2dtool.c | 1 - src/ast_accel.c | 1 - src/ast_cursor.c | 1 - src/ast_driver.c | 1 - src/ast_mode.c| 1 - src/ast_tool.c| 1 -

[PATCH driver/ast 02/15] Fix compiler warnings: get rid of deprecated IOADDRESS

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de For newer ABI versions IOADDRESS can be replaced by int which is big enough to hold an PIO offset. Signed-off-by: Egbert Eich e...@suse.de --- src/ast.h| 6 -- src/ast_driver.c | 18 -- 2 files changed, 12 insertions(+), 12

[PATCH driver/ast 13/15] Fix ABI version conflict in xf86UnclaimPciSlot()

2014-04-24 Thread Egbert Eich
From: Egbert Eich e...@suse.de xf86UnclaimPciSlot() received a GDevPtr as 2nd argument with ABI_VIDEODRV_VERSION 13. Signed-off-by: Egbert Eich e...@suse.de --- src/ast_driver.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/ast_driver.c b/src/ast_driver.c index

Re: [PATCH] Add a command line argument for disabling indirect GLX.

2014-04-24 Thread Eric Anholt
James Cloos cl...@jhcloos.com writes: Does preventing indirect glx prevent glx over x over tcp? (Not a criticism of the patch; I don't remember whether direct is possible w/o shared mem.) Yes. But if you wanted either performance functionality, you'd be rendering locally using direct