Re: [PATCH xserver 4/4] meson: Detect strlcat/strlcpy/reallocarray in libbsd

2017-05-09 Thread Eric Anholt
Peter Harris writes: > If we're linking with libbsd anyway, we might as well use the functions > it provides instead of compiling our replacements. > > Signed-off-by: Peter Harris This series is: Reviewed-by: Eric Anholt Let me know if you would like me to push them. signature.asc Descript

Re: [PATCH app/xdpyinfo] Use XRANDR 1.2 extension for reporting dimensions and resolution per output

2017-05-09 Thread Pali Rohár
On Monday 08 May 2017 09:55:08 Eric Anholt wrote: > That would also not be accepted. You don't get to rewrite output like > this in old CLI tools that have had consistent behavior for decades. It > causes breakage for systems expecting old behavior, even when that > behavior is unfortunate and wr

[PATCH xserver 4/4] meson: Detect strlcat/strlcpy/reallocarray in libbsd

2017-05-09 Thread Peter Harris
If we're linking with libbsd anyway, we might as well use the functions it provides instead of compiling our replacements. Signed-off-by: Peter Harris --- include/meson.build | 6 +++--- include/os.h| 6 ++ 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/include/meson.

[PATCH xserver 2/4] meson: Detect more functions

2017-05-09 Thread Peter Harris
Set HAVE_REALLOCARRAY, HAVE_SIGACTION, HAVE_STRCASESTR, HAVE_STRLCAT, HAVE_STRLCPY, HAVE_TIMINGSAFE_MEMCMP, and BUSFAULT. Signed-off-by: Peter Harris --- include/meson.build | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/include/meson.build b/include/meson.build i

[PATCH xserver 1/4] meson: Detect arc4random_buf

