Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: quoted-printable
From:   Qian Cai <cai@lca.pw>
Mime-Version: 1.0 (1.0)
Subject: Re: [PATCH v2 0/9] Introduce Data Access MONitor (DAMON)
Date:   Tue, 28 Jan 2020 05:20:35 -0500
Message-Id: <D20B234E-04EE-4410-9B27-FF63AB3E1808@lca.pw>
References: <20200128085742.14566-1-sjpark@amazon.com>
Cc:     akpm@linux-foundation.org, SeongJae Park <sjpark@amazon.de>,
        sj38.park@gmail.com, acme@kernel.org, amit@kernel.org,
        brendan.d.gregg@gmail.com, corbet@lwn.net, dwmw@amazon.com,
        mgorman@suse.de, rostedt@goodmis.org, kirill@shutemov.name,
        brendanhiggins@google.com, colin.king@canonical.com,
        minchan@kernel.org, vdavydov.dev@gmail.com, vdavydov@parallels.com,
        linux-mm@kvack.org, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org
In-Reply-To: <20200128085742.14566-1-sjpark@amazon.com>
To:     sjpark@amazon.com
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


> On Jan 28, 2020, at 3:58 AM, sjpark@amazon.com wrote:
>=20
> This patchset introduces a new kernel module for practical monitoring of d=
ata
> accesses, namely DAMON.
>=20
> The patches are organized in the following sequence.  The first four patch=
es
> implements the core logic of DAMON one by one.  After that, the fifth patc=
h
> implements DAMON's debugfs interface for users.  To provide a minimal refe=
rence
> to the low level interface and for more convenient use/tests of the DAMON,=
 the
> sixth patch implements an user space tool.  The seventh patch adds a docum=
ent
> for administrators of DAMON, and the eightth patch provides DAMON's kunit
> tests.  Finally, the ninth patch implements a tracepoint for DAMON.  As th=
e
> tracepoint prints every monitoring results, it will be easily integrated w=
ith
> other tracers supporting tracepoints including perf.

I am a bit surprised that this patchset did not include perf maintainers whi=
ch makes me wonder if there is any attempt to discuss first if we actually n=
eed a whole new subsystem for it or a existing tool can be enhanced.=