Re: [PATCH xinit v2 3/3] startx: Under Linux start X on the current VT

2014-04-04 Thread Julien Cristau
On Thu, Mar 27, 2014 at 14:18:14 +0100, Mark Kettenis wrote: What you're basically saying is that on a modern Linux systems, the historic behaviour where X allocates a new VT and occupies it is no longer wanted. This raises several questions: 1. Is there consensus among the various Linux

[PATCH xf86-video-vmware] configure: fix build without xatracker

2014-04-01 Thread Julien Cristau
ACTION-IF-NOT-FOUND is the fourth argument to PKG_CHECK_MODULES, not the fifth. Debian bug#743239 https://bugs.debian.org/743239 Signed-off-by: Julien Cristau jcris...@debian.org --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac

Re: [PATCH v2 2/2] xf86LogInit: log to XDG_DATA_HOME when not running as root

2014-04-01 Thread Julien Cristau
On Tue, Apr 1, 2014 at 17:33:08 +1000, Peter Hutterer wrote: the second point: 755 would be enough? any specific reason for 777? umask should take care of that. Either we consider the log private and should use 0700, or we don't and 0777 would be fine, IMO. Cheers, Julien

Re: [PATCH] Xephyr: restore initial window resize lost in xcb conversion

2014-03-31 Thread Julien Cristau
On Wed, Mar 26, 2014 at 22:37:04 -0700, Keith Packard wrote: Julien Cristau jcris...@debian.org writes: The XResizeWindow call wasn't replaced by the xcb equivalent, so we were no longer setting the initial window size, only wm size hints. Regression from commit a2b73da Xephyr: start

Re: [PATCH] Revert glx: Make sure we get an FBConfig the root window's visual.