2017-05-09 Thread Peter Harris
Signed-off-by: Peter Harris --- include/meson.build | 1 + 1 file changed, 1 insertion(+) diff --git a/include/meson.build b/include/meson.build index 761f158d7..d032fb37f 100644 --- a/include/meson.build +++ b/include/meson.build @@ -98,6 +98,7 @@ conf_data.set('HAVE_SYS_UTSNAME_H', cc.has_hea

[PATCH xserver 3/4] meson: Only detect each function once

2017-05-09 Thread Peter Harris
Use conf_data outside of include/ to avoid re-running detection of the same functions. Signed-off-by: Peter Harris --- os/meson.build | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/os/meson.build b/os/meson.build index 724e6d1a8..940c6f4d0 100644 --- a/os/

[PATCH xserver 0/4] A few meson fixes

2017-05-09 Thread Peter Harris
Mostly I was annoyed at the unused return value warning from the non-arc4random path. I noticed a few other minor things while I was in there. Peter Harris (4): meson: Detect arc4random_buf meson: Detect more functions meson: Only detect each function once meson: Detect str

Re: [PATCH xserver] xfree86: fix autoConfigDevice() regression that skipped all but the first driver match

2017-05-09 Thread Eric Anholt
Aaron Plattner writes: > On 05/09/2017 04:51 PM, Eric Anholt wrote: >> Aaron Plattner writes: >> >>> Commit 112d0d7d01b9 lost the initialization of the variable i in the loop >>> to add >>> secondary driver matches to the list of configs: >>> >>> @@ -398,8 +412,8 @@ autoConfigDevice(GDevPtr

Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs

2017-05-09 Thread Dave Airlie
> > Just please make sure that one (user configurable/opt-in if necessary) > policy from the beginning is to allow leasing out any output to > applications, not just HMDs. My type of scientific/medical applications > would benefit as soon as it has the option to get a drm lease for a given > output

Re: [PATCH xserver] xfree86: fix autoConfigDevice() regression that skipped all but the first driver match

2017-05-09 Thread Aaron Plattner
On 05/09/2017 04:51 PM, Eric Anholt wrote: Aaron Plattner writes: Commit 112d0d7d01b9 lost the initialization of the variable i in the loop to add secondary driver matches to the list of configs: @@ -398,8 +412,8 @@ autoConfigDevice(GDevPtr preconf_device) /* for each other

Re: [PATCH xserver] xfree86: fix autoConfigDevice() regression that skipped all but the first driver match

2017-05-09 Thread Eric Anholt
Aaron Plattner writes: > Commit 112d0d7d01b9 lost the initialization of the variable i in the loop to > add > secondary driver matches to the list of configs: > > @@ -398,8 +412,8 @@ autoConfigDevice(GDevPtr preconf_device) > > /* for each other driver found, copy the first screen

Re: [PATCH xserver] xfree86: fix autoConfigDevice() regression that skipped all but the first driver match

2017-05-09 Thread Peter Hutterer
On Tue, May 09, 2017 at 03:08:33PM -0700, Aaron Plattner wrote: > Commit 112d0d7d01b9 lost the initialization of the variable i in the loop to > add > secondary driver matches to the list of configs: > > @@ -398,8 +412,8 @@ autoConfigDevice(GDevPtr preconf_device) > > /* for each

[PATCH xserver] xfree86: fix autoConfigDevice() regression that skipped all but the first driver match

2017-05-09 Thread Aaron Plattner
Commit 112d0d7d01b9 lost the initialization of the variable i in the loop to add secondary driver matches to the list of configs: @@ -398,8 +412,8 @@ autoConfigDevice(GDevPtr preconf_device) /* for each other driver found, copy the first screen, insert it * into the

Re: [PATCH xserver 4/7] meson: Make driprotos and libdrm optional

2017-05-09 Thread Eric Anholt
Jon Turney writes: > On 08/05/2017 21:28, Jon Turney wrote: >> On 08/05/2017 18:12, Eric Anholt wrote: >>> Jon Turney writes: +libxserver_dri3 = '' >>> >>> Pretty sure this should be = [] >> >> Yeah, me too. But when I try it: >> >>> Meson encountered an error in file hw/kdrive/ephyr/meson.

Re: [PATCH] Improved autoconfig drivers matching

2017-05-09 Thread Eric Anholt
Adam Jackson writes: > On Mon, 2017-05-01 at 18:03 +0900, Michel Dänzer wrote: >> On 28/04/17 04:11 AM, Adam Jackson wrote: >> > Fixed up (and rebased and made meson-aware) and merged: >> > >> > To ssh://git.freedesktop.org/git/xorg/xserver >> >    1549e3037..112d0d7d0  master -> master >> >> T

Re: [PATCH] Improved autoconfig drivers matching

2017-05-09 Thread Aaron Plattner
On 05/03/2017 01:11 PM, Adam Jackson wrote: > On Mon, 2017-05-01 at 18:03 +0900, Michel Dänzer wrote: >> On 28/04/17 04:11 AM, Adam Jackson wrote: >>> Fixed up (and rebased and made meson-aware) and merged: >>> >>> To ssh://git.freedesktop.org/git/xorg/xserver >>>1549e3037..112d0d7d0 master ->

Re: [RFC PATCH xserver] xfree86: build a shared lib identical to Xorg

2017-05-09 Thread Eric Anholt
Peter Hutterer writes: > Because we can use a shared lib in dlsym() for the symbol loader test, but we > can't use the executable. > > Signed-off-by: Peter Hutterer Apparently I've had the test working just because my default amd64 build is PIE, such that the Xorg binary is effectively just a s

Re: [PATCH xserver 5/7] meson: Make XV optional

2017-05-09 Thread Olivier Fourdan
Hey, On 8 May 2017 at 19:13, Eric Anholt wrote: > > Optional XV seems a bit silly to me. Could we just do something > like: > > build_xv = build_xorg || build_dmx || build_ephyr? Does Xwayland fall in one of those 3 categories? If not, you'd need to add Xwayland to that list as it uses Xv wi

Re: [PATCH xserver 5/7] meson: Make XV optional

2017-05-09 Thread Jon Turney
On 08/05/2017 18:13, Eric Anholt wrote: Jon Turney writes: Signed-off-by: Jon Turney --- meson.build | 11 ++- meson_options.txt | 3 +++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/meson.build b/meson.build index c7c9c79c3..5f30b996c 100644 --- a/meson.buil

Re: [PATCH xserver 4/7] meson: Make driprotos and libdrm optional

2017-05-09 Thread Jon Turney
On 08/05/2017 21:28, Jon Turney wrote: On 08/05/2017 18:12, Eric Anholt wrote: Jon Turney writes: +libxserver_dri3 = '' Pretty sure this should be = [] Yeah, me too. But when I try it: Meson encountered an error in file hw/kdrive/ephyr/meson.build, line 40, column 0: Link target [] is no

Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs

2017-05-09 Thread Pekka Paalanen
On Mon, 08 May 2017 08:29:30 -0700 Keith Packard wrote: > Pekka Paalanen writes: > > > Thinking again, I believe we have to have a way to override > > database entries somehow. If we ship catch-all entries for, say, > > all Sony TVs, we are bound to get some assignment wrong for > > someone who