Re: [PATCH v4 xserver 0/4] modesetting: add DRI2 page flip support

2016-09-05 Thread Michel Dänzer
On 06/09/16 01:08 PM, Kenneth Graunke wrote: > > For that matter, why not delete DRI2 support entirely at this point? Some things (such as the VA-API backend for Intel GPUs IIRC, or the amdgpu-pro OpenGL driver in current releases) still only support DRI2. -- Earthling Michel Dänzer

Re: [PATCH v4 xserver 0/4] modesetting: add DRI2 page flip support

2016-09-05 Thread Kenneth Graunke
On Monday, August 22, 2016 6:14:14 PM PDT Qiang Yu wrote: > Hi guys, > > This patch set is for adding DRI2 page flip support to modesetting > driver. I mainly take reference of amdgpu DDX and reuse present > page flip code in the modesetting driver. > > V2: > 1. fix DRI2 page flip success handler

Re: Proposed X server 1.19 schedule

2016-09-05 Thread Peter Hutterer
On Sun, Sep 04, 2016 at 11:18:24AM +0200, Hans de Goede wrote: > Hi, > > On 02-09-16 07:33, Keith Packard wrote: > > > > Following the 'release early and often' mantra, we should probably get > > 1.19 out the door and let people start using the new code. > > > > I'd like to propose that we figur

Re: Proposed X server 1.19 schedule

2016-09-05 Thread Keith Packard
Hans de Goede writes: > That is quick :) Fine by me, but I've a whole bunch of > modesetting / prime patches (all bugfixes really) I would like to get in > before then. As you can see, bug fixes are welcome until 9-24. All that closes 9-10 is new feature development. If anyone has some feature

[PATCH v2 xserver] modesetting: fix compile error when --disable-glamor

2016-09-05 Thread Qiang Yu
Move ms_flush_drm_events out of GLAMOR ifdef. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97586 Signed-off-by: Qiang Yu Reviewed-by: Michel Dänzer --- hw/xfree86/drivers/modesetting/driver.h | 4 +- hw/xfree86/drivers/modesetting/pageflip.c | 68 +++ 2

Re: [PATCH xserver] cursor: add hw cursor support for prime

2016-09-05 Thread Keith Packard
Hans de Goede writes: > +if (!dixPrivatesCreated(PRIVATE_CURSOR)) > +dixRegisterScreenPrivateKey(&cursorScreenDevPriv, pScreen, > +PRIVATE_CURSOR, 0); > + I don't understand what you're trying to do here; you're only registering the screen private

Re: [PATCH xserver] modesetting: fix compile error when --disable-glamor

