incubator-mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jie Yu" <>
Subject Re: Review Request: Add an API to atomically kill all tasks in a cgroup.
Date Mon, 09 Jul 2012 18:50:25 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated July 9, 2012, 6:50 p.m.)

Review request for mesos, Benjamin Hindman and Vinod Kone.


Fix format issues. Remove sub-cgroups check in killTasks (no needed).


This patch leverages the freezer subsystem in cgroups to kill all the processes in a cgroup

The main idea is to freeze all the processes in a cgroup first, then send kill signal to all
the proceses. This avoids the need of walking the proc process tree to kill all processes
associated with an executor. In fact, the original killtree solution assumes that the user
processes haven't blocked the SIGSTOP signal, which may not be true in some cases.

Diffs (updated)

  src/linux/cgroups.hpp PRE-CREATION 
  src/linux/cgroups.cpp PRE-CREATION 
  src/tests/cgroups_tests.cpp PRE-CREATION 



On Linux machine, make check.


Jie Yu

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message