Re: Fwd: Re: [PATCH 0/2] Add New XWayland API

2013-10-19 Thread Axel Davy
I suggest putting the XWayland API in a separate library. There's still many things to fix (handling of the cursor for example), and if we keep XWayland outside of the Xserver, I believe it's going to be easier to update the API, and eventually break it when it is needed. I've looked closer at

Re: Fwd: Re: [PATCH 0/2] Add New XWayland API

2013-10-19 Thread Chris Wilson
On Sat, Oct 19, 2013 at 01:55:42PM +0200, Axel Davy wrote: > > On Sat, Oct 19, 2013 at 10:49:04AM +0200, Axel Davy wrote: > >> I've tried benchmarking AsyncSwap with the phoronix-test-suite, > >> and I was surprised to see a regression with Openarena and Xonotic. > >> According to dri devs, it is b

Re: Fwd: Re: [PATCH 0/2] Add New XWayland API

2013-10-19 Thread Axel Davy
> On Sat, Oct 19, 2013 at 10:49:04AM +0200, Axel Davy wrote: >> I've tried benchmarking AsyncSwap with the phoronix-test-suite, >> and I was surprised to see a regression with Openarena and Xonotic. >> According to dri devs, it is because, since I do an exchange, the >> application >> is fullscreen

Re: Fwd: Re: [PATCH 0/2] Add New XWayland API

2013-10-19 Thread Chris Wilson
On Sat, Oct 19, 2013 at 10:49:04AM +0200, Axel Davy wrote: > I've tried benchmarking AsyncSwap with the phoronix-test-suite, > and I was surprised to see a regression with Openarena and Xonotic. > According to dri devs, it is because, since I do an exchange, the > application > is fullscreen and th

Fwd: Re: [PATCH 0/2] Add New XWayland API

2013-10-19 Thread Axel Davy
Given the recent discussion on the xorg-devel mailing list, I think this new API shouldn't be merged within the X server. The clear goal of the xorg team is toward dri3, and I was told no patch enabling AsyncSwap for dri2 would be merged. To implement the Present extension and composite efficie

[PATCH 0/2] Add New XWayland API

2013-10-16 Thread Axel Davy
The current XWayland API has no functionality to help the DDXs implement ScheduleSwap. This new API proposal introduces functions to manipulate the frame event, the release event, and to manipulate the buffers that the Wayland compositor sees. The first patch is not linked directly to XWayland,