* On 03/12/2018 08:27 PM, Keith Packard wrote:
> This field was defined as a Bool in the protocol headers and BOOL in
> xcb. Bool is not a valid type for protocol fields. It is defined as
> 'int' by Xdefs.h, which we expect to be 32-bits on all machines.
>
> The protocol headers and xcb have patch
exa/exa.c:525:10: warning: initialization discards ‘const’ qualifier from
pointer target type [-Wdiscarded-qualifiers]
swap(pExaGC, pGC, funcs);
^
Signed-off-by: Eric Engestrom
---
exa/exa_priv.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/exa/exa_priv.h b
In function ‘doImageText’,
inlined from ‘ImageText’ at dix/dixfonts.c:1513:5:
dix/dixfonts.c:1492:9: warning: attempt to free a non-heap object
‘local_closure’ [-Wfree-nonheap-object]
free(c);
^
Signed-off-by: Eric Engestrom
---
dix/dixfonts.c | 23 ---
On 2018-03-13 11:56 AM, Eric Engestrom wrote:
> In function ‘doImageText’,
> inlined from ‘ImageText’ at dix/dixfonts.c:1513:5:
> dix/dixfonts.c:1492:9: warning: attempt to free a non-heap object
> ‘local_closure’ [-Wfree-nonheap-object]
> free(c);
> ^
>
> Signed-off-by: Eri
On Tuesday, 2018-03-13 12:09:40 +0100, Michel Dänzer wrote:
> On 2018-03-13 11:56 AM, Eric Engestrom wrote:
> > In function ‘doImageText’,
> > inlined from ‘ImageText’ at dix/dixfonts.c:1513:5:
> > dix/dixfonts.c:1492:9: warning: attempt to free a non-heap object
> > ‘local_closure’ [-Wfree-no
This revision rebases the patch set on current master and does changes
requested by Michel Dänzer.
Please note that while compilation works fine for me, I can't test the runtime
any longer because the latest changes to xserver master require a newer mesa
build, and although I tried extensively I c
As a preperation for future flip mode alternatives move most of the
functionality from 'present.c' into a separate file.
Leave some functions needed by future other flip modes in 'present.c'.
Signed-off-by: Roman Gilg
---
present/Makefile.am|1 +
present/meson.build|1 +
present
Add some basic function hooks to our future present-internal flip mode API,
that will allow us to share functionality in between modes and move more code
in separate files.
Signed-off-by: Roman Gilg
---
present/present_priv.h | 27 ---
present/present_scmd.c | 20
Make present_pixmap a common function callable by any
flip mode.
Signed-off-by: Roman Gilg
---
present/present.c | 57 ++
present/present_priv.h | 68 +++---
present/present_scmd.c | 54 ++---
Add 'queue_vblank', 'flush' and 're_execute' hooks, that
are supposed to be shared with other flip modes.
Signed-off-by: Roman Gilg
---
present/present_priv.h | 12
present/present_scmd.c | 3 +++
2 files changed, 15 insertions(+)
diff --git a/present/present_priv.h b/present/pres
With the new internal flip mode API move vblank creation
and so on into a seperate file, such that it can be shared
between flip modes.
Signed-off-by: Roman Gilg
---
present/Makefile.am | 3 +-
present/meson.build | 1 +
present/present_priv.h | 44 ++-
present/present_s
To be shared by multiple flip modes, refactor execute functionality,
such that logical chunks can go in new separate file.
Signed-off-by: Roman Gilg
---
present/Makefile.am | 1 +
present/meson.build | 1 +
present/present_execute.c | 120 +
To reduce future code duplication refactor timings adjustment out
as a separate function.
Signed-off-by: Roman Gilg
---
present/present.c | 38 ++
present/present_priv.h | 7 +++
present/present_scmd.c | 34 +-
3 files
Flip modes can now have different implementations of
present_can_window_flip.
Signed-off-by: Roman Gilg
---
present/present.c | 9 +
present/present_priv.h | 2 ++
present/present_scmd.c | 5 +++--
3 files changed, 14 insertions(+), 2 deletions(-)
diff --git a/present/present.c b/p
To initialize easily different flip modes, refactor
'present_screen_init'.
Signed-off-by: Roman Gilg
---
present/present_screen.c | 46 +-
1 file changed, 33 insertions(+), 13 deletions(-)
diff --git a/present/present_screen.c b/present/present_screen
Introduce vblank property for flip modes, that demand explicite
allowance by the driver for vblanks to become idle.
Signed-off-by: Roman Gilg
---
present/present_priv.h | 1 +
present/present_vblank.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/present/present_priv.h b/present/presen
For window flip mode data about flips needs to be stored per window.
Add properties to 'present_window_priv' and initialize them on creation.
Signed-off-by: Roman Gilg
---
present/present_priv.h | 10 ++
present/present_screen.c | 6 ++
2 files changed, 16 insertions(+)
diff --gi
Add hooks to query caps, get crtcs, abort vblanks and destroy
a flip.
Signed-off-by: Roman Gilg
---
present/present.c| 28
present/present_priv.h | 33 +
present/present_scmd.c | 38 ++
p
Flipping pixmaps per window needs additional arguments in the
flip mode API. Add these as preperation for window flip mode.
Signed-off-by: Roman Gilg
---
present/present_execute.c | 2 ++
present/present_priv.h| 10 +++---
present/present_scmd.c| 8 +---
present/present_screen.
In contrast to screen flip mode this mode:
* supports flips per windows (these windows currently need to have the same
size as their parent windows with the same pixmap),
* sends pixmap idle signals to the client only after the driver has given
an additional event notify.
This patch only intro
Calculate damage before trying to flip and report it to the driver.
This allows drivers to optimize their rendering.
Signed-off-by: Roman Gilg
---
present/present.h | 3 ++-
present/present_wnmd.c | 24 +---
2 files changed, 15 insertions(+), 12 deletions(-)
diff --git
Allow drivers now to initialize window flip mode.
Signed-off-by: Roman Gilg
---
present/present.h| 2 ++
present/present_screen.c | 23 ++-
2 files changed, 24 insertions(+), 1 deletion(-)
diff --git a/present/present.h b/present/present.h
index affb75f..3d0b972 100
Make sure that vblanks and windows get cleaned up correctly
in window flip mode.
Signed-off-by: Roman Gilg
---
present/present_screen.c | 33 -
present/present_vblank.c | 2 ++
2 files changed, 34 insertions(+), 1 deletion(-)
diff --git a/present/present_screen.
To enable special functionality of window flips introduce for window flips
a separate set of driver facing function hooks.
Signed-off-by: Roman Gilg
---
present/present.h | 43 +++
present/present_priv.h | 1 +
2 files changed, 44 insertions(+)
diff
Add arguments to give the caller more information and control
over the creation of a wl_buffer with GBM, in particular let
the caller determine the size of the buffer.
Signed-off-by: Roman Gilg
---
hw/xwayland/xwayland-glamor.c | 23 +--
hw/xwayland/xwayland.c| 5 +++
Introduce support for Present's window flip mode. The support is not yet
complete, but works reasonable well for the most important use case, that
is fullscreen applications.
We take a Present flip and if the xwl_window->window has the same dimensions
as the presenting window, the flip is represen
When the compositor is not sending frame callbacks while we still wait
on buffer release events fake a continuous msc counter with a timer.
Having this timer is a prerequisite for queuing events.
Signed-off-by: Roman Gilg
---
hw/xwayland/xwayland-present.c | 70 +
This ensures the same behavior as in Present's fake counter and xfree86.
At the moment clients might do put vblanks too far into the future, because
the fake vblank code in Present and the xfree86 driver tolerate cut off upper
32bit due to an 64 to 32bit conversion. Do this therefore here as well
Queue present events to msc values. Fake msc events with a refresh rate of
about 60fps when flips are not possible. When flips are executed rely on
frame callbacks with a slow updating timer as fallback.
This is important for applications, that want to limit their framerate.
Signed-off-by: Roman
Link the newly introduced support for Present flips. For now flips can only
be used in rootless mode together with Glamor.
Signed-off-by: Roman Gilg
---
hw/xwayland/xwayland.c | 22 ++
1 file changed, 22 insertions(+)
diff --git a/hw/xwayland/xwayland.c b/hw/xwayland/xwaylan
Eric Engestrom writes:
> I'll look at the code more closely to figure out when the free is
> needed, but I just saw this warning and had a look, this isn't code I'm
> familiar with *at all*, so I might just end up giving up if I can't
> figure it out easily enough :/
There's no free of stack mem
From: Emil Velikov
As per the protocol, the server should not return version greater than
the one supported by the client.
Add a spec quote and tweak the numbers accordingly.
Fixes: 563138298868 ("dri3: Add DRI3 extension")
Cc: Daniel Stone
Cc: Keith Packard
Signed-off-by: Emil Velikov
---
N
Antoine, thanks for the suggestion and the additional review. V2 patch
series forthcoming.
-- Ben
On Sat, Mar 10, 2018 at 11:58 PM, Antoine Martin
wrote:
> On 09/03/18 05:11, Ben Crocker wrote:
> > Sent that a little too soon; please consider "Reviewed-by: Ben Crocker
> > mailto:bcroc...@redh
From: Nicolai Hähnle
Signed-off-by: Nicolai Hähnle
Reviewed-by: Ben Crocker
Reviewed-by: Antoine Martin
Tested-by: Ben Crocker
---
hw/xfree86/common/xf86Config.c | 2 +-
hw/xfree86/common/xf86Init.c | 12 +++-
hw/xfree86/common/xf86Priv.h | 2 --
3 files changed, 4 insertions(+
From: Nicolai Hähnle
Signed-off-by: Nicolai Hähnle
Reviewed-by: Ben Crocker
Reviewed-by: Antoine Martin
Tested-by: Ben Crocker
---
os/utils.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/os/utils.c b/os/utils.c
index 4305dab26..6e3c16869 100644
--- a/os/utils.c
+++ b/o
From: Nicolai Hähnle
Having different types of code all trying to check for elevated privileges
is a bad idea. This implementation is the most thorough one.
Signed-off-by: Nicolai Hähnle
Reviewed-by: Ben Crocker
Reviewed-by: Antoine Martin
Tested-by: Ben Crocker
---
hw/xfree86/common/xf86In
Submit Nicolai Hähnle's four-patch series improving the checks for
elevated privileges and culminating in using the runtime environmental
control LIBGL_DRIVERS_PATH when searching for driver .so's, as
Mesa, and hence Glamor, already do.
The fourth patch prevents a conflict that can arise if, for e
From: Nicolai Hähnle
Allow switching to another driver build without a full installation.
Glamor already takes LIBGL_DRIVERS_PATH into account, so this change
makes sure that the same driver is used in both parts of the server.
Signed-off-by: Nicolai Hähnle
Reviewed-by: Ben Crocker
Reviewed-b
Mihai Moldovan writes:
> The actual changes (both server and proto) LGTM.
I'll take that as a 'Reviewed-by', unless you have some objection.
--
-keith
signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives:
39 matches
Mail list logo