projects
/
fio.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
diskutil: minor style cleanup
[fio.git]
/
cgroup.c
diff --git
a/cgroup.c
b/cgroup.c
index 0e3a71ebbfa75f57c9980ff6ba5c990b972d0c50..a297e2ac82e82f5145edfcb9cc509b23134a360b 100644
(file)
--- a/
cgroup.c
+++ b/
cgroup.c
@@
-4,6
+4,8
@@
#include <stdio.h>
#include <stdlib.h>
#include <mntent.h>
#include <stdio.h>
#include <stdlib.h>
#include <mntent.h>
+#include <sys/stat.h>
+#include <sys/types.h>
#include "fio.h"
#include "flist.h"
#include "cgroup.h"
#include "fio.h"
#include "flist.h"
#include "cgroup.h"
@@
-20,7
+22,8
@@
struct cgroup_member {
static char *find_cgroup_mnt(struct thread_data *td)
{
char *mntpoint = NULL;
static char *find_cgroup_mnt(struct thread_data *td)
{
char *mntpoint = NULL;
- struct mntent *mnt;
+ struct mntent *mnt, dummy;
+ char buf[256] = {0};
FILE *f;
f = setmntent("/proc/mounts", "r");
FILE *f;
f = setmntent("/proc/mounts", "r");
@@
-29,7
+32,7
@@
static char *find_cgroup_mnt(struct thread_data *td)
return NULL;
}
return NULL;
}
- while ((mnt = getmntent
(f
)) != NULL) {
+ while ((mnt = getmntent
_r(f, &dummy, buf, sizeof(buf)
)) != NULL) {
if (!strcmp(mnt->mnt_type, "cgroup") &&
strstr(mnt->mnt_opts, "blkio"))
break;
if (!strcmp(mnt->mnt_type, "cgroup") &&
strstr(mnt->mnt_opts, "blkio"))
break;
@@
-49,9
+52,22
@@
static void add_cgroup(struct thread_data *td, const char *name,
{
struct cgroup_member *cm;
{
struct cgroup_member *cm;
+ if (!lock)
+ return;
+
cm = smalloc(sizeof(*cm));
cm = smalloc(sizeof(*cm));
+ if (!cm) {
+err:
+ log_err("fio: failed to allocate cgroup member\n");
+ return;
+ }
+
INIT_FLIST_HEAD(&cm->list);
cm->root = smalloc_strdup(name);
INIT_FLIST_HEAD(&cm->list);
cm->root = smalloc_strdup(name);
+ if (!cm->root) {
+ sfree(cm);
+ goto err;
+ }
if (td->o.cgroup_nodelete)
cm->cgroup_nodelete = 1;
fio_mutex_down(lock);
if (td->o.cgroup_nodelete)
cm->cgroup_nodelete = 1;
fio_mutex_down(lock);
@@
-64,6
+80,9
@@
void cgroup_kill(struct flist_head *clist)
struct flist_head *n, *tmp;
struct cgroup_member *cm;
struct flist_head *n, *tmp;
struct cgroup_member *cm;
+ if (!lock)
+ return;
+
fio_mutex_down(lock);
flist_for_each_safe(n, tmp, clist) {
fio_mutex_down(lock);
flist_for_each_safe(n, tmp, clist) {
@@
-96,7
+115,7
@@
static int write_int_to_file(struct thread_data *td, const char *path,
{
char tmp[256];
FILE *f;
{
char tmp[256];
FILE *f;
-
+
sprintf(tmp, "%s/%s", path, filename);
f = fopen(tmp, "w");
if (!f) {
sprintf(tmp, "%s/%s", path, filename);
f = fopen(tmp, "w");
if (!f) {
@@
-179,7
+198,9
@@
void cgroup_shutdown(struct thread_data *td, char **mnt)
static void fio_init cgroup_init(void)
{
static void fio_init cgroup_init(void)
{
- lock = fio_mutex_init(1);
+ lock = fio_mutex_init(FIO_MUTEX_UNLOCKED);
+ if (!lock)
+ log_err("fio: failed to allocate cgroup lock\n");
}
static void fio_exit cgroup_exit(void)
}
static void fio_exit cgroup_exit(void)