On Fri, 31 Jan 2020 05:44:27 +0100 SeongJae Park <sj38.park@xxxxxxxxx> wrote: > On Thu, 30 Jan 2020 16:02:26 -0800 Brendan Higgins <brendanhiggins@xxxxxxxxxx> wrote: > > > On Tue, Jan 28, 2020 at 1:00 AM <sjpark@xxxxxxxxxx> wrote: > > > > > > From: SeongJae Park <sjpark@xxxxxxxxx> > > > > > > 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@xxxxxxxxx> > > > --- > > > 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@xxxxxxxxx> > > > L: linux-mm@xxxxxxxxx > > > S: Maintained > > > F: mm/damon.c > > > +F: tools/damon/* > > > > > > DAVICOM FAST ETHERNET (DMFE) NETWORK DRIVER > > > L: netdev@xxxxxxxxxxxxxxx > > > > 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 >
