lkml.org 
[lkml]   [2019]   [Aug]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[tip:perf/core] perf map: Use zalloc for map_groups
    Commit-ID:  ab6cd0e5276e24403751e0b3b8ed807738a8571f
    Gitweb: https://git.kernel.org/tip/ab6cd0e5276e24403751e0b3b8ed807738a8571f
    Author: John Keeping <john@metanate.com>
    AuthorDate: Thu, 15 Aug 2019 11:01:44 +0100
    Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
    CommitDate: Fri, 16 Aug 2019 12:25:23 -0300

    perf map: Use zalloc for map_groups

    In the next commit we will add new fields to map_groups and we need
    these to be null if no value is assigned. The simplest way to achieve
    this is to request zeroed memory from the allocator.

    Signed-off-by: John Keeping <john@metanate.com>
    Reviewed-by: Jiri Olsa <jolsa@kernel.org>
    Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
    Cc: Konstantin Khlebnikov <khlebnikov@yandex-team.ru>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: john keeping <john@metanate.com>
    Link: http://lkml.kernel.org/r/20190815100146.28842-1-john@metanate.com
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/perf/util/map.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/tools/perf/util/map.c b/tools/perf/util/map.c
    index 668410b1d426..44b556812e4b 100644
    --- a/tools/perf/util/map.c
    +++ b/tools/perf/util/map.c
    @@ -636,7 +636,7 @@ bool map_groups__empty(struct map_groups *mg)

    struct map_groups *map_groups__new(struct machine *machine)
    {
    - struct map_groups *mg = malloc(sizeof(*mg));
    + struct map_groups *mg = zalloc(sizeof(*mg));

    if (mg != NULL)
    map_groups__init(mg, machine);
    \
     
     \ /
      Last update: 2019-08-16 23:00    [W:4.967 / U:0.220 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site