Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-22 Thread Philipp Zabel
On Mon, 2018-08-20 at 23:00 +0300, Marius Vlad wrote: [...] > + > + > + This interface is used for managing zwp_kms_lease_v1 object. It is used > + to create a zwp_kms_lease_v1 object (the actual lease object) and also > to > + revoke the lease. > + > + > + > + >

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-22 Thread Philipp Zabel
Hi, On Wed, 2018-08-22 at 11:12 +, Marius-cristian Vlad wrote: [...] > > Why not just send the connectors one by one, a single event with all > > relevant information for each? > > Hmm, okay, I'll try do that. I'm wondering what should be used to identify a connector to a hypothetical Vulka

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-22 Thread Philipp Zabel
On Wed, 2018-08-22 at 09:24 -0700, Keith Packard wrote: > Marius-cristian Vlad writes: > > > > Can't the client query available modes on the passed connector via > > > the > > > leased fd? > > > > That's how the client does it now, it uses the leased fd to query > > available modes. Presumably t

Re: Wayland/weston opens files in suspend/resume

2018-08-22 Thread Teemu K
On Wed, Aug 22, 2018 at 12:51 PM Pekka Paalanen wrote: > > On Wed, 22 Aug 2018 12:18:00 +0300 > Teemu K wrote: > > > Hi, > > > > I have custom iMX6 based HW running image generated with Yocto 2.4 > > (Wayland 2.0.0) and Linux kernel 4.1.x that I've been testing with > > suspend/resume cycles. > >

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-22 Thread Keith Packard
Marius-cristian Vlad writes: >> Can't the client query available modes on the passed connector via >> the >> leased fd? > > That's how the client does it now, it uses the leased fd to query > available modes. Presumably the client should have/receive all the data > from the compositor Just

Re: EXT: Wayland/weston opens files in suspend/resume

2018-08-22 Thread Ray, Ian (GE Healthcare)
> On 22 Aug 2018, at 12.18, Teemu K wrote: > > Hi, > > I have custom iMX6 based HW running image generated with Yocto 2.4 > (Wayland 2.0.0) and Linux kernel 4.1.x that I've been testing with > suspend/resume cycles. > > I noticed that after each suspend/resume cycle open file count > increase

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-22 Thread Marius-cristian Vlad
On Wed, 2018-08-22 at 09:17 +0200, Philipp Zabel wrote: > Hi Marius, Hi, > > On Mon, 2018-08-20 at 23:00 +0300, Marius Vlad wrote: > > Simple protocol extension to manage DRM lease. Based on the work by > > Keith > > Packard in [1], respectively [2]. > > > > [1] https://emea01.safelinks.protect

Re: Wayland/weston opens files in suspend/resume

2018-08-22 Thread Pekka Paalanen
On Wed, 22 Aug 2018 12:18:00 +0300 Teemu K wrote: > Hi, > > I have custom iMX6 based HW running image generated with Yocto 2.4 > (Wayland 2.0.0) and Linux kernel 4.1.x that I've been testing with > suspend/resume cycles. > > I noticed that after each suspend/resume cycle open file count > incre

Wayland/weston opens files in suspend/resume

2018-08-22 Thread Teemu K
Hi, I have custom iMX6 based HW running image generated with Yocto 2.4 (Wayland 2.0.0) and Linux kernel 4.1.x that I've been testing with suspend/resume cycles. I noticed that after each suspend/resume cycle open file count increases. In my testing I got around 490 suspend/resume cycles until lib

Re: [PATCH wayland v2 1/3] tests: Demarshalling of very long array/string lengths.

2018-08-22 Thread Pekka Paalanen
On Tue, 21 Aug 2018 10:47:29 +0200 Michal Srb wrote: > Attempting to demarshal message with array or string longer than its > body should return failure. Handling the length correctly is tricky when > it gets to near-UINT32_MAX values. Unexpected overflows can cause > crashes and other security i

Re: [PATCH wayland-protocols v3] unstable/drm-lease: DRM lease protocol support

2018-08-22 Thread Philipp Zabel
Hi Marius, On Mon, 2018-08-20 at 23:00 +0300, Marius Vlad wrote: > Simple protocol extension to manage DRM lease. Based on the work by Keith > Packard in [1], respectively [2]. > > [1] > https://cgit.freedesktop.org/mesa/drm/commit/?id=c4171535389d72e9135c9615cecd07b346fd6d7e > [2] > https://gi