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

On Thu, 11 Aug 2022 at 11:41, Michael Ellerman wrote: > > Anders Roxell writes: > > Clang warns: > > > > arch/powerpc/mm/book3s64/radix_tlb.c:1191:23: error: variable 'hstart' is > > uninitialized when used here [-Werror,-Wuninitialized] > >

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

warnings. That will also simplify the 'else' path. Clang is getting confused with these warnings, but the warnings is a false-positive. Suggested-by: Arnd Bergmann Suggested-by: Nathan Chancellor Reviewed-by: Christophe Leroy Reviewed-by: Nathan Chancellor Signed-off-by: A

[PATCHv2 2/3] powerpc: fix build errors

s/PowerPC_002dPseudo.html#PowerPC_002dPseudo Cc: Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Reviewed-by: Segher Boessenkool Signed-off-by: Anders Roxell --- arch/powerpc/include/asm/io.h| 40 ++-- arch/powerpc/include/asm/uaccess.h | 3 ++

[PATCHv2 1/3] powerpc: lib: sstep: fix 'sthcx' instruction

emulation code") Reported-by: Arnd Bergmann Signed-off-by: Anders Roxell --- arch/powerpc/lib/sstep.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/lib/sstep.c b/arch/powerpc/lib/sstep.c index bd3734d5be89..d2d29243fa6d 100644 --- a/arch/powerpc/lib/sstep.c

[PATCHv2 3/3] powerpc: lib: sstep: fix build errors

re.org/binutils/docs/as/PowerPC_002dPseudo.html#PowerPC_002dPseudo Cc: Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Reviewed-by: Segher Boessenkool Signed-off-by: Anders Roxell --- arch/powerpc/lib/sstep.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/arch/powerp

Re: [PATCH 2/3] powerpc: fix build errors

On Thu, 24 Feb 2022 at 13:39, Michael Ellerman wrote: > > Hi Anders, Hi Michael, > > Thanks for these, just a few comments below ... I will resolve the comments below and resend a v2 shortly. Cheers, Anders > > Anders Roxell writes: > > Building tinyconfig with gc

[PATCH 2/3] powerpc: fix build errors

s/PowerPC_002dPseudo.html#PowerPC_002dPseudo Cc: Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Anders Roxell --- arch/powerpc/include/asm/io.h| 46 +++- arch/powerpc/include/asm/uaccess.h | 3 ++ arch/powerpc/platforms/powernv

[PATCH 1/3] powerpc: lib: sstep: fix 'sthcx' instruction

emulation code") Reported-by: Arnd Bergmann Signed-off-by: Anders Roxell --- arch/powerpc/lib/sstep.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/lib/sstep.c b/arch/powerpc/lib/sstep.c index bd3734d5be89..d2d29243fa6d 100644 --- a/arch/powerpc/lib/sstep.c

[PATCH 3/3] powerpc: lib: sstep: fix build errors

re.org/binutils/docs/as/PowerPC_002dPseudo.html#PowerPC_002dPseudo Cc: Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Anders Roxell --- arch/powerpc/lib/sstep.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/arch/powerpc/lib/sstep.c b/arch/powerpc/li

[PATCHv2] powerpc/lib/sstep: fix 'ptesync' build error

alyse_instr so it doesn't modify *regs"). Cc: sta...@vger.kernel.org # v4.14+ Fixes: 3cdfcbfd32b9 ("powerpc: Change analyse_instr so it doesn't modify *regs") Suggested-by: Arnd Bergmann Signed-off-by: Anders Roxell --- arch/powerpc/lib/sstep.c | 2 ++ 1 file changed, 2 insertions

[PATCH] powerpc/lib/sstep: fix 'ptesync' build error

r so it doesn't modify *regs"). Cc: sta...@vger.kernel.org # v4.14+ Fixes: 3cdfcbfd32b9 ("powerpc: Change analyse_instr so it doesn't modify *regs") Suggested-by: Arnd Bergmann Signed-off-by: Anders Roxell --- arch/powerpc/lib/sstep.c | 2 ++ 1 file changed, 2 insertions(+) diff

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

warnings. That will also simplify the 'else' path. Clang is getting confused with these warnings, but the warnings is a false-positive. Suggested-by: Arnd Bergmann Suggested-by: Nathan Chancellor Reviewed-by: Christophe Leroy Signed-off-by: Anders Roxell --- arch/powerpc/mm/book3s6

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

