Re: [PATCH xserver] glamor: Cannot use copies when accessing outside of composite source

2016-05-27 Thread Keith Packard
Michel Dänzer writes: > You know the details better than I do, but my understanding is that the > region can only be clipped to the destination in general, because > accessing source pictures outside of their boundaries is defined such > that it can change the contents of the

[PATCH xf86-video-ati resend 1/2] EXA/6xx/7xx: fast solid pixmap support

2016-05-27 Thread Tan Hu
Solid pixmaps are currently implemented with scratch pixmaps, which is slow. This replaces the hack with a proper implementation. The Composite shader can now either sample a src/mask or use a constant value. r6xx still be used on some machine, Ported from commit

Re: Re: [PATCH xf86-video-ati 1/2] EXA/6xx/7xx: fast solid pixmap support

2016-05-27 Thread tan . hu
Michel Dänzer wrote on 2016-05-24 16:52:23: > > Hi Tan, > > > unfortunately, your mailer seems to have mangled your patches, so they > can't be applied. Can you re-send the patches with git send-email, or > maybe provide access to a Git tree containing the changes? > I

[PATCH xf86-video-ati resend 2/2] EXA/6xx/7xx: accelerate PictOpOver with component alpha

2016-05-27 Thread Tan Hu
Subpixel text rendering is typically done with a solid src and a pixmap mask. Traditionally, this cannot be accelerated in a single pass and requires two passes [1]. However, we can cheat a little with a constant blend color. We can use: const.A = src.A / src.A const.R = src.R / src.A const.G =

Re: [PATCH xserver] glamor: Cannot use copies when accessing outside of composite source

2016-05-27 Thread Michel Dänzer
On 28.05.2016 05:29, Keith Packard wrote: > Michel Dänzer writes: >> From: Michel Dänzer >> >> Commit b64108fa ("glamor: Check for composite operations which are >> equivalent to copies") failed to copy conditions from exaComposite which >> ensure that

Re: [PATCH xserver] glamor: Cannot use copies when accessing outside of composite source

2016-05-27 Thread Michel Dänzer
On 26.05.2016 21:55, Hans de Goede wrote: > On 26-05-16 12:04, Michel Dänzer wrote: >> From: Michel Dänzer >> >> Commit b64108fa ("glamor: Check for composite operations which are >> equivalent to copies") failed to copy conditions from exaComposite which >> ensure that

Re: [PATCH xserver 17/23] os: Add ospoll interface

2016-05-27 Thread Keith Packard
Emil Velikov writes: > Hi Keith, > > Style question: do you/how many others refer having separate functions for > of ifdeffed code each vs a single with all the ifdeffs. Or in other words > having func_foo_poll and func_foo_epoll as opposed to having it all in >

[PATCH xserver] xfree86: Remove event reading code from xf86Wakeup

2016-05-27 Thread Keith Packard
Oops. This didn't get removed when xfree86 was converted over to use the input thread. Signed-off-by: Keith Packard --- hw/xfree86/common/xf86Events.c | 28 1 file changed, 28 deletions(-) diff --git a/hw/xfree86/common/xf86Events.c

[PATCH xserver] dix: Don't update current time in the middle of input event processing

2016-05-27 Thread Keith Packard
In patch 137ac094e7ab8c871f3b36e40ad826ac797f0e26, Adam moved an expensive call to UpdateCurrentTime out of the main dispatch loop. That's a good change as the original fix from Chase was a bit expensive. However, it breaks grab processing and so a couple of the calls to UpdateCurrenTime need to

Re: Remote OpenGL

2016-05-27 Thread Cook, Rich
> On May 27, 2016, at 4:54 PM, Jeremy Huddleston Sequoia > wrote: > > >> On May 27, 2016, at 09:27, Cook, Rich wrote: >> >> >>> On May 27, 2016, at 1:53 AM, Jeremy Huddleston Sequoia >>> wrote: >>> Another awkward

Re: Remote OpenGL

2016-05-27 Thread Jeremy Huddleston Sequoia
> On May 27, 2016, at 09:27, Cook, Rich wrote: > > >> On May 27, 2016, at 1:53 AM, Jeremy Huddleston Sequoia >> wrote: >> >>> Another awkward thing about this, I suppose, is that on OSX the X server >>> is started implicitly on demand and there's

Re: Remote OpenGL

2016-05-27 Thread Cook, Rich
Thanks for the pointer to the actual change. The commit message there also says, wrongly, "Almost every situation of someone running indirect GLX is a mistake that results in X Server crashes. Indirect GLX is the cause of regular security vulnerabilities, and rarely provides any capability

Re: [PATCH xserver 17/23] os: Add ospoll interface

