lkml.org 
[lkml]   [2010]   [Jun]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH v2] perf: close the opened directories.
    When I ran "perf kvm ... top", I encountered the following error output.

    Error: perfcounter syscall returned with -1 (Too many open files)

    Fatal: No CONFIG_PERF_EVENTS=y kernel support configured?

    Looking into perf, I found perf opens too many direcotries at initialization
    time, but forgets to close them. Here is the fix.

    Signef-off-by: Gui Jianfeng <guijianfeng@cn.fujitsu.com>
    ---
    tools/perf/util/symbol.c | 17 ++++++++++-------
    1 files changed, 10 insertions(+), 7 deletions(-)

    diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c
    index 7fd6b15..3609a20 100644
    --- a/tools/perf/util/symbol.c
    +++ b/tools/perf/util/symbol.c
    @@ -1443,6 +1443,7 @@ static int map_groups__set_modules_path_dir(struct map_groups *self,
    {
    struct dirent *dent;
    DIR *dir = opendir(dir_name);
    + int ret = 0;

    if (!dir) {
    pr_debug("%s: cannot open %s dir\n", __func__, dir_name);
    @@ -1465,8 +1466,9 @@ static int map_groups__set_modules_path_dir(struct map_groups *self,

    snprintf(path, sizeof(path), "%s/%s",
    dir_name, dent->d_name);
    - if (map_groups__set_modules_path_dir(self, path) < 0)
    - goto failure;
    + ret = map_groups__set_modules_path_dir(self, path);
    + if (ret < 0)
    + goto out;
    } else {
    char *dot = strrchr(dent->d_name, '.'),
    dso_name[PATH_MAX];
    @@ -1487,17 +1489,18 @@ static int map_groups__set_modules_path_dir(struct map_groups *self,
    dir_name, dent->d_name);

    long_name = strdup(path);
    - if (long_name == NULL)
    - goto failure;
    + if (long_name == NULL) {
    + ret = -1;
    + goto out;
    + }
    dso__set_long_name(map->dso, long_name);
    dso__kernel_module_get_build_id(map->dso, "");
    }
    }

    - return 0;
    -failure:
    +out:
    closedir(dir);
    - return -1;
    + return ret;
    }

    static char *get_kernel_version(const char *root_dir)
    --
    1.6.5.2


    \
     
     \ /
      Last update: 2010-06-24 09:07    [W:0.025 / U:1.288 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site