2014-03-26 Thread Julien Cristau
...@anholt.net --- hw/kdrive/ephyr/ephyr_glamor_glx.c | 1 - 1 file changed, 1 deletion(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info

Re: [PATCH 2/2] Fix build when configured --enable-debug

2014-03-26 Thread Julien Cristau
...@dronecode.org.uk --- hw/xfree86/parser/DRI.c| 1 + hw/xfree86/parser/Extensions.c | 1 + 2 files changed, 2 insertions(+) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives

Re: [PATCH xinit 2/3] startx: Under Linux start X on the current VT

2014-03-25 Thread Julien Cristau
On Tue, Mar 25, 2014 at 12:56:35 +0100, Hans de Goede wrote: When we let X allocate a new VT, systemd-logind will not recognize any processes running on this VT as belonging to a valid session (since there was no pam session opened on that tty). This causes problems like PolicyKit denials

Re: [PATCH xinit 3/3] Makefile.am: Give XINITDIR a default value

2014-03-25 Thread Julien Cristau
On Tue, Mar 25, 2014 at 12:56:36 +0100, Hans de Goede wrote: We are using XINITDIR, but never setting it. Lets give it a sane default. Signed-off-by: Hans de Goede hdego...@redhat.com --- Makefile.am | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Makefile.am b/Makefile.am index

Re: [PATCH] Enable subdir-objects

2014-02-12 Thread Julien Cristau
On Wed, Feb 12, 2014 at 17:15:26 +0100, Thierry Reding wrote: automake complains about the subdir-objects being missing. Enabling it, however, causes various build issues to pop up because $(srcdir), $(top_srcdir), $(builddir) and $(top_builddir) aren't handled properly. It's simple to work

Re: [PATCH libX11] Remove dead USE_OWN_COMPOSE-protected code

2014-02-11 Thread Julien Cristau
-- src/xkb/XKBBind.c | 102 -- 2 files changed, 174 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien signature.asc Description: Digital signature ___ xorg

Re: [PATCH 3/7] input: un-constify InputAttributes

2014-01-30 Thread Julien Cristau
On Thu, Jan 30, 2014 at 09:43:37 +0100, Hans de Goede wrote: Hi, On 01/30/2014 12:51 AM, Peter Hutterer wrote: Introduced in fecc7eb1cf66db64728ee2d68cd9443df7e70879 and reverts most of that but it's helpfully mixed with other stuff. InputAttributes are not const, they're strdup'd

Re: [PATCH 5/7] input: un-constify dev-name

2014-01-30 Thread Julien Cristau
On Thu, Jan 30, 2014 at 09:48:01 +0100, Hans de Goede wrote: Hi, On 01/30/2014 12:51 AM, Peter Hutterer wrote: Fallout from fecc7eb1cf66db64728ee2d68cd9443df7e70879, and reverts most of the rest of that patch. The device name is allocated and may even change during PreInit. The

Re: [PATCH 3/7] input: un-constify InputAttributes

2014-01-30 Thread Julien Cristau
On Thu, Jan 30, 2014 at 14:57:23 +0100, Hans de Goede wrote: Oops correction I should look closer myself, that is in 2/7 not 3/7 . Anyways I still believe that using XNFstrdup instead would be better. I don't see why sloppy coding (lacking error checking is sloppy coding) would be ok in test

Re: signed tar files (was: [ANNOUNCE] xf86-video-ati 7.3.0)

2014-01-25 Thread Julien Cristau
On Sat, Jan 25, 2014 at 10:46:03 -0800, Alan Coopersmith wrote: On 01/25/14 07:09 AM, Claus Assmann wrote: On Sat, Jan 25, 2014, Julien Cristau wrote: gpg: BAD signature from Alex Deucher alexdeuc...@gmail.com Same here. If this isn't just me, any chance to get the tarball checksums

Re: [ANNOUNCE] xf86-video-ati 7.3.0

2014-01-25 Thread Julien Cristau
On Sat, Jan 25, 2014 at 15:26:51 -0500, Alex Deucher wrote: I guess gmail mangled it. Does the attached work any better? Yes, looks good. Thanks! Cheers, Julien ___ xorg@lists.x.org: X.Org support Archives:

Re: signed tar files (was: [ANNOUNCE] xf86-video-ati 7.3.0)

2014-01-25 Thread Julien Cristau
On Sat, Jan 25, 2014 at 10:46:03 -0800, Alan Coopersmith wrote: On 01/25/14 07:09 AM, Claus Assmann wrote: On Sat, Jan 25, 2014, Julien Cristau wrote: gpg: BAD signature from Alex Deucher alexdeuc...@gmail.com Same here. If this isn't just me, any chance to get the tarball checksums

Re: [PATCH] Replace 'pointer' with the equivalent 'void *'.

2014-01-22 Thread Julien Cristau
.. Reviewed-by: Julien Cristau jcris...@debian.org [...] diff --git a/src/fontfile/catalogue.c b/src/fontfile/catalogue.c index 09ca2ae..81a1e13 100644 --- a/src/fontfile/catalogue.c +++ b/src/fontfile/catalogue.c @@ -40,7 +40,7 @@ static const char CataloguePrefix[] = catalogue:; static int

Re: Compilation of xserver broken ( git 2d2d49dab)

2014-01-19 Thread Julien Cristau
On Thu, Jan 16, 2014 at 13:21:21 +0100, Knut Petersen wrote: On 15.01.2014 21:01, Knut Petersen wrote: Hi Keith! I see a build failure on an openSuSE 13.1 system: Some more information: openSuSE 13.1, cpu: Pentium-M (32 bits), gcc (SUSE Linux) 4.8.1 20130909 [gcc-4_8-branch revision

Re: [PATCH:xf86-input-mouse] Use asprintf (or Xprintf on old servers) instead of strdup+sprintf

2014-01-05 Thread Julien Cristau
On Sat, Jan 4, 2014 at 20:35:06 +0100, Patrik Jakobsson wrote: asprintf needs _GNU_SOURCE to be defined or you'll get an implicit declaration and build failure. I fixed it with: diff --git a/src/mouse.c b/src/mouse.c index 2da2b4d..6678d24 100644 --- a/src/mouse.c +++ b/src/mouse.c @@

Bug#732514: [PATCH] fixes for Radeon KMS on kFreeBSD

2013-12-18 Thread Julien Cristau
On Wed, Dec 18, 2013 at 21:40:21 +0100, Robert Millan wrote: - Load radeonkms module instead of radeon. Actually, this was done at the wrong place. The first argument to drmOpen() call is a red herring. It's supposed to load a kernel module, but unless the module is already loaded this

Bug#732514: [PATCH] fixes for Radeon KMS on kFreeBSD

2013-12-18 Thread Julien Cristau
On Wed, Dec 18, 2013 at 22:05:27 +0100, Robert Millan wrote: Besides, when it comes to KMS drivers, is there a point in auto-loading them just because the hardware is present? AFAICS it makes a lot more sense to load them only when X is started and we know we will need them. Yes. At least

Re: [PATCH] exa: only draw valid trapezoids

2013-12-17 Thread Julien Cristau
On Wed, Oct 2, 2013 at 15:47:54 +0200, Maarten Lankhorst wrote: Fixes freedesktop.org bug https://bugs.freedesktop.org/show_bug.cgi?id=67484 If t-bottom is close to MIN_INT, removing top can wraparound, so do the check properly. Does the underflow affect users of xTrapezoidValid other

Bug#732216: xserver-xorg-video-radeon: Driver does not support Glamor 2D acceleration

2013-12-15 Thread Julien Cristau
On Sun, Dec 15, 2013 at 19:26:10 +0100, Jan Binder wrote: Package: xserver-xorg-video-radeon Version: 1:7.2.0-1+b2 Severity: important Dear Maintainer, On newer Radeon cards from the Southern Islands and Northern Islands family, Xorg does not really work out of the box due to missing

Re: [PATCH] xcb_aux_parse_color should take const char *

2013-12-11 Thread Julien Cristau
On Wed, Dec 11, 2013 at 12:10:16 -0800, Keith Packard wrote: This elimiantes warnings when passing string constants Signed-off-by: Keith Packard kei...@keithp.com --- src/xcb_aux.c | 2 +- src/xcb_aux.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Julien

[libxshmfence PATCH] Check return value from ftruncate

2013-12-03 Thread Julien Cristau
Silences compiler warning: xshmfence_alloc.c: In function 'xshmfence_alloc_shm': xshmfence_alloc.c:54:11: warning: ignoring return value of 'ftruncate', declared with attribute warn_unused_result [-Wunused-result] ftruncate(fd, sizeof (struct xshmfence)); ^ Signed-off-by: Julien

[PATCH libXft] Use FT_*_H macros instead of including freetype/*.h

2013-12-02 Thread Julien Cristau
The latter doesn't seem to work with recent freetype. Signed-off-by: Julien Cristau jcris...@debian.org --- src/xftglyphs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/xftglyphs.c b/src/xftglyphs.c index 2f3dc5a..4b5fb82 100644 --- a/src/xftglyphs.c +++ b/src

Re: [PATCH 06/14] composite: Automatically enable backing store support on the screen

2013-12-02 Thread Julien Cristau
On Thu, Nov 21, 2013 at 17:43:19 -0500, Adam Jackson wrote: ... unless you explicitly disabled it with -bs on the command line, or with the corresponding thing in xorg.conf. v2: Drop a bogus hunk from compChangeWindowAttributes [vsyrjala] Signed-off-by: Adam Jackson a...@redhat.com ---

Re: [PATCH 0/3] libxtrans + xserver: Add support for systemd socket activation

2013-12-02 Thread Julien Cristau
On Mon, Dec 2, 2013 at 20:50:33 +0100, Hans de Goede wrote: Given that this is mostly for more embedded setups and it does not work with xdm/gdm, I'm not sure shipping these as example is a good idea. They won't be usable ootb in anyway, integration work will need to be done by distros who

Re: [PATCH] Xdmx: Initialize DMX extension even if not built with GLX support

2013-11-25 Thread Julien Cristau
-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- hw/dmx/dmxinit.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development

Re: [PATCH] Describe the library better in the README file

2013-11-25 Thread Julien Cristau
On Mon, Nov 25, 2013 at 13:38:13 -0800, Keith Packard wrote: Signed-off-by: Keith Packard kei...@keithp.com --- Is this better? README | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) Looks good to me, thanks! Cheers, Julien

Re: [PATCH libxshmfence 2/2] Add the required README file

2013-11-24 Thread Julien Cristau
On Tue, Nov 5, 2013 at 09:28:25 -0500, Gaetan Nadon wrote: Signed-off-by: Gaetan Nadon mems...@videotron.ca --- README | 25 + 1 file changed, 25 insertions(+) create mode 100644 README diff --git a/README b/README new file mode 100644 index 000..66b36f0

Re: [PATCH libX11 1/2] xcb_io: Fix Xlib 32-bit request number wrapping

2013-11-18 Thread Julien Cristau
On Sun, Nov 17, 2013 at 21:30:44 +0100, Jonas Petersen wrote: Am 17.11.2013 20:20, schrieb Mouse: I guess the sizeof comparison would not be necessary since the condition should never meet with 64-bit longs. Unless it's in a code fragment that's used only on machines with 64-bit longs, it

Re: [PATCH] xext: Fix shmint.h to not use headers outside of sdk_HEADERS

2013-11-13 Thread Julien Cristau
. protocol-versions.h is not part of sdk_HEADERS, so instead of using that, just use XTRANS_SEND_FDS to choose whether to expose the fd passing requests directly. Reported-by: Julien Cristau jcris...@debian.org Signed-off-by: Keith Packard kei...@keithp.com v2: also avoid using protocol

Re: [PATCH 4/7] Add support for receiving fds in replies

2013-11-07 Thread Julien Cristau
On Thu, Nov 7, 2013 at 15:11:30 +0100, Daniel Martin wrote: Just a nitpick: On 6 November 2013 01:41, Keith Packard kei...@keithp.com wrote: Requests signal which replies will have fds, and the replies report how many fds they expect in byte 1. Signed-off-by: Keith Packard

Re: [PATCH] Add correct dependency on libxtrans.

2013-11-07 Thread Julien Cristau
On Wed, Nov 6, 2013 at 16:27:26 -0800, Eric Anholt wrote: The 1.3 release is when the new FD passing bits landed. Signed-off-by: Eric Anholt e...@anholt.net --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers

Re: [PATCH 1/7] Make XTrans FD passing support optional. Define only on Linux

2013-11-07 Thread Julien Cristau
On Thu, Nov 7, 2013 at 12:15:54 -0800, Keith Packard wrote: Until other operating systems have a libXtrans port for FD passing, disable this on non-Linux systems. Note that this define affects how libXtrans gets built into the X server, which is why it need only define the symbol

Re: [PATCH] os: Actually use the computed clockid in GetTimeInMicros

2013-11-05 Thread Julien Cristau
On Tue, Nov 5, 2013 at 07:10:27 -0800, Keith Packard wrote: The selection of which clock to use for this function was not actually getting used when fetching the final clock value. Reported-by: Julien Cristau jcris...@debian.org Signed-off-by: Keith Packard kei...@keithp.com --- os

Re: [PATCH] Trap SIGBUS to handle truncated shared memory segments

2013-11-05 Thread Julien Cristau
On Sun, Nov 3, 2013 at 14:05:05 -0800, Keith Packard wrote: +panic: +if (previous_busfault_sigaction) +(*previous_busfault_sigaction)(sig, info, param); +else +exit(1); +} + maybe change that exit() to FatalError()? Cheers, Julien

Re: [PATCH] xfree86: Bump video/input/extension ABIs for 1.15

2013-10-29 Thread Julien Cristau
On Tue, Oct 29, 2013 at 12:58:14 -0400, Adam Jackson wrote: Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/common/xf86Module.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [RFC] GLX dispatch rewrite

2013-10-24 Thread Julien Cristau
On Thu, Oct 24, 2013 at 14:11:04 +0800, Keith Packard wrote: Adam Jackson a...@redhat.com writes: I'd send this as a patch, but it's like 2M, so I figure that's rude. I'd love to see this in two pieces -- the first containing all of the actual interesting changes and the second just the

Re: any xserver stable branch nominations?

2013-10-19 Thread Julien Cristau
On Thu, Oct 17, 2013 at 17:29:26 -0600, Matt Dew wrote: Hi all, Today was the nominal 1.14.4-rc2 release. I've gotten no nominations, pull or cherry-pick requests since 1.14.3 was released. Wrong. I sent a pull request a month ago, in 20130915201100.gb5...@radis.cristau.org Cheers,

Re: [PATCH] glx: Only take a Pixmap reference if creating the GLXPixmap succeeded

2013-10-09 Thread Julien Cristau
On Wed, Oct 9, 2013 at 15:08:26 -0400, Adam Jackson wrote: Signed-off-by: Adam Jackson a...@redhat.com --- glx/glxcmds.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Bug#725409: xserver-xorg-video-radeon: Non-displayable screen-modes during system startup

2013-10-07 Thread Julien Cristau
On Sat, Oct 5, 2013 at 13:33:35 +0200, Andreas Glaeser wrote: At the moment I use the onboard graphics-solution ATI RS780 [Radeon HD 3200], as hardinfo tells me. The problem was mentioned already there: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=711941 Now a new kernel-version was

[git pull] hurd build fixes

2013-10-05 Thread Julien Cristau
Hi Keith, these are a couple of build fixes for hurd. Cheers, Julien The following changes since commit f0659a7f68a5a334879af01a6a63742eab1bf5de: Bump version to 1.14.99.2 (2013-10-04 16:44:46 -0700) are available in the git repository at: git://people.freedesktop.org/~jcristau/xserver

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 RESEND xserver] miext/shadow: missing c2p_core.h breaks make distcheck target.

2013-09-29 Thread Julien Cristau
Might want to keep them in alphabetical order, but other than that, Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman

Bug#724264: xserver-xorg-video-radeon: X server crashes in multiple programs attempting to view a jpg file

2013-09-25 Thread Julien Cristau
On Mon, Sep 23, 2013 at 12:21:33 +0930, Arthur Marsh wrote: Package: xserver-xorg-video-radeon Version: 1:7.2.0-1+b1 Severity: normal Dear Maintainer, *** Please consider answering these questions, where appropriate *** * What led up to the situation? Viewing a particular jpg file

Re: [PATCH libXaw 1/2] Makefile: use $(LN_S) for better code portability

2013-09-22 Thread Julien Cristau
On Sun, Sep 22, 2013 at 16:05:48 -0400, Gaetan Nadon wrote: AC_PROG_LN_S is brought in by AC_PROG_LIBTOOL Signed-off-by: Gaetan Nadon mems...@videotron.ca --- src/Makefile.am |8 1 file changed, 4 insertions(+), 4 deletions(-) OOI, which system we target doesn't have ln -s?

Bug#720329: xserver-xorg-video-radeon: Screen keeping black at start but for the cursor

2013-09-22 Thread Julien Cristau
On Sun, Sep 22, 2013 at 12:15:03 +0200, Samuel Hym wrote: Hello, Sorry for the slow reply. I finally tried setting the proposed kernel options, with the same result (or slightly worse, the cursor got blurred or something). I imagine the real problem is the fact that my card is not

Re: [PATCHES] fix build on GNU/Hurd with GCC 4.8

2013-09-20 Thread Julien Cristau
On Thu, Sep 19, 2013 at 21:43:02 -0600, Matt Dew wrote: On 09/18/2013 02:43 AM, Julien Cristau wrote: On Tue, Sep 17, 2013 at 22:50:46 -0600, Matt Dew wrote: Chris, I'm guessing this won't affect you but can you verify that? I don't want a nasty loop where fixing one things breaks

Re: [PATCHES] fix build on GNU/Hurd with GCC 4.8

2013-09-18 Thread Julien Cristau
On Tue, Sep 17, 2013 at 22:50:46 -0600, Matt Dew wrote: Chris, I'm guessing this won't affect you but can you verify that? I don't want a nasty loop where fixing one things breaks another. Err, what are you talking about? Cheers, Julien ___

[pull server-1.14] Build/test fixes

2013-09-15 Thread Julien Cristau
://anonscm.debian.org/users/jcristau/xorg-server server-1.14-branch for you to fetch changes up to a1985e892f5b5f0959235a631d8bcf676b78bfbc: dix: add missing include for DeleteWindowFromAnySelections (2013-09-15 22:03:45 +0200) Julien Cristau

Re: [PATCH][vmware] build: add explicit option to enable/disable VMWGFX

2013-09-13 Thread Julien Cristau
On Fri, Sep 13, 2013 at 09:08:22 +0100, Burton, Ross wrote: On 13 September 2013 06:01, Jakob Bornecrantz wallbra...@gmail.com wrote: If I'm reading this correctly this will cause the build to fail if you don't give it the disable flag if xatracker is missing? I'm on the fence if we want

[ANNOUNCE] xf86-video-dummy 0.3.7

2013-09-09 Thread Julien Cristau
This release contains a single patch to let the driver build against xserver 1.14. Adam Jackson (1): Remove mibstore.h Julien Cristau (1): Bump to 0.3.7 git tag: xf86-video-dummy-0.3.7 http://xorg.freedesktop.org/archive/individual/driver/xf86-video-dummy-0.3.7.tar.bz2 MD5

[ANNOUNCE] xf86-video-vesa 2.3.3

2013-09-09 Thread Julien Cristau
API Egbert Eich (1): Make bool option parsing clearer and more consistent Julien Cristau (1): Bump to 2.3.3 git tag: xf86-video-vesa-2.3.3 http://xorg.freedesktop.org/archive/individual/driver/xf86-video-vesa-2.3.3.tar.bz2 MD5: 3690a5356ed121b1a7dfb59a6dcf4bf9 xf86-video-vesa

Bug#721796: xserver-xorg-video-radeon: X not starting anymore after upgrade

2013-09-04 Thread Julien Cristau
On Wed, Sep 4, 2013 at 22:32:12 +0200, Eloi Notario wrote: El Dimecres, 4 de setembre de 2013, a les 12:47:32, Michel Dänzer va escriure: [...] Looks like the radeon kernel module doesn't get loaded before X starts. Current upstream xf86-video-ati only works with the radeon kernel

Re: [PATCH] fonts: Fix Xinerama sleep logic

2013-09-01 Thread Julien Cristau
On Fri, Aug 30, 2013 at 10:55:51 -0400, 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

Re: [PATCH 09/17] Remove the host/server split for XV attributes.

2013-08-31 Thread Julien Cristau
On Mon, Aug 26, 2013 at 13:26:15 -0700, Eric Anholt wrote: --- hw/kdrive/ephyr/ephyrhostvideo.c | 107 --- hw/kdrive/ephyr/ephyrhostvideo.h | 19 -- hw/kdrive/ephyr/ephyrvideo.c | 133 --- 3 files changed, 82

Re: Xephyr fixes patch series

2013-08-31 Thread Julien Cristau
On Mon, Aug 26, 2013 at 13:33:33 -0700, Eric Anholt wrote: The whole combined Xephyr series is at ephyr-fixes of my tree. Reviewed-by: Julien Cristau jcris...@debian.org for this series. Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development

Re: [PATCH 02/17] ephyr: Expose a single function for detecting extensions.

2013-08-30 Thread Julien Cristau
On Mon, Aug 26, 2013 at 13:26:08 -0700, Eric Anholt wrote: Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr/ephyr.c | 15 ++- hw/kdrive/ephyr/ephyrdriext.c | 7 +-- hw/kdrive/ephyr/ephyrglxext.c | 3 ++- hw/kdrive/ephyr/hostx.c | 29

Re: [PATCH 05/17] ephyr: Move event processing into ephyr.c.

2013-08-30 Thread Julien Cristau
On Mon, Aug 26, 2013 at 13:26:11 -0700, Eric Anholt wrote: No more extra event structure to translate between hostx.c and ephyr.c! --- hw/kdrive/ephyr/ephyr.c | 393 hw/kdrive/ephyr/hostx.c | 225 ---

Bug#720329: xserver-xorg-video-radeon: Screen keeping black at start but for the cursor

2013-08-30 Thread Julien Cristau
On Tue, Aug 20, 2013 at 16:34:44 +0200, Michel Dänzer wrote: Upstream xf86-video-ati 7.x only works with KMS anymore. You'll need to enable it with something like radeon.modeset=1 video=radeonfb:off on the kernel command line if you want to use a current version of the driver.

Re: [PATCH 16/19] Xephyr: move glx code to xcb

2013-08-28 Thread Julien Cristau
On Mon, Aug 26, 2013 at 13:21:00 -0700, Eric Anholt wrote: @@ -560,81 +366,70 @@ Bool ephyrHostGLXMakeCurrent(int a_drawable, int a_readable, int a_glx_ctxt_id, int a_old_ctxt_tag, int *a_ctxt_tag) { +xcb_connection_t *conn = hostx_get_xcbconn(); Bool

Re: [PATCH app-rstart] The scripts in the commands subdir are missing execute permissions

2013-08-21 Thread Julien Cristau
On Tue, Aug 20, 2013 at 20:54:37 -0400, Gaetan Nadon wrote: These were treated as _DATA and installed with -m 644 option on the /usr/bin/install program. Use _SCRIPTS Automake constructs for scripts. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- commands/Makefile.am |6

Re: [PATCH app-rstart] The scripts in the commands subdir are missing execute permissions

2013-08-21 Thread Julien Cristau
++--- 1 file changed, 6 insertions(+), 3 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Tested-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg

[PATCH imake] Replace fprintf with fputs

2013-08-17 Thread Julien Cristau
Fixes build failure with -Werror=format-security: imake.c:1008:5: error: format not a string literal and no format arguments [-Werror=format-security] fprintf (inFile, command); ^ Signed-off-by: Julien Cristau jcris...@debian.org --- imake.c |2 +- 1 file changed, 1 insertion

Re: [PATCH 1/9] ephyr: Fix const cast warnings in ephyrhostvideo.

2013-08-17 Thread Julien Cristau
On Sun, Aug 18, 2013 at 00:36:35 +0200, Eric Anholt wrote: --- hw/kdrive/ephyr/ephyrhostvideo.c | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

Re: [PATCH 1/9] ephyr: Fix const cast warnings in ephyrhostvideo.

2013-08-17 Thread Julien Cristau
On Sun, Aug 18, 2013 at 02:14:31 +0200, Julien Cristau wrote: On Sun, Aug 18, 2013 at 00:36:35 +0200, Eric Anholt wrote: --- hw/kdrive/ephyr/ephyrhostvideo.c | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) Reviewed-by: Julien Cristau

Re: [PATCH xgc] configure: look for gram.c in $srcdir

2013-08-14 Thread Julien Cristau
On Tue, Aug 13, 2013 at 14:44:30 -0700, Alan Coopersmith wrote: On 08/13/13 03:49 AM, Julien Cristau wrote: That's where it is when building from the tarball. Regression from ff668395fc408f540604bd534e323b483f52a4ea. --- configure.ac |2 +- 1 file changed, 1 insertion(+), 1 deletion

Re: [PATCH app-rstart 08/15] config: install commands using Automake

2013-08-14 Thread Julien Cristau
I know this patch is two and a half years old, but... On Thu, Jan 27, 2011 at 15:58:36 -0500, Gaetan Nadon wrote: This replaces the *FIXME* shell code which generates in place the commands rather than doing a proper installation using the $INSTALL automake specified program. Using

[PATCH xgc] configure: look for gram.c in $srcdir

2013-08-13 Thread Julien Cristau
That's where it is when building from the tarball. Regression from ff668395fc408f540604bd534e323b483f52a4ea. --- configure.ac |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 04f55f6..66560f9 100644 --- a/configure.ac +++ b/configure.ac @@

[PATCH libXfont] Make serverGeneration unsigned

2013-08-12 Thread Julien Cristau
Makes the definition match other declarations, and xserver's definition. Debian bug#689439 Reported-by: Michael Tautschnig m...@debian.org Signed-off-by: Julien Cristau jcris...@debian.org --- src/util/miscutil.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util

Re: [PATCH] xfree86: improve check for posix saved ids

2013-08-03 Thread Julien Cristau
On Sat, Aug 3, 2013 at 11:34:25 +0200, Mark Kettenis wrote: From: Julien Cristau jcris...@debian.org Date: Fri, 2 Aug 2013 20:07:36 +0200 Replace hardcoded SVR4 || linux || CSRG_BASED with an autoconf check and the _POSIX_SAVED_IDS macro. Suggested-by: Mark Kettenis mark.kette

Re: [PATCH] test: include dix-config.h in hashtabletest.c

2013-08-03 Thread Julien Cristau
On Fri, Aug 2, 2013 at 23:46:00 +0200, Julien Cristau wrote: Missing _XSERVER64 define caused inconsistent sizeof(XID) between the test and hashtable code, leading to test failures on 64bit big endian archs like s390x or ppc64. oops, forgot Signed-off-by: Julien Cristau jcris...@debian.org

Re: [PATCH 1/2] xfree86: Check for issetugid declaration

2013-08-02 Thread Julien Cristau
On Sun, Jul 28, 2013 at 13:34:44 +0200, Mark Kettenis wrote: Date: Sun, 28 Jul 2013 10:48:16 +0200 From: Julien Cristau jcris...@debian.org On Sat, Jul 27, 2013 at 23:25:22 +0200, Mark Kettenis wrote: From: Julien Cristau jcris...@debian.org Date: Sat, 27 Jul 2013 21:08:28

[PATCH] xfree86: improve check for posix saved ids

2013-08-02 Thread Julien Cristau
Replace hardcoded SVR4 || linux || CSRG_BASED with an autoconf check and the _POSIX_SAVED_IDS macro. Suggested-by: Mark Kettenis mark.kette...@xs4all.nl Signed-off-by: Julien Cristau jcris...@debian.org --- configure.ac |2 +- hw/xfree86/parser/write.c |3 ++- include/xorg

[PATCH] test: include dix-config.h in hashtabletest.c

2013-08-02 Thread Julien Cristau
Missing _XSERVER64 define caused inconsistent sizeof(XID) between the test and hashtable code, leading to test failures on 64bit big endian archs like s390x or ppc64. --- test/hashtabletest.c |4 1 file changed, 4 insertions(+) diff --git a/test/hashtabletest.c b/test/hashtabletest.c

Re: [PATCH 1/2] xfree86: Check for issetugid declaration

2013-07-28 Thread Julien Cristau
On Sat, Jul 27, 2013 at 23:25:22 +0200, Mark Kettenis wrote: From: Julien Cristau jcris...@debian.org Date: Sat, 27 Jul 2013 21:08:28 +0200 Avoids 'implicit function declaration' build error on gnu/kfreebsd. That's seriously backwards. You should make sure the right headers get

Re: [PATCH 2/2] xfree86: include os.h for FatalError

2013-07-28 Thread Julien Cristau
On Sat, Jul 27, 2013 at 23:38:43 +0200, Mark Kettenis wrote: From: Julien Cristau jcris...@debian.org Date: Sat, 27 Jul 2013 21:08:29 +0200 The os.h header should probably be included even in if HAS_SAVED_IDS_AND_SETEUID is defined, as in that case ErrorF gets called, which also lives

Re: [PATCH 1/2] xfree86: fix a memory leak in edidMakeAtom().

2013-07-28 Thread Julien Cristau
On Thu, Jul 11, 2013 at 18:27:14 -0400, Alex Deucher wrote: On Thu, Jul 11, 2013 at 6:02 PM, Julien Cristau jcris...@debian.org wrote: On Mon, Jun 17, 2013 at 12:39:20 -0400, alexdeuc...@gmail.com wrote: From: Leo Liu leo@amd.com leak happens when looping xrandr prop. Signed

[PATCH] dix: add missing include for DeleteWindowFromAnySelections

2013-07-27 Thread Julien Cristau
-by: Julien Cristau jcris...@debian.org --- dix/window.c |1 + 1 file changed, 1 insertion(+) diff --git a/dix/window.c b/dix/window.c index 8950f97..9fa51c2 100644 --- a/dix/window.c +++ b/dix/window.c @@ -126,6 +126,7 @@ Equipment Corporation. #ifdef COMPOSITE #include compint.h #endif +#include

[PATCH 1/2] xfree86: Check for issetugid declaration

2013-07-27 Thread Julien Cristau
Avoids 'implicit function declaration' build error on gnu/kfreebsd. Signed-off-by: Julien Cristau jcris...@debian.org --- configure.ac |1 + hw/xfree86/common/xf86Init.c |2 +- include/xorg-config.h.in |4 3 files changed, 6 insertions(+), 1 deletion(-) diff

[PATCH 2/2] xfree86: include os.h for FatalError

2013-07-27 Thread Julien Cristau
Signed-off-by: Julien Cristau jcris...@debian.org --- hw/xfree86/parser/write.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/xfree86/parser/write.c b/hw/xfree86/parser/write.c index 9c706a0..b525b11 100644 --- a/hw/xfree86/parser/write.c +++ b/hw/xfree86/parser/write.c @@ -67,6

Re: [PATCH:luit 0/3] configure.ac cleanups bug fix

2013-07-13 Thread Julien Cristau
- 2 files changed, 8 insertions(+), 18 deletions(-) for the series: Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http

Re: [PATCH 1/2] xfree86: fix a memory leak in edidMakeAtom().

2013-07-11 Thread Julien Cristau
On Mon, Jun 17, 2013 at 12:39:20 -0400, alexdeuc...@gmail.com wrote: From: Leo Liu leo@amd.com leak happens when looping xrandr prop. Signed-off-by: Leo Liu leo@amd.com --- hw/xfree86/common/xf86Helper.c |1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH libXRes] Compute number of bytes correctly in XResQueryClientPixmapBytes

2013-07-06 Thread Julien Cristau
Control: tags -1 fixed-upstream On Sat, Jul 6, 2013 at 11:29:52 -0700, Alan Coopersmith wrote: On 06/30/13 11:09 AM, Julien Cristau wrote: 64-bit servers send the upper 32-bit in bytes_overflow. We were multiplying by 2^32 - 1 instead of 2^32 when putting things back together. Debian

Re: [PATCH makedepend] Don't crash if an option's argument is missing

2013-07-02 Thread Julien Cristau
On Mon, Jul 1, 2013 at 16:20:29 -0700, Alan Coopersmith wrote: On 07/ 1/13 10:04 AM, Julien Cristau wrote: Avoid dereferencing argv[argc]. See http://www.forallsecure.com/bug-reports/011f1a55f79a5501b36008d6ee0d40e8b6644569/ Reported-by: Alexandre Rebert alexan...@cmu.edu Signed-off

Re: [PATCH xinput] Increase interactiveness of test* output

2013-07-02 Thread Julien Cristau
On Tue, Jul 2, 2013 at 18:13:53 +0100, Jon TURNEY wrote: On 20/08/2012 01:45, Peter Hutterer wrote: From: Nikolai Kondrashov spbnick-re5jqeeqqe8avxtiumw...@public.gmane.org Force line buffering for test and test_xi2 to increase interactiveness and avoid losing data with non-terminal

[PATCH makedepend] Don't crash if an option's argument is missing

2013-07-01 Thread Julien Cristau
Avoid dereferencing argv[argc]. See http://www.forallsecure.com/bug-reports/011f1a55f79a5501b36008d6ee0d40e8b6644569/ Reported-by: Alexandre Rebert alexan...@cmu.edu Signed-off-by: Julien Cristau jcris...@debian.org --- main.c | 16 1 file changed, 16 insertions(+) diff

[PATCH] xkb: don't call atoi(NULL) when parsing argv

2013-07-01 Thread Julien Cristau
If the -ardelay or -arinterval options have no argument, there's no point trying to read it. See http://www.forallsecure.com/bug-reports/feb3db57fc206d8df22ca53a6907f74973876272/ Reported-by: Alexandre Rebert alexan...@cmu.edu Signed-off-by: Julien Cristau jcris...@debian.org --- xkb/xkbInit.c

Re: [PATCH] xkb: don't call atoi(NULL) when parsing argv

2013-07-01 Thread Julien Cristau
On Mon, Jul 1, 2013 at 19:39:30 +0200, Julien Cristau wrote: If the -ardelay or -arinterval options have no argument, there's no point trying to read it. See http://www.forallsecure.com/bug-reports/feb3db57fc206d8df22ca53a6907f74973876272/ Reported-by: Alexandre Rebert alexan...@cmu.edu

[PATCH] Compute number of bytes correctly in XResQueryClientPixmapBytes

2013-06-30 Thread Julien Cristau
64-bit servers send the upper 32-bit in bytes_overflow. We were multiplying by 2^32 - 1 instead of 2^32 when putting things back together. Debian bug#621702 Reported-by: Kevin Ryde use...@zip.com.au Signed-off-by: Julien Cristau jcris...@debian.org --- src/XRes.c |2 +- 1 file changed, 1

Re: [PATCH libXRes] Compute number of bytes correctly in XResQueryClientPixmapBytes

2013-06-30 Thread Julien Cristau
On Sun, Jun 30, 2013 at 20:09:49 +0200, Julien Cristau wrote: 64-bit servers send the upper 32-bit in bytes_overflow. We were multiplying by 2^32 - 1 instead of 2^32 when putting things back together. Debian bug#621702 Reported-by: Kevin Ryde use...@zip.com.au Signed-off-by: Julien

Re: [PATCH:videoproto] Xvproto.h: fix xvQueryEncodingsReply to match actual wire protocol

2013-06-23 Thread Julien Cristau
the wire. Reported-by: Daniel Martin consume.no...@gmail.com Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- Xvproto.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien

[PATCH libX11] Add missing locales to configure.ac

2013-06-15 Thread Julien Cristau
Commits 40761898692e5063957bfa2518cca3d35b2e354a and f198c6aa98f88ff285d903175a3c4c0fd33a4575 added two new locales (sr_CS.UTF-8 and km_KH.UTF-8), but didn't list them in configure.ac, meaning they're not included in tarballs. Signed-off-by: Julien Cristau jcris...@debian.org --- configure.ac

Re: [PATCH videoproto] Xvproto: fix xvEncodingInfo to match actual wire protocol

2013-06-10 Thread Julien Cristau
On Mon, Jun 10, 2013 at 11:41:00 +0200, Daniel Martin wrote: On 9 June 2013 19:00, Julien Cristau jcris...@debian.org wrote: The padding is *before* the rate field, so the rate is placed on a 32bit boundary. This change adds explicit padding between height and rate, and removes extraneous

[PATCH videoproto] Xvproto: fix xvEncodingInfo to match actual wire protocol

2013-06-09 Thread Julien Cristau
that's not a big deal as clients only see the Xlib structure XvEncodingInfo. Debian bug#657215 Reported-by: Kevin Ryde use...@zip.com.au Signed-off-by: Julien Cristau jcris...@debian.org --- Xvproto.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Xvproto.h b/Xvproto.h index

[PATCH libXvMC] avoid overflowing by making nameLen and busIDLen addition overflow

2013-06-01 Thread Julien Cristau
Al Viro pointed this out on lwn: if nameLen + busIDLen overflows, we end up copying data from outside tmpBuf. Reported-by: Al Viro v...@zeniv.linux.org.uk Signed-off-by: Julien Cristau jcris...@debian.org --- src/XvMC.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 1/5] Xdmx: integer overflow in GetGLXVisualConfigs()

2013-06-01 Thread Julien Cristau
On Thu, May 23, 2013 at 09:27:26 -0700, Alan Coopersmith wrote: @@ -92,15 +96,16 @@ GetGLXVisualConfigs(Display * dpy, int screen, int *nconfigs) } /* Allocate memory for our config structure */ -config = (__GLXvisualConfig *) -Xmalloc(nvisuals *

Re: [PATCH 4/5] Xephyr: integer overflow in XF86DRIOpenConnection()

2013-06-01 Thread Julien Cristau
); SyncHandle(); We might still overflow in the _XEatData call, but I guess that's not so much an issue. Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org

<    1   2   3   4   5   6   7   8   9   10   >