[PATCH v2 2/2] powerpc/vdso: Link with ld.lld when requested

2022-05-11 Thread Nathan Chancellor
See the LLVM commit below for the full details of '--ld-path'. Link: https://github.com/ClangBuiltLinux/linux/issues/774 Link: https://github.com/llvm/llvm-project/commit/1bc5c84710a8c73ef21295e63c19d10a8c71f2f5 Reviewed-by: Nick Desaulniers Reviewed-by: Alexey Kardashevskiy Tested-by: Alexey

[PATCH v2 1/2] powerpc/vdso: Remove unused ENTRY in linker scripts

2022-05-11 Thread Nathan Chancellor
x0 Remove ENTRY to remove the warning, as it is unnecessary for the vDSO to function correctly. Reviewed-by: Nick Desaulniers Reviewed-by: Alexey Kardashevskiy Tested-by: Alexey Kardashevskiy Signed-off-by: Nathan Chancellor --- arch/powerpc/kernel/vdso/vdso32.lds.S | 1 - arch/powerpc/kernel/vdso/

[PATCH v2 0/2] Link the PowerPC vDSO with ld.lld

2022-05-11 Thread Nathan Chancellor
x27;s initial stab at converting to $(CC) + $(LD)) [Nick]. * Add Nick and Alexey's tags. Nathan Chancellor (2): powerpc/vdso: Remove unused ENTRY in linker scripts powerpc/vdso: Link with ld.lld when requested arch/powerpc/kernel/vdso/Makefile | 1 + arch/powerpc/kernel/vdso/vdso32.l

Re: [PATCH 2/2] powerpc/vdso: Link with ld.lld when requested

2022-05-10 Thread Nathan Chancellor
On Tue, May 10, 2022 at 04:22:12PM +1000, Alexey Kardashevskiy wrote: > > > On 5/10/22 06:46, Nathan Chancellor wrote: > > The PowerPC vDSO is linked with $(CC) instead of $(LD), which means the > > default linker of the compiler is used instead of the linker request

Re: [PATCH 2/2] powerpc/vdso: Link with ld.lld when requested

2022-05-09 Thread Nathan Chancellor
On Mon, May 09, 2022 at 02:58:09PM -0700, Nick Desaulniers wrote: > On Mon, May 9, 2022 at 2:47 PM Nathan Chancellor wrote: > > > > On Mon, May 09, 2022 at 02:24:40PM -0700, Nick Desaulniers wrote: > > > On Mon, May 9, 2022 at 1:47 PM Nathan Chancellor > > > w

Re: [PATCH v4 00/14] kbuild: yet another series of cleanups (modpost, LTO, MODULE_REL_CRCS, export.h)

2022-05-09 Thread Nathan Chancellor
[resending due to mailing list bounces from a large plain text attachment...] On Mon, May 09, 2022 at 01:24:33PM +0900, Masahiro Yamada wrote: > On Mon, May 9, 2022 at 4:09 AM Masahiro Yamada wrote: > > > > This is the third batch of cleanups in this development cycle. > > > > Major changes in v4

Re: [PATCH 2/2] powerpc/vdso: Link with ld.lld when requested

2022-05-09 Thread Nathan Chancellor
On Mon, May 09, 2022 at 02:24:40PM -0700, Nick Desaulniers wrote: > On Mon, May 9, 2022 at 1:47 PM Nathan Chancellor wrote: > > > > The PowerPC vDSO is linked with $(CC) instead of $(LD), which means the > > default linker of the compiler is used instead of the linker request

[PATCH 2/2] powerpc/vdso: Link with ld.lld when requested

2022-05-09 Thread Nathan Chancellor
e full details of '--ld-path'. Link: https://github.com/ClangBuiltLinux/linux/issues/774 Link: https://github.com/llvm/llvm-project/commit/1bc5c84710a8c73ef21295e63c19d10a8c71f2f5 Signed-off-by: Nathan Chancellor --- arch/powerpc/kernel/vdso/Makefile | 1 + 1 file changed, 1 inser

[PATCH 1/2] powerpc/vdso: Remove unused ENTRY in linker scripts

2022-05-09 Thread Nathan Chancellor
x0 Remove ENTRY to remove the warning, as it is unnecessary for the vDSO to function correctly. Signed-off-by: Nathan Chancellor --- arch/powerpc/kernel/vdso/vdso32.lds.S | 1 - arch/powerpc/kernel/vdso/vdso64.lds.S | 1 - 2 files changed, 2 deletions(-) diff --git a/arch/powerpc/kernel/vdso/vdso3

[PATCH 0/2] Link the PowerPC vDSO with ld.lld

2022-05-09 Thread Nathan Chancellor
qhdcpwjmniqqpvwojb9mbukf3rr0bl+h+da...@mail.gmail.com/ Nathan Chancellor (2): powerpc/vdso: Remove unused ENTRY in linker scripts powerpc/vdso: Link with ld.lld when requested arch/powerpc/kernel/vdso/Makefile | 1 + arch/powerpc/kernel/vdso/vdso32.lds.S | 1 - arch/powerpc/kernel/vdso/vds

Re: [PATCH kernel] powerpc/llvm/lto: Allow LLVM LTO builds

2022-05-09 Thread Nathan Chancellor
Hi Alexey, On Mon, May 09, 2022 at 05:42:59PM +1000, Alexey Kardashevskiy wrote: > > > On 5/9/22 15:18, Alexey Kardashevskiy wrote: > > > > > > On 5/4/22 07:21, Nick Desaulniers wrote: > > > On Thu, Apr 28, 2022 at 11:46 PM Alexey Kardashevskiy > > > wrote: > > > > > > > > This enables LTO_C

Re: [PATCH v3 00/15] kbuild: yet another series of cleanups (modpost, LTO, MODULE_REL_CRCS)

2022-05-06 Thread Nathan Chancellor
ment error I merged this series into mainline and tested an Arch Linux x86_64 configuration and Fedora aarch64 configuration with ThinLTO and saw no new warnings or issues. Modules loaded just fine in QEMU for Arch Linux and I did not notice any boot issues or warnings. Tested-by: Nathan Chancellor Cheers, Nathan