2016-05-27 Thread Emil Velikov
Hi Keith, Style question: do you/how many others refer having separate functions for of ifdeffed code each vs a single with all the ifdeffs. Or in other words having func_foo_poll and func_foo_epoll as opposed to having it all in func_foo. On Friday, 27 May 2016, Keith Packard

Re: [PATCH xserver] glamor: Adjust for drawable x/y in composite's copy optimization

2016-05-27 Thread Keith Packard
Alex Deucher writes: >> Signed-off-by: Keith Packard > > Reviewed-by: Alex Deucher Merged. 0d16a0c..8b9b438 master -> master -- -keith signature.asc Description: PGP signature

Re: [PATCH xserver] glamor: Cannot use copies when accessing outside of composite source

2016-05-27 Thread Keith Packard
Michel Dänzer writes: > From: Michel Dänzer > > Commit b64108fa ("glamor: Check for composite operations which are > equivalent to copies") failed to copy conditions from exaComposite which > ensure that the composite operation doesn't access outside

Re: Remote OpenGL

2016-05-27 Thread Alan Coopersmith
On 05/27/16 11:20 AM, Cook, Rich wrote: I'm glad to hear that one of the core X server maintainer is saying that it is not going "completely" away. The previous statement on IGLX when it was disabled was:

Re: [PATCH xserver] glamor: Adjust for drawable x/y in composite's copy optimization

2016-05-27 Thread Alex Deucher
On Fri, May 27, 2016 at 2:05 PM, Keith Packard wrote: > Patch b64108fa305e956e4edaae9d53071ff0abee268e added a short cut that > identifies composite operations that can be performed with a simple > copy instead. > > glamor_copy works in absolute coordinates, so the dx and dy

Re: [PATCH v5 00/13] PRIME Synchronization

2016-05-27 Thread Alex Goins
Got it. Thanks! Alex On Fri, 27 May 2016, Dave Airlie wrote: > On 20 May 2016 at 08:36, Alex Goins wrote: > > Hi Dave, > > > > Any update on this? Anything I can do to help? > > Hey, > > can you take a look at > > prime: clean up slave bo properly. (v3) > > I think with

Re: [PATCH xserver 2/2] xfree86/modes: Don't set xf86_config->cursor if loading the cursor fails

2016-05-27 Thread Keith Packard
Michel Dänzer writes: > From: Michel Dänzer > > If xf86_load_cursor_argb returns FALSE, the caller is expected to fall > back to SW cursor, so xf86_config->cursor shouldn't be updated. It doesn't matter -- xf86_config->cursor is supposed to be a

Re: [PATCH xserver] prime: clean up slave bo properly. (v3)

2016-05-27 Thread Alex Goins
Reviewed-by: Alex Goins Thanks, Alex On Fri, 6 May 2016, Dave Airlie wrote: > This is an ABI break, in that we now pass NULL to a function > that hasn't accepted it before. > > Alex Goins had a different patch for this but it wasn't symmetrical, > it freed something in a very different place

Re: [PATCH xserver 1/2] xfree86/modes: Assign xf86_config->cursor in xf86_load_cursor_image

2016-05-27 Thread Keith Packard
Michel Dänzer writes: > From: Michel Dänzer > > Fixes a crash on startup in the radeon driver's drmmode_show_cursor() > due to xf86_config->cursor == NULL, because no CRTC was enabled yet, so > xf86_crtc_load_cursor_image was never called. Good

Re: Remote OpenGL

2016-05-27 Thread Cook, Rich
I'm glad to hear that one of the core X server maintainer is saying that it is not going "completely" away. Also that he is adding an xorg.conf option to enable IGLX. Great news! We can rest easier. I could not for the life of me understand how this could be removed, but that was the

Re: Remote OpenGL

2016-05-27 Thread Alan Coopersmith
You do know that Adam is one of the core X server maintainers, right? If he says you're wrong, then I believe him and you should too. (Though the mail he points to says you're only half wrong - yes it is disabled by default today, but no, we've not announced any plans to make it completely go

[PATCH xserver] glamor: Adjust for drawable x/y in composite's copy optimization

2016-05-27 Thread Keith Packard
Patch b64108fa305e956e4edaae9d53071ff0abee268e added a short cut that identifies composite operations that can be performed with a simple copy instead. glamor_copy works in absolute coordinates, so the dx and dy values passed in need to be converted from drawable-relative to absolute by adding

Re: [PATCH xserver] os: Increase default client buffer to 16kB

2016-05-27 Thread Keith Packard
Hans de Goede writes: >> Signed-off-by: Keith Packard > > Ack. > > Reviewed-by: Hans de Goede Pushed. 7147361..0d16a0c master -> master -- -keith signature.asc Description: PGP signature

Re: [PATCH xserver 1/9] xfree86: Set xf86CrtcConfigRec cursor pointer to NULL in HideCursor

