Message ID | mhng-423e8bdb-977e-4b99-a1bb-b8c530664a51@palmerdabbelt-glaptop (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [GIT,PULL] RISC-V Patches for the 5.14 Merge Window, Part 1 | expand |
The pull request you sent on Fri, 09 Jul 2021 07:53:58 -0700 (PDT):
> git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git tags/riscv-for-linus-5.14-mw0
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/9b76d71fa8be8c52dbc855ab516754f0c93e2980
Thank you!
Le 9/07/2021 à 16:53, Palmer Dabbelt a écrit : > The following changes since commit 8a4102a0cf07cc76a18f373f6b49485258cc6af4: > > riscv: mm: Fix W+X mappings at boot (2021-06-01 21:15:09 -0700) > > are available in the Git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git tags/riscv-for-linus-5.14-mw0 > > for you to fetch changes up to 1958e5aef5098e28b7d6e6a2972649901ebecace: > > riscv: xip: Fix duplicate included asm/pgtable.h (2021-07-06 16:17:40 -0700) > > ---------------------------------------------------------------- > RISC-V Patches for the 5.14 Merge Window, Part 1 > > In addition to We have a handful of new features for 5.14: > > * Support for transparent huge pages. > * Support for generic PCI resources mapping. > * Support for the mem= kernel parameter. > * Support for KFENCE. > * A handful of fixes to avoid W+X mappings in the kernel. > * Support for VMAP_STACK based overflow detection. > * An optimized copy_{to,from}_user. > ---------------------------------------------------------------- > There are some Kconfig merge conflicts. They should be pretty > straight-forward, but we do have a symbol out of order -- I thought I had a > script to check for that, but I guess it doesn't work. I just sent out a patch > to fix it up. > > diff --cc arch/riscv/Kconfig > index 3590eb76000e,469a70bd8da6..d36f3c5029fd > --- a/arch/riscv/Kconfig > +++ b/arch/riscv/Kconfig > @@@ -60,12 -61,11 +61,12 @@@ config RISC > select GENERIC_TIME_VSYSCALL if MMU && 64BIT > select HANDLE_DOMAIN_IRQ > select HAVE_ARCH_AUDITSYSCALL > - select HAVE_ARCH_JUMP_LABEL > - select HAVE_ARCH_JUMP_LABEL_RELATIVE > + select HAVE_ARCH_JUMP_LABEL if !XIP_KERNEL > + select HAVE_ARCH_JUMP_LABEL_RELATIVE if !XIP_KERNEL > select HAVE_ARCH_KASAN if MMU && 64BIT > select HAVE_ARCH_KASAN_VMALLOC if MMU && 64BIT > + select HAVE_ARCH_KFENCE if MMU && 64BIT > - select HAVE_ARCH_KGDB > + select HAVE_ARCH_KGDB if !XIP_KERNEL > select HAVE_ARCH_KGDB_QXFER_PKT > select HAVE_ARCH_MMAP_RND_BITS if MMU > select HAVE_ARCH_SECCOMP_FILTER > @@@ -81,11 -80,9 +82,14 @@@ > select HAVE_GCC_PLUGINS > select HAVE_GENERIC_VDSO if MMU && 64BIT > select HAVE_IRQ_TIME_ACCOUNTING > + select HAVE_KPROBES > + select HAVE_KPROBES_ON_FTRACE > + select HAVE_KRETPROBES > + select HAVE_KPROBES if !XIP_KERNEL > + select HAVE_KPROBES_ON_FTRACE if !XIP_KERNEL > + select HAVE_KRETPROBES if !XIP_KERNEL > + select HAVE_MOVE_PMD > + select HAVE_MOVE_PUD > select HAVE_PCI > select HAVE_PERF_EVENTS > select HAVE_PERF_REGS > @@@ -108,7 -104,7 +112,8 @@@ > select SYSCTL_EXCEPTION_TRACE > select THREAD_INFO_IN_TASK > select UACCESS_MEMCPY if !MMU > + select HAVE_ARCH_TRANSPARENT_HUGEPAGE if 64BIT && MMU > + select ZONE_DMA32 if 64BIT > > config ARCH_MMAP_RND_BITS_MIN > default 18 if 64BIT > ---------------------------------------------------------------- > Akira Tsukamoto (1): > riscv: __asm_copy_to-from_user: Optimize unaligned memory access and pipeline stall > > Alexandre Ghiti (6): > riscv: Remove CONFIG_PHYS_RAM_BASE_FIXED > riscv: Simplify xip and !xip kernel address conversion macros @Palmer: As said in the thread of this patchset multiple times, those 2 patches should not be merged as it assumes that the base DRAM address is always 0x8000_0000 for all rv64 platforms: I don't think it is true, is it? > riscv: Introduce set_kernel_memory helper > riscv: Map the kernel with correct permissions the first time > riscv: Introduce structure that group all variables regarding kernel mapping > riscv: Fix PTDUMP output now BPF region moved back to module region > > Bixuan Cui (1): > riscv: fix build error when CONFIG_SMP is disabled > > Christoph Hellwig (1): > riscv: pass the mm_struct to __sbi_tlb_flush_range > > Guo Ren (3): > riscv: Use global mappings for kernel pages > riscv: Cleanup unused functions > riscv: add ASID-based tlbflushing methods > > Jeff Xie (1): > riscv: ptrace: add argn syntax > > Jiapeng Chong (1): > riscv: xip: Fix duplicate included asm/pgtable.h > > Jisheng Zhang (7): > riscv: mremap speedup - enable HAVE_MOVE_PUD and HAVE_MOVE_PMD > riscv: mm: Remove setup_zero_page() > riscv: Optimize switch_mm by passing "cpu" to flush_icache_deferred() > riscv: Turn has_fpu into a static key if FPU=y > riscv: kprobes: Remove redundant kprobe_step_ctx > riscv: Add __init section marker to some functions again > riscv: mm: init: Consolidate vars, functions > > Kefeng Wang (5): > riscv: Move setup_bootmem into paging_init > riscv: mm: Drop redundant _sdata and _edata declaration > riscv: mm: Use better bitmap_zalloc() > riscv: Only initialize swiotlb when necessary > riscv: Add mem kernel parameter support > > Liu Shixin (1): > riscv: Enable KFENCE for riscv64 > > Nanyong Sun (5): > riscv: mm: add _PAGE_LEAF macro > riscv: mm: make pmd_bad() check leaf condition > riscv: mm: add param stride for __sbi_tlb_flush_range > riscv: mm: add THP support on 64-bit > riscv: mm: fix build errors caused by mk_pmd() > > Palmer Dabbelt (2): > RISC-V: Use asm-generic for {in,out}{bwlq} > Merge branch 'riscv-wx-mappings' into for-next > > Randy Dunlap (1): > riscv: TRANSPARENT_HUGEPAGE: depends on MMU > > Stanislaw Kardach (1): > riscv: enable generic PCI resource mapping > > Tong Tiangen (1): > riscv: add VMAP_STACK overflow detection > > Vitaly Wool (1): > riscv: fix typo in init.c > > arch/riscv/Kconfig | 12 +- > arch/riscv/include/asm/asm-prototypes.h | 3 + > arch/riscv/include/asm/io.h | 13 -- > arch/riscv/include/asm/kfence.h | 63 +++++++ > arch/riscv/include/asm/kprobes.h | 7 - > arch/riscv/include/asm/mmu_context.h | 2 + > arch/riscv/include/asm/page.h | 81 +++++---- > arch/riscv/include/asm/pci.h | 2 + > arch/riscv/include/asm/pgtable-64.h | 5 +- > arch/riscv/include/asm/pgtable-bits.h | 5 + > arch/riscv/include/asm/pgtable.h | 171 ++++++++++++++++++- > arch/riscv/include/asm/ptrace.h | 31 ++++ > arch/riscv/include/asm/sections.h | 17 ++ > arch/riscv/include/asm/set_memory.h | 24 ++- > arch/riscv/include/asm/switch_to.h | 11 +- > arch/riscv/include/asm/thread_info.h | 15 ++ > arch/riscv/include/asm/tlbflush.h | 5 + > arch/riscv/kernel/asm-offsets.c | 2 + > arch/riscv/kernel/cpufeature.c | 6 +- > arch/riscv/kernel/entry.S | 108 ++++++++++++ > arch/riscv/kernel/head.S | 4 +- > arch/riscv/kernel/kexec_relocate.S | 4 +- > arch/riscv/kernel/machine_kexec.c | 2 +- > arch/riscv/kernel/probes/kprobes.c | 40 +---- > arch/riscv/kernel/process.c | 2 +- > arch/riscv/kernel/setup.c | 18 +- > arch/riscv/kernel/signal.c | 4 +- > arch/riscv/kernel/traps.c | 35 ++++ > arch/riscv/kernel/vmlinux-xip.lds.S | 1 - > arch/riscv/kernel/vmlinux.lds.S | 2 +- > arch/riscv/lib/uaccess.S | 181 ++++++++++++++++---- > arch/riscv/mm/context.c | 14 +- > arch/riscv/mm/fault.c | 11 +- > arch/riscv/mm/init.c | 283 +++++++++++++++----------------- > arch/riscv/mm/physaddr.c | 2 +- > arch/riscv/mm/ptdump.c | 6 +- > arch/riscv/mm/tlbflush.c | 69 ++++++-- > 37 files changed, 901 insertions(+), 360 deletions(-) > create mode 100644 arch/riscv/include/asm/kfence.h > > _______________________________________________ > linux-riscv mailing list > linux-riscv@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-riscv >
On Fri, 09 Jul 2021 12:58:10 PDT (-0700), alex@ghiti.fr wrote: > > > Le 9/07/2021 à 16:53, Palmer Dabbelt a écrit : >> The following changes since commit 8a4102a0cf07cc76a18f373f6b49485258cc6af4: >> >> riscv: mm: Fix W+X mappings at boot (2021-06-01 21:15:09 -0700) >> >> are available in the Git repository at: >> >> git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git tags/riscv-for-linus-5.14-mw0 >> >> for you to fetch changes up to 1958e5aef5098e28b7d6e6a2972649901ebecace: >> >> riscv: xip: Fix duplicate included asm/pgtable.h (2021-07-06 16:17:40 -0700) >> >> ---------------------------------------------------------------- >> RISC-V Patches for the 5.14 Merge Window, Part 1 >> >> In addition to We have a handful of new features for 5.14: >> >> * Support for transparent huge pages. >> * Support for generic PCI resources mapping. >> * Support for the mem= kernel parameter. >> * Support for KFENCE. >> * A handful of fixes to avoid W+X mappings in the kernel. >> * Support for VMAP_STACK based overflow detection. >> * An optimized copy_{to,from}_user. >> ---------------------------------------------------------------- >> There are some Kconfig merge conflicts. They should be pretty >> straight-forward, but we do have a symbol out of order -- I thought I had a >> script to check for that, but I guess it doesn't work. I just sent out a patch >> to fix it up. >> >> diff --cc arch/riscv/Kconfig >> index 3590eb76000e,469a70bd8da6..d36f3c5029fd >> --- a/arch/riscv/Kconfig >> +++ b/arch/riscv/Kconfig >> @@@ -60,12 -61,11 +61,12 @@@ config RISC >> select GENERIC_TIME_VSYSCALL if MMU && 64BIT >> select HANDLE_DOMAIN_IRQ >> select HAVE_ARCH_AUDITSYSCALL >> - select HAVE_ARCH_JUMP_LABEL >> - select HAVE_ARCH_JUMP_LABEL_RELATIVE >> + select HAVE_ARCH_JUMP_LABEL if !XIP_KERNEL >> + select HAVE_ARCH_JUMP_LABEL_RELATIVE if !XIP_KERNEL >> select HAVE_ARCH_KASAN if MMU && 64BIT >> select HAVE_ARCH_KASAN_VMALLOC if MMU && 64BIT >> + select HAVE_ARCH_KFENCE if MMU && 64BIT >> - select HAVE_ARCH_KGDB >> + select HAVE_ARCH_KGDB if !XIP_KERNEL >> select HAVE_ARCH_KGDB_QXFER_PKT >> select HAVE_ARCH_MMAP_RND_BITS if MMU >> select HAVE_ARCH_SECCOMP_FILTER >> @@@ -81,11 -80,9 +82,14 @@@ >> select HAVE_GCC_PLUGINS >> select HAVE_GENERIC_VDSO if MMU && 64BIT >> select HAVE_IRQ_TIME_ACCOUNTING >> + select HAVE_KPROBES >> + select HAVE_KPROBES_ON_FTRACE >> + select HAVE_KRETPROBES >> + select HAVE_KPROBES if !XIP_KERNEL >> + select HAVE_KPROBES_ON_FTRACE if !XIP_KERNEL >> + select HAVE_KRETPROBES if !XIP_KERNEL >> + select HAVE_MOVE_PMD >> + select HAVE_MOVE_PUD >> select HAVE_PCI >> select HAVE_PERF_EVENTS >> select HAVE_PERF_REGS >> @@@ -108,7 -104,7 +112,8 @@@ >> select SYSCTL_EXCEPTION_TRACE >> select THREAD_INFO_IN_TASK >> select UACCESS_MEMCPY if !MMU >> + select HAVE_ARCH_TRANSPARENT_HUGEPAGE if 64BIT && MMU >> + select ZONE_DMA32 if 64BIT >> >> config ARCH_MMAP_RND_BITS_MIN >> default 18 if 64BIT >> ---------------------------------------------------------------- >> Akira Tsukamoto (1): >> riscv: __asm_copy_to-from_user: Optimize unaligned memory access and pipeline stall >> >> Alexandre Ghiti (6): >> riscv: Remove CONFIG_PHYS_RAM_BASE_FIXED >> riscv: Simplify xip and !xip kernel address conversion macros > > @Palmer: As said in the thread of this patchset multiple times, those 2 > patches should not be merged as it assumes that the base DRAM address is > always 0x8000_0000 for all rv64 platforms: I don't think it is true, is it? Sorry, I remember saying something about that but must have missed the resposeses. Do you have a pointer to the discussion? If this break stuff I'm happy to revert it, which can be done post-rc1. I just need to see what's actually broken first, because IIUC this was de-facto how things worked already. > >> riscv: Introduce set_kernel_memory helper >> riscv: Map the kernel with correct permissions the first time >> riscv: Introduce structure that group all variables regarding kernel mapping >> riscv: Fix PTDUMP output now BPF region moved back to module region >> >> Bixuan Cui (1): >> riscv: fix build error when CONFIG_SMP is disabled >> >> Christoph Hellwig (1): >> riscv: pass the mm_struct to __sbi_tlb_flush_range >> >> Guo Ren (3): >> riscv: Use global mappings for kernel pages >> riscv: Cleanup unused functions >> riscv: add ASID-based tlbflushing methods >> >> Jeff Xie (1): >> riscv: ptrace: add argn syntax >> >> Jiapeng Chong (1): >> riscv: xip: Fix duplicate included asm/pgtable.h >> >> Jisheng Zhang (7): >> riscv: mremap speedup - enable HAVE_MOVE_PUD and HAVE_MOVE_PMD >> riscv: mm: Remove setup_zero_page() >> riscv: Optimize switch_mm by passing "cpu" to flush_icache_deferred() >> riscv: Turn has_fpu into a static key if FPU=y >> riscv: kprobes: Remove redundant kprobe_step_ctx >> riscv: Add __init section marker to some functions again >> riscv: mm: init: Consolidate vars, functions >> >> Kefeng Wang (5): >> riscv: Move setup_bootmem into paging_init >> riscv: mm: Drop redundant _sdata and _edata declaration >> riscv: mm: Use better bitmap_zalloc() >> riscv: Only initialize swiotlb when necessary >> riscv: Add mem kernel parameter support >> >> Liu Shixin (1): >> riscv: Enable KFENCE for riscv64 >> >> Nanyong Sun (5): >> riscv: mm: add _PAGE_LEAF macro >> riscv: mm: make pmd_bad() check leaf condition >> riscv: mm: add param stride for __sbi_tlb_flush_range >> riscv: mm: add THP support on 64-bit >> riscv: mm: fix build errors caused by mk_pmd() >> >> Palmer Dabbelt (2): >> RISC-V: Use asm-generic for {in,out}{bwlq} >> Merge branch 'riscv-wx-mappings' into for-next >> >> Randy Dunlap (1): >> riscv: TRANSPARENT_HUGEPAGE: depends on MMU >> >> Stanislaw Kardach (1): >> riscv: enable generic PCI resource mapping >> >> Tong Tiangen (1): >> riscv: add VMAP_STACK overflow detection >> >> Vitaly Wool (1): >> riscv: fix typo in init.c >> >> arch/riscv/Kconfig | 12 +- >> arch/riscv/include/asm/asm-prototypes.h | 3 + >> arch/riscv/include/asm/io.h | 13 -- >> arch/riscv/include/asm/kfence.h | 63 +++++++ >> arch/riscv/include/asm/kprobes.h | 7 - >> arch/riscv/include/asm/mmu_context.h | 2 + >> arch/riscv/include/asm/page.h | 81 +++++---- >> arch/riscv/include/asm/pci.h | 2 + >> arch/riscv/include/asm/pgtable-64.h | 5 +- >> arch/riscv/include/asm/pgtable-bits.h | 5 + >> arch/riscv/include/asm/pgtable.h | 171 ++++++++++++++++++- >> arch/riscv/include/asm/ptrace.h | 31 ++++ >> arch/riscv/include/asm/sections.h | 17 ++ >> arch/riscv/include/asm/set_memory.h | 24 ++- >> arch/riscv/include/asm/switch_to.h | 11 +- >> arch/riscv/include/asm/thread_info.h | 15 ++ >> arch/riscv/include/asm/tlbflush.h | 5 + >> arch/riscv/kernel/asm-offsets.c | 2 + >> arch/riscv/kernel/cpufeature.c | 6 +- >> arch/riscv/kernel/entry.S | 108 ++++++++++++ >> arch/riscv/kernel/head.S | 4 +- >> arch/riscv/kernel/kexec_relocate.S | 4 +- >> arch/riscv/kernel/machine_kexec.c | 2 +- >> arch/riscv/kernel/probes/kprobes.c | 40 +---- >> arch/riscv/kernel/process.c | 2 +- >> arch/riscv/kernel/setup.c | 18 +- >> arch/riscv/kernel/signal.c | 4 +- >> arch/riscv/kernel/traps.c | 35 ++++ >> arch/riscv/kernel/vmlinux-xip.lds.S | 1 - >> arch/riscv/kernel/vmlinux.lds.S | 2 +- >> arch/riscv/lib/uaccess.S | 181 ++++++++++++++++---- >> arch/riscv/mm/context.c | 14 +- >> arch/riscv/mm/fault.c | 11 +- >> arch/riscv/mm/init.c | 283 +++++++++++++++----------------- >> arch/riscv/mm/physaddr.c | 2 +- >> arch/riscv/mm/ptdump.c | 6 +- >> arch/riscv/mm/tlbflush.c | 69 ++++++-- >> 37 files changed, 901 insertions(+), 360 deletions(-) >> create mode 100644 arch/riscv/include/asm/kfence.h >> >> _______________________________________________ >> linux-riscv mailing list >> linux-riscv@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-riscv >>
Hi Palmer, Le 11/07/2021 à 20:45, Palmer Dabbelt a écrit : > On Fri, 09 Jul 2021 12:58:10 PDT (-0700), alex@ghiti.fr wrote: >> >> >> Le 9/07/2021 à 16:53, Palmer Dabbelt a écrit : >>> The following changes since commit >>> 8a4102a0cf07cc76a18f373f6b49485258cc6af4: >>> >>> riscv: mm: Fix W+X mappings at boot (2021-06-01 21:15:09 -0700) >>> >>> are available in the Git repository at: >>> >>> git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git >>> tags/riscv-for-linus-5.14-mw0 >>> >>> for you to fetch changes up to 1958e5aef5098e28b7d6e6a2972649901ebecace: >>> >>> riscv: xip: Fix duplicate included asm/pgtable.h (2021-07-06 >>> 16:17:40 -0700) >>> >>> ---------------------------------------------------------------- >>> RISC-V Patches for the 5.14 Merge Window, Part 1 >>> >>> In addition to We have a handful of new features for 5.14: >>> >>> * Support for transparent huge pages. >>> * Support for generic PCI resources mapping. >>> * Support for the mem= kernel parameter. >>> * Support for KFENCE. >>> * A handful of fixes to avoid W+X mappings in the kernel. >>> * Support for VMAP_STACK based overflow detection. >>> * An optimized copy_{to,from}_user. >>> ---------------------------------------------------------------- >>> There are some Kconfig merge conflicts. They should be pretty >>> straight-forward, but we do have a symbol out of order -- I thought I >>> had a >>> script to check for that, but I guess it doesn't work. I just sent >>> out a patch >>> to fix it up. >>> >>> diff --cc arch/riscv/Kconfig >>> index 3590eb76000e,469a70bd8da6..d36f3c5029fd >>> --- a/arch/riscv/Kconfig >>> +++ b/arch/riscv/Kconfig >>> @@@ -60,12 -61,11 +61,12 @@@ config RISC >>> select GENERIC_TIME_VSYSCALL if MMU && 64BIT >>> select HANDLE_DOMAIN_IRQ >>> select HAVE_ARCH_AUDITSYSCALL >>> - select HAVE_ARCH_JUMP_LABEL >>> - select HAVE_ARCH_JUMP_LABEL_RELATIVE >>> + select HAVE_ARCH_JUMP_LABEL if !XIP_KERNEL >>> + select HAVE_ARCH_JUMP_LABEL_RELATIVE if !XIP_KERNEL >>> select HAVE_ARCH_KASAN if MMU && 64BIT >>> select HAVE_ARCH_KASAN_VMALLOC if MMU && 64BIT >>> + select HAVE_ARCH_KFENCE if MMU && 64BIT >>> - select HAVE_ARCH_KGDB >>> + select HAVE_ARCH_KGDB if !XIP_KERNEL >>> select HAVE_ARCH_KGDB_QXFER_PKT >>> select HAVE_ARCH_MMAP_RND_BITS if MMU >>> select HAVE_ARCH_SECCOMP_FILTER >>> @@@ -81,11 -80,9 +82,14 @@@ >>> select HAVE_GCC_PLUGINS >>> select HAVE_GENERIC_VDSO if MMU && 64BIT >>> select HAVE_IRQ_TIME_ACCOUNTING >>> + select HAVE_KPROBES >>> + select HAVE_KPROBES_ON_FTRACE >>> + select HAVE_KRETPROBES >>> + select HAVE_KPROBES if !XIP_KERNEL >>> + select HAVE_KPROBES_ON_FTRACE if !XIP_KERNEL >>> + select HAVE_KRETPROBES if !XIP_KERNEL >>> + select HAVE_MOVE_PMD >>> + select HAVE_MOVE_PUD >>> select HAVE_PCI >>> select HAVE_PERF_EVENTS >>> select HAVE_PERF_REGS >>> @@@ -108,7 -104,7 +112,8 @@@ >>> select SYSCTL_EXCEPTION_TRACE >>> select THREAD_INFO_IN_TASK >>> select UACCESS_MEMCPY if !MMU >>> + select HAVE_ARCH_TRANSPARENT_HUGEPAGE if 64BIT && MMU >>> + select ZONE_DMA32 if 64BIT >>> >>> config ARCH_MMAP_RND_BITS_MIN >>> default 18 if 64BIT >>> ---------------------------------------------------------------- >>> Akira Tsukamoto (1): >>> riscv: __asm_copy_to-from_user: Optimize unaligned memory >>> access and pipeline stall >>> >>> Alexandre Ghiti (6): >>> riscv: Remove CONFIG_PHYS_RAM_BASE_FIXED >>> riscv: Simplify xip and !xip kernel address conversion macros >> >> @Palmer: As said in the thread of this patchset multiple times, those 2 >> patches should not be merged as it assumes that the base DRAM address is >> always 0x8000_0000 for all rv64 platforms: I don't think it is true, >> is it? > > Sorry, I remember saying something about that but must have missed the > resposeses. Do you have a pointer to the discussion? If this break > stuff I'm happy to revert it, which can be done post-rc1. I just need > to see what's actually broken first, because IIUC this was de-facto how > things worked already. > Really sorry about my response delay. The thing is that removing CONFIG_PHYS_RAM_BASE_FIXED defines CONFIG_PHYS_RAM_BASE to 0x8000_0000 for all rv64 chips, but I believe this is implementation specific: for now, this base address was passed into the device tree, and here it makes this value static. This issue with my patch was originally pointed by Jisheng, Geert and Arnd. If this is not a problem and you have a pointer to a document that specifies this, I would be very happy to have the link :) Thanks and again sorry about my response delay, Alex >> >>> riscv: Introduce set_kernel_memory helper >>> riscv: Map the kernel with correct permissions the first time >>> riscv: Introduce structure that group all variables regarding >>> kernel mapping >>> riscv: Fix PTDUMP output now BPF region moved back to module >>> region >>> >>> Bixuan Cui (1): >>> riscv: fix build error when CONFIG_SMP is disabled >>> >>> Christoph Hellwig (1): >>> riscv: pass the mm_struct to __sbi_tlb_flush_range >>> >>> Guo Ren (3): >>> riscv: Use global mappings for kernel pages >>> riscv: Cleanup unused functions >>> riscv: add ASID-based tlbflushing methods >>> >>> Jeff Xie (1): >>> riscv: ptrace: add argn syntax >>> >>> Jiapeng Chong (1): >>> riscv: xip: Fix duplicate included asm/pgtable.h >>> >>> Jisheng Zhang (7): >>> riscv: mremap speedup - enable HAVE_MOVE_PUD and HAVE_MOVE_PMD >>> riscv: mm: Remove setup_zero_page() >>> riscv: Optimize switch_mm by passing "cpu" to >>> flush_icache_deferred() >>> riscv: Turn has_fpu into a static key if FPU=y >>> riscv: kprobes: Remove redundant kprobe_step_ctx >>> riscv: Add __init section marker to some functions again >>> riscv: mm: init: Consolidate vars, functions >>> >>> Kefeng Wang (5): >>> riscv: Move setup_bootmem into paging_init >>> riscv: mm: Drop redundant _sdata and _edata declaration >>> riscv: mm: Use better bitmap_zalloc() >>> riscv: Only initialize swiotlb when necessary >>> riscv: Add mem kernel parameter support >>> >>> Liu Shixin (1): >>> riscv: Enable KFENCE for riscv64 >>> >>> Nanyong Sun (5): >>> riscv: mm: add _PAGE_LEAF macro >>> riscv: mm: make pmd_bad() check leaf condition >>> riscv: mm: add param stride for __sbi_tlb_flush_range >>> riscv: mm: add THP support on 64-bit >>> riscv: mm: fix build errors caused by mk_pmd() >>> >>> Palmer Dabbelt (2): >>> RISC-V: Use asm-generic for {in,out}{bwlq} >>> Merge branch 'riscv-wx-mappings' into for-next >>> >>> Randy Dunlap (1): >>> riscv: TRANSPARENT_HUGEPAGE: depends on MMU >>> >>> Stanislaw Kardach (1): >>> riscv: enable generic PCI resource mapping >>> >>> Tong Tiangen (1): >>> riscv: add VMAP_STACK overflow detection >>> >>> Vitaly Wool (1): >>> riscv: fix typo in init.c >>> >>> arch/riscv/Kconfig | 12 +- >>> arch/riscv/include/asm/asm-prototypes.h | 3 + >>> arch/riscv/include/asm/io.h | 13 -- >>> arch/riscv/include/asm/kfence.h | 63 +++++++ >>> arch/riscv/include/asm/kprobes.h | 7 - >>> arch/riscv/include/asm/mmu_context.h | 2 + >>> arch/riscv/include/asm/page.h | 81 +++++---- >>> arch/riscv/include/asm/pci.h | 2 + >>> arch/riscv/include/asm/pgtable-64.h | 5 +- >>> arch/riscv/include/asm/pgtable-bits.h | 5 + >>> arch/riscv/include/asm/pgtable.h | 171 ++++++++++++++++++- >>> arch/riscv/include/asm/ptrace.h | 31 ++++ >>> arch/riscv/include/asm/sections.h | 17 ++ >>> arch/riscv/include/asm/set_memory.h | 24 ++- >>> arch/riscv/include/asm/switch_to.h | 11 +- >>> arch/riscv/include/asm/thread_info.h | 15 ++ >>> arch/riscv/include/asm/tlbflush.h | 5 + >>> arch/riscv/kernel/asm-offsets.c | 2 + >>> arch/riscv/kernel/cpufeature.c | 6 +- >>> arch/riscv/kernel/entry.S | 108 ++++++++++++ >>> arch/riscv/kernel/head.S | 4 +- >>> arch/riscv/kernel/kexec_relocate.S | 4 +- >>> arch/riscv/kernel/machine_kexec.c | 2 +- >>> arch/riscv/kernel/probes/kprobes.c | 40 +---- >>> arch/riscv/kernel/process.c | 2 +- >>> arch/riscv/kernel/setup.c | 18 +- >>> arch/riscv/kernel/signal.c | 4 +- >>> arch/riscv/kernel/traps.c | 35 ++++ >>> arch/riscv/kernel/vmlinux-xip.lds.S | 1 - >>> arch/riscv/kernel/vmlinux.lds.S | 2 +- >>> arch/riscv/lib/uaccess.S | 181 ++++++++++++++++---- >>> arch/riscv/mm/context.c | 14 +- >>> arch/riscv/mm/fault.c | 11 +- >>> arch/riscv/mm/init.c | 283 >>> +++++++++++++++----------------- >>> arch/riscv/mm/physaddr.c | 2 +- >>> arch/riscv/mm/ptdump.c | 6 +- >>> arch/riscv/mm/tlbflush.c | 69 ++++++-- >>> 37 files changed, 901 insertions(+), 360 deletions(-) >>> create mode 100644 arch/riscv/include/asm/kfence.h >>> >>> _______________________________________________ >>> linux-riscv mailing list >>> linux-riscv@lists.infradead.org >>> http://lists.infradead.org/mailman/listinfo/linux-riscv >>> > > _______________________________________________ > linux-riscv mailing list > linux-riscv@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-riscv
Hi Palmer, On Fri, Jul 9, 2021 at 10:54 PM Palmer Dabbelt <palmer@dabbelt.com> wrote: > > The following changes since commit 8a4102a0cf07cc76a18f373f6b49485258cc6af4: > > riscv: mm: Fix W+X mappings at boot (2021-06-01 21:15:09 -0700) > > are available in the Git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git tags/riscv-for-linus-5.14-mw0 > > for you to fetch changes up to 1958e5aef5098e28b7d6e6a2972649901ebecace: > > riscv: xip: Fix duplicate included asm/pgtable.h (2021-07-06 16:17:40 -0700) > > ---------------------------------------------------------------- > RISC-V Patches for the 5.14 Merge Window, Part 1 > > In addition to We have a handful of new features for 5.14: > > * Support for transparent huge pages. > * Support for generic PCI resources mapping. > * Support for the mem= kernel parameter. > * Support for KFENCE. > * A handful of fixes to avoid W+X mappings in the kernel. > * Support for VMAP_STACK based overflow detection. > * An optimized copy_{to,from}_user. > ---------------------------------------------------------------- > There are some Kconfig merge conflicts. They should be pretty > straight-forward, but we do have a symbol out of order -- I thought I had a > script to check for that, but I guess it doesn't work. I just sent out a patch > to fix it up. > > diff --cc arch/riscv/Kconfig > index 3590eb76000e,469a70bd8da6..d36f3c5029fd > --- a/arch/riscv/Kconfig > +++ b/arch/riscv/Kconfig > @@@ -60,12 -61,11 +61,12 @@@ config RISC > select GENERIC_TIME_VSYSCALL if MMU && 64BIT > select HANDLE_DOMAIN_IRQ > select HAVE_ARCH_AUDITSYSCALL > - select HAVE_ARCH_JUMP_LABEL > - select HAVE_ARCH_JUMP_LABEL_RELATIVE > + select HAVE_ARCH_JUMP_LABEL if !XIP_KERNEL > + select HAVE_ARCH_JUMP_LABEL_RELATIVE if !XIP_KERNEL > select HAVE_ARCH_KASAN if MMU && 64BIT > select HAVE_ARCH_KASAN_VMALLOC if MMU && 64BIT > + select HAVE_ARCH_KFENCE if MMU && 64BIT > - select HAVE_ARCH_KGDB > + select HAVE_ARCH_KGDB if !XIP_KERNEL > select HAVE_ARCH_KGDB_QXFER_PKT > select HAVE_ARCH_MMAP_RND_BITS if MMU > select HAVE_ARCH_SECCOMP_FILTER > @@@ -81,11 -80,9 +82,14 @@@ > select HAVE_GCC_PLUGINS > select HAVE_GENERIC_VDSO if MMU && 64BIT > select HAVE_IRQ_TIME_ACCOUNTING > + select HAVE_KPROBES > + select HAVE_KPROBES_ON_FTRACE > + select HAVE_KRETPROBES > + select HAVE_KPROBES if !XIP_KERNEL > + select HAVE_KPROBES_ON_FTRACE if !XIP_KERNEL > + select HAVE_KRETPROBES if !XIP_KERNEL > + select HAVE_MOVE_PMD > + select HAVE_MOVE_PUD > select HAVE_PCI > select HAVE_PERF_EVENTS > select HAVE_PERF_REGS > @@@ -108,7 -104,7 +112,8 @@@ > select SYSCTL_EXCEPTION_TRACE > select THREAD_INFO_IN_TASK > select UACCESS_MEMCPY if !MMU > + select HAVE_ARCH_TRANSPARENT_HUGEPAGE if 64BIT && MMU > + select ZONE_DMA32 if 64BIT > > config ARCH_MMAP_RND_BITS_MIN > default 18 if 64BIT > ---------------------------------------------------------------- > Akira Tsukamoto (1): > riscv: __asm_copy_to-from_user: Optimize unaligned memory access and pipeline stall > > Alexandre Ghiti (6): > riscv: Remove CONFIG_PHYS_RAM_BASE_FIXED > riscv: Simplify xip and !xip kernel address conversion macros > riscv: Introduce set_kernel_memory helper > riscv: Map the kernel with correct permissions the first time > riscv: Introduce structure that group all variables regarding kernel mapping > riscv: Fix PTDUMP output now BPF region moved back to module region > > Bixuan Cui (1): > riscv: fix build error when CONFIG_SMP is disabled > > Christoph Hellwig (1): > riscv: pass the mm_struct to __sbi_tlb_flush_range > > Guo Ren (3): > riscv: Use global mappings for kernel pages > riscv: Cleanup unused functions > riscv: add ASID-based tlbflushing methods > > Jeff Xie (1): > riscv: ptrace: add argn syntax > > Jiapeng Chong (1): > riscv: xip: Fix duplicate included asm/pgtable.h > > Jisheng Zhang (7): > riscv: mremap speedup - enable HAVE_MOVE_PUD and HAVE_MOVE_PMD > riscv: mm: Remove setup_zero_page() > riscv: Optimize switch_mm by passing "cpu" to flush_icache_deferred() > riscv: Turn has_fpu into a static key if FPU=y > riscv: kprobes: Remove redundant kprobe_step_ctx > riscv: Add __init section marker to some functions again > riscv: mm: init: Consolidate vars, functions > > Kefeng Wang (5): > riscv: Move setup_bootmem into paging_init > riscv: mm: Drop redundant _sdata and _edata declaration > riscv: mm: Use better bitmap_zalloc() > riscv: Only initialize swiotlb when necessary > riscv: Add mem kernel parameter support This patch should not be merged as it mixed two things into one patch, as discussed here: http://lists.infradead.org/pipermail/linux-riscv/2021-July/007486.html You remained silent on my "ping", sigh ... > > Liu Shixin (1): > riscv: Enable KFENCE for riscv64 > > Nanyong Sun (5): > riscv: mm: add _PAGE_LEAF macro > riscv: mm: make pmd_bad() check leaf condition > riscv: mm: add param stride for __sbi_tlb_flush_range > riscv: mm: add THP support on 64-bit > riscv: mm: fix build errors caused by mk_pmd() > > Palmer Dabbelt (2): > RISC-V: Use asm-generic for {in,out}{bwlq} > Merge branch 'riscv-wx-mappings' into for-next > > Randy Dunlap (1): > riscv: TRANSPARENT_HUGEPAGE: depends on MMU > > Stanislaw Kardach (1): > riscv: enable generic PCI resource mapping > > Tong Tiangen (1): > riscv: add VMAP_STACK overflow detection > > Vitaly Wool (1): > riscv: fix typo in init.c Regards, Bin
On Thu, 15 Jul 2021 09:11:14 +0200 Alex Ghiti <alex@ghiti.fr> wrote: > Hi Palmer, > > Le 11/07/2021 à 20:45, Palmer Dabbelt a écrit : > > On Fri, 09 Jul 2021 12:58:10 PDT (-0700), alex@ghiti.fr wrote: > >> > >> > >> Le 9/07/2021 à 16:53, Palmer Dabbelt a écrit : > >>> The following changes since commit > >>> 8a4102a0cf07cc76a18f373f6b49485258cc6af4: > >>> > >>> riscv: mm: Fix W+X mappings at boot (2021-06-01 21:15:09 -0700) > >>> > >>> are available in the Git repository at: > >>> > >>> git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git > >>> tags/riscv-for-linus-5.14-mw0 > >>> > >>> for you to fetch changes up to 1958e5aef5098e28b7d6e6a2972649901ebecace: > >>> > >>> riscv: xip: Fix duplicate included asm/pgtable.h (2021-07-06 > >>> 16:17:40 -0700) > >>> > >>> ---------------------------------------------------------------- > >>> RISC-V Patches for the 5.14 Merge Window, Part 1 > >>> > >>> In addition to We have a handful of new features for 5.14: > >>> > >>> * Support for transparent huge pages. > >>> * Support for generic PCI resources mapping. > >>> * Support for the mem= kernel parameter. > >>> * Support for KFENCE. > >>> * A handful of fixes to avoid W+X mappings in the kernel. > >>> * Support for VMAP_STACK based overflow detection. > >>> * An optimized copy_{to,from}_user. > >>> ---------------------------------------------------------------- > >>> There are some Kconfig merge conflicts. They should be pretty > >>> straight-forward, but we do have a symbol out of order -- I thought I > >>> had a > >>> script to check for that, but I guess it doesn't work. I just sent > >>> out a patch > >>> to fix it up. > >>> > >>> diff --cc arch/riscv/Kconfig > >>> index 3590eb76000e,469a70bd8da6..d36f3c5029fd > >>> --- a/arch/riscv/Kconfig > >>> +++ b/arch/riscv/Kconfig > >>> @@@ -60,12 -61,11 +61,12 @@@ config RISC > >>> select GENERIC_TIME_VSYSCALL if MMU && 64BIT > >>> select HANDLE_DOMAIN_IRQ > >>> select HAVE_ARCH_AUDITSYSCALL > >>> - select HAVE_ARCH_JUMP_LABEL > >>> - select HAVE_ARCH_JUMP_LABEL_RELATIVE > >>> + select HAVE_ARCH_JUMP_LABEL if !XIP_KERNEL > >>> + select HAVE_ARCH_JUMP_LABEL_RELATIVE if !XIP_KERNEL > >>> select HAVE_ARCH_KASAN if MMU && 64BIT > >>> select HAVE_ARCH_KASAN_VMALLOC if MMU && 64BIT > >>> + select HAVE_ARCH_KFENCE if MMU && 64BIT > >>> - select HAVE_ARCH_KGDB > >>> + select HAVE_ARCH_KGDB if !XIP_KERNEL > >>> select HAVE_ARCH_KGDB_QXFER_PKT > >>> select HAVE_ARCH_MMAP_RND_BITS if MMU > >>> select HAVE_ARCH_SECCOMP_FILTER > >>> @@@ -81,11 -80,9 +82,14 @@@ > >>> select HAVE_GCC_PLUGINS > >>> select HAVE_GENERIC_VDSO if MMU && 64BIT > >>> select HAVE_IRQ_TIME_ACCOUNTING > >>> + select HAVE_KPROBES > >>> + select HAVE_KPROBES_ON_FTRACE > >>> + select HAVE_KRETPROBES > >>> + select HAVE_KPROBES if !XIP_KERNEL > >>> + select HAVE_KPROBES_ON_FTRACE if !XIP_KERNEL > >>> + select HAVE_KRETPROBES if !XIP_KERNEL > >>> + select HAVE_MOVE_PMD > >>> + select HAVE_MOVE_PUD > >>> select HAVE_PCI > >>> select HAVE_PERF_EVENTS > >>> select HAVE_PERF_REGS > >>> @@@ -108,7 -104,7 +112,8 @@@ > >>> select SYSCTL_EXCEPTION_TRACE > >>> select THREAD_INFO_IN_TASK > >>> select UACCESS_MEMCPY if !MMU > >>> + select HAVE_ARCH_TRANSPARENT_HUGEPAGE if 64BIT && MMU > >>> + select ZONE_DMA32 if 64BIT > >>> > >>> config ARCH_MMAP_RND_BITS_MIN > >>> default 18 if 64BIT > >>> ---------------------------------------------------------------- > >>> Akira Tsukamoto (1): > >>> riscv: __asm_copy_to-from_user: Optimize unaligned memory > >>> access and pipeline stall > >>> > >>> Alexandre Ghiti (6): > >>> riscv: Remove CONFIG_PHYS_RAM_BASE_FIXED > >>> riscv: Simplify xip and !xip kernel address conversion macros > >> > >> @Palmer: As said in the thread of this patchset multiple times, those 2 > >> patches should not be merged as it assumes that the base DRAM address is > >> always 0x8000_0000 for all rv64 platforms: I don't think it is true, > >> is it? > > > > Sorry, I remember saying something about that but must have missed the > > resposeses. Do you have a pointer to the discussion? If this break > > stuff I'm happy to revert it, which can be done post-rc1. I just need > > to see what's actually broken first, because IIUC this was de-facto how > > things worked already. > > > > Really sorry about my response delay. > > The thing is that removing CONFIG_PHYS_RAM_BASE_FIXED defines > CONFIG_PHYS_RAM_BASE to 0x8000_0000 for all rv64 chips, but I believe > this is implementation specific: for now, this base address was passed > into the device tree, and here it makes this value static. > > This issue with my patch was originally pointed by Jisheng, Geert and Arnd. > > If this is not a problem and you have a pointer to a document that > specifies this, I would be very happy to have the link :) > > Thanks and again sorry about my response delay, > Hi Alex, Since the PR has been merged into linux-5.14-rc1, directly reverting seems not a good idea. IMHO, it's better to send patch(es) against current 5.14-rc1 tree. what do you think? Thanks
Hi Jisheng, Le 16/07/2021 à 14:57, Jisheng Zhang a écrit : > On Thu, 15 Jul 2021 09:11:14 +0200 > Alex Ghiti <alex@ghiti.fr> wrote: > >> Hi Palmer, >> >> Le 11/07/2021 à 20:45, Palmer Dabbelt a écrit : >>> On Fri, 09 Jul 2021 12:58:10 PDT (-0700), alex@ghiti.fr wrote: >>>> >>>> >>>> Le 9/07/2021 à 16:53, Palmer Dabbelt a écrit : >>>>> The following changes since commit >>>>> 8a4102a0cf07cc76a18f373f6b49485258cc6af4: >>>>> >>>>> riscv: mm: Fix W+X mappings at boot (2021-06-01 21:15:09 -0700) >>>>> >>>>> are available in the Git repository at: >>>>> >>>>> git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git >>>>> tags/riscv-for-linus-5.14-mw0 >>>>> >>>>> for you to fetch changes up to 1958e5aef5098e28b7d6e6a2972649901ebecace: >>>>> >>>>> riscv: xip: Fix duplicate included asm/pgtable.h (2021-07-06 >>>>> 16:17:40 -0700) >>>>> >>>>> ---------------------------------------------------------------- >>>>> RISC-V Patches for the 5.14 Merge Window, Part 1 >>>>> >>>>> In addition to We have a handful of new features for 5.14: >>>>> >>>>> * Support for transparent huge pages. >>>>> * Support for generic PCI resources mapping. >>>>> * Support for the mem= kernel parameter. >>>>> * Support for KFENCE. >>>>> * A handful of fixes to avoid W+X mappings in the kernel. >>>>> * Support for VMAP_STACK based overflow detection. >>>>> * An optimized copy_{to,from}_user. >>>>> ---------------------------------------------------------------- >>>>> There are some Kconfig merge conflicts. They should be pretty >>>>> straight-forward, but we do have a symbol out of order -- I thought I >>>>> had a >>>>> script to check for that, but I guess it doesn't work. I just sent >>>>> out a patch >>>>> to fix it up. >>>>> >>>>> diff --cc arch/riscv/Kconfig >>>>> index 3590eb76000e,469a70bd8da6..d36f3c5029fd >>>>> --- a/arch/riscv/Kconfig >>>>> +++ b/arch/riscv/Kconfig >>>>> @@@ -60,12 -61,11 +61,12 @@@ config RISC >>>>> select GENERIC_TIME_VSYSCALL if MMU && 64BIT >>>>> select HANDLE_DOMAIN_IRQ >>>>> select HAVE_ARCH_AUDITSYSCALL >>>>> - select HAVE_ARCH_JUMP_LABEL >>>>> - select HAVE_ARCH_JUMP_LABEL_RELATIVE >>>>> + select HAVE_ARCH_JUMP_LABEL if !XIP_KERNEL >>>>> + select HAVE_ARCH_JUMP_LABEL_RELATIVE if !XIP_KERNEL >>>>> select HAVE_ARCH_KASAN if MMU && 64BIT >>>>> select HAVE_ARCH_KASAN_VMALLOC if MMU && 64BIT >>>>> + select HAVE_ARCH_KFENCE if MMU && 64BIT >>>>> - select HAVE_ARCH_KGDB >>>>> + select HAVE_ARCH_KGDB if !XIP_KERNEL >>>>> select HAVE_ARCH_KGDB_QXFER_PKT >>>>> select HAVE_ARCH_MMAP_RND_BITS if MMU >>>>> select HAVE_ARCH_SECCOMP_FILTER >>>>> @@@ -81,11 -80,9 +82,14 @@@ >>>>> select HAVE_GCC_PLUGINS >>>>> select HAVE_GENERIC_VDSO if MMU && 64BIT >>>>> select HAVE_IRQ_TIME_ACCOUNTING >>>>> + select HAVE_KPROBES >>>>> + select HAVE_KPROBES_ON_FTRACE >>>>> + select HAVE_KRETPROBES >>>>> + select HAVE_KPROBES if !XIP_KERNEL >>>>> + select HAVE_KPROBES_ON_FTRACE if !XIP_KERNEL >>>>> + select HAVE_KRETPROBES if !XIP_KERNEL >>>>> + select HAVE_MOVE_PMD >>>>> + select HAVE_MOVE_PUD >>>>> select HAVE_PCI >>>>> select HAVE_PERF_EVENTS >>>>> select HAVE_PERF_REGS >>>>> @@@ -108,7 -104,7 +112,8 @@@ >>>>> select SYSCTL_EXCEPTION_TRACE >>>>> select THREAD_INFO_IN_TASK >>>>> select UACCESS_MEMCPY if !MMU >>>>> + select HAVE_ARCH_TRANSPARENT_HUGEPAGE if 64BIT && MMU >>>>> + select ZONE_DMA32 if 64BIT >>>>> >>>>> config ARCH_MMAP_RND_BITS_MIN >>>>> default 18 if 64BIT >>>>> ---------------------------------------------------------------- >>>>> Akira Tsukamoto (1): >>>>> riscv: __asm_copy_to-from_user: Optimize unaligned memory >>>>> access and pipeline stall >>>>> >>>>> Alexandre Ghiti (6): >>>>> riscv: Remove CONFIG_PHYS_RAM_BASE_FIXED >>>>> riscv: Simplify xip and !xip kernel address conversion macros >>>> >>>> @Palmer: As said in the thread of this patchset multiple times, those 2 >>>> patches should not be merged as it assumes that the base DRAM address is >>>> always 0x8000_0000 for all rv64 platforms: I don't think it is true, >>>> is it? >>> >>> Sorry, I remember saying something about that but must have missed the >>> resposeses. Do you have a pointer to the discussion? If this break >>> stuff I'm happy to revert it, which can be done post-rc1. I just need >>> to see what's actually broken first, because IIUC this was de-facto how >>> things worked already. >>> >> >> Really sorry about my response delay. >> >> The thing is that removing CONFIG_PHYS_RAM_BASE_FIXED defines >> CONFIG_PHYS_RAM_BASE to 0x8000_0000 for all rv64 chips, but I believe >> this is implementation specific: for now, this base address was passed >> into the device tree, and here it makes this value static. >> >> This issue with my patch was originally pointed by Jisheng, Geert and Arnd. >> >> If this is not a problem and you have a pointer to a document that >> specifies this, I would be very happy to have the link :) >> >> Thanks and again sorry about my response delay, >> > > Hi Alex, > > Since the PR has been merged into linux-5.14-rc1, directly reverting seems > not a good idea. IMHO, it's better to send patch(es) against current 5.14-rc1 > tree. what do you think? > > Thanks I have a patchset that fixes this problem cleanly, I have to check but I think it depends on an another larger patchset that cleans the code a lot, I'll see if I can extract only this part and submit it as a fix. I take a look and get back ASAP! Thanks, Alex > > > _______________________________________________ > linux-riscv mailing list > linux-riscv@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-riscv >
diff --cc arch/riscv/Kconfig index 3590eb76000e,469a70bd8da6..d36f3c5029fd --- a/arch/riscv/Kconfig