Re: [PATCH] powerpc/mm: Remove DEBUG_VM_PGTABLE support on ppc64

On 9/1/20 2:40 PM, Christophe Leroy wrote:

Le 01/09/2020 à 10:15, Christophe Leroy a écrit :

quoted

Le 01/09/2020 à 10:12, Aneesh Kumar K.V a écrit :

quoted

On 9/1/20 1:40 PM, Christophe Leroy wrote:

quoted

Le 01/09/2020 à 10:02, Aneesh Kumar K.V a écrit :

quoted

The test is broken w.r.t page table update rules and results in kernel
crash as below. Disable the support untill we get the tests updated.


Signed-off-by: Aneesh Kumar K.V <redacted>

Any Fixes: tag ?

quoted

---
  arch/powerpc/Kconfig | 1 -
  1 file changed, 1 deletion(-)

diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 65bed1fdeaad..787e829b6f25 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig

@@ -116,7 +116,6 @@ config PPC

      #
      select ARCH_32BIT_OFF_T if PPC32
      select ARCH_HAS_DEBUG_VIRTUAL
-    select ARCH_HAS_DEBUG_VM_PGTABLE

You say you remove support for ppc64 but you are removing it for both PPC64 and PPC32. Should you replace the line by:

Does it work on PPC32 with DEBUG_VM enabled? I was expecting it will be broken there too.

I was wondering. I have just started a build to test that on my 8xx. I'll tell you before noon (Paris).

There are warnings but it boots OK. So up to you, but if you deactivate it for both PPC32 and PPC64, say so in the subject like.

I will update the subject line to indicate powerpc instead of ppc64

-aneesh