2016-05-27 Thread Keith Packard
Michel Dänzer writes: > [ Unknown signature status ] > > The patch is > > Reviewed-by: Michel Dänzer Thanks. Pushed. e69061e..7147361 master -> master -- -keith signature.asc Description: PGP signature

Re: Remote OpenGL

2016-05-27 Thread Alan Coopersmith
On 05/26/16 06:56 PM, Christopher Barry wrote: I have certain software that must use opengl, and it runs on windows. I had to stand up a separate box just for that application, as I run Linux everywhere else. I could not make it work in a KVM VM due to no opengl capability in KVM (I won't use

Re: Remote OpenGL

2016-05-27 Thread Cook, Rich
> On May 27, 2016, at 1:53 AM, Jeremy Huddleston Sequoia > wrote: > >> Another awkward thing about this, I suppose, is that on OSX the X server >> is started implicitly on demand and there's no reasonable way to configure >> things like command line options. > > Most

GLX

2016-05-27 Thread John Allison
Hi I need GLX in my work - radiation modelling in particle physics, nuclear physics, medicine (radiation therapy) and spacecraft design. Radiation modelling takes lots of remote CPU and we display results locally via X11 with GLX using relatively low bandwidth. Using Quartz on Mac. John

Re: Can I change the X rotate the xf86-video-fbdev during execution ?

2016-05-27 Thread Alex Deucher
Runtime rotation using randr only works with randr 1.2+ capable drivers. I don't think the fbdev supports that. On supported drivers: xrandr --output --rotate right|left|inverted|normal Alex On Thu, May 26, 2016 at 1:46 AM, 권태영 wrote: > > > Good morning. > > > > I

[PATCH xserver v2] xrandrprovider: Do not use separate lists for unbound / source / offload slaves

2016-05-27 Thread Hans de Goede
A single provider can be both a offload and source slave at the same time, the use of seperate lists breaks in this case e.g. : xrandr --listproviders Providers: number : 2 Provider 0: id: 0x7b cap: 0xf, Source Output, Sink Output, Source Offload, Sink Offload crtcs: 3 outputs: 2 associated

Re: Remote OpenGL

2016-05-27 Thread Jeremy Huddleston Sequoia
> On May 26, 2016, at 15:56, Adam Jackson wrote: > > On Thu, 2016-05-26 at 09:14 +0200, Laurent Lamalle wrote: >> Hello, >> >> I have just discovered, through the x11-us...@lists.apple.com mailing >> list, that Xorg deprecated some software component (IGLX?) necessary for >>

Re: xserver: Branch 'master' - 10 commits

2016-05-27 Thread Keith Packard
Michel Dänzer writes: > On 27.05.2016 08:08, Keith Packard wrote: >> >> commit f84703b50cc908a127f4ad923ebbf56f8f244c0d >> Author: Keith Packard >> Date: Tue Dec 8 14:20:21 2015 -0800 >> >> dix: Reallocate touchpoint buffer at input event time [v2]

Re: Black window with Gnome3 (xf86-video-ati 1:7.6.1-1)

2016-05-27 Thread Michel Dänzer
On 27.05.2016 04:49, Oscar Megia wrote: > > I updated the driver to xf86-video-ati 1:7.7.0-1 and I still have the > same issue. > > Could you try to solve this issue, please? FWIW, the black text in LibreOffice should be fixed in current xserver Git master. As for the black windows, I'm not

Re: Black window with Gnome3 (xf86-video-ati 1:7.6.1-1)

2016-05-27 Thread Oscar Megia
Hi I updated the driver to xf86-video-ati 1:7.7.0-1 and I still have the same issue. Could you try to solve this issue, please? Regards Oscar 2016-01-30 15:18 GMT+01:00 Oscar Megia : > Hi > > Sorry for my last email. I though that the issue was solved, but this week > I

Re: xserver: Branch 'master' - 10 commits

2016-05-27 Thread Michel Dänzer
On 27.05.2016 08:08, Keith Packard wrote: > > commit f84703b50cc908a127f4ad923ebbf56f8f244c0d > Author: Keith Packard > Date: Tue Dec 8 14:20:21 2015 -0800 > > dix: Reallocate touchpoint buffer at input event time [v2] > > Now that input is threaded, malloc can

Re: [PATCH xserver 1/9] xfree86: Set xf86CrtcConfigRec cursor pointer to NULL in HideCursor

2016-05-27 Thread Michel Dänzer
On 12.05.2016 05:54, Keith Packard wrote: > This makes the cursor pointer held by xf86Cursors.c get reset to NULL > whenever the cursor isn't displayed, and means that the reference > count held in xf86Cursor.c is sufficient to cover the reference in > xf86Cursors.c. > > As HideCursor may be