* [PATCH -next] khugepaged: selftests: remove debug_cow
@ 2021-04-30 5:11 Nanyong Sun
2021-05-28 2:06 ` Sunnanyong (Nanyong Sun, Intelligent Computing Solution Development Dep)
2021-05-28 11:01 ` Kirill A. Shutemov
0 siblings, 2 replies; 3+ messages in thread
From: Nanyong Sun @ 2021-04-30 5:11 UTC (permalink / raw)
To: shuah, sfr, linux-kselftest
Cc: linux-kernel, kirill.shutemov, yang.shi, sunnanyong, wangkefeng.wang
The debug_cow attribute had been removed since commit 4958e4d86ecb01
("mm: thp: remove debug_cow switch"), so remove it in selftest code too,
otherwise the khugepaged test will fail.
Signed-off-by: Nanyong Sun <sunnanyong@huawei.com>
---
tools/testing/selftests/vm/khugepaged.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/tools/testing/selftests/vm/khugepaged.c b/tools/testing/selftests/vm/khugepaged.c
index 8b7582130..155120b67 100644
--- a/tools/testing/selftests/vm/khugepaged.c
+++ b/tools/testing/selftests/vm/khugepaged.c
@@ -86,7 +86,6 @@ struct settings {
enum thp_enabled thp_enabled;
enum thp_defrag thp_defrag;
enum shmem_enabled shmem_enabled;
- bool debug_cow;
bool use_zero_page;
struct khugepaged_settings khugepaged;
};
@@ -95,7 +94,6 @@ static struct settings default_settings = {
.thp_enabled = THP_MADVISE,
.thp_defrag = THP_DEFRAG_ALWAYS,
.shmem_enabled = SHMEM_NEVER,
- .debug_cow = 0,
.use_zero_page = 0,
.khugepaged = {
.defrag = 1,
@@ -268,7 +266,6 @@ static void write_settings(struct settings *settings)
write_string("defrag", thp_defrag_strings[settings->thp_defrag]);
write_string("shmem_enabled",
shmem_enabled_strings[settings->shmem_enabled]);
- write_num("debug_cow", settings->debug_cow);
write_num("use_zero_page", settings->use_zero_page);
write_num("khugepaged/defrag", khugepaged->defrag);
@@ -304,7 +301,6 @@ static void save_settings(void)
.thp_defrag = read_string("defrag", thp_defrag_strings),
.shmem_enabled =
read_string("shmem_enabled", shmem_enabled_strings),
- .debug_cow = read_num("debug_cow"),
.use_zero_page = read_num("use_zero_page"),
};
saved_settings.khugepaged = (struct khugepaged_settings) {
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH -next] khugepaged: selftests: remove debug_cow
2021-04-30 5:11 [PATCH -next] khugepaged: selftests: remove debug_cow Nanyong Sun
@ 2021-05-28 2:06 ` Sunnanyong (Nanyong Sun, Intelligent Computing Solution Development Dep)
2021-05-28 11:01 ` Kirill A. Shutemov
1 sibling, 0 replies; 3+ messages in thread
From: Sunnanyong (Nanyong Sun, Intelligent Computing Solution Development Dep) @ 2021-05-28 2:06 UTC (permalink / raw)
To: shuah, sfr, akpm
Cc: linux-kernel, kirill.shutemov, yang.shi, wangkefeng.wang,
linux-kselftest
On 2021/4/30 13:11, Nanyong Sun wrote:
> The debug_cow attribute had been removed since commit 4958e4d86ecb01
> ("mm: thp: remove debug_cow switch"), so remove it in selftest code too,
> otherwise the khugepaged test will fail.
>
> Signed-off-by: Nanyong Sun <sunnanyong@huawei.com>
> ---
> tools/testing/selftests/vm/khugepaged.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/tools/testing/selftests/vm/khugepaged.c b/tools/testing/selftests/vm/khugepaged.c
> index 8b7582130..155120b67 100644
> --- a/tools/testing/selftests/vm/khugepaged.c
> +++ b/tools/testing/selftests/vm/khugepaged.c
> @@ -86,7 +86,6 @@ struct settings {
> enum thp_enabled thp_enabled;
> enum thp_defrag thp_defrag;
> enum shmem_enabled shmem_enabled;
> - bool debug_cow;
> bool use_zero_page;
> struct khugepaged_settings khugepaged;
> };
> @@ -95,7 +94,6 @@ static struct settings default_settings = {
> .thp_enabled = THP_MADVISE,
> .thp_defrag = THP_DEFRAG_ALWAYS,
> .shmem_enabled = SHMEM_NEVER,
> - .debug_cow = 0,
> .use_zero_page = 0,
> .khugepaged = {
> .defrag = 1,
> @@ -268,7 +266,6 @@ static void write_settings(struct settings *settings)
> write_string("defrag", thp_defrag_strings[settings->thp_defrag]);
> write_string("shmem_enabled",
> shmem_enabled_strings[settings->shmem_enabled]);
> - write_num("debug_cow", settings->debug_cow);
> write_num("use_zero_page", settings->use_zero_page);
>
> write_num("khugepaged/defrag", khugepaged->defrag);
> @@ -304,7 +301,6 @@ static void save_settings(void)
> .thp_defrag = read_string("defrag", thp_defrag_strings),
> .shmem_enabled =
> read_string("shmem_enabled", shmem_enabled_strings),
> - .debug_cow = read_num("debug_cow"),
> .use_zero_page = read_num("use_zero_page"),
> };
> saved_settings.khugepaged = (struct khugepaged_settings) {
This patch has not been responded for a long time.
Could someone please help to deal with it ?
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH -next] khugepaged: selftests: remove debug_cow
2021-04-30 5:11 [PATCH -next] khugepaged: selftests: remove debug_cow Nanyong Sun
2021-05-28 2:06 ` Sunnanyong (Nanyong Sun, Intelligent Computing Solution Development Dep)
@ 2021-05-28 11:01 ` Kirill A. Shutemov
1 sibling, 0 replies; 3+ messages in thread
From: Kirill A. Shutemov @ 2021-05-28 11:01 UTC (permalink / raw)
To: Nanyong Sun
Cc: shuah, sfr, linux-kselftest, linux-kernel, kirill.shutemov,
yang.shi, wangkefeng.wang
On Fri, Apr 30, 2021 at 01:11:17PM +0800, Nanyong Sun wrote:
> The debug_cow attribute had been removed since commit 4958e4d86ecb01
> ("mm: thp: remove debug_cow switch"), so remove it in selftest code too,
> otherwise the khugepaged test will fail.
>
> Signed-off-by: Nanyong Sun <sunnanyong@huawei.com>
Acked-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
--
Kirill A. Shutemov
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-05-28 11:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-30 5:11 [PATCH -next] khugepaged: selftests: remove debug_cow Nanyong Sun
2021-05-28 2:06 ` Sunnanyong (Nanyong Sun, Intelligent Computing Solution Development Dep)
2021-05-28 11:01 ` Kirill A. Shutemov
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.