2016-09-05 Thread Michel Dänzer
On 05/09/16 07:05 PM, Qiang Yu wrote: > Move ms_flush_drm_events out of GLAMOR ifdef. > Fix Bug 97586. The second line should be written like this: Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97586 (with an empty line above it, so it's grouped with other tags such as Signed-off-by.

Re: [PATCH xserver] cursor: add hw cursor support for prime

2016-09-05 Thread Michel Dänzer
On 05/09/16 10:42 PM, Hans de Goede wrote: > Hi, > > Subject should have contained v4, resending so that we've > the right version in patchwork. Sorry. Hmm, for some reason I'm not receiving your recent patches from the mailing list, though I can see them in the list archives and in patchwork.

Re: [PATCH xserver 4/4] xfree86: promote one GPU screen if (NumScreens == 0 && NumGPUScreens > 0)

2016-09-05 Thread Gerd Hoffmann
Hi, > > Why is this otherwise correct, generally speaking ? AFAIK virtio-gpu-pci > > is needed for Virgil, so I would expect it to become the default graphical > > card in vms in the future, or is that what virtio-vga is for ? > > Yes. Both virtio-vga and virtio-gpu-pci share the VirtIO GPU Dev

Re: [PATCH xserver 4/4] xfree86: promote one GPU screen if (NumScreens == 0 && NumGPUScreens > 0)

2016-09-05 Thread Dave Airlie
On 5 September 2016 at 22:39, Laszlo Ersek wrote: > On 09/04/16 11:01, Hans de Goede wrote: >> HI, >> >> On 04-09-16 03:11, Laszlo Ersek wrote: >>> Aarch64/KVM virtual machines cannot use emulated graphics cards with >>> linear framebuffers, due to architectural cache coherency issues. (For >>> th

Re: [PATCH xserver 4/4] xfree86: promote one GPU screen if (NumScreens == 0 && NumGPUScreens > 0)

2016-09-05 Thread Marcin Juszkiewicz
W dniu 05.09.2016 o 14:39, Laszlo Ersek pisze: > I didn't want to mess with the definition of "primary graphics card", > because that seems to have ties to platforms and graphics cards that I > have no clue about. Instead, picking an unspecified secondary graphics > card automatically, when there

Re: [PATCH xserver] cursor: add hw cursor support for prime

2016-09-05 Thread Hans de Goede
Hi, Subject should have contained v4, resending so that we've the right version in patchwork. Sorry. Regards, Hans On 05-09-16 15:37, Hans de Goede wrote: From: Dave Airlie Currently with PRIME if we detect a secondary GPU, we switch to using SW cursors, this isn't optimal, esp for the int

[PATCH v4 xserver] cursor: add hw cursor support for prime

2016-09-05 Thread Hans de Goede
From: Dave Airlie Currently with PRIME if we detect a secondary GPU, we switch to using SW cursors, this isn't optimal, esp for the intel/nvidia combinations, we have no choice for the USB offload devices. This patch checks on each slave screen if hw cursors are enabled, and also calls set curso

[PATCH xserver] glamor: Fix crash when master gpu is using glamor and another gpu is hotplugged

2016-09-05 Thread Hans de Goede
When a GPU gets hotplugged while X is already running, glamor_egl_init() gets called and changes the current egl context at a point where glamor does not expect this. This causes glamor to e.g. crash in the next glamor_create_pixmap() call (caled through the master's screen->CreatePixmap), note th

[PATCH xserver] cursor: add hw cursor support for prime

2016-09-05 Thread Hans de Goede
From: Dave Airlie Currently with PRIME if we detect a secondary GPU, we switch to using SW cursors, this isn't optimal, esp for the intel/nvidia combinations, we have no choice for the USB offload devices. This patch checks on each slave screen if hw cursors are enabled, and also calls set curso

Re: [PATCH xserver 4/4] xfree86: promote one GPU screen if (NumScreens == 0 && NumGPUScreens > 0)

2016-09-05 Thread Laszlo Ersek
On 09/04/16 11:01, Hans de Goede wrote: > HI, > > On 04-09-16 03:11, Laszlo Ersek wrote: >> Aarch64/KVM virtual machines cannot use emulated graphics cards with >> linear framebuffers, due to architectural cache coherency issues. (For >> this reason, "qemu-system-aarch64" doesn't even include the

Missing Xwayland patch for X server 1.19

2016-09-05 Thread Olivier Fourdan
Hi Peter, Adam, Do you think we could land Rui's patch for Xwayland for 1.19?: https://patchwork.freedesktop.org/patch/95244/ It's been reviewed by Daniel and tested by myself (and others) for some time. If needed, I can prepare a pull request if that's more convenient for you. Thanks Olivier

[PATCH xserver] modesetting: fix compile error when --disable-glamor

2016-09-05 Thread Qiang Yu
Move ms_flush_drm_events out of GLAMOR ifdef. Fix Bug 97586. Signed-off-by: Qiang Yu --- hw/xfree86/drivers/modesetting/driver.h | 4 +- hw/xfree86/drivers/modesetting/pageflip.c | 68 +++ 2 files changed, 36 insertions(+), 36 deletions(-) diff --git a/hw/xfree86/