RE: [PATCH weston] ivi-layout: Add missing free() in ivi_view_create

2017-03-29 Thread Ucan, Emre (ADITG/SW1)
Hi, Thank you for your patch. Reviewed-by: Emre Ucan Best regards Emre Ucan Software Group I (ADITG/SW1) Tel. +49 5121 49 6937 > -Original Message- > From: wayland-devel [mailto:wayland-devel- > boun...@lists.freedesktop.org] On Behalf Of Raúl Peñacoba > Sent: Mittwoch, 29. März 2017

[PATCH weston] wcap: Prevent fd leak in wcap_decoder_create() fail path

2017-03-29 Thread Sergi Granell
Signed-off-by: Sergi Granell --- wcap/wcap-decode.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/wcap/wcap-decode.c b/wcap/wcap-decode.c index e3b8985f..7e8c8477 100644 --- a/wcap/wcap-decode.c +++ b/wcap/wcap-decode.c @@ -131,6 +131,7 @@ wcap_decoder_create(const char *filename)

Re: [PATCH weston v2 10/11] [RFC] Account for very large repaint window misses

2017-03-29 Thread Mario Kleiner
On 03/29/2017 12:30 PM, Pekka Paalanen wrote: On Tue, 28 Mar 2017 20:53:53 +0200 Mario Kleiner wrote: ... Agreed. The adjustment can never be more than 1 refresh period, ergo doesn't need a loop, unless we'd get the very unlikely case of Weston getting preempted for a long time exactly durin

[PATCH weston] compositor-wayland: Properly dealloc mmap data using munmap

2017-03-29 Thread Raúl Peñacoba
Signed-off-by: Raúl Peñacoba --- libweston/compositor-wayland.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libweston/compositor-wayland.c b/libweston/compositor-wayland.c index a76dd08e..14f2c8db 100644 --- a/libweston/compositor-wayland.c +++ b/libweston/compositor-wayla

Re: [PATCH weston v9 14/62] compositor-drm: Introduce fb_last member

2017-03-29 Thread Daniel Stone
Hi Pekka, On 24 March 2017 at 10:30, Pekka Paalanen wrote: > On Thu, 23 Mar 2017 15:56:55 + > Daniel Stone wrote: >> On 22 March 2017 at 14:35, Pekka Paalanen wrote: >> > I think this patch could use a better explanation, particularly if all >> > my speculations were in fact incorrect. ;-)

[PATCH weston] editor: Add missing free() and display_destroy() in main

2017-03-29 Thread Raúl Peñacoba
Signed-off-by: Raúl Peñacoba --- clients/editor.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/clients/editor.c b/clients/editor.c index a0cc97af..b63c5628 100644 --- a/clients/editor.c +++ b/clients/editor.c @@ -1607,6 +1607,7 @@ main(int argc, char *argv[]) editor.display = di

[PATCH weston] ivi-layout: Add missing free() in ivi_view_create

2017-03-29 Thread Raúl Peñacoba
Signed-off-by: Raúl Peñacoba --- ivi-shell/ivi-layout.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ivi-shell/ivi-layout.c b/ivi-shell/ivi-layout.c index 64e4ead8..298e18ea 100644 --- a/ivi-shell/ivi-layout.c +++ b/ivi-shell/ivi-layout.c @@ -173,6 +173,7 @@ ivi_view_create(struct ivi_layo

Re: [PATCH weston v2 10/11] [RFC] Account for very large repaint window misses

2017-03-29 Thread Pekka Paalanen
On Wed, 1 Mar 2017 11:34:09 + Daniel Stone wrote: > At the bottom of weston_output_finish_frame(), code exists to account > for flips which have missed the repaint window, by shifting them to lock > on to the next repaint window rather than repainting immediately. > > This code only account

Re: [RFC] Interface for injection of input events

2017-03-29 Thread Pekka Paalanen
On Wed, 22 Mar 2017 12:23:46 +1000 Peter Hutterer wrote: > Hi all, > > This is an RFC for a new interface to generate input events from arbitrary > clients. Note that this is only a few days old, so **do not** assume this is > anything more a thought experiment right now. This email is supposed

Re: [PATCH weston v2 10/11] [RFC] Account for very large repaint window misses

2017-03-29 Thread Pekka Paalanen
On Wed, 29 Mar 2017 11:24:49 +0900 Michel Dänzer wrote: > On 28/03/17 08:02 PM, Pekka Paalanen wrote: > > > > There are also other developments, like priority based, pre-emptive > > task scheduling in the GPUs, which I'm told AMD already has written: > > https://lists.freedesktop.org/archives/am

Re: [PATCH weston v2 10/11] [RFC] Account for very large repaint window misses

2017-03-29 Thread Pekka Paalanen
On Tue, 28 Mar 2017 20:53:53 +0200 Mario Kleiner wrote: > On 03/28/2017 01:02 PM, Pekka Paalanen wrote: > > Hi Mario, > > > > I'm glad to hear from you, it's this kind of details we really > > appreciate you for. :-) > > > > On Tue, 28 Mar 2017 00:59:41 +0200 > > Mario Kleiner wrote: > > > >>