linux-kernel - Re: Re: Re: [PATCH v2 6/9] mm/damon: Add minimal user-space tools

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 linux-cve-announce 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:   Sat,  1 Feb 2020 09:52:33 +0100
From:   SeongJae Park <sj38.park@...il.com>
To:     SeongJae Park <sj38.park@...il.com>
Cc:     Brendan Higgins <brendanhiggins@...gle.com>,
        SeongJae Park <sjpark@...zon.com>,
        Andrew Morton <akpm@...ux-foundation.org>,
        SeongJae Park <sjpark@...zon.de>, acme@...nel.org,
        amit@...nel.org, brendan.d.gregg@...il.com,
        Jonathan Corbet <corbet@....net>, dwmw@...zon.com,
        mgorman@...e.de, Steven Rostedt <rostedt@...dmis.org>,
        kirill@...temov.name, colin.king@...onical.com, minchan@...nel.org,
        vdavydov.dev@...il.com, vdavydov@...allels.com, linux-mm@...ck.org,
        "open list:DOCUMENTATION" <linux-doc@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: Re: Re: [PATCH v2 6/9] mm/damon: Add minimal user-space tools

On Fri, 31 Jan 2020 05:44:27 +0100 SeongJae Park <sj38.park@...il.com> wrote:

> On Thu, 30 Jan 2020 16:02:26 -0800 Brendan Higgins <brendanhiggins@...gle.com> wrote:
> 
> > On Tue, Jan 28, 2020 at 1:00 AM <sjpark@...zon.com> wrote:
> > >
> > > From: SeongJae Park <sjpark@...zon.de>
> > >
> > > This commit adds a shallow wrapper python script, ``/tools/damon/damo``
> > > that provides more convenient interface.  Note that it is only aimed to
> > > be used for minimal reference of the DAMON's raw interfaces and for
> > > debugging of the DAMON itself.  Based on the debugfs interface, you can
> > > create another cool and more convenient user space tools.
> > >
> > > Signed-off-by: SeongJae Park <sjpark@...zon.de>
> > > ---
> > >  MAINTAINERS               |   1 +
> > >  tools/damon/.gitignore    |   1 +
> > >  tools/damon/_dist.py      |  35 ++++
> > >  tools/damon/bin2txt.py    |  64 +++++++
> > >  tools/damon/damo          |  37 ++++
> > >  tools/damon/heats.py      | 358 ++++++++++++++++++++++++++++++++++++++
> > >  tools/damon/nr_regions.py |  88 ++++++++++
> > >  tools/damon/record.py     | 194 +++++++++++++++++++++
> > >  tools/damon/report.py     |  45 +++++
> > >  tools/damon/wss.py        |  94 ++++++++++
> > >  10 files changed, 917 insertions(+)
> > >  create mode 100644 tools/damon/.gitignore
> > >  create mode 100644 tools/damon/_dist.py
> > >  create mode 100644 tools/damon/bin2txt.py
> > >  create mode 100755 tools/damon/damo
> > >  create mode 100644 tools/damon/heats.py
> > >  create mode 100644 tools/damon/nr_regions.py
> > >  create mode 100644 tools/damon/record.py
> > >  create mode 100644 tools/damon/report.py
> > >  create mode 100644 tools/damon/wss.py
> > >
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 5a4db07cad33..95729c138d34 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -4616,6 +4616,7 @@ M:        SeongJae Park <sjpark@...zon.de>
> > >  L:     linux-mm@...ck.org
> > >  S:     Maintained
> > >  F:     mm/damon.c
> > > +F:     tools/damon/*
> > >
> > >  DAVICOM FAST ETHERNET (DMFE) NETWORK DRIVER
> > >  L:     netdev@...r.kernel.org
> > 
> > Another reason to put the MAINTAINERS update at the end; that way you
> > don't have multiple edits sprinkled around your patchset.
> 
> I made this change here due to the warning from 'checkpatch.pl' (WARNING:
> added, moved or deleted file(s), does MAINTAINERS need updating?).  But, as it
> is just a warning, I think simply ignore it and and make this change at the end
> of the patchset would not be a problem, anyway.  What would you prefer?

I think I was too worrying for just warnings.  Will ignore the warnings and
move the MAINTAINERS changes to last patch, as you suggested.


Thanks,
SeongJae Park

> 
> 
> Thanks,
> SeongJae Park
>