On Fri, 28 Jan 2022 at 11:14, Christophe Leroy wrote: > > > > Le 28/01/2022 à 11:08, Anders Roxell a écrit : > > Clang warns: > > > > arch/powerpc/mm/book3s64/radix_tlb.c:1191:23: error: variable 'hstart' is > > uniniti

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

warnings. That will also simplify the 'else' path. Clang is getting confused with these warnings, but the warnings is a false-positive. Suggested-by: Arnd Bergmann Suggested-by: Nathan Chancellor Signed-off-by: Anders Roxell --- arch/powerpc/mm/book3s64/radix_tlb.c | 9 +++-- 1 file

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

the warning. Fixes: 6e83985b0f6e ("powerpc/cbe: Do not process external or decremeter interrupts from sreset") Reported-by: Naresh Kamboju Signed-off-by: Anders Roxell --- arch/powerpc/platforms/cell/pervasive.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/platforms/ce

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

gs. That will also simplify the 'else' path. Clang is getting confused with these warnings, but the warnings is a false-positive. Suggested-by: Arnd Bergmann Signed-off-by: Anders Roxell --- arch/powerpc/mm/book3s64/radix_tlb.c | 31 +--- 1 file changed, 24

[PATCH] arm64: defconfig: enable modern virtio pci device

Since patch ("virtio-pci: introduce modern device module") got added it is not possible to boot a defconfig kernel in qemu with a virtio pci device. Add CONFIG_VIRTIO_PCI_MODERN=y fragment makes the kernel able to boot. Signed-off-by: Anders Roxell --- arch/arm/configs/multi_v7

Re: [PATCH v2 1/2] kbuild: introduce ccflags-remove-y and asflags-remove-y

09ded0 ("ftrace: do not trace library functions") > excluded too much. In later commit, I will try to remove ccflags-remove-y > from sub-directory Makefiles. > > Suggested-by: Sami Tolvanen > Signed-off-by: Masahiro Yamada > Acked-by: Steven Rostedt (VMware) >

Re: [PATCH] kbuild: introduce ccflags-remove-y and asflags-remove-y

On Mon, 6 Jul 2020 at 13:24, Anders Roxell wrote: > > Hi, > > When I built an allmodconfig kernel for arm64, and boot that in qemu > guest I see the following issues: > > [...] > [ 10.451561][T1] Testing tracer function: PASSED > [ 33.087895][T1] Testing dy

Re: [PATCH] kbuild: introduce ccflags-remove-y and asflags-remove-y

Hi, When I built an allmodconfig kernel for arm64, and boot that in qemu guest I see the following issues: [...] [ 10.451561][T1] Testing tracer function: PASSED [ 33.087895][T1] Testing dynamic ftrace: .. filter did not filter .. FAILED! [ 51.127094][T1] [ cut here

Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

On Mon, 20 Apr 2020 at 23:43, Mike Kravetz wrote: > > On 4/20/20 1:29 PM, Anders Roxell wrote: > > On Mon, 20 Apr 2020 at 20:23, Mike Kravetz wrote: > >> On 4/20/20 8:34 AM, Qian Cai wrote: > >>> > >>> Reverted this series fixed many undefi

Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

On Mon, 20 Apr 2020 at 20:23, Mike Kravetz wrote: > > On 4/20/20 8:34 AM, Qian Cai wrote: > > > > > >> On Apr 17, 2020, at 2:50 PM, Mike Kravetz wrote: > >> > >> Longpeng(Mike) reported a weird message from hugetlb command line > >> processing > >> and proposed a solution [1]. While the propose

Re: [PATCH v3 3/4] hugetlbfs: remove hugetlb_add_hstate() warning for existing hstate

On Fri, 17 Apr 2020 at 20:52, Mike Kravetz wrote: > > The routine hugetlb_add_hstate prints a warning if the hstate already > exists. This was originally done as part of kernel command line > parsing. If 'hugepagesz=' was specified more than once, the warning > pr_warn("hugepagesz= speci

Re: [PATCH] selftests/powerpc: Fix Makefiles for headers_install change

../scripts/subarch.include'. > make[2]: Failed to remake makefile '../../../../scripts/subarch.include'. > Makefile:38: recipe for target 'alignment' failed > > Fix it by setting top_srcdir in the affected Makefiles. > > Fixes: b2d35fa5fc80 ("selfte