Re: Apply d799769188529abc6cbf035a10087a51f7832b6b to 5.17 and 5.15?

2022-05-03 Thread Nathan Chancellor
On Thu, Apr 21, 2022 at 08:13:13AM -0700, Nathan Chancellor wrote: > On Thu, Apr 21, 2022 at 05:46:52PM +1000, Michael Ellerman wrote: > > Nathan Chancellor writes: > > > Hi Greg, Sasha, and Michael, > > > > > > Commit d79976918852 ("powerpc/64: Add UADDR6

Re: Apply d799769188529abc6cbf035a10087a51f7832b6b to 5.17 and 5.15?

2022-04-21 Thread Nathan Chancellor
On Thu, Apr 21, 2022 at 05:46:52PM +1000, Michael Ellerman wrote: > Nathan Chancellor writes: > > Hi Greg, Sasha, and Michael, > > > > Commit d79976918852 ("powerpc/64: Add UADDR64 relocation support") fixes > > a boot failure with CONFIG_RELOCATABLE=y kerne

Apply d799769188529abc6cbf035a10087a51f7832b6b to 5.17 and 5.15?

2022-04-19 Thread Nathan Chancellor
Hi Greg, Sasha, and Michael, Commit d79976918852 ("powerpc/64: Add UADDR64 relocation support") fixes a boot failure with CONFIG_RELOCATABLE=y kernels linked with recent versions of ld.lld [1]. Additionally, it resolves a separate boot failure that Paul Menzel reported [2] with ld.lld 13.0.0. Is t

Re: [PATCH] powerpc: Replace ppc64 DT_RELACOUNT usage with DT_RELASZ/24

2022-03-09 Thread Nathan Chancellor
gt; Link: https://github.com/ClangBuiltLinux/linux/issues/1581 > Reported-by: Nathan Chancellor > Signed-off-by: Fangrui Song Thank you a lot for the fix! I tested this with several different configurations with tip of tree LLVM and they all booted in QEMU. I also tested one configuration with G

Re: [PATCHv3] powerpc: mm: radix_tlb: rearrange the if-else block

2022-01-28 Thread Nathan Chancellor
= 0 > 2 errors generated. > > Rework the 'if (IS_ENABLE(CONFIG_TRANSPARENT_HUGEPAGE))' so hstart/hend > always gets initialized, this will silence the warnings. That will also > simplify the 'else' path. Clang is getting confused with these warnings, >

Re: [patch V3 28/35] PCI/MSI: Simplify pci_irq_get_affinity()

2021-12-18 Thread Nathan Chancellor
On Sat, Dec 18, 2021 at 11:25:14AM +0100, Thomas Gleixner wrote: > On Fri, Dec 17 2021 at 15:30, Nathan Chancellor wrote: > > On Fri, Dec 10, 2021 at 11:19:26PM +0100, Thomas Gleixner wrote: > > I just bisected a boot failure on my AMD test desktop to this patch as > > comm

Re: [patch V3 28/35] PCI/MSI: Simplify pci_irq_get_affinity()

2021-12-17 Thread Nathan Chancellor
Hi Thomas, On Fri, Dec 10, 2021 at 11:19:26PM +0100, Thomas Gleixner wrote: > From: Thomas Gleixner > > Replace open coded MSI descriptor chasing and use the proper accessor > functions instead. > > Signed-off-by: Thomas Gleixner > Reviewed-by: Greg Kroah-Hartman > Reviewed-by: Jason Gunthorp

Re: [PATCH] powerpc: platforms: cell: pervasive: fix clang -Wimplicit-fallthrough

2021-12-07 Thread Nathan Chancellor
> interrupts from sreset") > Reported-by: Naresh Kamboju > Signed-off-by: Anders Roxell Reviewed-by: Nathan Chancellor > --- > arch/powerpc/platforms/cell/pervasive.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/powerpc/platforms/cell/pervasiv

Re: [PATCH v5 5/5] powerpc/inst: Optimise copy_inst_from_kernel_nofault()

2021-12-06 Thread Nathan Chancellor
On Tue, Dec 07, 2021 at 05:45:08AM +, Christophe Leroy wrote: > > > Le 07/12/2021 à 05:48, Nathan Chancellor a écrit : > > On Tue, Dec 07, 2021 at 02:37:26PM +1100, Michael Ellerman wrote: > >> Bill Wendling writes: > >>> On Tue, Nov 30, 2021 at 10:38 AM

Re: [PATCH v5 5/5] powerpc/inst: Optimise copy_inst_from_kernel_nofault()

2021-12-06 Thread Nathan Chancellor
On Tue, Dec 07, 2021 at 02:37:26PM +1100, Michael Ellerman wrote: > Bill Wendling writes: > > On Tue, Nov 30, 2021 at 10:38 AM Bill Wendling wrote: > >> On Tue, Nov 30, 2021 at 10:17 AM Nathan Chancellor > >> wrote: > >> > On Tue, Nov 30, 2021 at 1

Re: [PATCH v5 5/5] powerpc/inst: Optimise copy_inst_from_kernel_nofault()

2021-11-30 Thread Nathan Chancellor
On Tue, Nov 30, 2021 at 10:25:43PM +1100, Michael Ellerman wrote: > Christophe Leroy writes: > > Le 29/11/2021 à 23:55, kernel test robot a écrit : > >> Hi Christophe, > >> > >> I love your patch! Perhaps something to improve: > >> > >> [auto build test WARNING on powerpc/next] > >> [also build

Re: [PATCH] powerpc: mm: radix_tlb: rearrange the if-else block

2021-11-26 Thread Nathan Chancellor
On Fri, Nov 26, 2021 at 02:59:29PM +0100, Arnd Bergmann wrote: > On Fri, Nov 26, 2021 at 2:43 PM Christophe Leroy > wrote: > > Le 25/11/2021 à 16:44, Anders Roxell a écrit : > > Can't you fix CLANG instead :) ? > > > > Or just add an else to the IS_ENABLED(CONFIG_TRANSPARENT_HUGEPAGE) that > > set

Re: [PATCH v3] lib/zlib_inflate/inffast: Check config in C to avoid unused function warning

2021-09-20 Thread Nathan Chancellor
on] > get_unaligned16(const unsigned short *p) > ^ > 1 warning generated. > > Fix it, by moving the check from the preprocessor to C, so the compiler > sees the use. > > Signed-off-by: Paul Menzel Reviewed-by: Nathan Chancellor Tested-by: Nathan Chancellor

Re: [PATCH] lib/zlib_inflate/inffast: Check config in C to avoid unused function warning

2021-09-16 Thread Nathan Chancellor
Hi Paul, On 9/16/2021 7:22 AM, Paul Menzel wrote: Building Linux for ppc64le with Ubuntu clang version 12.0.0-3ubuntu1~21.04.1 shows the warning below. arch/powerpc/boot/inffast.c:20:1: warning: unused function 'get_unaligned16' [-Wunused-function] get_unaligned16(const unsigned shor

Re: [PATCH] powerpc: clean up UPD_CONSTR

2021-09-14 Thread Nathan Chancellor
On 9/14/2021 9:17 AM, Nick Desaulniers wrote: UPD_CONSTR was previously a preprocessor define for an old GCC 4.9 inline asm bug with m<> constraints. Fixes: 6563139d90ad ("powerpc: remove GCC version check for UPD_CONSTR") Suggested-by: Nathan Chancellor Suggested-by:

Re: [PATCH 06/10] powerpc: remove GCC version check for UPD_CONSTR

2021-09-10 Thread Nathan Chancellor
On 9/10/2021 4:40 PM, Nick Desaulniers wrote: Now that GCC 5.1 is the minimum supported version, we can drop this workaround for older versions of GCC. This adversely affected clang, too. Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Segher Boessenkool Cc: Christophe

Re: [PATCH] powerpc/ptdump: Fix generic ptdump for 64-bit

2021-08-31 Thread Nathan Chancellor
ss immediately past the end of the kernel > page tables, to limit the page table walk to valid addresses. > > Fixes: e084728393a5 ("powerpc/ptdump: Convert powerpc to GENERIC_PTDUMP") > Reported-by: Nathan Chancellor > Signed-off-by: Michael Ellerman Tested-by: Nathan Chancel

Re: [PATCH] powerpc/bug: Cast to unsigned long before passing to inline asm

2021-08-31 Thread Nathan Chancellor
e register (32 or > 64-bit) we need to ensure the value passed to the inline asm has all > bits defined. So fix it by casting to long. > > We also already cast to long for the similar case in BUG_ENTRY(), which > it turns out was added to fix a similar bug in 2005 in commit > 32818

Re: [PATCH v4 4/4] powerpc/ptdump: Convert powerpc to GENERIC_PTDUMP

2021-08-29 Thread Nathan Chancellor
On Sun, Aug 29, 2021 at 09:11:47PM +0200, Christophe Leroy wrote: > Hi Nathan, > > Le 29/08/2021 à 20:55, Nathan Chancellor a écrit : > > Hi Christophe, > > > > On Thu, Jul 08, 2021 at 04:49:43PM +, Christophe Leroy wrote: > > > This patch co

Re: [PATCH v4 4/4] powerpc/ptdump: Convert powerpc to GENERIC_PTDUMP

2021-08-29 Thread Nathan Chancellor
Hi Christophe, On Thu, Jul 08, 2021 at 04:49:43PM +, Christophe Leroy wrote: > This patch converts powerpc to the generic PTDUMP implementation. > > Signed-off-by: Christophe Leroy This patch as commit e084728393a5 ("powerpc/ptdump: Convert powerpc to GENERIC_PTDUMP") in powerpc/next causes

Re: [PATCH v2 2/2] powerpc/bug: Provide better flexibility to WARN_ON/__WARN_FLAGS() with asm goto

2021-08-26 Thread Nathan Chancellor
On Thu, Aug 26, 2021 at 11:54:17AM -0700, Nathan Chancellor wrote: > On Thu, Aug 26, 2021 at 01:21:39PM +1000, Michael Ellerman wrote: > > Nathan Chancellor writes: > > > On Tue, Apr 13, 2021 at 04:38:10PM +, Christophe Leroy wrote: > > >> Using asm goto in __WAR

Re: [PATCH v2 2/2] powerpc/bug: Provide better flexibility to WARN_ON/__WARN_FLAGS() with asm goto

2021-08-26 Thread Nathan Chancellor
On Thu, Aug 26, 2021 at 01:21:39PM +1000, Michael Ellerman wrote: > Nathan Chancellor writes: > > On Tue, Apr 13, 2021 at 04:38:10PM +, Christophe Leroy wrote: > >> Using asm goto in __WARN_FLAGS() and WARN_ON() allows more > >> flexibility to GCC. > ..

Re: [PATCH v2 2/2] powerpc/bug: Provide better flexibility to WARN_ON/__WARN_FLAGS() with asm goto

2021-08-25 Thread Nathan Chancellor
Hi Christophe, On Tue, Apr 13, 2021 at 04:38:10PM +, Christophe Leroy wrote: > Using asm goto in __WARN_FLAGS() and WARN_ON() allows more > flexibility to GCC. > > For that add an entry to the exception table so that > program_check_exception() knowns where to resume execution > after a WARNI

Re: [PATCH 3/7] powerpc: replace cc-option-yn uses with cc-option

2021-08-16 Thread Nathan Chancellor
On 8/16/2021 5:21 PM, 'Nick Desaulniers' via Clang Built Linux wrote: cc-option-yn can be replaced with cc-option. ie. Checking for support: ifeq ($(call cc-option-yn,$(FLAG)),y) becomes: ifneq ($(call cc-option,$(FLAG)),) Checking for lack of support: ifeq ($(call cc-option-yn,$(FLAG)),n) be

[PATCH] powerpc/xive: Do not mark xive_request_ipi() as __init

2021-08-16 Thread Nathan Chancellor
warning. Fixes: cbc06f051c52 ("powerpc/xive: Do not skip CPU-less nodes when creating the IPIs") Signed-off-by: Nathan Chancellor --- arch/powerpc/sysdev/xive/common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/sysdev/xive/common.c b/arch/powerpc/s

Re: [PATCH v2] scripts/Makefile.clang: default to LLVM_IAS=1

2021-08-06 Thread Nathan Chancellor
t going to object much further than this. Regardless of that concern, this patch does what it says so: Reviewed-by: Nathan Chancellor > --- > Changes v1 -> v2: > * Drop "Currently" from Documentation/, as per Matthew. > * Drop Makefile and riscv Makefile, rebase on >

[PATCH] cpuidle: pseries: Mark pseries_idle_proble() as __init

2021-08-03 Thread Nathan Chancellor
("cpuidle: pseries: Add function to parse extended CEDE records") Signed-off-by: Nathan Chancellor --- drivers/cpuidle/cpuidle-pseries.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpuidle/cpuidle-pseries.c b/drivers/cpuidle/cpuidle-pseries.c index bba449

Re: [PATCH 3/3] isystem: delete global -isystem compile option

2021-08-02 Thread Nathan Chancellor
On 8/2/2021 1:32 PM, Alexey Dobriyan wrote: On Mon, Aug 02, 2021 at 11:18:32AM -0700, Nathan Chancellor wrote: On Sun, Aug 01, 2021 at 11:13:36PM +0300, Alexey Dobriyan wrote: In theory, it enables "leakage" of userspace headers into kernel which may present licensing problem. I

Re: [PATCH 3/3] isystem: delete global -isystem compile option

2021-08-02 Thread Nathan Chancellor
stem is selectively reenabled for some files. > > Not compile tested on hexagon. With this series on top of v5.14-rc4 and a tangential patch to fix another issue, ARCH=hexagon defconfig and allmodconfig show no issues. Tested-by: Nathan Chancellor # build (hexagon) > Compile tested on:

Re: [powerpc][next-20210727] Boot failure - kernel BUG at arch/powerpc/kernel/interrupt.c:98!

2021-07-29 Thread Nathan Chancellor
On 7/29/2021 9:35 AM, Konrad Rzeszutek Wilk wrote: On Thu, Jul 29, 2021 at 05:13:36PM +0100, Will Deacon wrote: On Wed, Jul 28, 2021 at 10:35:34AM -0700, Nathan Chancellor wrote: On Wed, Jul 28, 2021 at 01:31:06PM +0530, Sachin Sant wrote: next-20210723 was good. The boot failure seems to

Re: [powerpc][next-20210727] Boot failure - kernel BUG at arch/powerpc/kernel/interrupt.c:98!

2021-07-28 Thread Nathan Chancellor
On Wed, Jul 28, 2021 at 01:31:06PM +0530, Sachin Sant wrote: > linux-next fails to boot on Power server (POWER8/POWER9). Following traces > are seen during boot > > [0.010799] software IO TLB: tearing down default memory pool > [0.010805] [ cut here ] > [0.01080

Re: [PATCH v15 06/12] swiotlb: Use is_swiotlb_force_bounce for swiotlb data bouncing

2021-07-06 Thread Nathan Chancellor
Hi Will and Robin, On 7/6/2021 10:06 AM, Will Deacon wrote: On Tue, Jul 06, 2021 at 04:39:11PM +0100, Robin Murphy wrote: On 2021-07-06 15:05, Christoph Hellwig wrote: On Tue, Jul 06, 2021 at 03:01:04PM +0100, Robin Murphy wrote: FWIW I was pondering the question of whether to do something al

Re: [PATCH v15 06/12] swiotlb: Use is_swiotlb_force_bounce for swiotlb data bouncing

2021-07-02 Thread Nathan Chancellor
Hi Will and Robin, On Fri, Jul 02, 2021 at 04:13:50PM +0100, Robin Murphy wrote: > On 2021-07-02 14:58, Will Deacon wrote: > > Hi Nathan, > > > > On Thu, Jul 01, 2021 at 12:52:20AM -0700, Nathan Chancellor wrote: > > > On 7/1/2021 12:40 AM, Will Deacon wrote: >

Re: [PATCH v15 06/12] swiotlb: Use is_swiotlb_force_bounce for swiotlb data bouncing

2021-06-30 Thread Nathan Chancellor
Hi Will and Claire, On Wed, Jun 30, 2021 at 12:43:48PM +0100, Will Deacon wrote: > On Wed, Jun 30, 2021 at 05:17:27PM +0800, Claire Chang wrote: > > On Wed, Jun 30, 2021 at 9:43 AM Nathan Chancellor wrote: > > > > > > On Thu, Jun 24, 2021 at 11:55:20P

[PATCH] KVM: PPC: Book3S HV: Workaround high stack usage with clang

2021-06-21 Thread Nathan Chancellor
: https://github.com/ClangBuiltLinux/linux/issues/1292 Link: https://bugs.llvm.org/show_bug.cgi?id=49610 Link: https://lore.kernel.org/r/202104031853.vdt0qjqj-...@intel.com/ Link: https://gist.github.com/ba710e3703bf45043a31e2806c843ffd Reported-by: kernel test robot Signed-off-by: Nathan Chancellor -

Re: arch/powerpc/kvm/book3s_hv_nested.c:264:6: error: stack frame size of 2304 bytes in function 'kvmhv_enter_nested_guest'

2021-06-21 Thread Nathan Chancellor
On Mon, Jun 21, 2021 at 07:46:03PM +1000, Michael Ellerman wrote: > Nathan Chancellor writes: > > On 6/20/2021 4:59 PM, Nicholas Piggin wrote: > >> Excerpts from kernel test robot's message of April 3, 2021 8:47 pm: > >>> tree: > >>> https://g

Re: [PATCH v8 4/6] KVM: PPC: Book3S HV: Nested support in H_RPT_INVALIDATE

2021-06-21 Thread Nathan Chancellor
On Mon, Jun 21, 2021 at 02:20:01PM +0530, Bharata B Rao wrote: > Enable support for process-scoped invalidations from nested > guests and partition-scoped invalidations for nested guests. > > Process-scoped invalidations for any level of nested guests > are handled by implementing H_RPT_INVALIDATE

Re: arch/powerpc/kvm/book3s_hv_nested.c:264:6: error: stack frame size of 2304 bytes in function 'kvmhv_enter_nested_guest'

2021-06-20 Thread Nathan Chancellor
On 6/20/2021 4:59 PM, Nicholas Piggin wrote: Excerpts from kernel test robot's message of April 3, 2021 8:47 pm: tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: d93a0d43e3d0ba9e19387be4dae4a8d5b175a8d7 commit: 97e4910232fa1f81e806aa60c25a0450276d99a2 li

[PATCH] powerpc/barrier: Avoid collision with clang's __lwsync macro

2021-05-28 Thread Nathan Chancellor
ernel.org Link: https://github.com/ClangBuiltLinux/linux/issues/1386 Link: https://github.com/llvm/llvm-project/commit/62b5df7fe2b3fda1772befeda15598fbef96a614 Signed-off-by: Nathan Chancellor --- arch/powerpc/include/asm/barrier.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/pow

Re: [PATCH v5 3/9] mm/mremap: Use pmd/pud_poplulate to update page table entries

2021-05-19 Thread Nathan Chancellor
On 5/18/2021 9:46 PM, Aneesh Kumar K.V wrote: Nathan Chancellor writes: Hi Aneesh, On Thu, Apr 22, 2021 at 11:13:17AM +0530, Aneesh Kumar K.V wrote: pmd/pud_populate is the right interface to be used to set the respective page table entries. Some architectures like ppc64 do assume that

Re: [PATCH] powerpc: Kconfig: disable CONFIG_COMPAT for clang < 12

2021-05-18 Thread Nathan Chancellor
: Nathan Chancellor Signed-off-by: Nick Desaulniers Reviewed-by: Nathan Chancellor --- arch/powerpc/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig index ce3f59531b51..2a02784b7ef0 100644 --- a/arch/powerpc/Kconfig +++ b/arch/powerpc

Re: [PATCH v2] powerpc/powernv/pci: fix header guard

2021-05-18 Thread Nathan Chancellor
ed on upstream discussion, it's not clear yet that CONFIG_EEH should be enabled by default. For now, simply fix the -Wundef warning. Suggested-by: Nathan Chancellor Suggested-by: Joe Perches Link: https://github.com/ClangBuiltLinux/linux/issues/570 Link: https://lore.kerne

Re: [PATCH v5 3/9] mm/mremap: Use pmd/pud_poplulate to update page table entries

2021-05-18 Thread Nathan Chancellor
Hi Aneesh, On Thu, Apr 22, 2021 at 11:13:17AM +0530, Aneesh Kumar K.V wrote: > pmd/pud_populate is the right interface to be used to set the respective > page table entries. Some architectures like ppc64 do assume that > set_pmd/pud_at > can only be used to set a hugepage PTE. Since we are not se

Re: [PATCH] arm64: Define only {pud/pmd}_{set/clear}_huge when usefull

2021-05-14 Thread Nathan Chancellor
On Fri, May 14, 2021 at 02:42:00PM -0700, Andrew Morton wrote: > On Fri, 14 May 2021 11:08:53 + (UTC) Christophe Leroy > wrote: > > > When PUD and/or PMD are folded, those functions are useless > > and we now have a stub in linux/pgtable.h > > OK, help me out here please. What patch does t

Re: [PATCH kernel v3] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-13 Thread Nathan Chancellor
On 5/13/2021 4:59 AM, Alexey Kardashevskiy wrote: The $(CPP) (do only preprocessing) macro is already defined in Makefile. However POWERPC redefines it and adds $(KBUILD_CFLAGS) which results in flags duplication. Which is not a big deal by itself except for the flags which depend on other flags

Re: [PATCH kernel v2] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-11 Thread Nathan Chancellor
On 5/10/2021 9:48 PM, Alexey Kardashevskiy wrote: The $(CPP) (do only preprocessing) macro is already defined in Makefile. However POWERPC redefines it and adds $(KBUILD_CFLAGS) which results in flags duplication. Which is not a big deal by itself except for the flags which depend on other flags

Re: [PATCH] powerpc: Avoid clang uninitialized warning in __get_user_size_allowed

2021-04-28 Thread Nathan Chancellor
On Tue, Apr 27, 2021 at 07:05:12AM +0200, Christophe Leroy wrote: > > > Le 26/04/2021 à 22:35, Nathan Chancellor a écrit : > > Commit 9975f852ce1b ("powerpc/uaccess: Remove calls to __get_user_bad() > > and __put_user_bad()") switch to BUILD_BUG() in the d

[PATCH] powerpc: Avoid clang uninitialized warning in __get_user_size_allowed

2021-04-26 Thread Nathan Chancellor
k: https://github.com/ClangBuiltLinux/linux/issues/1359 Signed-off-by: Nathan Chancellor --- arch/powerpc/include/asm/uaccess.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/include/asm/uaccess.h b/arch/powerpc/include/asm/uaccess.h index a4e791bcd3fe..a09e4240c5

Re: [PATCH v7] powerpc/irq: Inline call_do_irq() and call_do_softirq()

2021-04-26 Thread Nathan Chancellor
On Sat, Mar 20, 2021 at 11:22:27PM +1100, Michael Ellerman wrote: > From: Christophe Leroy > > call_do_irq() and call_do_softirq() are simple enough to be > worth inlining. > > Inlining them avoids an mflr/mtlr pair plus a save/reload on stack. It > also allows GCC to keep the saved ksp_limit in

Re: Clang: powerpc: kvm/book3s_hv_nested.c:264:6: error: stack frame size of 2480 bytes in function 'kvmhv_enter_nested_guest'

2021-03-19 Thread Nathan Chancellor
Hi Naresh, Thank you for the testing and report. On Fri, Mar 19, 2021 at 12:50:42PM +0530, Naresh Kamboju wrote: > Linux mainline master build breaks for powerpc defconfig. > There are multiple errors / warnings with clang-12 and clang-11 and 10. > - powerpc (defconfig) with clang-12 > - powerp

[PATCH] powerpc/prom: Mark identical_pvr_fixup as __init

2021-03-02 Thread Nathan Chancellor
;). Fixes: 14b3d926a22b ("[POWERPC] 4xx: update 440EP(x)/440GR(x) identical PVR issue workaround") Link: https://github.com/ClangBuiltLinux/linux/issues/1316 Signed-off-by: Nathan Chancellor --- arch/powerpc/kernel/prom.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

[PATCH] powerpc/fadump: Mark fadump_calculate_reserve_size as __init

2021-03-02 Thread Nathan Chancellor
worthwhile since commit 889b3c1245de ("compiler: remove CONFIG_OPTIMIZE_INLINING entirely"). Fixes: 11550dc0a00b ("powerpc/fadump: reuse crashkernel parameter for fadump memory reservation") Link: https://github.com/ClangBuiltLinux/linux/issues/1300 Signed-off-by: Nathan Chancel

Re: [PATCH v2] powerpc: Handle .text.{hot,unlikely}.* in linker script

2021-01-16 Thread Nathan Chancellor
On Sat, Jan 16, 2021 at 12:44:38PM -0600, Segher Boessenkool wrote: > Hi! > > Very late of course, and the patch is fine, but: > > On Mon, Jan 04, 2021 at 01:59:53PM -0700, Nathan Chancellor wrote: > > Commit eff8728fe698 ("vmlinux.lds.h: Add PGO and AutoFDO

[PATCH v2] powerpc: Handle .text.{hot,unlikely}.* in linker script

2021-01-04 Thread Nathan Chancellor
e these sections like in the main linker script so there is no more build failure. [1]: https://reviews.llvm.org/D79600 [2]: https://reviews.llvm.org/D92493 Cc: sta...@vger.kernel.org Fixes: 83a092cf95f2 ("powerpc: Link warning for orphan sections") Link: https://github.com/ClangBuiltLinux

Re: [PATCH] powerpc: Handle .text.{hot,unlikely}.* in linker script

2021-01-04 Thread Nathan Chancellor
On Mon, Jan 04, 2021 at 09:55:20PM +0100, Sedat Dilek wrote: > On Mon, Jan 4, 2021 at 9:49 PM Nathan Chancellor > wrote: > > > > Commit eff8728fe698 ("vmlinux.lds.h: Add PGO and AutoFDO input > > sections") added ".text.unlikely.*" and ".text.ho

[PATCH] powerpc: Handle .text.{hot,unlikely}.* in linker script

2021-01-04 Thread Nathan Chancellor
e these sections like in the main linker script so there is no more build failure. Cc: sta...@vger.kernel.org Fixes: 83a092cf95f2 ("powerpc: Link warning for orphan sections") Link: https://github.com/ClangBuiltLinux/linux/issues/1218 Signed-off-by: Nathan Chancellor --- arch/powerpc/ke

Re: [PATCH] ASoC: fsl: fix -Wmaybe-uninitialized warning

2020-12-31 Thread Nathan Chancellor
gt; return ret; > > The driver returns -EINVAL for other broken DT properties, so do > it the same way here. > > Fixes: 6a5f850aa83a ("ASoC: fsl: Add imx-hdmi machine driver") > Signed-off-by: Arnd Bergmann Reviewed-by: Nathan Chancellor > --- > sou

Re: [PATCH 3/3] ibmvfc: use correlation token to tag commands

2020-12-21 Thread Nathan Chancellor
On Tue, Nov 17, 2020 at 12:50:31PM -0600, Tyrel Datwyler wrote: > The vfcFrame correlation field is 64bit handle that is intended to trace > I/O operations through both the client stack and VIOS stack when the > underlying physical FC adapter supports tagging. > > Tag vfcFrames with the associated

[PATCH v2 2/2] kbuild: Disable CONFIG_LD_ORPHAN_WARN for ld.lld 10.0.1

2020-11-19 Thread Nathan Chancellor
ClangBuiltLinux/linux/issues/1187 Link: https://github.com/ClangBuiltLinux/linux/issues/1193 Reported-by: Arvind Sankar Reported-by: kernelci.org bot Reported-by: Mark Brown Reviewed-by: Kees Cook Signed-off-by: Nathan Chancellor --- v1 -> v2: * Add condition as a depends on line (Kees Cook)

[PATCH v2 1/2] kbuild: Hoist '--orphan-handling' into Kconfig

2020-11-19 Thread Nathan Chancellor
esaulniers Signed-off-by: Nathan Chancellor --- v1 -> v2: * Change ifeq ($(CONFIG_LD_ORPHAN_WARN),y) to ifdef CONFIG_LD_ORPHAN_WARN to improve readability (Michael Ellerman) * Separate conditions for CONFIG_LD_ORPHAN warn to improve readability (Kees Cook) * Pick up tags from Kees,

Re: [PATCH 2/2] kbuild: Disable CONFIG_LD_ORPHAN_WARN for ld.lld 10.0.1

2020-11-17 Thread Nathan Chancellor
On Tue, Nov 17, 2020 at 01:51:43PM -0800, Kees Cook wrote: > On Fri, Nov 13, 2020 at 12:55:53PM -0700, Nathan Chancellor wrote: > > config LD_ORPHAN_WARN > > - def_bool ARCH_WANT_LD_ORPHAN_WARN && $(ld-option,--orphan-handling=warn) > > + def_bool ARCH_WANT_LD_OR

Re: [PATCH 2/2] kbuild: Disable CONFIG_LD_ORPHAN_WARN for ld.lld 10.0.1

2020-11-17 Thread Nathan Chancellor
On Tue, Nov 17, 2020 at 11:41:15AM -0800, Nick Desaulniers wrote: > On Fri, Nov 13, 2020 at 11:56 AM Nathan Chancellor > wrote: > > > > ld.lld 10.0.1 spews a bunch of various warnings about .rela sections, > > along with a few others. Newer versions of ld.lld do not have

Re: [PATCH 1/2] kbuild: Hoist '--orphan-handling' into Kconfig

2020-11-17 Thread Nathan Chancellor
On Tue, Nov 17, 2020 at 10:03:29PM +1100, Michael Ellerman wrote: > Nathan Chancellor writes: > > Currently, '--orphan-handling=warn' is spread out across four different > > architectures in their respective Makefiles, which makes it a little > > unruly to d

Re: [PATCH 3/3] powerpc: fix -Wimplicit-fallthrough

2020-11-16 Thread Nathan Chancellor
se cases. > > Link: https://github.com/ClangBuiltLinux/linux/issues/236 > Signed-off-by: Nick Desaulniers Reviewed-by: Nathan Chancellor Tested-by: Nathan Chancellor > --- > arch/powerpc/kernel/prom_init.c | 1 + > arch/powerpc/kernel/uprobes.c | 1 + > arch/powerpc/per

Re: [PATCH 2/3] Revert "lib: Revert use of fallthrough pseudo-keyword in lib/"

2020-11-16 Thread Nathan Chancellor
or lib/. > > Link: https://github.com/ClangBuiltLinux/linux/issues/236 > Signed-off-by: Nick Desaulniers Reviewed-by: Nathan Chancellor Tested-by: Nathan Chancellor > --- > lib/asn1_decoder.c | 4 ++-- > lib/assoc_array.c | 2 +- > lib/bootconfig.c

Re: [PATCH 1/3] powerpc: boot: include compiler_attributes.h

2020-11-16 Thread Nathan Chancellor
thub.com/ClangBuiltLinux/linux/issues/236 > Signed-off-by: Nick Desaulniers Reviewed-by: Nathan Chancellor Tested-by: Nathan Chancellor > --- > We could just `#include "include/linux/compiler_types.h"` in the few .c > sources used from lib/ (there are proper header guards in >

Re: [PATCH 1/2] kbuild: Hoist '--orphan-handling' into Kconfig

2020-11-16 Thread Nathan Chancellor
On Mon, Nov 16, 2020 at 05:41:58PM -0800, Nick Desaulniers wrote: > On Fri, Nov 13, 2020 at 11:56 AM Nathan Chancellor > wrote: > > > > Currently, '--orphan-handling=warn' is spread out across four different > > architectures in their respective Makefiles, whi

Re: Error: invalid switch -me200

2020-11-13 Thread Nathan Chancellor
On Fri, Nov 13, 2020 at 11:42:03AM -0800, Nick Desaulniers wrote: > + MPE, PPC > > On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor > wrote: > > > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > > On Thu, Nov 12, 2020 at 7:22 PM kernel test

[PATCH 1/2] kbuild: Hoist '--orphan-handling' into Kconfig

2020-11-13 Thread Nathan Chancellor
ally after all of the sections are specified and size asserted. A special thanks to Kees Cook for the help text on this config. Link: https://github.com/ClangBuiltLinux/linux/issues/1187 Signed-off-by: Nathan Chancellor --- Makefile | 6 ++ arch/Kconfig

[PATCH 2/2] kbuild: Disable CONFIG_LD_ORPHAN_WARN for ld.lld 10.0.1

2020-11-13 Thread Nathan Chancellor
ar Reported-by: kernelci.org bot Reported-by: Mark Brown Link: https://github.com/ClangBuiltLinux/linux/issues/1187 Link: https://github.com/ClangBuiltLinux/linux/issues/1193 Signed-off-by: Nathan Chancellor --- MAINTAINERS| 1 + init/Kconfig | 6 +- scripts/lld-

Re: [PATCH] treewide: Convert macro and uses of __section(foo) to __section("foo")

2020-10-26 Thread Nathan Chancellor
On Mon, Oct 26, 2020 at 12:36:52PM -0700, Guenter Roeck wrote: > On Wed, Oct 21, 2020 at 07:36:07PM -0700, Joe Perches wrote: > > Use a more generic form for __section that requires quotes to avoid > > complications with clang and gcc differences. > > > > Remove the quote operator # from compiler_

Re: [PATCH v4 13/13] mm/debug_vm_pgtable: Avoid none pte in pte_clear_test

2020-09-10 Thread Nathan Chancellor
On Wed, Sep 02, 2020 at 05:12:22PM +0530, Aneesh Kumar K.V wrote: > pte_clear_tests operate on an existing pte entry. Make sure that > is not a none pte entry. > > Signed-off-by: Aneesh Kumar K.V > --- > mm/debug_vm_pgtable.c | 7 --- > 1 file changed, 4 insertions(+), 3 deletions(-) > > di

Re: [PATCH 0/2] link vdso with linker

2020-09-01 Thread Nathan Chancellor
On Tue, Sep 01, 2020 at 03:25:21PM -0700, Nick Desaulniers wrote: > Kees Cook is working on series that adds --orphan-section=warn to arm, > arm64, and x86. I noticed that ppc vdso were still using cc-ldoption > for these which I removed. It seems this results in that flag being > silently droppe

Re: [GIT PULL] fallthrough pseudo-keyword macro conversions for 5.9-rc3

2020-08-24 Thread Nathan Chancellor
On Sun, Aug 23, 2020 at 10:48:41PM -0500, Gustavo A. R. Silva wrote: > Hi Linus, > > Not sure what the problem was with my pull-request for -rc2. So, I'm giving > this a second try because I think it is worth it. > > I have build-tested this patch on 10 different architectures: x86_64, i386, > ar

Re: [PATCH v2 15/16] powerpc/powernv/sriov: Make single PE mode a per-BAR setting

2020-08-02 Thread Nathan Chancellor
On Sun, Aug 02, 2020 at 11:12:23PM +1000, Michael Ellerman wrote: > Nathan Chancellor writes: > > On Wed, Jul 22, 2020 at 04:57:14PM +1000, Oliver O'Halloran wrote: > >> Using single PE BARs to map an SR-IOV BAR is really a choice about what > >> strategy to us

Re: [PATCH v2 15/16] powerpc/powernv/sriov: Make single PE mode a per-BAR setting

2020-07-31 Thread Nathan Chancellor
On Wed, Jul 22, 2020 at 04:57:14PM +1000, Oliver O'Halloran wrote: > Using single PE BARs to map an SR-IOV BAR is really a choice about what > strategy to use when mapping a BAR. It doesn't make much sense for this to > be a global setting since a device might have one large BAR which needs to > be

Re: [PATCH] powerpc/boot: Use address-of operator on section symbols

2020-07-18 Thread Nathan Chancellor
On Sat, Jul 18, 2020 at 09:50:50AM +0200, Geert Uytterhoeven wrote: > Hi Nathan, > > On Wed, Jun 24, 2020 at 6:02 AM Nathan Chancellor > wrote: > > arch/powerpc/boot/main.c:107:18: warning: array comparison always > > evaluates to a constant [-Wtautological-compare] >

[PATCH] powerpc/boot: Use address-of operator on section symbols

2020-06-23 Thread Nathan Chancellor
y with either clang/ld.lld or gcc/ld (tested with diff + objdump -Dr). Link: https://github.com/ClangBuiltLinux/linux/issues/212 Reported-by: Joel Stanley Signed-off-by: Nathan Chancellor --- arch/powerpc/boot/main.c | 4 ++-- arch/powerpc/boot/ps3.c | 2 +- 2 files changed, 3 insertions(+), 3 del

Re: [PATCH v5 01/13] powerpc: Remove Xilinx PPC405/PPC440 support

2020-06-17 Thread Nathan Chancellor
On Thu, Jun 18, 2020 at 10:48:21AM +1000, Michael Ellerman wrote: > Nick Desaulniers writes: > > On Wed, Jun 17, 2020 at 3:20 AM Michael Ellerman > > wrote: > >> Michael Ellerman writes: > >> > Michal Simek writes: > >> > >> > >> >> Or if bamboo requires uImage to be built by default you can

Re: [PATCH v5 01/13] powerpc: Remove Xilinx PPC405/PPC440 support

2020-06-16 Thread Nathan Chancellor
Hi Michal, On Tue, Jun 16, 2020 at 04:45:20PM +0200, Michal Simek wrote: > > > On 16. 06. 20 2:27, Nathan Chancellor wrote: > > On Thu, May 21, 2020 at 04:55:52PM +, Christophe Leroy wrote: > >> From: Michal Simek > >> > >> The latest Xilin

Re: [PATCH v5 01/13] powerpc: Remove Xilinx PPC405/PPC440 support

2020-06-15 Thread Nathan Chancellor
On Thu, May 21, 2020 at 04:55:52PM +, Christophe Leroy wrote: > From: Michal Simek > > The latest Xilinx design tools called ISE and EDK has been released in > October 2013. New tool doesn't support any PPC405/PPC440 new designs. > These platforms are no longer supported and tested. > > Powe

Re: [PATCH] media: omap3isp: Shuffle cacheflush.h and include mm.h

2020-05-27 Thread Nathan Chancellor
Hi Geert, On Wed, May 27, 2020 at 09:02:51AM +0200, Geert Uytterhoeven wrote: > Hi Nathan, > > CC Laurent > > On Wed, May 27, 2020 at 6:37 AM Nathan Chancellor > wrote: > > After mm.h was removed from the asm-generic version of cacheflush.h, > > s390 allyesconfig

[PATCH] media: omap3isp: Shuffle cacheflush.h and include mm.h

2020-05-26 Thread Nathan Chancellor
licitly in this file and shuffle cacheflush.h below it. Fixes: 19c0054597a0 ("asm-generic: don't include in cacheflush.h") Signed-off-by: Nathan Chancellor --- I am aware the fixes tag is kind of irrelevant because that SHA will change in the next linux-next revision and this will

[PATCH v2] powerpc/wii: Fix declaration made after definition

2020-05-26 Thread Nathan Chancellor
esaulniers Signed-off-by: Nathan Chancellor --- v1 -> v2: * s/is_machine/machine_is/ (Nick) * Add Nick's reviewed-by tag. arch/powerpc/platforms/embedded6xx/wii.c | 25 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/arch/powerpc/platforms/em

Re: Fwd: [CRON] Broken: ClangBuiltLinux/continuous-integration#1432 (master - 0aceafc)

2020-05-21 Thread Nathan Chancellor
On Thu, May 21, 2020 at 03:23:11PM -0700, Nick Desaulniers wrote: > On Thu, May 21, 2020 at 6:00 AM Michael Ellerman wrote: > > > > Nathan Chancellor writes: > > > On Tue, May 19, 2020 at 05:56:32PM -0700, 'Nick Desaulniers' via Clang > > > Built Lin

Re: Fwd: [CRON] Broken: ClangBuiltLinux/continuous-integration#1432 (master - 0aceafc)

2020-05-19 Thread Nathan Chancellor
On Tue, May 19, 2020 at 05:56:32PM -0700, 'Nick Desaulniers' via Clang Built Linux wrote: > Looks like our CI is still red from this: > > https://travis-ci.com/github/ClangBuiltLinux/continuous-integration/builds/166854584 > > Filing a bug to follow up on: > https://github.com/ClangBuiltLinux/li

[PATCH] input: i8042: Remove special PowerPC handling

2020-05-18 Thread Nathan Chancellor
ast 12 years, there is not anyone actually trying to use this driver so we can just remove this special walnut code and use the generic header so it builds for all configurations. Fixes: 917f0af9e5a9 ("powerpc: Remove arch/ppc and include/asm-ppc") Reported-by: kbuild test robot Signe

Re: [PATCH] powerpc/wii: Fix declaration made after definition

2020-05-18 Thread Nathan Chancellor
On Mon, Apr 13, 2020 at 12:06:45PM -0700, Nathan Chancellor wrote: > A 0day randconfig uncovered an error with clang, trimmed for brevity: > > arch/powerpc/platforms/embedded6xx/wii.c:195:7: error: attribute > declaration must precede definition [-Werror,-Wignored-attributes]

<    1   2   3   >