You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@mesos.apache.org by "Neil Conway (JIRA)" <ji...@apache.org> on 2015/08/28 02:11:46 UTC

[jira] [Created] (MESOS-3326) Make use of C++11 atomics

Neil Conway created MESOS-3326:
----------------------------------

             Summary: Make use of C++11 atomics
                 Key: MESOS-3326
                 URL: https://issues.apache.org/jira/browse/MESOS-3326
             Project: Mesos
          Issue Type: Bug
            Reporter: Neil Conway
            Assignee: Neil Conway


Now that we require C++11, we can make use of std::atomic. For example:

* libprocess/process.cpp uses a bare int + __sync_synchronize() for "running"
* __sync_synchronize() is used in logging.hpp in libprocess and fork.hpp in stout
* sched/sched.cpp uses a volatile int for "running" -- this is wrong, "volatile" is not sufficient to ensure safe concurrent access
* "volatile" is used in a few other places -- most are probably dubious but I haven't looked closely



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)