On Wed, Dec 24, 2014 at 04:15:00PM +0900, Namhyung Kim wrote: SNIP
> #endif /* __PERF_DATA_H */ > diff --git a/tools/perf/util/util.c b/tools/perf/util/util.c > index d5eab3f3323f..a5046d52e311 100644 > --- a/tools/perf/util/util.c > +++ b/tools/perf/util/util.c > @@ -72,6 +72,49 @@ int mkdir_p(char *path, mode_t mode) > return (stat(path, &st) && mkdir(path, mode)) ? -1 : 0; > } > > +int rm_rf(char *path) > +{ > + DIR *dir; > + int ret = 0; > + struct dirent *d; > + char namebuf[PATH_MAX]; > + > + dir = opendir(path); > + if (dir == NULL) > + return 0; > + > + while ((d = readdir(dir)) != NULL && !ret) { > + struct stat statbuf; > + > + if (d->d_name[0] == '.') > + continue; Could you check for '.' and for '..' to support '.*' removal? I know tha we will probably not have any '.*' files in perf.data.dir, but this function could be used later like for total cache clean ;-) also please submit 'rm_rf' in separate patch thanks, jirka -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/