Re: RADEON(0): [drm] Failed to open DRM device: No such file or directory error

On Saturday, March 8, 2014 9:01 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Sat, Mar 8, 2014 at 2:56 AM, stompdagg...@yahoo.com stompdagg...@yahoo.com wrote: greetings, since I've updated my kernel to 3.13.x I'm unable to start X on my radeon cards. the setup works great in kernel

[PATCH xf86-input-evdev] evdev: Add support for server managed fds

Signed-off-by: Hans de Goede hdego...@redhat.com --- src/evdev.c | 49 - 1 file changed, 28 insertions(+), 21 deletions(-) diff --git a/src/evdev.c b/src/evdev.c index d1ed9ee..ed84f0f 100644 --- a/src/evdev.c +++ b/src/evdev.c @@ -97,6 +97,10 @@

[PATCH xf86-input-synaptics] synaptics: Add support for server managed fds

Signed-off-by: Hans de Goede hdego...@redhat.com --- src/eventcomm.c | 10 -- src/synaptics.c | 30 +- src/synproto.h | 4 3 files changed, 33 insertions(+), 11 deletions(-) diff --git a/src/eventcomm.c b/src/eventcomm.c index a0a2af0..49a8a3f 100644

[PATCH] xserver: fix build since system-logind.h

on tinderbox and irc Signed-off-by: Dave Airlie airl...@redhat.com --- include/systemd-logind.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/systemd-logind.h b/include/systemd-logind.h index 8b09081..06dd031 100644 --- a/include/systemd-logind.h +++

Re: [PATCH] xserver: fix build since system-logind.h

Hi, On 03/08/2014 11:39 AM, Dave Airlie wrote: on tinderbox and irc Signed-off-by: Dave Airlie airl...@redhat.com Oops, sorry about that. Reviewed-by: Hans de Goede hdego...@redhat.com Regards, Hans --- include/systemd-logind.h | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [PATCH] xserver: fix build since system-logind.h

On Sat, Mar 08, 2014 at 08:39:30PM +1000, Dave Airlie wrote: on tinderbox and irc Signed-off-by: Dave Airlie airl...@redhat.com Reviewed-by: Matthieu Herrb matth...@herrb.eu --- include/systemd-logind.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH libxtrans] configure: under glibc define _GNU_SOURCE rather then _BSD_SOURCE

Hi all, On 03/03/2014 03:14 PM, Daniel Stone wrote: Hi, On 3 March 2014 07:56, Hans de Goede hdego...@redhat.com wrote: The latest glibc considers _BSD_SOURCE deprecated, leading to the following warning being issued for pretty much every C-file in the xserver: In file included from

Re: [PATCH libxtrans] configure: under glibc define _GNU_SOURCE rather then _BSD_SOURCE

On 14-03-08 09:03 AM, Hans de Goede wrote: So I've just been looking into using AC_USE_SYSTEM_EXTENSIONS but that seem pretty useless, it only adds the relevant defines to confdefs.h, not into cflags, and in libxtrans we want to add them to the .pc file, so we need them in a variable. I'm

Re: [PATCH libxtrans] configure: under glibc define _GNU_SOURCE rather then _BSD_SOURCE

Date: Sat, 08 Mar 2014 17:11:42 +0100 From: Hans de Goede hdego...@redhat.com Hi, On 03/08/2014 05:07 PM, Gaetan Nadon wrote: On 14-03-08 09:03 AM, Hans de Goede wrote: So I've just been looking into using AC_USE_SYSTEM_EXTENSIONS but that seem pretty useless, it only adds the

Re: [PATCH] xserver: fix build since system-logind.h

On 03/ 8/14 02:39 AM, Dave Airlie wrote: on tinderbox and irc Signed-off-by: Dave Airlie airl...@redhat.com Matches the changes I'd made to my tree last night to make the build work on Solaris, but hadn't gotten committed yet. Reviewed-by: Alan Coopersmith alan.coopersm...@oracle.com --

Re: [PATCH libxtrans] configure: under glibc define _GNU_SOURCE rather then _BSD_SOURCE

Hi, On 03/08/2014 05:31 PM, Mark Kettenis wrote: Date: Sat, 08 Mar 2014 17:11:42 +0100 From: Hans de Goede hdego...@redhat.com Hi, On 03/08/2014 05:07 PM, Gaetan Nadon wrote: On 14-03-08 09:03 AM, Hans de Goede wrote: So I've just been looking into using AC_USE_SYSTEM_EXTENSIONS but that

[PATCH 1/2] glamor_*gl_has_extension: Move strlen after check for NULL string

Flagged by cppcheck 1.64: [glamor/glamor_core.c:536] - [glamor/glamor_core.c:540]: (warning) Possible null pointer dereference: extension - otherwise it is redundant to check it against null. [glamor/glamor_egl.c:620] - [glamor/glamor_egl.c:622]: (warning) Possible null

