Re: [PATCH 10/19] xfree86: Remove Option TextClockFreq

2014-09-25 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: No modern driver pays attention to this. Presumably there existed hardware once where you couldn't just read the right values out of the CRTC. Reviewed-by: Eric Anholt e...@anholt.net pgpNUpue8mlHq.pgp Description: PGP signature

Re: [PATCH 11/19] xfree86: Remove spurious xf86ConfigError

2014-09-25 Thread Eric Anholt
(\nConfig Error:\n); -va_start(ap, msg); -VErrorF(msg, ap); -va_end(ap); -ErrorF(\n); -return; -} Yeah, doesn't seem useful, really. (also, the return; as the last statement is just the best) Reviewed-by: Eric Anholt e...@anholt.net pgpHykxJBKd1a.pgp Description: PGP

Re: [PATCH 12/19] xfree86: Remove useless back-pointer to pScrn from colormap code

2014-09-25 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/common/xf86cmap.c | 46 +++- 1 file changed, 20 insertions(+), 26 deletions(-) Looks nicer to me. Reviewed-by: Eric Anholt e...@anholt.net

Re: [PATCH 13/19] xfree86: Remove DisplayID support

2014-09-25 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: Not actually wired up so it's fairly useless. Reviewed-by: Eric Anholt e...@anholt.net pgpyp44Uj2jSa.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org

Re: [PATCH 14/19] xfree86: Remove xf86Info.log

2014-09-25 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: SUBCLASS ALL THE THINGS Isn't the Log enum dead now? Other than that, Reviewed-by: Eric Anholt e...@anholt.net pgp4QRTLe7D2i.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives

Re: [PATCH 15/19] xfree86: Remove unused xf86Info.useDefaultFontPathFrom

2014-09-25 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: Signed-off-by: Adam Jackson a...@redhat.com Reviewed-by: Eric Anholt e...@anholt.net pgpNeLTmeBy5p.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org

Re: [PATCH 16/19] xfree86: configServerFlags never fails, make it return void

2014-09-25 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: Signed-off-by: Adam Jackson a...@redhat.com Patches 16, 17, and 19 are also: Reviewed-by: Eric Anholt e...@anholt.net pgpwWyi_3X7og.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development

Re: [PATCH 3/3] xfree86: Delete file modinit.h

2014-10-01 Thread Eric Anholt
Daniel Martin consume.no...@gmail.com writes: All references to modinit.h have been remove with: a1d41e3 Move extension initialisation prototypes into extinit.h Signed-off-by: Daniel Martin consume.no...@gmail.com Reviewed-by: Eric Anholt e...@anholt.net pgpby1o6A9iXE.pgp Description

[PATCH 2/4] modesetting: Fix some weird formatting after x-indent-all.sh.

2014-10-08 Thread Eric Anholt
Signed-off-by: Eric Anholt e...@anholt.net --- hw/xfree86/drivers/modesetting/drmmode_display.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/hw/xfree86/drivers/modesetting/drmmode_display.c b/hw/xfree86/drivers/modesetting/drmmode_display.c index ca02a4d..d959837

[PATCH 4/4] modesetting: Add support for rendering using glamor.

2014-10-08 Thread Eric Anholt
By default modesetting now tries to enable X acceleration using glamor, but falls back to normal shadowfb if GL fails to initialize. Signed-off-by: Eric Anholt e...@anholt.net --- hw/xfree86/drivers/modesetting/Makefile.am | 1 + hw/xfree86/drivers/modesetting/driver.c| 101

[PATCH 3/4] modesetting: Deduplicate some scrn setup.

2014-10-08 Thread Eric Anholt
Signed-off-by: Eric Anholt e...@anholt.net --- hw/xfree86/drivers/modesetting/driver.c | 55 + 1 file changed, 22 insertions(+), 33 deletions(-) diff --git a/hw/xfree86/drivers/modesetting/driver.c b/hw/xfree86/drivers/modesetting/driver.c index f509a19..c713746

[PATCH 1/4] modesetting: Run x-indent-all.sh.

2014-10-08 Thread Eric Anholt
As I was editing code, the top-level .dir-locals.el was making my new stuff conflict with the existing style. Make it consistently use the xorg style, instead. Signed-off-by: Eric Anholt e...@anholt.net --- hw/xfree86/drivers/modesetting/driver.c | 579 ++--- hw/xfree86/drivers

[PATCH 2/4] dri2: Automatically fill in the driver name if the DDX doesn't provide it.

2014-10-10 Thread Eric Anholt
, except for the lack of nouveau_vieux support (which requires a predicate on the device). Signed-off-by: Eric Anholt e...@anholt.net --- hw/xfree86/dri2/dri2.c | 61 ++ 1 file changed, 61 insertions(+) diff --git a/hw/xfree86/dri2/dri2.c b/hw/xfree86

