lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Tue, 21 Feb 2017 19:33:58 +0530
From:   Hari Bathini <hbathini@...ux.vnet.ibm.com>
To:     Jiri Olsa <jolsa@...hat.com>
Cc:     ast@...com, peterz@...radead.org,
        lkml <linux-kernel@...r.kernel.org>, acme@...nel.org,
        alexander.shishkin@...ux.intel.com, mingo@...hat.com,
        daniel@...earbox.net, rostedt@...dmis.org,
        Ananth N Mavinakayanahalli <ananth@...ux.vnet.ibm.com>,
        ebiederm@...ssion.com, sargun@...gun.me,
        Aravinda Prasad <aravinda@...ux.vnet.ibm.com>,
        brendan.d.gregg@...il.com
Subject: Re: [PATCH v6 2/3] perf tool: add PERF_RECORD_NAMESPACES to include
 namespaces related info

Hi Jiri,


On Wednesday 08 February 2017 06:27 PM, Jiri Olsa wrote:
> On Wed, Feb 08, 2017 at 02:01:49PM +0530, Hari Bathini wrote:
>> This patch updates perf tool to examine PERF_RECORD_NAMESPACES events
>> emitted by the kernel when fork, clone, setns or unshare are invoked.
>> Also, it synthesizes PERF_RECORD_NAMESPACES events for processes that
>> were running prior to invocation of perf record, the data for which
>> is taken from /proc/$PID/ns. These changes make way for analyzing
>> events with regard to namespaces.
>>
>> Signed-off-by: Hari Bathini <hbathini@...ux.vnet.ibm.com>
>> ---
>>   tools/include/uapi/linux/perf_event.h |   38 +++++++++
>>   tools/perf/builtin-annotate.c         |    1
>>   tools/perf/builtin-diff.c             |    1
>>   tools/perf/builtin-inject.c           |   14 +++
>>   tools/perf/builtin-kmem.c             |    1
>>   tools/perf/builtin-kvm.c              |    2
>>   tools/perf/builtin-lock.c             |    1
>>   tools/perf/builtin-mem.c              |    1
>>   tools/perf/builtin-record.c           |   33 +++++++-
>>   tools/perf/builtin-report.c           |    1
>>   tools/perf/builtin-sched.c            |    1
>>   tools/perf/builtin-script.c           |   41 +++++++++
>>   tools/perf/builtin-trace.c            |    3 -
>>   tools/perf/perf.h                     |    1
>>   tools/perf/util/Build                 |    1
>>   tools/perf/util/data-convert-bt.c     |    2
>>   tools/perf/util/event.c               |  143 ++++++++++++++++++++++++++++++++-
>>   tools/perf/util/event.h               |   19 ++++
>>   tools/perf/util/evsel.c               |    3 +
>>   tools/perf/util/machine.c             |   34 ++++++++
>>   tools/perf/util/machine.h             |    3 +
>>   tools/perf/util/namespaces.c          |   35 ++++++++
>>   tools/perf/util/namespaces.h          |   26 ++++++
>>   tools/perf/util/session.c             |    7 ++
>>   tools/perf/util/thread.c              |   44 ++++++++++
>>   tools/perf/util/thread.h              |    6 +
>>   tools/perf/util/tool.h                |    2
>>   27 files changed, 450 insertions(+), 14 deletions(-)
>>   create mode 100644 tools/perf/util/namespaces.c
>>   create mode 100644 tools/perf/util/namespaces.h
> could you please split this patch into logically separated changes, like:
>    - adding PERF_RECORD_NAMESPACES event with perf_event__process_namespaces
>      machinery
>    - synthesize support (perf_event__synthesize_namespaces) for record
>    - script print support
>    - perf_event__fprintf print support
>
> or any other split that'd make sense, it's too many changes for one patch now
>
>

Thanks for the review.
Posted v7 with the changes included..

- Hari

Powered by blists - more mailing lists