[PATCH 7/7] sync: split updating and triggering a counter up

Keith Packard keithp at keithp.com
Thu Oct 17 23:40:36 PDT 2013


Peter Hutterer <peter.hutterer at who-t.net> writes:

> Now that the brackets are always the nearest change points (regardless of
> transition) we need to update the counters whenever we check for any updates.
>
> Otherwise we end up with a situation where counter->value is out of date and
> an alarm doesn't trigger because we're still using the value from last time
> something actually triggered.

The only significant change here is that the old value of the timer will
be different the next time SyncChangeCounter is called, which could
affect the Transition triggers. However, the brackets will ensure that
every edge is always caught as it will force each trigger to be checked
at each trigger set value.

Reviewed-by: Keith Packard <keithp at keithp.com>

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20131017/6e2c3414/attachment.pgp>


More information about the xorg-devel mailing list