[PATCH 4/4] modesetting: Add support for DRI2 with glamor.

2014-10-10 Thread Eric Anholt
-rendering that require presence in GLX (which aren't supported in glxdriswrast.c). Signed-off-by: Eric Anholt e...@anholt.net --- hw/xfree86/drivers/modesetting/Makefile.am |6 +- hw/xfree86/drivers/modesetting/dri2.c| 1122 ++ hw/xfree86/drivers/modesetting

[PATCH 1/4] dri2: Import a copy of Mesa's PCI ID - driver name mappings.

2014-10-10 Thread Eric Anholt
. (Some day, when AIGLX transitions to EGL, we can drop the DRI2 filename setup entirely). Signed-off-by: Eric Anholt e...@anholt.net --- hw/xfree86/dri2/pci_ids/i810_pci_ids.h | 4 + hw/xfree86/dri2/pci_ids/i915_pci_ids.h | 15 ++ hw/xfree86/dri2/pci_ids/i965_pci_ids.h | 115

Re: [PATCH 1/4] dri2: Import a copy of Mesa's PCI ID - driver name mappings.

2014-10-10 Thread Eric Anholt
Eric Anholt e...@anholt.net writes: This comes from Mesa commit acdcef6788beaa2a1532e13ff84c3e246b8025ed I forgot to mention, this series is on the modesetting-dri2 branch of my tree. I've done minor cleanups on 4/4 as I started looking into Present. pgpi3bhmRt3Jz.pgp Description: PGP

Re: [PATCH] Xephyr: option to disable grabbing the host

2014-10-11 Thread Eric Anholt
William ML Leslie william.leslie@gmail.com writes: This patch makes it possible to use C-S key combinations within Xephyr without losing access to the host window manager's commands. Reviewed-by: Eric Anholt e...@anholt.net pgpaBuaCz4BPX.pgp Description: PGP signature

Re: [PATCH 2/4] dri2: Automatically fill in the driver name if the DDX doesn't provide it.

2014-10-11 Thread Eric Anholt
Mark Marshall markmarshal...@gmail.com writes: Just a minor thing On 10 October 2014 11:09, Eric Anholt e...@anholt.net wrote: This will be used by the modesetting driver to support DRI2 across all hardware that can support glamor, and could potentially be used by other drivers that have

[PATCH 4/4 v3] modesetting: Add support for DRI2 with glamor.

2014-10-15 Thread Eric Anholt
(not that the normal 2D driver is stable with pageflipping for me), and I won't get it fixed before the merge window. It now passes all of the OML_sync_control tests from Jamey and Theo (except for occasional warns in timing -fullscreen -divisor 2). Signed-off-by: Eric Anholt e

Re: [PATCH 1/4] dri2: Import a copy of Mesa's PCI ID - driver name mappings.

2014-10-17 Thread Eric Anholt
Adam Jackson a...@nwnk.net writes: On Fri, 2014-10-10 at 11:09 +0200, Eric Anholt wrote: +CHIPSET(0xA011, IGD_GM, Intel(R) IGD) +CHIPSET(0xA001, IGD_G,Intel(R) IGD) Can we not? Pineview, right? Or Atom D4xx/D5xx/N4xx/N5xx, or whatever, but IGD is worse than useless. +CHIPSET

Re: [PATCH 1/4] dri2: Import a copy of Mesa's PCI ID - driver name mappings.

2014-10-19 Thread Eric Anholt
Alex Deucher alexdeuc...@gmail.com writes: On Fri, Oct 17, 2014 at 12:08 PM, Eric Anholt e...@anholt.net wrote: Adam Jackson a...@nwnk.net writes: On Fri, 2014-10-10 at 11:09 +0200, Eric Anholt wrote: +CHIPSET(0xA011, IGD_GM, Intel(R) IGD) +CHIPSET(0xA001, IGD_G,Intel(R) IGD) Can

[PATCH 3/4] modesetting: Export two functions I want to reuse from DRI2/Present.

2014-10-24 Thread Eric Anholt
This renames dumb_get_bo_from_handle(), since it wasn't using a handle (GEM terminology) but a dmabuf fd. Signed-off-by: Eric Anholt e...@anholt.net --- hw/xfree86/drivers/modesetting/drmmode_display.c | 8 hw/xfree86/drivers/modesetting/drmmode_display.h | 3 +++ 2 files changed, 7

[PULL] modesetting DRI2 support

2014-10-27 Thread Eric Anholt
to cac4b064f9f66435430f61568c6a516c54bf3c40: modesetting: Add support for DRI2 with glamor. (2014-10-27 13:16:43 -0700) Eric Anholt ( dri2: Import a copy of Mesa's PCI ID - driver name mappings. dri2: Automatically fill

Re: [PATCH] os: Server terminated successfully is not an error

2014-11-24 Thread Eric Anholt
: Reviewed-by: Keith Packard kei...@keithp.com With X_INFO, Reviewed-by: Eric Anholt e...@anholt.net pgp5eco4yj70a.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http

Re: [PATCH] glx: Revert part of 9838b703

2012-07-02 Thread Eric Anholt
Ian Romanick i...@freedesktop.org writes: From: Ian Romanick ian.d.roman...@intel.com The server does not want GL extension prototypes. It never links with anything that could possibly provide implementations of these functions. It *is* the provide, and it does not provde these symbols.

Re: [PATCH:intel-gpu-tools 2/4] Rename NOPID to NO_PID to avoid conflict with Solaris NOPID

2012-08-24 Thread Eric Anholt
Alan Coopersmith alan.coopersm...@oracle.com writes: Solaris sys/types.h already has #define NOPID (pid_t)(-1) Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com NOPID is the ID that was in a MI_NOOP instruction, so NOP_ID would be better. pgpSQUQ5EnVhN.pgp Description: PGP

Re: [PATCH] configure: Stop using AM_MAINTAINER_MODE

2012-09-27 Thread Eric Anholt
-in to correct behavior, instead of just having correct behavior all the time. It should die. Reviewed-by: Eric Anholt e...@anholt.net pgpAMn8ri3SOS.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives

Re: [PATCH] Update dri2proto.txt to reflect current extension contents

2012-10-01 Thread Eric Anholt
no effect on the X server, but + presumably it should cause the server to stop delivering + InvalidateBuffers events? Spec text with question marks. I love this extension! No substantive review. Looks good. Reviewed-by: Eric Anholt e...@anholt.net pgpO6I0egIHHH.pgp Description: PGP

Re: [PATCH 11/12] dri2: Don't block GLX clients on VT switch

2012-10-16 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: This was necessary in DRI1 for a variety of reasons (SAREA ownership, VRAM layout, etc). None of that is relevant to DRI2. Which is good, because VT switch is an xf86-ism and I'd like to make DRI2 ddx-independent. Signed-off-by: Adam Jackson

[PATCH] dri2: Don't short-circuit to blitting for vblank_mode=0.

2012-11-13 Thread Eric Anholt
When someone asks for no vblank syncing, they're also going for high performance, which forcing a blit is not. Leave this up to the driver's normal SwapBuffers path, which can choose something appropriate to do based off of the target/divisor/remainder. Note that this currently reduces

[PATCH 1/2] xv: Fix warnings in drivers using string literals in XV structs.

2012-11-17 Thread Eric Anholt
The server doesn't need to mutate these strings, nor do we expect drivers to. --- hw/xfree86/common/xf86xv.h |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/xfree86/common/xf86xv.h b/hw/xfree86/common/xf86xv.h index 091efca..f393369 100644 ---

[PATCH 2/2] xf86: Mark a few driver-provided strings const.

2012-11-17 Thread Eric Anholt
This cleans up const-cast warnings in the intel driver's screen setup. I left the chipset/board/etc. fields non-const, because I could *possibly* imagine a driver changing those out of the screen struct at runtime. --- hw/xfree86/common/xf86str.h |6 +++--- 1 file changed, 3 insertions(+), 3

Re: [PATCH 4/5] glx: Implement GLX_PRESERVED_CONTENTS drawable attribute

2013-02-25 Thread Eric Anholt
always have a preserved buffer -- we always behave that way, right? Patches 1, 2, 3, and 5 are: Reviewed-by: Eric Anholt e...@anholt.net pgpIZ2hpxf5lV.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http

[PATCH] Revert DRI2: re-allocate DRI2 drawable if pixmap serial changes

2013-06-05 Thread Eric Anholt
This reverts commit 3209b094a3b1466b579e8020e12a4f3fa78a5f3f. After a long debug session by Paul Berry, it appears that this was the commit that has been producing sporadic failures in piglit front buffer rendering tests for the last several years. GetBuffers may return fresh buffers with

[PATCH] Revert DRI2: re-allocate DRI2 drawable if pixmap serial changes

2013-06-17 Thread Eric Anholt
. Signed-off-by: Eric Anholt e...@anholt.net Reviewed-by: Keith Packard kei...@keithp.com Reviewed-by: Chris Wilson ch...@chris-wilson.co.uk Tested-by: Chris Wilson ch...@chris-wilson.co.uk --- hw/xfree86/dri2/dri2.c | 19 +-- 1 file changed, 1 insertion(+), 18 deletions(-) diff

[PATCH 6/9] kdrive: Fix const cast warnings in driver name handling.

2013-08-17 Thread Eric Anholt
--- hw/kdrive/linux/mouse.c | 2 +- hw/kdrive/src/kdrive.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/kdrive/linux/mouse.c b/hw/kdrive/linux/mouse.c index aaab0cd..2bfe7f2 100644 --- a/hw/kdrive/linux/mouse.c +++ b/hw/kdrive/linux/mouse.c @@ -193,7 +193,7 @@

[PATCH 7/9] kdrive: Fix const cast warnings in arguments processing.

2013-08-17 Thread Eric Anholt
--- hw/kdrive/ephyr/ephyrinit.c | 4 ++-- hw/kdrive/ephyr/os.c| 2 +- hw/kdrive/src/kdrive.c | 6 +++--- hw/kdrive/src/kdrive.h | 9 + hw/kdrive/src/kinput.c | 4 ++-- 5 files changed, 13 insertions(+), 12 deletions(-) diff --git a/hw/kdrive/ephyr/ephyrinit.c

[PATCH 3/9] ephyr: Fix dead code warning.

2013-08-17 Thread Eric Anholt
--- hw/kdrive/ephyr/ephyrhostglx.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/hw/kdrive/ephyr/ephyrhostglx.c b/hw/kdrive/ephyr/ephyrhostglx.c index 6a4392f..5ecb02d 100644 --- a/hw/kdrive/ephyr/ephyrhostglx.c +++ b/hw/kdrive/ephyr/ephyrhostglx.c @@ -274,7 +274,6 @@

[PATCH 4/9] kdrive: Drop dead code.

2013-08-17 Thread Eric Anholt
This is unused as of the introduction of a helper in e7150db5350bc2113ff4126019b489847a4dc217 --- hw/kdrive/src/kinput.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/hw/kdrive/src/kinput.c b/hw/kdrive/src/kinput.c index c30f170..3aad87a 100644 --- a/hw/kdrive/src/kinput.c +++

[PATCH 5/9] kdrive: Fix a few easy cast warnings.

2013-08-17 Thread Eric Anholt
--- hw/kdrive/linux/evdev.c | 2 +- hw/kdrive/linux/linux.c | 2 +- hw/kdrive/linux/mouse.c | 4 ++-- hw/kdrive/linux/ps2.c | 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/hw/kdrive/linux/evdev.c b/hw/kdrive/linux/evdev.c index 9982fad..63e8409 100644 ---

[PATCH 9/9] ephyr: Fix warning about XID vs unsigned long * by changing function args

2013-08-17 Thread Eric Anholt
There's no reason to pass the data back out to the caller, since the caller was dropping it on the floor. The original data is a CARD32, so no need to mess with weird unsigned longs. --- hw/kdrive/ephyr/ephyrdri.c| 5 +++-- hw/kdrive/ephyr/ephyrdri.h| 2 +- hw/kdrive/ephyr/ephyrdriext.c

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

2013-08-17 Thread Eric Anholt
--- hw/kdrive/ephyr/ephyrhostvideo.c | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/hw/kdrive/ephyr/ephyrhostvideo.c b/hw/kdrive/ephyr/ephyrhostvideo.c index 05e9ad9..99f0f50 100644 --- a/hw/kdrive/ephyr/ephyrhostvideo.c +++

[PATCH 2/9] ephyr: Fix const-cast warnings for setting window title.

2013-08-17 Thread Eric Anholt
--- hw/kdrive/ephyr/hostx.c | 2 +- hw/kdrive/ephyr/hostx.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/kdrive/ephyr/hostx.c b/hw/kdrive/ephyr/hostx.c index f2b458d..5071289 100644 --- a/hw/kdrive/ephyr/hostx.c +++ b/hw/kdrive/ephyr/hostx.c @@ -215,7 +215,7 @@

[PATCH 8/9] ephyr: Drop is_ok logging.

2013-08-17 Thread Eric Anholt
The only cases that lead to !is_ok are already EPHYR_LOG_ERROR, and it fixes set-but-unused compiler warnings. --- hw/kdrive/ephyr/ephyrdriext.c | 19 --- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/hw/kdrive/ephyr/ephyrdriext.c b/hw/kdrive/ephyr/ephyrdriext.c

[PULL] Xephyr warning fixes

2013-08-21 Thread Eric Anholt
up to e01a3f65d3e6d92f92ef2992b338cc9625bde082: ephyr: Fix warning about XID vs unsigned long * by changing function args (2013-08-21 09:10:49 -0700) Eric Anholt (9): ephyr: Fix const cast warnings in ephyrhostvideo

Re: [PATCH 03/19] glxproxy: Convert to private-based client tracking

2013-08-21 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: This is mostly to make things look more like glx/. Note that the currentDrawables and currentContexts tracking is gone from glx. We can't get rid of them yet in DMX because we're not actually running with a full GLX under us, so there's not

Re: [PATCH 09/19] glx: Implement GLX_FBCONFIG_ID in GetDrawableAttributes

2013-08-21 Thread Eric Anholt
] = GLX_FBCONFIG_ID; +attributes[11] = pGlxDraw-config-fbconfigID; +numAttribs++; Would be nice to see a piglit test checking that this works. Similarly for patch 19. Other than that, all but patch 3 are: Reviewed-by: Eric Anholt e...@anholt.net pgpYX1s1NGIOx.pgp Description: PGP

[PATCH 03/19] Xephyr: use xcb-shape instead of XShape*

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Julien Cristau jcris...@debian.org --- hw/kdrive/ephyr/hostx.c | 40 +++- 1 file changed, 27 insertions(+), 13

[PATCH 01/19] Xephyr: stop loading the host's keymap

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org This isn't used anywhere. v2: Rebase to the top of the patch series (anholt) Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net (v1) Signed-off-by: Julien Cristau jcris...@debian.org Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr

[PATCH 04/19] Xephyr: no need for XDisplayKeycodes

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Julien Cristau jcris...@debian.org --- hw/kdrive/ephyr/hostx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/kdrive

[PATCH 05/19] Xephyr: xcb-ify pointer/keyboard grab

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Julien Cristau jcris...@debian.org --- hw/kdrive/ephyr/hostx.c | 45 - 1 file changed, 32 insertions

[PATCH 02/19] Xephyr: start converting hostx.c over to xcb

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org v2: Dropped the hostx_load_keymap changes, now that that function is gutted (anholt). Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net (v1) Signed-off-by: Julien Cristau jcris...@debian.org Signed-off-by: Eric Anholt e...@anholt.net

[PATCH 07/19] Xephyr: delete unused proxy code

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Julien Cristau jcris...@debian.org --- hw/kdrive/ephyr/ephyrhostproxy.c | 91 --- hw/kdrive/ephyr/ephyrhostproxy.h

[PATCH 09/19] Xephyr: replace XKeycodeToKeysym with xcb-keysyms

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Julien Cristau jcris...@debian.org --- configure.ac| 2 +- hw/kdrive/ephyr/hostx.c | 13 - 2 files changed, 9 insertions

[PATCH 06/19] Xephyr: xcb-ify visual list construction

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Julien Cristau jcris...@debian.org --- hw/kdrive/ephyr/hostx.c | 59 +++-- 1 file changed, 33

[PATCH 11/19] Xephyr: some more hostx.c xcb-ification

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Julien Cristau jcris...@debian.org --- hw/kdrive/ephyr/hostx.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/hw

[PATCH 12/19] Xephyr: use xcb for event handling

2013-08-26 Thread Eric Anholt
Gusarov dotted...@dottedmag.net (v1) Signed-off-by: Julien Cristau jcris...@debian.org Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr/hostx.c | 284 ++-- 1 file changed, 155 insertions(+), 129 deletions(-) diff --git a/hw/kdrive/ephyr

[PATCH 10/19] Xephyr: move HostX.visual to xcb

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Julien Cristau jcris...@debian.org --- hw/kdrive/ephyr/hostx.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/hw

[PATCH 18/19] Xephyr: handle errors in event loop

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Signed-off-by: Julien Cristau jcris...@debian.org Reviewed-by: Eric Anholt e...@anholt.net Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net --- hw/kdrive/ephyr/hostx.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/hw/kdrive

[PATCH 17/19] Xephyr: drop remaining Xlib dependency

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Julien Cristau jcris...@debian.org --- configure.ac| 4 ++-- hw/kdrive/ephyr/hostx.c | 64

[PATCH 19/19] Xephyr: we're not using Xlib anymore, no need to undef _XSERVER64

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Drop obsolete comments about interaction between Xlib code and that macro, and stop undefining it. Signed-off-by: Julien Cristau jcris...@debian.org Reviewed-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr/ephyrhostglx.c | 11 --- hw/kdrive

[PATCH 08/19] Xephyr: delete unused hostx_get_extension_info function

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org v2: Also remove the prototype (anholt) Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net (v1) Signed-off-by: Julien Cristau jcris...@debian.org Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr/hostx.c | 15 --- hw/kdrive

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

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org v2: Rebase on master, notably adding XCB for X_GLXvop_MakeCurrentReadSGI (anholt). Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net (v1) Signed-off-by: Julien Cristau jcris...@debian.org Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive

[PATCH 13/19] Xephyr: use xcb-xv instead of libXv

2013-08-26 Thread Eric Anholt
Cristau jcris...@debian.org Signed-off-by: Eric Anholt e...@anholt.net --- configure.ac | 2 +- hw/kdrive/ephyr/ephyrhostvideo.c | 892 +++ hw/kdrive/ephyr/ephyrhostvideo.h | 45 +- hw/kdrive/ephyr/ephyrvideo.c | 45 +- 4 files changed, 382

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

2013-08-26 Thread Eric Anholt
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 - hw/kdrive/ephyr/hostx.h | 3 ++- 5 files

[PATCH 03/17] ephyr: Rename and use the proper type for what was host_screen-info.

2013-08-26 Thread Eric Anholt
Now that we can include server headers in talking to host X, we don't need to hide any more. --- hw/kdrive/ephyr/hostx.c | 41 + hw/kdrive/ephyr/hostx.h | 26 +- 2 files changed, 34 insertions(+), 33 deletions(-) diff --git

[PATCH 14/19] Xephyr: move ephyrdri over to xcb

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Require new xcb-xf86dri to get fixed GetDrawableInfo. Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Signed-off-by: Julien Cristau jcris...@debian.org --- configure.ac | 2 +- hw/kdrive/ephyr/ephyrdri.c | 201

[PATCH 01/17] ephyr: Drop dead ephyrHostAtomToLocal code.

2013-08-26 Thread Eric Anholt
Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr/ephyrvideo.c | 34 -- 1 file changed, 34 deletions(-) diff --git a/hw/kdrive/ephyr/ephyrvideo.c b/hw/kdrive/ephyr/ephyrvideo.c index 345c7b9..5813634 100644 --- a/hw/kdrive/ephyr/ephyrvideo.c +++ b/hw

xephyr continue deleting code series.

2013-08-26 Thread Eric Anholt
You can also find these two series in my personal tree, http://cgit.freedesktop.org/~anholt/xserver/log/?h=xephyr-xcb-rebase http://cgit.freedesktop.org/~anholt/xserver/log/?h=ephyr-continue-deleting ___ xorg-devel@lists.x.org: X.Org development

[PATCH 04/17] ephyr: Move the host screen info into the kdrive screen private.

2013-08-26 Thread Eric Anholt
We can include xcb bits from the same place as server headers, so there's no need to hide them any more. --- hw/kdrive/ephyr/ephyr.c | 26 +--- hw/kdrive/ephyr/ephyr.h | 20 ++- hw/kdrive/ephyr/ephyrinit.c | 5 +- hw/kdrive/ephyr/hostx.c | 372

[PATCH 15/19] Xephyr: remove unused DRI1 code

2013-08-26 Thread Eric Anholt
From: Julien Cristau jcris...@debian.org Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net Signed-off-by: Julien Cristau jcris...@debian.org --- hw/kdrive/ephyr/Makefile.am | 2 - hw/kdrive/ephyr/XF86dri.c | 654 hw/kdrive/ephyr/xf86dri.h |

[PATCH 10/17] ephyr: Remove the host/server split for XV formats.

2013-08-26 Thread Eric Anholt
--- hw/kdrive/ephyr/ephyrhostvideo.c | 25 - hw/kdrive/ephyr/ephyrhostvideo.h | 11 --- hw/kdrive/ephyr/ephyrvideo.c | 22 +++--- 3 files changed, 15 insertions(+), 43 deletions(-) diff --git a/hw/kdrive/ephyr/ephyrhostvideo.c

[PATCH 11/17] ephyr: Remove the host/server split for video encodings.

2013-08-26 Thread Eric Anholt
--- hw/kdrive/ephyr/ephyrhostvideo.c | 52 hw/kdrive/ephyr/ephyrhostvideo.h | 22 -- hw/kdrive/ephyr/ephyrvideo.c | 64 3 files changed, 39 insertions(+), 99 deletions(-) diff --git

[PATCH 07/17] ephyr: Refactor XV adaptor feature detection.

2013-08-26 Thread Eric Anholt
This obviously wanted a helper function beforehand, but even more so now that we have XCB. --- hw/kdrive/ephyr/ephyrhostvideo.c | 68 hw/kdrive/ephyr/ephyrhostvideo.h | 11 --- hw/kdrive/ephyr/ephyrvideo.c | 48 ++-- 3 files

[PATCH 08/17] ephyr: Remove the host/server split for XV adaptors.

2013-08-26 Thread Eric Anholt
--- hw/kdrive/ephyr/ephyrhostvideo.c | 69 hw/kdrive/ephyr/ephyrhostvideo.h | 11 --- hw/kdrive/ephyr/ephyrvideo.c | 44 +++-- 3 files changed, 28 insertions(+), 96 deletions(-) diff --git a/hw/kdrive/ephyr/ephyrhostvideo.c

[PATCH 06/17] ephyr: Remove some pointless indirection in the XV code.

2013-08-26 Thread Eric Anholt
Now that we have XCB on the server side, we don't need to split these functions out. --- hw/kdrive/ephyr/ephyrhostvideo.c | 54 hw/kdrive/ephyr/ephyrhostvideo.h | 13 -- hw/kdrive/ephyr/ephyrvideo.c | 23 ++--- 3 files changed, 7

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

2013-08-26 Thread Eric Anholt
--- hw/kdrive/ephyr/ephyrhostvideo.c | 107 --- hw/kdrive/ephyr/ephyrhostvideo.h | 19 -- hw/kdrive/ephyr/ephyrvideo.c | 133 --- 3 files changed, 82 insertions(+), 177 deletions(-) diff --git

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

2013-08-26 Thread Eric Anholt
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 --- hw/kdrive/ephyr/hostx.h | 64 ++-- 3 files changed, 310 insertions(+), 372

[PATCH 17/17] ephyr: Garbage collect some DOA host window clipping code.

2013-08-26 Thread Eric Anholt
Introduced in 79782726617d4b5a4f9b376f21936d035fc870e1 but never used. --- hw/kdrive/ephyr/hostx.c | 40 hw/kdrive/ephyr/hostx.h | 3 --- 2 files changed, 43 deletions(-) diff --git a/hw/kdrive/ephyr/hostx.c b/hw/kdrive/ephyr/hostx.c index

[PATCH 12/17] ephyr: Remove another host/server split for XV image formats.

2013-08-26 Thread Eric Anholt
--- hw/kdrive/ephyr/ephyrhostvideo.c | 55 --- hw/kdrive/ephyr/ephyrhostvideo.h | 37 --- hw/kdrive/ephyr/ephyrvideo.c | 63 +++- 3 files changed, 56 insertions(+), 99 deletions(-) diff --git

[PATCH 16/17] ephyr: Garbage collect some dead XV clipping code.

2013-08-26 Thread Eric Anholt
--- hw/kdrive/ephyr/ephyrvideo.c | 117 ++- 1 file changed, 4 insertions(+), 113 deletions(-) diff --git a/hw/kdrive/ephyr/ephyrvideo.c b/hw/kdrive/ephyr/ephyrvideo.c index fe69043..e78f18d 100644 --- a/hw/kdrive/ephyr/ephyrvideo.c +++

[PATCH 15/17] ephyr: Remove the helper libs for each of the optional components.

2013-08-26 Thread Eric Anholt
Note that EXTRA_DIST was always unnecessary, because automake notices conditional compile of source files and includes them. Copyright header is added because git noted that this was a 61% rewrite. --- hw/kdrive/ephyr/Makefile.am | 87 - 1 file

[PATCH 14/17] ephyr: Remove the remaining bits of host/server XV split.

2013-08-26 Thread Eric Anholt
--- hw/kdrive/ephyr/Makefile.am | 4 +- hw/kdrive/ephyr/ephyrhostvideo.c | 130 --- hw/kdrive/ephyr/ephyrhostvideo.h | 53 hw/kdrive/ephyr/ephyrvideo.c | 48 ++- 4 files changed, 33 insertions(+), 202 deletions(-)

[PATCH 13/17] ephyr: Remove the host/server split of the actual XV video operations.

2013-08-26 Thread Eric Anholt
I suspect there's more cleanup possible in ephyrHostXVPutImage() by sticking a bunch of the args in the port priv earlier. --- hw/kdrive/ephyr/ephyrhostvideo.c | 189 --- hw/kdrive/ephyr/ephyrhostvideo.h | 47 -- hw/kdrive/ephyr/ephyrvideo.c | 171

Xephyr fixes patch series

2013-08-26 Thread Eric Anholt
The whole combined Xephyr series is at ephyr-fixes of my tree. ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

[PATCH 2/4] ephyr: Flush the X connection when updating the window title.

2013-08-26 Thread Eric Anholt
Otherwise when you're doing the ctrl-shift mouse grab thing, you don't know what state you're in until the next rendering occurs. Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr/hostx.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/kdrive/ephyr/hostx.c b/hw/kdrive/ephyr

[PATCH 1/4] ephyr: Use host (HW) cursors by default.

2013-08-26 Thread Eric Anholt
Unless you're working on the sw cursor rendering code, you surely want to have real hardware cursors. Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr/ephyrinit.c | 8 ++-- hw/kdrive/ephyr/hostx.c | 8 hw/kdrive/ephyr/hostx.h | 2 +- 3 files changed, 11

[PATCH 3/4] ephyr: Do grab/ungrab for ctrl+shift, not just shift+ctrl.

2013-08-26 Thread Eric Anholt
Given that the window title says ctrl+shift, having pressing those keys in that order not ungrab you is fairly mean. Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr/ephyr.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/hw/kdrive/ephyr/ephyr.c b/hw

[PATCH 4/4] kdrive: initialize GLX for xephyr

2013-08-26 Thread Eric Anholt
://bugs.freedesktop.org/show_bug.cgi?id=62346 Reviewed-by: Eric Anholt e...@anholt.net Signed-off-by: Eric Anholt e...@anholt.net --- hw/kdrive/ephyr/ephyrinit.c | 20 1 file changed, 20 insertions(+) diff --git a/hw/kdrive/ephyr/ephyrinit.c b/hw/kdrive/ephyr/ephyrinit.c index

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

2013-09-03 Thread Eric Anholt
Julien Cristau jcris...@debian.org writes: On Mon, Aug 26, 2013 at 13:26:11 -0700, Eric Anholt wrote: -struct EphyrHostXEvent { -EphyrHostXEventType type; -union { -struct mouse_motion { -int x; -int y; -int screen; -int

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

2013-09-03 Thread Eric Anholt
Julien Cristau jcris...@debian.org writes: 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

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

2013-09-03 Thread Eric Anholt
Julien Cristau jcris...@debian.org writes: 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

Re: [PATCH 2/7] damage: Simplify DamageUnregister

2013-09-03 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: You can only register one drawable on a given damage, so there's no reason to require the caller to specify the drawable, the damage is enough. The implementation would do something fairly horrible if you _did_ pass mismatched drawable and damage, so

Re: [PATCH 7/7] damageext: Xineramify

2013-09-03 Thread Eric Anholt
, 3, 4, and 6 are: Reviewed-by: Eric Anholt e...@anholt.net pgptHnR7JyAqT.pgp Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

[PULL] xephyr xcb conversion, code deletion, and bugfixes

2013-09-03 Thread Eric Anholt
changes up to abc2bfca16adcd1e5657d4ce54f4e1335d5ed53f: kdrive: initialize GLX for xephyr (2013-09-03 14:35:39 -0700) Eric Anholt (20): ephyr: Drop dead ephyrHostAtomToLocal code. ephyr: Expose a single function

Re: [PATCH RESEND xserver 3/6] Xorg binary: use install-exec-hook rather than install-exec-local

2013-09-28 Thread Eric Anholt
, it is possible to perform post-installation modifications using an install hook. With the -local targets, there is no particular guarantee of execution order; typically, they are run early, but with parallel make, there is no way to be sure of that. Reviewed-by: Eric Anholt e...@anholt.net

Re: [PATCH 2/3] glx: Fix memory leak in context garbage collection

2013-09-30 Thread Eric Anholt
, so that we have a back-pointer to chase when walking the list of contexts when ClientStateGone happens. Patch 1/3 is: Reviewed-by: Eric Anholt e...@anholt.net But I'm lost on this one. loseCurrent is just -core-unbindContext, which is mesa's dri_util.c dereferencing the drawables and calling

Re: [PATCH] loader: Use RTLD_DEEPBIND if available to prefer local symbols

2013-10-04 Thread Eric Anholt
Keith Packard kei...@keithp.com writes: Egbert Eich e...@freedesktop.org writes: If there are namespace clashes among different drivers it would be preferrable if each driver used its local symbols. Use the RTLD_DEEPBIND if available to achive this. Eric and Adam are doing a lot of work in

[PATCH] glx: Add support for the new DRI loader entrypoint.

2013-10-11 Thread Eric Anholt
This is going to be exposed (and not the old entrypoint) for some DRI drivers once the megadrivers series lands, and the plan is to eventually transition all drivers to that. Hopefully this is unobtrusive enough to merge to stable X servers so that they can be compatible with new Mesa versions.

Re: [PATCH] glx: Remove DRI1 AIGLX

2013-10-22 Thread Eric Anholt
there's no EGL implemented for DRI1 drivers, this would already doom AIGLX on DRI1 (short of entirely forking the GLX implementation, which I'm not enthusiastic about). I think you also want to drop the DRI1 checks for AIGLX and AIGLX_DRI_LOADER in configure.ac. If you do: Reviewed-by: Eric Anholt

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