----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38639/#review100046 -----------------------------------------------------------
Ship it! 3rdparty/libprocess/3rdparty/stout/include/stout/os/linux.hpp (line 73) <https://reviews.apache.org/r/38639/#comment157135> We should delete the stack only if CLONE_VM was not specified in the flags or at least have a comment mentioning the fact that this call is creating a new process and not a new thread. Having a CLONE_VM will create a new thread, not a new process. - Kapil Arya On Sept. 22, 2015, 2:36 p.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38639/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2015, 2:36 p.m.) > > > Review request for mesos, Jie Yu and Cong Wang. > > > Bugs: MESOS-3474 > https://issues.apache.org/jira/browse/MESOS-3474 > > > Repository: mesos > > > Description > ------- > > Add os::clone function in stout. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/linux.hpp > b994b13941628947c9d12b8baae155d5da1ec7bd > > Diff: https://reviews.apache.org/r/38639/diff/ > > > Testing > ------- > > > Thanks, > > haosdent huang > >