eina/lock: rework meaning of EINA_DEBUG_THREADS variable
authorMike Blumenkrantz <zmike@samsung.com>
Mon, 25 Jun 2018 19:15:34 +0000 (15:15 -0400)
committerWonki Kim <wonki_.kim@samsung.com>
Wed, 27 Jun 2018 07:54:55 +0000 (16:54 +0900)
commitb9184e8642e34af6fc66588b96909c1768691175
tree5d01c4f5491f17d3ad95c0f95c995dc2338e4f6f
parentd774fdf91008f08017f80c2506d6373362f35f1f
eina/lock: rework meaning of EINA_DEBUG_THREADS variable

Summary:
previously this used to mean 'the number of ms that a lock can wait for
until abort() is called once the lock is acquired' and it was useful
when trying to find contention issues with locks

unfortunately this required a bit of reading into the code to determine,
and it made the common case of setting values to 1 fail in some cases,
as this is a very short amount of time. also the documentation explicitly
gives '1' as an example setting for this variable, which will cause immediate
abort() in most cases when debugging was enabled since things are much slower

this variable now is the number of usec that a lock can wait for before abort()
is called, and the lowest value that will be checked for abort()ing is 100, meaning
that '1' is valid again
Depends on D6375

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

Differential Revision: https://phab.enlightenment.org/D6376
src/lib/eina/eina_inline_lock_posix.x