diff options
author | Jan Safranek <jsafrane@redhat.com> | 2009-07-29 15:14:24 +0200 |
---|---|---|
committer | Jan Safranek <jsafrane@redhat.com> | 2009-07-29 15:14:24 +0200 |
commit | c2cb41865e7691a615485abcb21e19536bd13148 (patch) | |
tree | 3bb86e25aa7557b33a8b66c72daf974f023da24f /configure.in | |
parent | 9b269475dbc9d9a7527c7ff512bf2a5a0aa493a4 (diff) | |
download | libcg-c2cb41865e7691a615485abcb21e19536bd13148.tar.gz libcg-c2cb41865e7691a615485abcb21e19536bd13148.tar.xz libcg-c2cb41865e7691a615485abcb21e19536bd13148.zip |
Split cgroup removal to separate functions.
cgroup_delete_cgroup does all the stuff on it's own. This patch refactors
the function and moves the parts with complete error handling.
The new functions are cg_delete_cgroup_controller and cg_move_task_files.
They are not necessarily needed, but it simplifies error handling and clearly
distinguishes functionality of cgroup_delete_cgroup.
Purpose of the functions should be clearly specified in the doxygen comments.
Especially note that cg_delete_cgroup_controller wants already opened file,
where all tasks should be moved - this is optimization, which will be used in
subsequent patches. I could use cgroup_get_task_* and cgroup_attach_task, which
would look definitely nicer, but it would lead to lot of fopen() and fclose()
calls. Let me know what do you think about it.
Signed-off-by: Jan Safranek <jsafrane@redhat.com>
Diffstat (limited to 'configure.in')
0 files changed, 0 insertions, 0 deletions