Re: [PATCH xserver 00/10] Implement RandR 1.6 (leases and non-desktop)

2018-01-16 Thread Keith Packard 
Adam Jackson  writes:

>> Changes since the last series:
>> 
>>  * Merged non-desktop and lease support into a single patch
>>sequence. They have a dependency on reporting output connection
>>status, so it's something of a pain to keep them separate.
>> 
>>  * Added RandR lease tests
>
> Is there userspace for this? Patched version of xrandr(1) perhaps?

I've got xcb patches for the protocol bits, and mesa patches out for
review that use them. I'm not sure how you'd expect xrandr to take
advantage of leasing though? As for non-desktop support, xrandr happily
ignores non-desktop monitors, so that at least works?

I think one useful user-space thing I could do that wasn't mesa would be
to create an X server that used a leased output. I'm not sure how I'd
get input devices though.

I'm happy to create more use cases if you can think of some that would
be helpful.

-- 
-keith


signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH xserver 00/10] Implement RandR 1.6 (leases and non-desktop)

2018-01-16 Thread Adam Jackson 
On Thu, 2017-12-21 at 18:54 -0800, Keith Packard wrote:
> Here's the latest version of the DRM lease and non-desktop output code
> for the X server. This time, I've gone ahead and implemented a pile of
> tests for leases.
> 
> Changes since the last series:
> 
>  * Merged non-desktop and lease support into a single patch
>sequence. They have a dependency on reporting output connection
>status, so it's something of a pain to keep them separate.
> 
>  * Added RandR lease tests

Is there userspace for this? Patched version of xrandr(1) perhaps?

- ajax
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH xserver] modesetting: Use seq instead of msc in ms_queue_vblank failure path

2018-01-16 Thread Adam Jackson 
On Tue, 2018-01-16 at 12:48 -0800, Eric Anholt wrote:
> Keith Packard  writes:
> 
> > When the call to queue a vblank event fails, we need to clean up by
> > removing the user-space queue entry. That is indexed by the local
> > sequence number, not by the kernel vblank count. The call in this
> > case was just passing the wrong value.
> > 
> > Signed-off-by: Keith Packard 
> 
> Reviewed-by: Eric Anholt 

remote: I: patch #198077 updated using rev 
32b42627217917337553121191e9bc3b423b8d99.
remote: I: 1 patch(es) updated to state Accepted.
To ssh://git.freedesktop.org/git/xorg/xserver
   15d91df474..32b4262721  master -> master

- ajax
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH xserver] x86emu: Teach the debug code about varargs

2018-01-16 Thread Eric Anholt 
Adam Jackson  writes:

> With -Wformat-nonliteral and a debug build you'd get yelled at here:
>
> ../hw/xfree86/x86emu/x86emu/debug.h:188:9: warning: format not a string 
> literal, argument types not checked [-Wformat-nonliteral]
>
> To fix this, rewrite the printf code to actually use varargs and the
> appropriate format attribute. All callers of DECODE_PRINTF() pass a
> string with no % specifiers, so we pass that as the argument to
> printf("%s"). For DECODE_PRINTF2() we just pass the args through.
>
> Signed-off-by: Adam Jackson 

Reviewed-by: Eric Anholt 


signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH xserver 1/2] meson: Enable SUN-DES-1 auth code

2018-01-16 Thread Eric Anholt 
Adam Jackson  writes:

> Signed-off-by: Adam Jackson 

These two are:

Reviewed-by: Eric Anholt 


signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH xserver] modesetting: Use seq instead of msc in ms_queue_vblank failure path

2018-01-16 Thread Eric Anholt 
Keith Packard  writes:

> When the call to queue a vblank event fails, we need to clean up by
> removing the user-space queue entry. That is indexed by the local
> sequence number, not by the kernel vblank count. The call in this
> case was just passing the wrong value.
>
> Signed-off-by: Keith Packard 

Reviewed-by: Eric Anholt 


signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

[PATCH xserver] modesetting: Use seq instead of msc in ms_queue_vblank failure path

2018-01-16 Thread Keith Packard 
When the call to queue a vblank event fails, we need to clean up by
removing the user-space queue entry. That is indexed by the local
sequence number, not by the kernel vblank count. The call in this
case was just passing the wrong value.

Signed-off-by: Keith Packard 
---
 hw/xfree86/drivers/modesetting/vblank.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/xfree86/drivers/modesetting/vblank.c 
b/hw/xfree86/drivers/modesetting/vblank.c
index fe046e48b..3f6568997 100644
--- a/hw/xfree86/drivers/modesetting/vblank.c
+++ b/hw/xfree86/drivers/modesetting/vblank.c
@@ -229,7 +229,7 @@ ms_queue_vblank(xf86CrtcPtr crtc, ms_queue_flag flags,
 return TRUE;
 }
 if (errno != EBUSY) {
-ms_drm_abort_seq(scrn, msc);
+ms_drm_abort_seq(scrn, seq);
 return FALSE;
 }
 ms_flush_drm_events(screen);
-- 
2.15.1

___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH] xfree86: add default modes for 16:9 and 16:10

2018-01-16 Thread Martin Wilck 
On Tue, 2018-01-09 at 20:33 +0100, Martin Wilck wrote:
> Improve the user experience for users with wide screens by adding
> standard
> 16:9 and 16:10 modes to extramodes, as suggested previously
> (https://lists.x.org/archives/xorg-devel/2016-February/048866.html).
> Tested successfully on my laptop. Feedback welcome.
> 
> See also https://bugs.freedesktop.org/show_bug.cgi?id=37858.
> 
> Signed-off-by: Martin Wilck 
> ---
>  hw/xfree86/common/extramodes | 142
> +++
>  1 file changed, 142 insertions(+)

No opinions on this patch?

Regards
Martin

___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel