Re: [PATCH weston] compositor: Assign new views to the primary plane

2016-12-16 Thread Daniel Stone
Hi, On 14 December 2016 at 15:21, Pekka Paalanen wrote: > On Wed, 14 Dec 2016 14:53:37 + Daniel Stone wrote: >> On 14 December 2016 at 14:17, Pekka Paalanen >> wrote: >> > So, reading both versions of the commit message, I think I was able >> > reconstruct enough of the idea to see what's g

Re: [PATCH weston] compositor: Assign new views to the primary plane

2016-12-14 Thread Pekka Paalanen
On Wed, 14 Dec 2016 14:53:37 + Daniel Stone wrote: > Hi Pekka, > > On 14 December 2016 at 14:17, Pekka Paalanen > wrote: > > On Fri, 9 Dec 2016 17:27:13 + Daniel Stone > > wrote: > >> However, this is undesirable for DRM. With multi-output, when > >> assign_planes() is called, any v

Re: [PATCH weston] compositor: Assign new views to the primary plane

2016-12-14 Thread Daniel Stone
Hi Pekka, On 14 December 2016 at 14:17, Pekka Paalanen wrote: > On Fri, 9 Dec 2016 17:27:13 + Daniel Stone wrote: >> However, this is undesirable for DRM. With multi-output, when >> assign_planes() is called, any view which wasn't on a plane for our >> putput was moved to the primary plane,

Re: [PATCH weston] compositor: Assign new views to the primary plane

2016-12-14 Thread Pekka Paalanen
On Fri, 9 Dec 2016 17:27:13 + Daniel Stone wrote: > Hi, > > On 9 December 2016 at 16:35, Daniel Stone wrote: > > However, this is undesirable for DRM. In a multi-output situation, > > we would see a view only visible on another output, reasonably decide we > > didn't want it in a plane on o

Re: [PATCH weston] compositor: Assign new views to the primary plane

2016-12-09 Thread Daniel Stone
Hi, On 9 December 2016 at 16:35, Daniel Stone wrote: > However, this is undesirable for DRM. In a multi-output situation, > we would see a view only visible on another output, reasonably decide we > didn't want it in a plane on our output, and move it to the primary > plane, causing damage, and a

[PATCH weston] compositor: Assign new views to the primary plane

2016-12-09 Thread Daniel Stone
When we create a new view, assign it to the primary plane from the beginning. This made no difference before, since the next surface repaint would forcibly assign all views to a plane, either through DRM's assign_planes() hook, or the fallback inside core repaint. However, this is undesirable for