Re: [PATCH xinit] Remove workaround for xterm -L (#89653)

Date: Tue, 24 Mar 2015 11:33:28 +0100 From: Hans de Goede hdego...@redhat.com Hi, On 24-03-15 05:49, Peter Hutterer wrote: The -L flag was removed in 1989. This enables the legacy keyboard driver again when the server is started with -keeptty (bd6cacdd3661) X.Org Bug 89653

Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

On Mon, Mar 23, 2015 at 02:16:13PM +, Jon TURNEY wrote: On 20/03/2015 17:16, Ray Strode wrote: On Fri, Mar 20, 2015 at 10:02 AM, Hans de Goede hdego...@redhat.com wrote: If a separate /tmp per user is used the existing auto display select code does not work, add an extra check for the

Re: [PATCH xinit 1/2] startx: Fix startx picking an already used display number when -nolock is used

On Fri, Mar 20, 2015 at 03:02:35PM +0100, Hans de Goede wrote: Currently startx relies on /tmp/.X?-lock being present for automatically picking a free display number. This does not work if -nolock is used when starting the server, or if the server is started with -displayfd as -displayfd

[PATCH xinit] startx: don't init defaultdisplay to :0

This is misleading - it is always overwritten after looping through the lock files. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- startx.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/startx.cpp b/startx.cpp index cc8be90..636d55e 100644 --- a/startx.cpp +++

RE: [PATCH] glamor: add fallback if NULL EGLConfig is not supported

Hi Eric, Thanks for your response. I think some binary blobs don't support EGL_MESA_configless extension. We can argue that they should add this support, but it will be good if we can also have some fallback there. Regards, Jammy -Original Message- From: Eric Anholt

[PATCH keyboard] linux: bring our process group to the foreground (#89653)

Required to make the driver receive keyboard events from the console. X.Org Bug http://bugs.freedesktop.org/show_bug.cgi?id=89653 Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- Mark was right, this seems to do the trick. I don't know why this is needed now though when it wasn't in

Re: [PATCH] glamor: Perform texture2D() separately from swizzle.

Matt Turner matts...@gmail.com writes: The texture2D() happens in each branch, so we may as well do it as early as possible and hide some of its latency in the branching instructions. Moving it outside the (uniform) control flow reduces the number of instructions in the fs_source shader from

Re: [PATCH] glamor: Fix up indentation

Adam Jackson a...@redhat.com writes: Signed-off-by: Adam Jackson a...@redhat.com Reviewed and applied to -next. signature.asc Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel

Re: [PATCH] glamor: add fallback if NULL EGLConfig is not supported

Jammy Zhou jammy.z...@amd.com writes: Signed-off-by: Kai Guo kai@amd.com Signed-off-by: Jammy Zhou jammy.z...@amd.com What driver does this apply to? Is there a good reason for that driver not supporting EGL_MESA_configless context? It's a very nice extension, and trivial to implement.

Re: [PATCH] ephyr: Avoid a segfault with 'DISPLAY= Xephy -glamor'

Daniel Martin consume.no...@gmail.com writes: On 9 February 2015 at 15:09, Jon TURNEY jon.tur...@dronecode.org.uk wrote: On 09/02/2015 14:03, Daniel Martin wrote: On 9 February 2015 at 14:45, Jon TURNEY jon.tur...@dronecode.org.uk wrote: ephyr_glamor_connect() returns NULL if we failed,

Re: [PATCH] Byte-swapping and size-checking generator all-in-one

Good morning all and Peter, I replied to your comments below. On 17.03.15 09:25, Peter Hutterer wrote: On Wed, Mar 11, 2015 at 08:46:41PM +0200,asalle@gmail.com wrote: --- .gitignore | 3 + Makefile.am | 4 +- Xext/Makefile.am| 2 +-

Re: [PATCH] Byte-swapping and size-checking generator all-in-one

Hi Asal, On Tue, Mar 24, 2015 at 08:16:44AM +0200, Asal Mirzaieva wrote: Good morning all and Peter, I replied to your comments below. On 17.03.15 09:25, Peter Hutterer wrote: [...] diff --git a/configure.ac b/configure.ac index 96524c5..5b89705 100644 --- a/configure.ac +++

glamor cleanups

Here are some documentation fixes and dead code removal from glamor, as well as a tiny patch from trying to figure out how some of the large pixmap code works. This can be found on the glamor-cleanups branch of my tree. ___ xorg-devel@lists.x.org:

[PULL] glamor-next

The following changes since commit 0a78b599b34cc8b5fe6fe82f90e90234e8ab7a56: int10: Fix error check for pci_device_map_legacy (2015-03-13 12:31:21 +1000) are available in the git repository at: git://people.freedesktop.org/~anholt/xserver glamor-next for you to fetch changes up to

[PATCH 4/7] glamor: Move doxygen for private structs into the structs.

This should hopefully keep the comments more up to date with the structure comments. While I'm here, I've reworded a few of them to be more accurate, and dropped a bunch of stale comments. Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_priv.h | 149

[PATCH 7/7] glamor: Simplify some temp pixmap extents calculations.

Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_largepixmap.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/glamor/glamor_largepixmap.c b/glamor/glamor_largepixmap.c index 391f376..4c2534d 100644 --- a/glamor/glamor_largepixmap.c +++

[PATCH 6/7] glamor: Drop a dead flag to glamor_create_fbo_array().

Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_fbo.c | 35 +++ 1 file changed, 11 insertions(+), 24 deletions(-) diff --git a/glamor/glamor_fbo.c b/glamor/glamor_fbo.c index 5d59913..ffe8ef9 100644 --- a/glamor/glamor_fbo.c +++ b/glamor/glamor_fbo.c

[PATCH 3/7] glamor: Drop the dead PBO field in an FBO cache struct.

PBOs are only used at the whole pixmap level. Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_fbo.c | 2 -- glamor/glamor_priv.h | 2 -- 2 files changed, 4 deletions(-) diff --git a/glamor/glamor_fbo.c b/glamor/glamor_fbo.c index aedc804..5d59913 100644 --- a/glamor/glamor_fbo.c

[PATCH 5/7] glamor: Drop dead GLAMOR_FBO_DOWNLOADED flag.

Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_priv.h | 5 - glamor/glamor_utils.h | 1 - 2 files changed, 6 deletions(-) diff --git a/glamor/glamor_priv.h b/glamor/glamor_priv.h index 0960055..b9bd1ba 100644 --- a/glamor/glamor_priv.h +++ b/glamor/glamor_priv.h @@ -334,11

[PATCH 1/7] glamor: Propagate the fact that pbo_valid is never set.

The code to set it was deleted in keithp's big rewrite. Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_pixmap.c | 18 ++ glamor/glamor_priv.h | 2 -- 2 files changed, 2 insertions(+), 18 deletions(-) diff --git a/glamor/glamor_pixmap.c b/glamor/glamor_pixmap.c

[PATCH 2/7] glamor: Remove dead gl_tex flag in the fbo struct.

This used to be used in the old copy_area path. --- glamor/glamor_fbo.c | 6 -- glamor/glamor_priv.h | 2 -- 2 files changed, 8 deletions(-) diff --git a/glamor/glamor_fbo.c b/glamor/glamor_fbo.c index ea0e801..aedc804 100644 --- a/glamor/glamor_fbo.c +++ b/glamor/glamor_fbo.c @@ -476,12

[PULL] randr cleanups et al

Hi Keith, Various patches from me. Please consider pulling into master. The following changes since commit 0a78b599b34cc8b5fe6fe82f90e90234e8ab7a56: int10: Fix error check for pci_device_map_legacy (2015-03-13 12:31:21 +1000) are available in the git repository at:

Re: [PATCH synaptics] eventcomm: ignore fake and broken MT devices

Hi, On 24-03-15 06:45, Peter Hutterer wrote: An MT device without X/Y is not a touchpad. And neither are fake MT devices. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net Looks good: Reviewed-by: Hans de Goede hdego...@redhat.com Regards, Hans --- src/eventcomm.c | 9 +