All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] xtensa: enable ARCH_HAS_DEBUG_VM_PGTABLE
@ 2022-04-25 18:05 Max Filippov
  2022-04-26  6:08 ` Anshuman Khandual
  2022-04-29  4:21 ` Anshuman Khandual
  0 siblings, 2 replies; 5+ messages in thread
From: Max Filippov @ 2022-04-25 18:05 UTC (permalink / raw)
  To: linux-xtensa; +Cc: Chris Zankel, linux-kernel, Anshuman Khandual, Max Filippov

xtensa kernels successfully build and run with
CONFIG_DEBUG_VM_PGTABLE=y, enable arch support for it.

Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
---
 Documentation/features/debug/debug-vm-pgtable/arch-support.txt | 2 +-
 arch/xtensa/Kconfig                                            | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/features/debug/debug-vm-pgtable/arch-support.txt b/Documentation/features/debug/debug-vm-pgtable/arch-support.txt
index 83eafe1a7f68..ff21a83abe62 100644
--- a/Documentation/features/debug/debug-vm-pgtable/arch-support.txt
+++ b/Documentation/features/debug/debug-vm-pgtable/arch-support.txt
@@ -27,5 +27,5 @@
     |       sparc: | TODO |
     |          um: | TODO |
     |         x86: |  ok  |
-    |      xtensa: | TODO |
+    |      xtensa: |  ok  |
     -----------------------
diff --git a/arch/xtensa/Kconfig b/arch/xtensa/Kconfig
index 2ed897299f19..903b910ebc10 100644
--- a/arch/xtensa/Kconfig
+++ b/arch/xtensa/Kconfig
@@ -4,6 +4,7 @@ config XTENSA
 	select ARCH_32BIT_OFF_T
 	select ARCH_HAS_BINFMT_FLAT if !MMU
 	select ARCH_HAS_CURRENT_STACK_POINTER
+	select ARCH_HAS_DEBUG_VM_PGTABLE
 	select ARCH_HAS_DMA_PREP_COHERENT if MMU
 	select ARCH_HAS_SYNC_DMA_FOR_CPU if MMU
 	select ARCH_HAS_SYNC_DMA_FOR_DEVICE if MMU
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] xtensa: enable ARCH_HAS_DEBUG_VM_PGTABLE
  2022-04-25 18:05 [PATCH] xtensa: enable ARCH_HAS_DEBUG_VM_PGTABLE Max Filippov
@ 2022-04-26  6:08 ` Anshuman Khandual
  2022-04-26 12:42   ` Max Filippov
  2022-04-29  4:21 ` Anshuman Khandual
  1 sibling, 1 reply; 5+ messages in thread
From: Anshuman Khandual @ 2022-04-26  6:08 UTC (permalink / raw)
  To: Max Filippov, linux-xtensa; +Cc: Chris Zankel, linux-kernel

Hi Max,

Thanks for the patch. Just wondering if you ran with different config
options (which might impact test case selection) on/off during testing ?

- Anshuman

On 4/25/22 23:35, Max Filippov wrote:
> xtensa kernels successfully build and run with
> CONFIG_DEBUG_VM_PGTABLE=y, enable arch support for it.
> 
> Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
> ---
>  Documentation/features/debug/debug-vm-pgtable/arch-support.txt | 2 +-
>  arch/xtensa/Kconfig                                            | 1 +
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/features/debug/debug-vm-pgtable/arch-support.txt b/Documentation/features/debug/debug-vm-pgtable/arch-support.txt
> index 83eafe1a7f68..ff21a83abe62 100644
> --- a/Documentation/features/debug/debug-vm-pgtable/arch-support.txt
> +++ b/Documentation/features/debug/debug-vm-pgtable/arch-support.txt
> @@ -27,5 +27,5 @@
>      |       sparc: | TODO |
>      |          um: | TODO |
>      |         x86: |  ok  |
> -    |      xtensa: | TODO |
> +    |      xtensa: |  ok  |
>      -----------------------
> diff --git a/arch/xtensa/Kconfig b/arch/xtensa/Kconfig
> index 2ed897299f19..903b910ebc10 100644
> --- a/arch/xtensa/Kconfig
> +++ b/arch/xtensa/Kconfig
> @@ -4,6 +4,7 @@ config XTENSA
>  	select ARCH_32BIT_OFF_T
>  	select ARCH_HAS_BINFMT_FLAT if !MMU
>  	select ARCH_HAS_CURRENT_STACK_POINTER
> +	select ARCH_HAS_DEBUG_VM_PGTABLE
>  	select ARCH_HAS_DMA_PREP_COHERENT if MMU
>  	select ARCH_HAS_SYNC_DMA_FOR_CPU if MMU
>  	select ARCH_HAS_SYNC_DMA_FOR_DEVICE if MMU

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] xtensa: enable ARCH_HAS_DEBUG_VM_PGTABLE
  2022-04-26  6:08 ` Anshuman Khandual
