* [PATCH V1 1/2] mm/damon/dbgfs: Put "paddr" check into damon_pa_check() func
[not found] <cover.1642722660.git.xhao@linux.alibaba.com>
@ 2022-01-20 16:04 ` Xin Hao
2022-01-20 17:47 ` SeongJae Park
2022-01-20 16:04 ` [PATCH V1 2/2] mm/damon/dbgfs: Modify Damon dbfs interface dependency in Kconfig Xin Hao
1 sibling, 1 reply; 6+ messages in thread
From: Xin Hao @ 2022-01-20 16:04 UTC (permalink / raw)
To: sj; +Cc: xhao, akpm, linux-mm, linux-kernel
The purpose of putting "paddr" check into damon_pa_check() func
is to better solve "DAMON_DBGFS" dependency on both "DAMON_VADDR"
and "DAMON_PADDR".
Signed-off-by: Xin Hao <xhao@linux.alibaba.com>
---
include/linux/damon.h | 1 +
mm/damon/dbgfs.c | 2 +-
mm/damon/paddr.c | 8 ++++++++
3 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/include/linux/damon.h b/include/linux/damon.h
index 5e1e3a128b77..794b5f453662 100644
--- a/include/linux/damon.h
+++ b/include/linux/damon.h
@@ -504,6 +504,7 @@ void damon_va_set_primitives(struct damon_ctx *ctx);
#endif /* CONFIG_DAMON_VADDR */
#ifdef CONFIG_DAMON_PADDR
+bool damon_pa_check(char *buf, size_t count);
bool damon_pa_target_valid(void *t);
void damon_pa_set_primitives(struct damon_ctx *ctx);
#endif /* CONFIG_DAMON_PADDR */
diff --git a/mm/damon/dbgfs.c b/mm/damon/dbgfs.c
index 5b899601e56c..cdb9c1e2ab29 100644
--- a/mm/damon/dbgfs.c
+++ b/mm/damon/dbgfs.c
@@ -374,7 +374,7 @@ static ssize_t dbgfs_target_ids_write(struct file *file,
if (IS_ERR(kbuf))
return PTR_ERR(kbuf);
- if (!strncmp(kbuf, "paddr\n", count)) {
+ if (damon_pa_check(kbuf, count)) {
id_is_pid = false;
/* target id is meaningless here, but we set it just for fun */
scnprintf(kbuf, count, "42 ");
diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c
index 5e8244f65a1a..14a6bf2a2adb 100644
--- a/mm/damon/paddr.c
+++ b/mm/damon/paddr.c
@@ -208,6 +208,14 @@ static unsigned int damon_pa_check_accesses(struct damon_ctx *ctx)
return max_nr_accesses;
}
+bool damon_pa_check(char *buf, size_t count)
+{
+ if (!strncmp(buf, "paddr\n", count))
+ return true;
+
+ return false;
+}
+
bool damon_pa_target_valid(void *t)
{
return true;
--
2.27.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH V1 2/2] mm/damon/dbgfs: Modify Damon dbfs interface dependency in Kconfig
[not found] <cover.1642722660.git.xhao@linux.alibaba.com>
2022-01-20 16:04 ` [PATCH V1 1/2] mm/damon/dbgfs: Put "paddr" check into damon_pa_check() func Xin Hao
@ 2022-01-20 16:04 ` Xin Hao
2022-01-20 17:46 ` SeongJae Park
1 sibling, 1 reply; 6+ messages in thread
From: Xin Hao @ 2022-01-20 16:04 UTC (permalink / raw)
To: sj; +Cc: xhao, akpm, linux-mm, linux-kernel
If you want to support "DAMON_DBGFS" in config file, it only depends on
any one of "DAMON_VADDR" and "DAMON_PADDR", and sometimes we just want to
use damon virtual address function, but it is unreasonable to include
"DAMON_PADDR" in config file which cause the damon/paddr.c be compiled.
So there just do little change to fix it.
Signed-off-by: Xin Hao <xhao@linux.alibaba.com>
---
include/linux/damon.h | 16 ++++++++++++++++
mm/damon/Kconfig | 2 +-
2 files changed, 17 insertions(+), 1 deletion(-)
diff --git a/include/linux/damon.h b/include/linux/damon.h
index 794b5f453662..c83e4588aa45 100644
--- a/include/linux/damon.h
+++ b/include/linux/damon.h
@@ -501,12 +501,28 @@ int damon_stop(struct damon_ctx **ctxs, int nr_ctxs);
#ifdef CONFIG_DAMON_VADDR
bool damon_va_target_valid(void *t);
void damon_va_set_primitives(struct damon_ctx *ctx);
+#else
+static inline void damon_va_set_primitives(struct damon_ctx *ctx) { }
+static inline bool damon_va_target_valid(void *t)
+{
+ return false;
+}
#endif /* CONFIG_DAMON_VADDR */
#ifdef CONFIG_DAMON_PADDR
bool damon_pa_check(char *buf, size_t count);
bool damon_pa_target_valid(void *t);
void damon_pa_set_primitives(struct damon_ctx *ctx);
+#else
+static inline bool damon_pa_check(char *buf, size_t count)
+{
+ return false;
+}
+static inline void damon_pa_set_primitives(struct damon_ctx *ctx) { }
+static inline bool damon_pa_target_valid(void *t)
+{
+ return false;
+}
#endif /* CONFIG_DAMON_PADDR */
#endif /* _DAMON_H */
diff --git a/mm/damon/Kconfig b/mm/damon/Kconfig
index 5bcf05851ad0..a8952af3c8bf 100644
--- a/mm/damon/Kconfig
+++ b/mm/damon/Kconfig
@@ -54,7 +54,7 @@ config DAMON_VADDR_KUNIT_TEST
config DAMON_DBGFS
bool "DAMON debugfs interface"
- depends on DAMON_VADDR && DAMON_PADDR && DEBUG_FS
+ depends on (DAMON_VADDR || DAMON_PADDR) && DEBUG_FS
help
This builds the debugfs interface for DAMON. The user space admins
can use the interface for arbitrary data access monitoring.
--
2.27.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH V1 2/2] mm/damon/dbgfs: Modify Damon dbfs interface dependency in Kconfig
2022-01-20 16:04 ` [PATCH V1 2/2] mm/damon/dbgfs: Modify Damon dbfs interface dependency in Kconfig Xin Hao
@ 2022-01-20 17:46 ` SeongJae Park
2022-01-21 2:14 ` Xin Hao
0 siblings, 1 reply; 6+ messages in thread
From: SeongJae Park @ 2022-01-20 17:46 UTC (permalink / raw)
To: Xin Hao; +Cc: sj, akpm, linux-mm, linux-kernel
Hi Xin,
I guess this should be at least v3, as you posted v1[1] and v2[2] before.
Could you please also summarize what changes are made from v2?
On Fri, 21 Jan 2022 00:04:28 +0800 Xin Hao <xhao@linux.alibaba.com> wrote:
> If you want to support "DAMON_DBGFS" in config file, it only depends on
> any one of "DAMON_VADDR" and "DAMON_PADDR", and sometimes we just want to
> use damon virtual address function, but it is unreasonable to include
> "DAMON_PADDR" in config file which cause the damon/paddr.c be compiled.
My comment[3] to the v2 was,
So, unless you make me believe the benefit is big enough and all possible
corner cases are well handled and sufficiently tested, I'm sorry but I would
not be convinced with this change.
I don't find answers to the requests here. Could you please add those if you
have?
[1] https://lore.kernel.org/linux-mm/fbf27c39e8f23d12d5474b9b659d3d45f8dd38ca.1637429074.git.xhao@lin
ux.alibaba.com/
[2] https://lore.kernel.org/linux-mm/b6bd0063-efb2-62ce-53ee-0193f63262d8@linux.alibaba.com/
[3] https://lore.kernel.org/linux-mm/20211123103855.12592-1-sj@kernel.org/
Thanks,
SJ
>
> So there just do little change to fix it.
>
> Signed-off-by: Xin Hao <xhao@linux.alibaba.com>
> ---
> include/linux/damon.h | 16 ++++++++++++++++
> mm/damon/Kconfig | 2 +-
> 2 files changed, 17 insertions(+), 1 deletion(-)
>
> diff --git a/include/linux/damon.h b/include/linux/damon.h
> index 794b5f453662..c83e4588aa45 100644
> --- a/include/linux/damon.h
> +++ b/include/linux/damon.h
> @@ -501,12 +501,28 @@ int damon_stop(struct damon_ctx **ctxs, int nr_ctxs);
> #ifdef CONFIG_DAMON_VADDR
> bool damon_va_target_valid(void *t);
> void damon_va_set_primitives(struct damon_ctx *ctx);
> +#else
> +static inline void damon_va_set_primitives(struct damon_ctx *ctx) { }
> +static inline bool damon_va_target_valid(void *t)
> +{
> + return false;
> +}
> #endif /* CONFIG_DAMON_VADDR */
>
> #ifdef CONFIG_DAMON_PADDR
> bool damon_pa_check(char *buf, size_t count);
> bool damon_pa_target_valid(void *t);
> void damon_pa_set_primitives(struct damon_ctx *ctx);
> +#else
> +static inline bool damon_pa_check(char *buf, size_t count)
> +{
> + return false;
> +}
> +static inline void damon_pa_set_primitives(struct damon_ctx *ctx) { }
> +static inline bool damon_pa_target_valid(void *t)
> +{
> + return false;
> +}
> #endif /* CONFIG_DAMON_PADDR */
>
> #endif /* _DAMON_H */
> diff --git a/mm/damon/Kconfig b/mm/damon/Kconfig
> index 5bcf05851ad0..a8952af3c8bf 100644
> --- a/mm/damon/Kconfig
> +++ b/mm/damon/Kconfig
> @@ -54,7 +54,7 @@ config DAMON_VADDR_KUNIT_TEST
>
> config DAMON_DBGFS
> bool "DAMON debugfs interface"
> - depends on DAMON_VADDR && DAMON_PADDR && DEBUG_FS
> + depends on (DAMON_VADDR || DAMON_PADDR) && DEBUG_FS
> help
> This builds the debugfs interface for DAMON. The user space admins
> can use the interface for arbitrary data access monitoring.
> --
> 2.27.0
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V1 1/2] mm/damon/dbgfs: Put "paddr" check into damon_pa_check() func
2022-01-20 16:04 ` [PATCH V1 1/2] mm/damon/dbgfs: Put "paddr" check into damon_pa_check() func Xin Hao
@ 2022-01-20 17:47 ` SeongJae Park
0 siblings, 0 replies; 6+ messages in thread
From: SeongJae Park @ 2022-01-20 17:47 UTC (permalink / raw)
To: Xin Hao; +Cc: sj, akpm, linux-mm, linux-kernel
Hi Xin,
On Fri, 21 Jan 2022 00:04:27 +0800 Xin Hao <xhao@linux.alibaba.com> wrote:
> The purpose of putting "paddr" check into damon_pa_check() func
> is to better solve "DAMON_DBGFS" dependency on both "DAMON_VADDR"
> and "DAMON_PADDR".
Syntax of "paddr" is dbgfs's own. I don't think handling it in paddr is a
right thing to do.
Thanks,
SJ
>
> Signed-off-by: Xin Hao <xhao@linux.alibaba.com>
> ---
> include/linux/damon.h | 1 +
> mm/damon/dbgfs.c | 2 +-
> mm/damon/paddr.c | 8 ++++++++
> 3 files changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/include/linux/damon.h b/include/linux/damon.h
> index 5e1e3a128b77..794b5f453662 100644
> --- a/include/linux/damon.h
> +++ b/include/linux/damon.h
> @@ -504,6 +504,7 @@ void damon_va_set_primitives(struct damon_ctx *ctx);
> #endif /* CONFIG_DAMON_VADDR */
>
> #ifdef CONFIG_DAMON_PADDR
> +bool damon_pa_check(char *buf, size_t count);
> bool damon_pa_target_valid(void *t);
> void damon_pa_set_primitives(struct damon_ctx *ctx);
> #endif /* CONFIG_DAMON_PADDR */
> diff --git a/mm/damon/dbgfs.c b/mm/damon/dbgfs.c
> index 5b899601e56c..cdb9c1e2ab29 100644
> --- a/mm/damon/dbgfs.c
> +++ b/mm/damon/dbgfs.c
> @@ -374,7 +374,7 @@ static ssize_t dbgfs_target_ids_write(struct file *file,
> if (IS_ERR(kbuf))
> return PTR_ERR(kbuf);
>
> - if (!strncmp(kbuf, "paddr\n", count)) {
> + if (damon_pa_check(kbuf, count)) {
> id_is_pid = false;
> /* target id is meaningless here, but we set it just for fun */
> scnprintf(kbuf, count, "42 ");
> diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c
> index 5e8244f65a1a..14a6bf2a2adb 100644
> --- a/mm/damon/paddr.c
> +++ b/mm/damon/paddr.c
> @@ -208,6 +208,14 @@ static unsigned int damon_pa_check_accesses(struct damon_ctx *ctx)
> return max_nr_accesses;
> }
>
> +bool damon_pa_check(char *buf, size_t count)
> +{
> + if (!strncmp(buf, "paddr\n", count))
> + return true;
> +
> + return false;
> +}
> +
> bool damon_pa_target_valid(void *t)
> {
> return true;
> --
> 2.27.0
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V1 2/2] mm/damon/dbgfs: Modify Damon dbfs interface dependency in Kconfig
2022-01-20 17:46 ` SeongJae Park
@ 2022-01-21 2:14 ` Xin Hao
2022-01-21 9:45 ` SeongJae Park
0 siblings, 1 reply; 6+ messages in thread
From: Xin Hao @ 2022-01-21 2:14 UTC (permalink / raw)
To: SeongJae Park; +Cc: akpm, linux-mm, linux-kernel
On 1/21/22 1:46 AM, SeongJae Park wrote:
> Hi Xin,
>
>
> I guess this should be at least v3, as you posted v1[1] and v2[2] before.
> Could you please also summarize what changes are made from v2?
>
> On Fri, 21 Jan 2022 00:04:28 +0800 Xin Hao <xhao@linux.alibaba.com> wrote:
>
>> If you want to support "DAMON_DBGFS" in config file, it only depends on
>> any one of "DAMON_VADDR" and "DAMON_PADDR", and sometimes we just want to
>> use damon virtual address function, but it is unreasonable to include
>> "DAMON_PADDR" in config file which cause the damon/paddr.c be compiled.
> My comment[3] to the v2 was,
>
> So, unless you make me believe the benefit is big enough and all possible
> corner cases are well handled and sufficiently tested, I'm sorry but I would
> not be convinced with this change.
This has really bothered me for a long time. I don't send patches just
for the sake of sending patches. I integrate DAMON patches into my
kernel, and one obvious problem is that I don't use paddr at all. But my
config had to select DAMON_PADDR so that my DAMON codes would compile
and work. I wanted DAMON code to be as simple and clean as possible,
which is why I sent the patch for three times.
I quite agree with what you said, it is necessary to conduct sufficient
test and verification, which I have not done enough. However, I will
conduct further test later. I don't think we should not modify the
previous test case just because it will affect it. I can guarantee the
tools/testing/selftests/Damon and kunit Damon test right before i send a
new version.
>
> I don't find answers to the requests here. Could you please add those if you
> have?
>
> [1] https://lore.kernel.org/linux-mm/fbf27c39e8f23d12d5474b9b659d3d45f8dd38ca.1637429074.git.xhao@lin
> ux.alibaba.com/
> [2] https://lore.kernel.org/linux-mm/b6bd0063-efb2-62ce-53ee-0193f63262d8@linux.alibaba.com/
> [3] https://lore.kernel.org/linux-mm/20211123103855.12592-1-sj@kernel.org/
>
>
> Thanks,
> SJ
>
>> So there just do little change to fix it.
>>
>> Signed-off-by: Xin Hao <xhao@linux.alibaba.com>
>> ---
>> include/linux/damon.h | 16 ++++++++++++++++
>> mm/damon/Kconfig | 2 +-
>> 2 files changed, 17 insertions(+), 1 deletion(-)
>>
>> diff --git a/include/linux/damon.h b/include/linux/damon.h
>> index 794b5f453662..c83e4588aa45 100644
>> --- a/include/linux/damon.h
>> +++ b/include/linux/damon.h
>> @@ -501,12 +501,28 @@ int damon_stop(struct damon_ctx **ctxs, int nr_ctxs);
>> #ifdef CONFIG_DAMON_VADDR
>> bool damon_va_target_valid(void *t);
>> void damon_va_set_primitives(struct damon_ctx *ctx);
>> +#else
>> +static inline void damon_va_set_primitives(struct damon_ctx *ctx) { }
>> +static inline bool damon_va_target_valid(void *t)
>> +{
>> + return false;
>> +}
>> #endif /* CONFIG_DAMON_VADDR */
>>
>> #ifdef CONFIG_DAMON_PADDR
>> bool damon_pa_check(char *buf, size_t count);
>> bool damon_pa_target_valid(void *t);
>> void damon_pa_set_primitives(struct damon_ctx *ctx);
>> +#else
>> +static inline bool damon_pa_check(char *buf, size_t count)
>> +{
>> + return false;
>> +}
>> +static inline void damon_pa_set_primitives(struct damon_ctx *ctx) { }
>> +static inline bool damon_pa_target_valid(void *t)
>> +{
>> + return false;
>> +}
>> #endif /* CONFIG_DAMON_PADDR */
>>
>> #endif /* _DAMON_H */
>> diff --git a/mm/damon/Kconfig b/mm/damon/Kconfig
>> index 5bcf05851ad0..a8952af3c8bf 100644
>> --- a/mm/damon/Kconfig
>> +++ b/mm/damon/Kconfig
>> @@ -54,7 +54,7 @@ config DAMON_VADDR_KUNIT_TEST
>>
>> config DAMON_DBGFS
>> bool "DAMON debugfs interface"
>> - depends on DAMON_VADDR && DAMON_PADDR && DEBUG_FS
>> + depends on (DAMON_VADDR || DAMON_PADDR) && DEBUG_FS
>> help
>> This builds the debugfs interface for DAMON. The user space admins
>> can use the interface for arbitrary data access monitoring.
>> --
>> 2.27.0
>>
--
Best Regards!
Xin Hao
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V1 2/2] mm/damon/dbgfs: Modify Damon dbfs interface dependency in Kconfig
2022-01-21 2:14 ` Xin Hao
@ 2022-01-21 9:45 ` SeongJae Park
0 siblings, 0 replies; 6+ messages in thread
From: SeongJae Park @ 2022-01-21 9:45 UTC (permalink / raw)
To: Xin Hao; +Cc: SeongJae Park, akpm, linux-mm, linux-kernel
Hi Xin,
On Fri, 21 Jan 2022 10:14:36 +0800 Xin Hao <xhao@linux.alibaba.com> wrote:
>
> On 1/21/22 1:46 AM, SeongJae Park wrote:
> > Hi Xin,
> >
> >
> > I guess this should be at least v3, as you posted v1[1] and v2[2] before.
> > Could you please also summarize what changes are made from v2?
> >
> > On Fri, 21 Jan 2022 00:04:28 +0800 Xin Hao <xhao@linux.alibaba.com> wrote:
> >
> >> If you want to support "DAMON_DBGFS" in config file, it only depends on
> >> any one of "DAMON_VADDR" and "DAMON_PADDR", and sometimes we just want to
> >> use damon virtual address function, but it is unreasonable to include
> >> "DAMON_PADDR" in config file which cause the damon/paddr.c be compiled.
> > My comment[3] to the v2 was,
> >
> > So, unless you make me believe the benefit is big enough and all possible
> > corner cases are well handled and sufficiently tested, I'm sorry but I would
> > not be convinced with this change.
>
> This has really bothered me for a long time. I don't send patches just
> for the sake of sending patches. I integrate DAMON patches into my
> kernel, and one obvious problem is that I don't use paddr at all. But my
> config had to select DAMON_PADDR so that my DAMON codes would compile
> and work. I wanted DAMON code to be as simple and clean as possible,
> which is why I sent the patch for three times.
Thank you for the clarification. I know you really want to improve DAMON and
made great progress already. I really appreciate that. I can also understand
your frustration. Of course, it's optimal to compile only what we need.
However, this code is for the community, not for specific users. And, I still
unsure how much pain is incurred by the compilation of one additional source
file for you, and others in the community.
And, sometimes we can compile some of unnecessary code and thus make the binary
a tiny bit bigger to make the code simple. If it seems the benefit of dropping
DAMON_PADDR dependency is clearly bigger than increased complexity of dbgfs and
the change doesn't break any other things to me, I will happily give you my
'Reviewed-by:'. Else, I'm sorry but I wouldn't.
Also, please note that I have a plan[1] to make more reliable DAMON user
interface based on sysfs, and it would not have such unnecessary dependencies.
[1] https://lore.kernel.org/linux-mm/20220119133110.24901-1-sj@kernel.org/
>
>
> I quite agree with what you said, it is necessary to conduct sufficient
> test and verification, which I have not done enough. However, I will
> conduct further test later. I don't think we should not modify the
> previous test case just because it will affect it. I can guarantee the
> tools/testing/selftests/Damon and kunit Damon test right before i send a
> new version.
Those are for minimal tests. I'd like to suggest you to do
https://github.com/awslabs/damon-tests/tree/master/corr, or at least
https://github.com/awslabs/damo/blob/next/tests/run.sh in addition to those.
Also, please ensure you are caring not only DAMON_VADDR && DEBUG_FS, but also
DAMON_PADDR && DEBUG_FS. This also reveals another downside of this patch.
This requires DAMON tests to cover the one more configuration case. And
actually it's currently affecting you. I hope the benefit of next patch
clearly outweighs the effort for that.
Of course, it would be even greter if you can clearly guarantee the patch will
not break other things with the code or comments.
Thanks,
SJ
>
> >
> > I don't find answers to the requests here. Could you please add those if you
> > have?
> >
> > [1] https://lore.kernel.org/linux-mm/fbf27c39e8f23d12d5474b9b659d3d45f8dd38ca.1637429074.git.xhao@lin
> > ux.alibaba.com/
> > [2] https://lore.kernel.org/linux-mm/b6bd0063-efb2-62ce-53ee-0193f63262d8@linux.alibaba.com/
> > [3] https://lore.kernel.org/linux-mm/20211123103855.12592-1-sj@kernel.org/
> >
> >
> > Thanks,
> > SJ
> >
> >> So there just do little change to fix it.
> >>
> >> Signed-off-by: Xin Hao <xhao@linux.alibaba.com>
> >> ---
> >> include/linux/damon.h | 16 ++++++++++++++++
> >> mm/damon/Kconfig | 2 +-
> >> 2 files changed, 17 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/include/linux/damon.h b/include/linux/damon.h
> >> index 794b5f453662..c83e4588aa45 100644
> >> --- a/include/linux/damon.h
> >> +++ b/include/linux/damon.h
> >> @@ -501,12 +501,28 @@ int damon_stop(struct damon_ctx **ctxs, int nr_ctxs);
> >> #ifdef CONFIG_DAMON_VADDR
> >> bool damon_va_target_valid(void *t);
> >> void damon_va_set_primitives(struct damon_ctx *ctx);
> >> +#else
> >> +static inline void damon_va_set_primitives(struct damon_ctx *ctx) { }
> >> +static inline bool damon_va_target_valid(void *t)
> >> +{
> >> + return false;
> >> +}
> >> #endif /* CONFIG_DAMON_VADDR */
> >>
> >> #ifdef CONFIG_DAMON_PADDR
> >> bool damon_pa_check(char *buf, size_t count);
> >> bool damon_pa_target_valid(void *t);
> >> void damon_pa_set_primitives(struct damon_ctx *ctx);
> >> +#else
> >> +static inline bool damon_pa_check(char *buf, size_t count)
> >> +{
> >> + return false;
> >> +}
> >> +static inline void damon_pa_set_primitives(struct damon_ctx *ctx) { }
> >> +static inline bool damon_pa_target_valid(void *t)
> >> +{
> >> + return false;
> >> +}
> >> #endif /* CONFIG_DAMON_PADDR */
> >>
> >> #endif /* _DAMON_H */
> >> diff --git a/mm/damon/Kconfig b/mm/damon/Kconfig
> >> index 5bcf05851ad0..a8952af3c8bf 100644
> >> --- a/mm/damon/Kconfig
> >> +++ b/mm/damon/Kconfig
> >> @@ -54,7 +54,7 @@ config DAMON_VADDR_KUNIT_TEST
> >>
> >> config DAMON_DBGFS
> >> bool "DAMON debugfs interface"
> >> - depends on DAMON_VADDR && DAMON_PADDR && DEBUG_FS
> >> + depends on (DAMON_VADDR || DAMON_PADDR) && DEBUG_FS
> >> help
> >> This builds the debugfs interface for DAMON. The user space admins
> >> can use the interface for arbitrary data access monitoring.
> >> --
> >> 2.27.0
> >>
> --
> Best Regards!
> Xin Hao
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-01-21 9:45 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <cover.1642722660.git.xhao@linux.alibaba.com>
2022-01-20 16:04 ` [PATCH V1 1/2] mm/damon/dbgfs: Put "paddr" check into damon_pa_check() func Xin Hao
2022-01-20 17:47 ` SeongJae Park
2022-01-20 16:04 ` [PATCH V1 2/2] mm/damon/dbgfs: Modify Damon dbfs interface dependency in Kconfig Xin Hao
2022-01-20 17:46 ` SeongJae Park
2022-01-21 2:14 ` Xin Hao
2022-01-21 9:45 ` SeongJae Park
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).