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

2017-03-30 Thread Pekka Paalanen
On Wed, 29 Mar 2017 18:46:03 +0100 Daniel Stone wrote: > 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 explana

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. ;-)

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

2017-03-24 Thread Pekka Paalanen
On Thu, 23 Mar 2017 15:56:55 + Daniel Stone wrote: > Hi, > > On 22 March 2017 at 14:35, Pekka Paalanen wrote: > > On Fri, 3 Mar 2017 23:05:25 + > > Daniel Stone wrote: > >> Clean up some ambiguity around current/next: current could previously > >> have referred to a buffer which was

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

2017-03-23 Thread Daniel Stone
Hi, On 22 March 2017 at 14:35, Pekka Paalanen wrote: > On Fri, 3 Mar 2017 23:05:25 + > Daniel Stone wrote: >> Clean up some ambiguity around current/next: current could previously >> have referred to a buffer which was being displayed, or the last buffer >> being displayed whilst we waited

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

2017-03-22 Thread Pekka Paalanen
On Fri, 3 Mar 2017 23:05:25 + Daniel Stone wrote: > Clean up some ambiguity around current/next: current could previously > have referred to a buffer which was being displayed, or the last buffer > being displayed whilst we waited for a configuration we'd requested to > take effect. > > Int

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

2017-03-03 Thread Daniel Stone
Clean up some ambiguity around current/next: current could previously have referred to a buffer which was being displayed, or the last buffer being displayed whilst we waited for a configuration we'd requested to take effect. Introduce a new variable, fb_last, which exclusively holds the latter ca