, let's just use that here,
instead.
v2: Fix alarm delta changes.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
Xext/sync.c | 302 +++-
Xext/syncsrv.h | 23 ++--
include/misc.h | 25
miext/sync/misync.c
This would be squashed into the other tests.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
test/sync/sync.c | 71 ++--
1 file changed, 69 insertions(+), 2 deletions(-)
diff --git a/test/sync/sync.c b/test/sync/sync.c
index c39774
Signed-off-by: Eric Anholt <e...@anholt.net>
---
test/scripts/build-travis-deps.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/scripts/build-travis-deps.sh
b/test/scripts/build-travis-deps.sh
index 213b51f6f161..b87c65d3e397 100755
--- a/test/scripts/build-
on Travis.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
.travis.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index 00d7a485a29f..edeed0d90972 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -7,7 +7,7 @@ before_install:
- if [[ "$TRA
Keith Packard <kei...@keithp.com> writes:
> [ Unknown signature status ]
> Eric Anholt <e...@anholt.net> writes:
>
>> I couldn't find any, and I was modifying the implementation, so I had
>> to write some. I would like the test to end with a "make sure there
Keith Packard <kei...@keithp.com> writes:
> [ Unknown signature status ]
> Eric Anholt <e...@anholt.net> writes:
>
>> The extension was using the name CARD64 to represent 64-bit values,
>> with a #define from CARD64 to XSyncValue, a struct with a pair of
>>
-comppixwin10 -repeat 1 -reps 1
on i965 by 3.21191% +/- 1.79977% (n=50).
v2: Make the jump to the exit land after scissor disable.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_render.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/
) to avoid overhead on desktop GL.
No performance difference on i965 with x11perf -rect1 -repeat 1 -reps
1 (n=50)
v2: Clamp rectangle bounds addition.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_rects.c | 26 ++
glamor/glamor_utils.
Like the previous fix to rectangles, this reduces the area drawn on
tiled renderers by letting the CPU-side tile setup know what tiles
might be drawn at all.
Surprisingly, it improves x11perf -copypixwin1 -repeat 1 -reps 1
on i965 by 2.93185% +/- 1.5561% (n=90).
Signed-off-by: Eric Anholt &l
Daniel Stone writes:
> On 28 July 2017 at 16:51, Keith Packard wrote:
>> Michel Dänzer writes:
>>> Declaring where? Once a pixmap is created, it only has a depth, no
>>> format, so there's nothing to base on that e.g. CopyArea
The extension was using the name CARD64 to represent 64-bit values,
with a #define from CARD64 to XSyncValue, a struct with a pair of
32-bit values representing a signed 64-bit value. Now that stdint.h
exists, let's just use that instead.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
Signed-off-by: Eric Anholt <e...@anholt.net>
---
miext/sync/misyncstr.h | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/miext/sync/misyncstr.h b/miext/sync/misyncstr.h
index 084ca4c82d9d..a0a0a98146b4 100644
--- a/miext/sync/misyncstr.h
+++ b/miex
Autotools also had it as an option.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
hw/meson.build| 4 +++-
meson_options.txt | 2 ++
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/hw/meson.build b/hw/meson.build
index c0d2db3f5c85..96c1559c348b 100644
--- a/hw/meson
I want to be able to call client tests with simple-xinit, so assertion
failures should be an error.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
test/simple-xinit.c | 8
1 file changed, 8 insertions(+)
diff --git a/test/simple-xinit.c b/test/simple-xinit.c
index 89189a
The Xvfb tests are passing and Xephyr-glamor is failing for me, but it
fails identically on autotools.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
meson.build | 1 +
test/meson.build | 24
test/scripts/xephyr-
Signed-off-by: Eric Anholt <e...@anholt.net>
---
test/Makefile.am | 1 -
test/ddxstubs.c | 97
2 files changed, 98 deletions(-)
delete mode 100644 test/ddxstubs.c
diff --git a/test/Makefile.am b/test/Makefile.am
index 15f0b5
I couldn't find any, and I was modifying the implementation, so I had
to write some. I would like the test to end with a "make sure there
weren't any stray unchecked errors", but I didn't figure out how to do
that.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
hw/vfb/meso
Michel Dänzer <mic...@daenzer.net> writes:
> On 02/08/17 05:59 AM, Eric Anholt wrote:
>> Scissors provide a critical hint to tiled renderers as to what tiles
>> need to be load/stored because they could be modified by the
>> rendering.
>>
>> The bounds calc
-comppixwin10 -repeat 1 -reps 1
on i965 by 3.21191% +/- 1.79977% (n=50).
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_render.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/glamor/glamor_render.c b/glamor/glamor_render.c
index 52f073
Like the previous fix to rectangles, this reduces the area drawn on
tiled renderers by letting the CPU-side tile setup know what tiles
might be drawn at all.
Surprisingly, it improves x11perf -copypixwin1 -repeat 1 -reps 1
on i965 by 2.93185% +/- 1.5561% (n=90).
Signed-off-by: Eric Anholt &l
) to avoid overhead on desktop GL.
No performance difference on i965 with x11perf -rect1 -repeat 1 -reps
1 (n=50)
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_rects.c | 26 ++
glamor/glamor_utils.h | 35 +++
2 files chang
The version detect was erroring out with 1.9 protos installed, and we
weren't building the new code.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
hw/xwayland/meson.build | 3 +++
meson.build | 2 +-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/hw/xw
Adam Jackson <a...@redhat.com> writes:
> On Fri, 2017-07-28 at 09:56 -0700, Eric Anholt wrote:
>> Adam Jackson <a...@redhat.com> writes:
>>
>> > For direct contexts, most context attributes don't require any
>> > particular awarenes
Adam Jackson writes:
> For direct contexts, most context attributes don't require any
> particular awareness on the part of the server. Examples include
> GLX_ARB_create_context_no_error and GLX_ARB_context_flush_control, where
> all of the behavior change lives in the renderer;
Keith Packard <kei...@keithp.com> writes:
> [ Unknown signature status ]
> Eric Anholt <e...@anholt.net> writes:
>
>> That's what both patch 5 of this series and pixman say the depth is for
>> bgrx. I think things have only worked because nobody uses bgrx8
Daniel Stone writes:
> From: Louis-Francis Ratté-Boulianne
>
> Signed-off-by: Louis-Francis Ratté-Boulianne
> Signed-off-by: Daniel Stone
> ---
> +{ .format = DRM_FORMAT_ARGB2101010, .depth = 30,
Michel Dänzer <mic...@daenzer.net> writes:
> [ Unknown signature status ]
> On 27/07/17 07:38 AM, Eric Anholt wrote:
>> Michel Dänzer <mic...@daenzer.net> writes:
>>
>>> [ Unknown signature status ]
>>> On 26/07/17 06:20 AM, Eric Anholt wrote:
Michel Dänzer <mic...@daenzer.net> writes:
> [ Unknown signature status ]
> On 26/07/17 06:20 AM, Eric Anholt wrote:
>> Daniel Stone <dani...@collabora.com> writes:
>>
>>> DRI3 version 1.1 adds support for explicit format modifiers, including
>>>
Daniel Stone writes:
> DRI3 version 1.1 adds support for explicit format modifiers, including
> multi-planar buffers.
I still want proper 64-bit values, and I don't think the little XSync
mess will be much of a blocker.
> Signed-off-by: Daniel Stone
Daniel Stone <dan...@fooishbar.org> writes:
> On 17 June 2017 at 02:21, Eric Anholt <e...@anholt.net> wrote:
>> With the Present extension, we started putting CARD64s on the wire.
>> Let's use them here, too.
>
> Mm, we did have that originally, but anyone includi
Emil Velikov writes:
> From: Emil Velikov
>
> The code that needed it was introduced with commit 7cfd9cc2327 ("Add
> DRI3 support to glamor") back in 2013. And was nuked a couple of years
> ago with commit 51984dddfcc ("glamor: Delay making
Jon Turney <jon.tur...@dronecode.org.uk> writes:
> Don't build them on platforms where they aren't meaningful.
>
> Note that autoconf defines WITH_VGAHW when building the VGAHW module, but
> that doesn't seem to be used anywhere, so we just drop that.
both patches are:
Review
Daniel Stone writes:
> DRI3 version 1.1 adds support for explicit format modifiers, including
> multi-planar buffers.
>
> Signed-off-by: Daniel Stone
> ---
> dri3proto.h | 142
> +++-
> 1
Keith Packard <kei...@keithp.com> writes:
> [ Unknown signature status ]
> Eric Anholt <e...@anholt.net> writes:
>
>> All that was left here was updating the FBO's size. However, the FBO
>> size was always set correctly already through
>> glamor_set_pix
Our top-level glx.h include already provides all of the tokens we use,
and fixes redefinition warnings in the meson build.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
hw/dmx/glxProxy/glxcmds.c | 1 -
hw/dmx/glxProxy/glxcmdsswap.c | 1 -
2 files changed, 2 deletions(-)
diff --gi
Aaron Plattner <aplatt...@nvidia.com> writes:
> On 06/01/2017 01:59 PM, Eric Anholt wrote:
>> We bind-mount the cache directory into the container. Cuts build time
>> from about 4 minutes to 2.
>>
>> Signed-off-by: Eric Anholt <e...@anholt.net>
>&g
Peter Hutterer <peter.hutte...@who-t.net> writes:
> On Thu, Jun 01, 2017 at 04:48:07PM -0700, Eric Anholt wrote:
>> I misspelled the enable flag, so DRI3 would throw BadImplementation
>> when you tried to start any GL app. Same as in
>> c7be7a688a78a34f61b90c0d95914e
I misspelled the enable flag, so DRI3 would throw BadImplementation
when you tried to start any GL app. Same as in
c7be7a688a78a34f61b90c0d95914e14b90b0cdc, we also convert it to #ifdef
for consistency.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
Xext/sync.c | 4 ++--
i
Keith Packard <kei...@keithp.com> writes:
> [ Unknown signature status ]
> Eric Anholt <e...@anholt.net> writes:
>
>> All that was left here was updating the FBO's size. However, the FBO
>> size was always set correctly already through
>> glamor_set_pix
We bind-mount the cache directory into the container. Cuts build time
from about 4 minutes to 2.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
.travis.yml | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index e10cfe1e134f..0a136b
This currently does a meson build using a docker image I've prepared.
The Dockerfile source is at:
https://github.com/anholt/xserver-travis
Docker proved to be necessary to cut the build time per Travis push.
If some day we end up using meson in more of the X stack, we may be
able to move
This matches the test we use for going into the glamor_egl subdir in
../../meson.build.
---
hw/xfree86/drivers/modesetting/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/xfree86/drivers/modesetting/meson.build
b/hw/xfree86/drivers/modesetting/meson.build
index
Michel Dänzer <mic...@daenzer.net> writes:
> [ Unknown signature status ]
> On 01/06/17 03:00 AM, Eric Anholt wrote:
>> Michel Dänzer <mic...@daenzer.net> writes:
>>
>>> From: Michel Dänzer <michel.daen...@amd.com>
>>>
>>> Fixes sub
All that was left here was updating the FBO's size. However, the FBO
size was always set correctly already through
glamor_set_pixmap_texture() from whoever had attached a new BO to the
pixmap.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor.c
The usage of this died with the old core rendering code.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_debug.h | 29 -
glamor/glamor_priv.h | 2 --
2 files changed, 31 deletions(-)
diff --git a/glamor/glamor_debug.h b/glamor/glamor_debug.h
We don't use them for anything.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index 354c30f68ea5..193a04409ee8 100644
--- a/glamor/glamor_egl.c
+++ b/
This means we no longer get "s" for on-screen drawing in glamor_debug,
and there's only "m" (CPU memory) or "f" (Any GPU memory, aka FBOs).
That seems fine to me.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor.c | 1 -
glamor/glamor_core.c
Jon Turney <jon.tur...@dronecode.org.uk> writes:
> On 23/05/2017 00:17, Eric Anholt wrote:
>> Jon Turney <jon.tur...@dronecode.org.uk> writes:
>>
>>> This needs a meson with PRs #1784, #1792 and #1794
>>>
>>> Future work: remove co
Michel Dänzer writes:
> From: Michel Dänzer
>
> Fixes subtle breakage which could sometimes trigger after a server reset
> with multiple screens using glamor:
>
> Screen A enters glamor_close_screen last and calls various cleanup
> functions, which at
Jon Turney writes:
> This needs a meson with PRs #1784, #1792 and #1794
>
> Future work: remove conditionals which are always on, and simplify redundant
> CYGDEBUG conditionals
This looks pretty good! Just a few little cleanups I see.
> ---
> hw/meson.build
Jon Turney <jon.tur...@dronecode.org.uk> writes:
That's some seriously unhelpful stub code!
These first two are:
Reviewed-by: Eric Anholt <e...@anholt.net>
signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org
Adam Jackson <a...@redhat.com> writes:
> Unused since Keith ported misprite to damage in '04.
Nice. Both patches are:
Reviewed-by: Eric Anholt <e...@anholt.net>
signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.
Those extensions don't exist. There's only surfaceless_context.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index a82c1f790997..9527caad2b37
GLES2 support has been requested multiple times, and we've had this
code laying around trying to implement it. The GLES2 implementation
is not quite there yet (some pixel transfer failures), but it
shouldn't take much fixing at this point.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
Michel Dänzer <mic...@daenzer.net> writes:
> On 17/05/17 03:04 AM, Eric Anholt wrote:
>> Since 5064ffab631 (2014), glamor's implementation of _ext just drops the
>> back_pixmap arg, which we were passing NULL (the default) to anyway.
>>
>> Signed-off-by: Eric An
I couldn't find it being used anywhere in the history of the code.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index 21cf3c801242..c59aaa832f63 100644
--- a/glamor/glamor
Using flink is banned on render nodes, and they needlessly expose our
screen pixmap contents to any authenticated client. This also
incidentally drops the dependency on EGL_MESA_drm_image.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.
The function hasn't been doing anything useful since keithp's resource
freeing fixes in 2014.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor.h | 14 --
glamor/glamor_egl.c | 8
2 files changed, 4 insertions(+), 18 deletions(-)
diff --git a/
It's been unused since the initial import.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index abfdbf2a3129..ea047b4c422e 100644
--- a/glamor/glamor_egl.c
+++ b/glamor/glamor
This was replaced in 4afe15d8bfd575c010ed1868697a7922a37ab378, but not
deleted.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index c59aaa832f63..abfdbf2a3129 100644
--- a/
to implement as texture import.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 55 ++---
1 file changed, 27 insertions(+), 28 deletions(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index bf23dc8ed16c..21cf3c
GLES2 support has been requested multiple times, and we've had this
code laying around trying to implement it. The GLES2 implementation
is not quite there yet (some pixel transfer failures), but it
shouldn't take much fixing at this point.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
The glamor_egl module that the GLAMOR paths are using is only built if
GLAMOR_HAS_GBM is true, and there's no plan for implementing the
module without GBM. Simplify modesetting's code as a result.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
hw/xfree86/drivers/modesetting/
The EGL version is not used anywhere in the glamor code, so it's not
interesting. And when saying that we've started using GL
acceleration, it's nice to know what GL we're actually using.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 17 +++--
1 file c
Indirect GLX uses its own context and doesn't care what glamor is
using.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index 6bc9b5383f08..a82c1f790997 100644
--- a/
We're using GBM, so we know we've got GEM.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 35 +++
1 file changed, 7 insertions(+), 28 deletions(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index f68d153114df..bf23dc
We only build this code with GBM, and supporting non-GBM well would be
invasive.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_egl.c | 34 --
1 file changed, 34 deletions(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
Since 5064ffab631 (2014), glamor's implementation of _ext just drops the
back_pixmap arg, which we were passing NULL (the default) to anyway.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
src/radeon_glamor.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git
Julien Cristau <jcris...@debian.org> writes:
> On Mon, May 8, 2017 at 10:13:52 -0700, 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?
>
> FW
Peter Harris <phar...@opentext.com> 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 <phar...@opentext.com>
This series is:
Reviewed-by: Eric Anholt &
Aaron Plattner <aplatt...@nvidia.com> writes:
> On 05/09/2017 04:51 PM, Eric Anholt wrote:
>> Aaron Plattner <aplatt...@nvidia.com> writes:
>>
>>> Commit 112d0d7d01b9 lost the initialization of the variable i in the loop
>>> to add
>&g
he while loop back to a for loop.
>
> Reported-by: Michel Dänzer <mic...@daenzer.net>
> Reported-by: Peter Hutterer <peter.hutte...@who-t.net>
> Reported-by: Eric Anholt <e...@anholt.net>
> Cc: Adam Jackson <a...@redhat.com>
> Fixes: 112d0d7d01b9 ("xfree86:
Jon Turney <jon.tur...@dronecode.org.uk> 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 = []
>>
>> Y
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
cy(link_with : xorgserver_lib)
Elsewhere we have no ' ' before ':'.
Other than that,
Reviewed-by: Eric Anholt <e...@anholt.net>
signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.or
I clearly hadn't run ninja test, since fb no longer had any of the fb
symbols in it.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
hw/xfree86/dixmods/meson.build | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/xfree86/dixmods/meson.build b/hw/xfree86/d
e)
> if gbm_dep.found()
We've certainly got a cleanup of gbm_dep to do, but I'm fine with this
for now. Meson folks agreed that this was the best workaround, though
"[].found() == false" may be available in the future.
Reviewed-by: Eric Anholt <e...@anholt.net>
signature.asc
he contents of
> dbus-core.h are turned off.
I got confused by dbus/hal/systemd requirements several times. Thanks
for fixing it up.
Acked-by: Eric Anholt <e...@anholt.net>
signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.
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
y stated as:
build_modesetting = libdrm_dep.version().version_compare('>= 2.4.46')
It's a bit of an odd choice (modesetting driver if you have dri1/2/3
enabled, or if libdrm happened to be found anyway), but that should be
true so often I don't think it's worth worrying about.
> # XXX: A
Pali Rohár writes:
> [ Unknown signature status ]
> On Saturday 06 May 2017 13:28:17 Julien Cristau wrote:
>> On Fri, May 5, 2017 at 10:08:14 +0200, Pali Rohár wrote:
>> > PING.
>> >
>> > I would like to know if there is some problem with this and
>> > something needs to
Hans de Goede writes:
> The modesetting driver (which now often is used with Intel GPUs),
> relies on dri2_probe_driver_name() to get the dri and vdpau driver
> names, before this commit it would always assign the same name to
> the 2 names. But the vdpau driver for i965
Signed-off-by: Eric Anholt <e...@anholt.net>
---
hw/kdrive/ephyr/meson.build | 1 -
hw/kdrive/src/meson.build | 1 -
2 files changed, 2 deletions(-)
diff --git a/hw/kdrive/ephyr/meson.build b/hw/kdrive/ephyr/meson.build
index cf1cfd5e0c6f..615649865c1e 100644
--- a/hw/kdrive/ephyr/meson
Adam Jackson <a...@redhat.com> writes:
> Signed-off-by: Adam Jackson <a...@redhat.com>
Cute.
Reviewed-by: Eric Anholt <e...@anholt.net>
signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org devel
Peter Hutterer <peter.hutte...@who-t.net> writes:
> While we're shipping both build systems, we'll likely have 'make dist'
> generated tarballs - those tarballs should be buildable with meson to
> have more exposure.
>
> Signed-off-by: Peter Hutterer <peter.hutte...@who-t
I converted Xorg when meson 0.40 came out, but fumbled my grep and
forgot that I had instances of the .extract_all_objects() workaround
in dixmods, too.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
hw/xfree86/dixmods/meson.build | 8
1 file changed, 4 insertions(+), 4 del
A common meson pattern is to use empty arrays for optional sources and
dependencies, since it generally iterates over trees of arrays looking
for values. A value of '' will actually go looking for ''.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glx/meson.build | 2 +-
1 file chan
ly
reasonable solution when using autotool's make dist. Hopefully we can
stop that soon.
Reviewed-by: Eric Anholt <e...@anholt.net>
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
Adam Jackson <a...@nwnk.net> writes:
> On Tue, 2017-04-25 at 16:02 -0700, Eric Anholt wrote:
>
>> meson build/ -Dxephyr=true # ./configure equivalent
>> mesonconf build/ -Ddmx=yes # Turns on a feature later.
>
> I also found useful:
>
> mesonconf build
well, the fact that you can then pass it as a link_with to
> another static library and have it work like you expect makes up for it.
I'd say that pattern was the hardest thing to adjust to with meson, for
me.
One whitespace comment, two optional changes, otherwise it's all:
Reviewed-by: E
Adam Jackson writes:
> Signed-off-by: Adam Jackson
> ---
> fb/meson.build | 10 ++
> hw/xfree86/dixmods/meson.build | 12
> 2 files changed, 22 insertions(+)
>
> diff --git a/fb/meson.build b/fb/meson.build
> index
Adam Jackson writes:
> Signed-off-by: Adam Jackson
> ---
> glx/meson.build| 5 +
> hw/xfree86/dixmods/meson.build | 15 +++
> meson.build| 2 ++
> 3 files changed, 22 insertions(+)
>
> diff --git
Aaron Plattner writes:
> We try to do exactly the opposite in our internal driver build, because
> it's too easy to accidentally do something like
>
> #ifdef GLAMOUR_HAS_GBM
>
> And mistakes like that don't always cause obvious build failures like
> this would. So we build
dded to their lists, too:
- xf86Init.c's notrapSignals case
- quartz.c's QuartzInitOutput() that also tries to not trap signals.
With that fixed, it will be:
Reviewed-by: Eric Anholt <e...@anholt.net>
signature.asc
Description: PGP signature
___
replace autotools. meson is faster
to generate the build, faster to run the bulid, shorter to write the
build files in, and less error-prone than autotools.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
Xext/meson.build | 61 +
Xi/meson
/ # make equivalent
ninja -C build/ install # make install equivalent
ninja -C build/ test# make check equivalent
Eric Anholt (3):
dix: Remove a redundant #define
Use #ifdef instead of #if for features to make Meson easier.
Add a Meson build system alongside autotools.
Xext
We mostly use #ifdef throughout the tree, and this lets the generated
config.h files just be #define TOKEN instead of #define TOKEN 1.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
glamor/glamor_priv.h| 4 ++--
hw/xfree86/common/xf86.h| 2 +-
hw/x
Found when the meson conversion set the symbol to defined, instead of
defined to 1.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
dix/dispatch.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/dix/dispatch.c b/dix/dispatch.c
index 78ac095b1fb8..a2df1e0e6dca
The number for it was merged to drm_mode.h in kernel 4.7, and the
output_names[] array just requires that we slot in new strings in
order.
Signed-off-by: Eric Anholt <e...@anholt.net>
---
hw/xfree86/drivers/modesetting/drmmode_display.c | 1 +
1 file changed, 1 insertion(+)
diff --gi
Keith Packard <kei...@keithp.com> writes:
> [ Unknown signature status ]
> Eric Anholt <e...@anholt.net> writes:
>
>> It doesn't find anything.
>
> Sounds like we can just remove the memcpy then.
So, is that a reviewed-by for #1, since #3 removes the memcpy?
101 - 200 of 1093 matches
Mail list logo