@ 2022-04-26 12:42   ` Max Filippov
  2022-04-29  3:33     ` Anshuman Khandual
  0 siblings, 1 reply; 5+ messages in thread
From: Max Filippov @ 2022-04-26 12:42 UTC (permalink / raw)
  To: Anshuman Khandual
  Cc: open list:TENSILICA XTENSA PORT (xtensa), Chris Zankel, LKML

Anshuman,

On Mon, Apr 25, 2022 at 11:07 PM Anshuman Khandual
<anshuman.khandual@arm.com> wrote:
> Thanks for the patch. Just wondering if you ran with different config
> options (which might impact test case selection) on/off during testing ?

I went through the code in the mm/debug_vm_pgtable.c and saw that most
variations are related to the huge page support which xtensa doesn't have.
I've tested it with and without CMA.
I haven't found anything else interesting that I could tweak -- have I missed
anything?

-- 
Thanks.
-- Max

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] xtensa: enable ARCH_HAS_DEBUG_VM_PGTABLE
  2022-04-26 12:42   ` Max Filippov
@ 2022-04-29  3:33     ` Anshuman Khandual
  0 siblings, 0 replies; 5+ messages in thread
From: Anshuman Khandual @ 2022-04-29  3:33 UTC (permalink / raw)
  To: Max Filippov; +Cc: open list:TENSILICA XTENSA PORT (xtensa), Chris Zankel, LKML



On 4/26/22 18:12, Max Filippov wrote:
> Anshuman,
> 
> On Mon, Apr 25, 2022 at 11:07 PM Anshuman Khandual
> <anshuman.khandual@arm.com> wrote:
>> Thanks for the patch. Just wondering if you ran with different config
>> options (which might impact test case selection) on/off during testing ?
> 
> I went through the code in the mm/debug_vm_pgtable.c and saw that most
> variations are related to the huge page support which xtensa doesn't have.
> I've tested it with and without CMA.
> I haven't found anything else interesting that I could tweak -- have I missed
> anything?

I dont think so, but thanks for confirming.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] xtensa: enable ARCH_HAS_DEBUG_VM_PGTABLE
  2022-04-25 18:05 [PATCH] xtensa: enable ARCH_HAS_DEBUG_VM_PGTABLE Max Filippov
  2022-04-26  6:08 ` Anshuman Khandual
@ 2022-04-29  4:21 ` Anshuman Khandual
  1 sibling, 0 replies; 5+ messages in thread
From: Anshuman Khandual @ 2022-04-29  4:21 UTC (permalink / raw)
  To: Max Filippov, linux-xtensa; +Cc: Chris Zankel, linux-kernel



On 4/25/22 23:35, Max Filippov wrote:
> xtensa kernels successfully build and run with
> CONFIG_DEBUG_VM_PGTABLE=y, enable arch support for it.
> 
> Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>

Reviewed-by: Anshuman Khandual <anshuman.khandual@arm.com>

> ---
>  Documentation/features/debug/debug-vm-pgtable/arch-support.txt | 2 +-
>  arch/xtensa/Kconfig                                            | 1 +
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/features/debug/debug-vm-pgtable/arch-support.txt b/Documentation/features/debug/debug-vm-pgtable/arch-support.txt
> index 83eafe1a7f68..ff21a83abe62 100644
> --- a/Documentation/features/debug/debug-vm-pgtable/arch-support.txt
> +++ b/Documentation/features/debug/debug-vm-pgtable/arch-support.txt
> @@ -27,5 +27,5 @@
>      |       sparc: | TODO |
>      |          um: | TODO |
>      |         x86: |  ok  |
> -    |      xtensa: | TODO |
> +    |      xtensa: |  ok  |
>      -----------------------
> diff --git a/arch/xtensa/Kconfig b/arch/xtensa/Kconfig
> index 2ed897299f19..903b910ebc10 100644
> --- a/arch/xtensa/Kconfig
> +++ b/arch/xtensa/Kconfig
> @@ -4,6 +4,7 @@ config XTENSA
>  	select ARCH_32BIT_OFF_T
>  	select ARCH_HAS_BINFMT_FLAT if !MMU
>  	select ARCH_HAS_CURRENT_STACK_POINTER
> +	select ARCH_HAS_DEBUG_VM_PGTABLE
>  	select ARCH_HAS_DMA_PREP_COHERENT if MMU
>  	select ARCH_HAS_SYNC_DMA_FOR_CPU if MMU
>  	select ARCH_HAS_SYNC_DMA_FOR_DEVICE if MMU

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2022-04-29  4:20 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-25 18:05 [PATCH] xtensa: enable ARCH_HAS_DEBUG_VM_PGTABLE Max Filippov
2022-04-26  6:08 ` Anshuman Khandual
2022-04-26 12:42   ` Max Filippov
2022-04-29  3:33     ` Anshuman Khandual
2022-04-29  4:21 ` Anshuman Khandual

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.