[PATCH 2/2] Remove duplicate assignment of repeat_type_uniform_location

Flagged by cppcheck 1.64: [glamor/glamor_gradient.c:987] - [glamor/glamor_gradient.c:991]: (performance) Variable 'repeat_type_uniform_location' is reassigned a value before the old one has been used. Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com ---

Re: [PATCH 2/2] Remove duplicate assignment of repeat_type_uniform_location

cppcheck also flagged another instance in glamor for which I wasn't sure which value was right: [glamor/glamor_xv.c:301] - [glamor/glamor_xv.c:316]: (performance) Variable 'gamma' is reassigned a value before the old one has been used. In glamor_display_textured_video() in this function, the

[PATCH:xkbcomp 4/4] Initialize nMatch even if WIN32 is defined

Flagged by cppcheck 1.64: Checking app/xkbcomp/listing.c: WIN32... [app/xkbcomp/listing.c:335]: (error) Uninitialized variable: nMatch Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- listing.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/listing.c

[PATCH:xkbcomp 3/4] Remove useless assignment to 'outline' variable

Flagged by cppcheck 1.64: [app/xkbcomp/geometry.c:2426] - [app/xkbcomp/geometry.c:2427]: (performance) Variable 'outline' is reassigned a value before the old one has been used. Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- geometry.c |1 - 1 file changed, 1

[PATCH:xkbcomp 1/4] Remove useless checks for NULL before free in OverlayKeyCreate()

There is no need to ensure the pointers passed to free are not NULL, especially right after passing them to strncpy without checking for NULL. Flagged by cppcheck 1.64: [app/xkbcomp/parseutils.c:557] - [app/xkbcomp/parseutils.c:559]: (warning) Possible null pointer dereference: over - otherwise

[PATCH:xkbcomp 2/4] Don't dereference the pointer whose allocation failed

Flagged by cppcheck 1.64: [app/xkbcomp/keycodes.c:264] - [app/xkbcomp/keycodes.c:262]: (warning) Possible null pointer dereference: new - otherwise it is redundant to check it against null. [app/xkbcomp/keytypes.c:600] - [app/xkbcomp/keytypes.c:597]: (warning) Possible null pointer

[PATCH 8/9] glamor: Fix requested composite VBO size.

The argument to setup_composte_vbo is the number of verts. Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_render.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/glamor/glamor_render.c b/glamor/glamor_render.c index 7829977..63bddfd 100644 ---

[PATCH 5/9] glamor: Track the next vertex offset as we go for non-AA traps.

Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_trapezoid.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/glamor/glamor_trapezoid.c b/glamor/glamor_trapezoid.c index 7bbee4b..a774c37 100644 --- a/glamor/glamor_trapezoid.c +++

[PATCH 9/9] glamor: Use buffer_storage

v2: - Make the default buffer size a #define. (by Markus Wick) - Fix the return offset for mapping with buffer_storage. (oops!) v3: - Avoid GL error at first rendering from unmapping no buffer. - Rebase on the glBindBuffer(GL_ARRAY_BUFFER, 0) change. v4: Rebase on Markus's vbo init

[PATCH 1/9] glamor: Drop the set of the context to NULL at the end of glamor ops.

The theory here was that it (which I copied from EGL) existed to fix up context switching with indirect GLX. But indirect GLX won't even try to set the context again unless its lastContext field is cleared, so we need to solve this a different way. Signed-off-by: Eric Anholt e...@anholt.net ---

[PATCH 2/9] glamor: Don't forget to set GL_INVALIDATE_RANGE_BIT on GL_ARB_mbr.

We don't need any current contents of the buffer, and this allows an implementation to make a temporary BO for a streamed upload if it wants to. Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_render.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 6/9] glamor: Track the next vertex offset as we go for AA traps.

Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_trapezoid.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/glamor/glamor_trapezoid.c b/glamor/glamor_trapezoid.c index a774c37..4d2360d 100644 --- a/glamor/glamor_trapezoid.c +++

[PATCH 7/9] glamor: Extract the streamed vertex data code used by Render.

We should be uploading any vertex data using this kind of upload style, since it saves a bunch of extra copies of our vertex data. v2: - Add a simple comment about what the function does. - Use get_vbo_space()'s return in trapezoids, instead of dereffing glamor_priv-vb (by Markus Wick).

[PATCH:libXext] XeviGetVisualInfo: Free clear *evi_return, not evi_return pointer

evi_return is passed in as a pointer to a location into which XeviGetVisualInfo is expected to write a pointer to the memory it allocated for the returned structures. If we're failing and bailing out, we need to dispose of the pointer we set, not the one passed into us (which the caller may have