~fginther/+junk/mir-staging

Viewing all changes in revision 1049.

  • Committer: Tarmac
  • Author(s): Daniel van Vugt
  • Date: 2013-09-03 10:52:21 UTC
  • mfrom: (1045.2.4 order)
  • Revision ID: tarmac-20130903105221-uc3ph9qekm6zwz8i
SwitchingBundle: Simplify and clarify guarantees that compositor_acquire
always has a buffer to return without blocking or throwing an exception.

The trade-off is that to enforce the guarantee we need to permanently
reserve one buffer for compositing. This reduces the flexibility of
SwitchingBundle a little, such that minimum nbuffers is now 2.

This was originally requested by Alexandros, as the potential throw concerned
him. Although, it was logically guaranteed to never happen for other reasons.

The second reason for doing this is to eliminate recycling logic, which
while safe and correct, was quite confusing. So this change further proves
that that logic (now removed) is not to blame for frame ordering bugs.
.

Approved by Robert Ancell, Alan Griffiths, PS Jenkins bot.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: