[PATCH] Do not delete cgroups upon completion of fio job

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Currently upon fio job completion, we delete the cgroups. But in the
process we also loose important debugging information presetn in cgroup
files. For example time information in blkio.time, sector information in
blkio.sectors and how many a times a group was dequeued blkio.dequeue.
Leaving the cgroups as it is, allows one to go and inspect those files
after the fio job is complete.

Secondly, cgroups might have already been created by user before fio job
started. In that case it is not very right to delete cgroups after job
completion.

Signed-off-by: Vivek Goyal <vgoyal@xxxxxxxxxx>
---
 cgroup.c |    1 -
 1 file changed, 1 deletion(-)

Index: fio/cgroup.c
===================================================================
--- fio.orig/cgroup.c	2010-03-31 13:29:49.000000000 -0400
+++ fio/cgroup.c	2010-03-31 13:31:55.788002620 -0400
@@ -65,7 +65,6 @@ void cgroup_kill(struct flist_head *clis
 
 	flist_for_each_safe(n, tmp, clist) {
 		cm = flist_entry(n, struct cgroup_member, list);
-		rmdir(cm->root);
 		flist_del(&cm->list);
 		sfree(cm->root);
 		sfree(cm);


--
To unsubscribe from this list: send the line "unsubscribe fio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Linux SCSI]     [Linux IDE]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux