Re: A question about XCompositeRedirectSubwindows

2018-06-20 Thread Jasper St. Pierre
On Wed, Jun 20, 2018 at 8:19 AM Adam Jackson wrote: > On Tue, 2018-06-19 at 17:35 -0700, Jasper St. Pierre wrote: > > > First off, there's a super high latency cost here. > > There is a latency cost, but I think it's worth being honest about it. > I'm on whatever iteration of the Dell XPS laptop

Re: [Linux-graphics-maintainer] [PATCH] glamor: Work around GEM usage v2

2018-06-20 Thread Lyude Paul
Added to the modesetting pull request: https://gitlab.freedesktop.org/lyudess/xserver/commit/49a2dae2dc45b4a144ae69be47a90ce6f4ca27f8 Thanks! On Wed, 2018-06-20 at 20:18 +0200, Thomas Hellstrom wrote: > On 06/20/2018 07:58 PM, Deepak Singh Rawat wrote: > > The patch looks good to me just one

Re: [PATCH xserver v2 2/2] randr: Scream when creating a shared pixmap fails

2018-06-20 Thread Dave Airlie
On 21 June 2018 at 09:12, Lyude Paul wrote: > This seems like a problem worth screaming about. > > Signed-off-by: Lyude Paul For the series: Reviewed-by: Dave Airlie > --- > randr/rrcrtc.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/randr/rrcrtc.c b/randr/rrcrtc.c > index

Re: Gathering EGLStream related changed for Xwayland for a pull request

2018-06-20 Thread Lyude Paul
On Mon, 2018-06-18 at 14:50 -0400, Lyude Paul wrote: > Hey guys! So, talking to Ajax he said that something which would probably > help > out with getting a bugfix release out there for X is if people started > getting > their changes together into pull requests so that he doesn't need to go >

[PATCH xserver v2 2/2] randr: Scream when creating a shared pixmap fails

2018-06-20 Thread Lyude Paul
This seems like a problem worth screaming about. Signed-off-by: Lyude Paul --- randr/rrcrtc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/randr/rrcrtc.c b/randr/rrcrtc.c index d7d937e80..74dc5a265 100644 --- a/randr/rrcrtc.c +++ b/randr/rrcrtc.c @@ -534,6 +534,7 @@

[PATCH xserver v2 1/2] glamor: Unbreak glamor_fd_from_pixmap()

2018-06-20 Thread Lyude Paul
When support for allocating GBM BOs with modifiers was added, glamor_fd_from_pixmap() was changed so that it would return an error if it got a bo with modifiers set from glamor_fds_from_pixmap(). The problem is that on systems that support BOs with modifiers, glamor_fds_from_pixmap() will always

[PATCH xserver 1/2] glamor: Unbreak glamor_fd_from_pixmap()

2018-06-20 Thread Lyude Paul
When support for allocating GBM BOs with modifiers was added, glamor_fd_from_pixmap() was changed so that it would return an error if it got a bo with modifiers set from glamor_fds_from_pixmap(). The problem is that on systems that support BOs with modifiers, glamor_fds_from_pixmap() will always

[PATCH xserver 2/2] randr: Scream when creating a shared pixmap fails

2018-06-20 Thread Lyude Paul
This seems like a problem worth screaming about. Signed-off-by: Lyude Paul --- randr/rrcrtc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/randr/rrcrtc.c b/randr/rrcrtc.c index d7d937e80..74dc5a265 100644 --- a/randr/rrcrtc.c +++ b/randr/rrcrtc.c @@ -534,6 +534,7 @@

Re: [Linux-graphics-maintainer] [PATCH] glamor: Work around GEM usage v2

2018-06-20 Thread Thomas Hellstrom
On 06/20/2018 07:58 PM, Deepak Singh Rawat wrote: The patch looks good to me just one question, I see that flink name is returned as fd, so handle is what vmwgfx expecting in that case ? Yes. vmwgfx doesn't differentiate between handles and names. /Thomas Reviewed-by: Deepak Rawat KMS

Re: Grabbing modesetting ddx related patches for pull request

2018-06-20 Thread Thomas Hellstrom
On 06/20/2018 06:53 PM, Lyude Paul wrote: On Wed, 2018-06-20 at 16:46 +0200, Thomas Hellstrom wrote: On 06/18/2018 10:48 PM, Lyude Paul wrote: To help ajax out with getting a bug release out for Xorg, we figured it would be a good idea for me to go through the stuff I needed to get upstream

[PATCH] glamor: Work around GEM usage v2

2018-06-20 Thread Thomas Hellstrom
KMS drivers are not required to support GEM. In particular, vmwgfx doesn't support flink and handles and names are identical. Getting a bo name should really be part of a lower level API, if needed, but in the mean time work around this by setting the name identical to the handle if GEM isn't

Re: Grabbing modesetting ddx related patches for pull request

2018-06-20 Thread Lyude Paul
On Wed, 2018-06-20 at 16:46 +0200, Thomas Hellstrom wrote: > On 06/18/2018 10:48 PM, Lyude Paul wrote: > > To help ajax out with getting a bug release out for Xorg, we figured it > > would > > be a good idea for me to go through the stuff I needed to get upstream and > > file pull requests for all

Re: Grabbing modesetting ddx related patches for pull request

2018-06-20 Thread Lyude Paul
On Wed, 2018-06-20 at 16:46 +0200, Thomas Hellstrom wrote: > On 06/18/2018 10:48 PM, Lyude Paul wrote: > > To help ajax out with getting a bug release out for Xorg, we figured it > > would > > be a good idea for me to go through the stuff I needed to get upstream and > > file pull requests for all

Re: A question about XCompositeRedirectSubwindows

2018-06-20 Thread Adam Jackson
On Tue, 2018-06-19 at 17:35 -0700, Jasper St. Pierre wrote: > First off, there's a super high latency cost here. There is a latency cost, but I think it's worth being honest about it. I'm on whatever iteration of the Dell XPS laptop was a Skylake, and: 200 trep @ 0.0131 msec (

Re: Grabbing modesetting ddx related patches for pull request

2018-06-20 Thread Thomas Hellstrom
On 06/18/2018 10:48 PM, Lyude Paul wrote: To help ajax out with getting a bug release out for Xorg, we figured it would be a good idea for me to go through the stuff I needed to get upstream and file pull requests for all of it. This is pretty much the same thing as what I'm doing for EGLStreams

Re: [PATCH xserver] : bug 106963 : change the DPMS initialization to be conditional on not set from config

2018-06-20 Thread Emil Velikov
Hi John, Please don't drop the ML when replying. On 19 June 2018 at 20:32, John Lumby wrote: > I've tried to conform to those guidelines and resent - > however I cannot use git send-email from this email account because I would > have to send it from my dev system and the mailing-list server

Re: [PATCH] release: fix mesa url detection after migration to gitlab

2018-06-20 Thread Emil Velikov
On 19 June 2018 at 16:37, Dylan Baker wrote: > Quoting Dylan Baker (2018-06-15 14:12:55) >> Mesa's migration to gitlab has changed the URL and made things not work. >> --- >> release.sh | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/release.sh b/release.sh >> index

Re: A question about XCompositeRedirectSubwindows

2018-06-20 Thread Egil Möller
Thank you for helping me figure this out, and I should also add thanks for that article series you created and linked! It's really good, and the interactive examples are really cool. Must have taken forever to build them (and the abstractions on top of the DOM)! Impressive! It's nice to see