On 03/07/2018 11:23 AM, Vasyl Vavrychuk wrote:
As far as I understand connman is single-threaded but then why it
to use built-in functions for atomic memory access such as
__sync_synchronize, __sync_fetch_and_sub, etc.
ConnMan had support for multithreaded plugins. We ripped that out long
time ago and the atomics stayed. It has no technical purpose, it is more
a documentation thingy. Newer code (mostly the refcounting bits) keep
the spirit of the old code. Not sure if it is worth to remove all the
existing atomics at this point. It just introduces a lot of code churn.
Something for 2.x.