From 5e0a47a2382b58e15336a742017a57e24611375f Mon Sep 17 00:00:00 2001 From: Dhaval Giani Date: Fri, 9 Jan 2009 16:27:48 +0000 Subject: libcgrouptest: let check_task() receive a pid as argument From: Sudhir Kumar The patch would get a reject because of modifications in an earlier patch. Hence updating the patch by fixing the reject. This patch makes the function check_task() capable of checking if a pid other than the calling process is attached to a particlular group. Earlier it was checking for the calling process only. For optimization purpose 0 is used to indicate that the task to be attached is the current task and hence call cgroup_gettid() only once. Signed-off-by: Sudhir Kumar Acked-by: Dhaval Giani Signed-off-by: Dhaval Giani git-svn-id: https://libcg.svn.sourceforge.net/svnroot/libcg/trunk@311 4f4bb910-9a46-0410-90c8-c897d4f1cd53 --- tests/libcgrouptest.h | 2 +- tests/test_functions.c | 12 ++++++++---- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/tests/libcgrouptest.h b/tests/libcgrouptest.h index 36edc26..00765bd 100644 --- a/tests/libcgrouptest.h +++ b/tests/libcgrouptest.h @@ -138,7 +138,7 @@ struct cgroup *new_cgroup(char *group, char *controller_name, char *control_file, int value_type, struct cntl_val_t cval, struct uid_gid_t ids, int i); int check_fsmounted(int multimnt); -int check_task(char *tasksfile); +int check_task(char *tasksfile, pid_t pid); /* function to print messages in better format */ void message(int num, int pass, const char *api, int ret, char *extra); diff --git a/tests/test_functions.c b/tests/test_functions.c index 4b8bcbb..fcdd308 100644 --- a/tests/test_functions.c +++ b/tests/test_functions.c @@ -96,12 +96,12 @@ void test_cgroup_attach_task(int retcode, struct cgroup *cgrp, build_path(tasksfile, mountpoint, group1, "tasks"); - if (check_task(tasksfile)) { + if (check_task(tasksfile, 0)) { if (fs_mounted == 2) { /* multiple mounts */ build_path(tasksfile2, mountpoint2, group2, "tasks"); - if (check_task(tasksfile2)) { + if (check_task(tasksfile2, 0)) { message(i, PASS, "attach_task()", retval, info[TASKINGRP]); } else { @@ -613,7 +613,7 @@ error: * Checks if the current task belongs to the given tasks file * @param tasksfile the task file to be tested for the task */ -int check_task(char *tasksfile) +int check_task(char *tasksfile, pid_t pid) { FILE *file; pid_t curr_tid, tid; @@ -626,7 +626,11 @@ int check_task(char *tasksfile) exit(1); } - curr_tid = cgrouptest_gettid(); + if (pid) + curr_tid = pid; + else + curr_tid = cgrouptest_gettid(); + while (!feof(file)) { fscanf(file, "%u", &tid); if (tid == curr_tid) { -- cgit