Re: [PATCH RFC] x86: remove toolchain check for X32 ABI capability

2021-02-27 Thread Nathan Chancellor
On Sun, Feb 28, 2021 at 12:15:16PM +0900, Masahiro Yamada wrote: > On Sun, Feb 28, 2021 at 3:41 AM Masahiro Yamada wrote: > > > > This commit reverts 0bf6276392e9 ("x32: Warn and disable rather than > > error if binutils too old"). > > > > The help text in arch/x86/Kconfig says enabling the X32

Re: [PATCH] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*

2021-02-25 Thread Nathan Chancellor
On Thu, Feb 25, 2021 at 12:14:17PM -0800, Kees Cook wrote: > On Thu, Feb 25, 2021 at 12:06:37PM -0800, Andrew Morton wrote: > > On Thu, 25 Feb 2021 12:03:48 -0800 Kees Cook wrote: > > > > > On Thu, Feb 25, 2021 at 05:45:09PM +0100, Arnd Bergmann wrote: > > > > From: Arnd Bergmann > > > > > > >

Re: [PATCH] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*

2021-02-25 Thread Nathan Chancellor
6c ("include/linux/compiler*.h: make compiler-*.h mutually > exclusive") > Signed-off-by: Arnd Bergmann Reviewed-by: Nathan Chancellor > --- > include/linux/compiler-clang.h | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/include/linux/com

Re: [PATCH v3] vmlinux.lds.h: Define SANTIZER_DISCARDS with CONFIG_GCOV_KERNEL=y

2021-02-25 Thread Nathan Chancellor
On Fri, Jan 29, 2021 at 05:46:51PM -0700, Nathan Chancellor wrote: > clang produces .eh_frame sections when CONFIG_GCOV_KERNEL is enabled, > even when -fno-asynchronous-unwind-tables is in KBUILD_CFLAGS: > > $ make CC=clang vmlinux > ... > ld: warning: orphan section `.eh_frame'

Re: [kbuild-all] Re: WARNING: modpost: vmlinux.o(.text+0x1a8edb8): Section mismatch in reference from the function stop_machine() to the function .init.text:intel_rng_hw_init()

2021-02-25 Thread Nathan Chancellor
On Thu, Feb 25, 2021 at 04:17:21PM +0800, Rong Chen wrote: > > > On 2/24/21 10:26 PM, Jürgen Groß wrote: > > On 24.02.21 15:20, kernel test robot wrote: > > > tree: > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > master > > > head:  

Re: [PATCH 1/2] mt76: mt7915: fix unused 'mode' variable

2021-02-25 Thread Nathan Chancellor
On Thu, Feb 25, 2021 at 03:59:14PM +0100, Arnd Bergmann wrote: > From: Arnd Bergmann > > clang points out a possible corner case in the mt7915_tm_set_tx_cont() > function if called with invalid arguments: > > drivers/net/wireless/mediatek/mt76/mt7915/testmode.c:593:2: warning: variable >

Re: [PATCH V2] init/Kconfig: Fix a typo in CC_VERSION_TEXT help text

2021-02-24 Thread Nathan Chancellor
On Thu, Feb 25, 2021 at 04:03:25AM +0530, Bhaskar Chowdhury wrote: > > s/compier/compiler/ > > Signed-off-by: Bhaskar Chowdhury Reviewed-by: Nathan Chancellor > --- > Changes from V1: > Nathan and Randy, suggested better subject line texts,so incorporated. > >

Re: [PATCH] Makefile: reuse CC_VERSION_TEXT

2021-02-24 Thread Nathan Chancellor
On Wed, Feb 24, 2021 at 12:14:04PM +0900, Masahiro Yamada wrote: > On Sat, Feb 6, 2021 at 12:46 PM Sedat Dilek wrote: > > > > On Sat, Feb 6, 2021 at 2:49 AM Masahiro Yamada wrote: > > > > > > On Sat, Feb 6, 2021 at 7:01 AM 'Nick Desaulniers' via Clang Built > > > Linux wrote: > > > > > > > > I

Re: [PATCH] init: Kconfig: Fix a spelling compier to compiler in the file init/Kconfig

2021-02-24 Thread Nathan Chancellor
ccinct and descriptive. With that change: Reviewed-by: Nathan Chancellor > --- > init/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/init/Kconfig b/init/Kconfig > index b77c60f8b963..739c3425777b 100644 > --- a/init/Kconfig > +++ b/init/Kcon

Re: [PATCH] drm/amdgpu/display: initialize the variable 'i'

2021-02-22 Thread Nathan Chancellor
On Mon, Feb 22, 2021 at 11:50:06PM +, Simon Ser wrote: > On Tuesday, February 23rd, 2021 at 12:44 AM, Nathan Chancellor > wrote: > > > On Mon, Feb 22, 2021 at 11:05:17PM +, Simon Ser wrote: > > > On Monday, February 22nd, 2021 at 8:25 PM, Soupti

Re: [PATCH] drm/amdgpu/display: initialize the variable 'i'

2021-02-22 Thread Nathan Chancellor
On Mon, Feb 22, 2021 at 11:05:17PM +, Simon Ser wrote: > On Monday, February 22nd, 2021 at 8:25 PM, Souptick Joarder > wrote: > > > >> drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:9804:38: > > >> warning: variable 'i' is uninitialized when used here > > >> [-Wuninitialized] >

Re: clang-12: tun: Unknown symbol _GLOBAL_OFFSET_TABLE_ (err -2)

2021-02-22 Thread Nathan Chancellor
On Tue, Feb 23, 2021 at 01:01:16AM +0530, Naresh Kamboju wrote: > Following test regressions reported after upgrade to clang-12 from clang-10. > Regression detected on Linux mainline and stable-rc 5.4, 5.10 and 5.11. > > qemu-x86_64-clang-12 failed test cases, > ltp-containers-tests: > *

Re: [PATCH] qemu_fw_cfg: Make fw_cfg_rev_attr a proper kobj_attribute

2021-02-22 Thread Nathan Chancellor
On Mon, Feb 22, 2021 at 11:02:34AM -0800, Nick Desaulniers wrote: > Did this happen to get picked up already? EOM I have not gotten an email saying it has been picked up nor does it appear to be in -next. Cheers, Nathan > On Thu, Feb 11, 2021 at 11:43 AM Nathan Chancellor

Re: [git pull] drm for 5.12-rc1

2021-02-21 Thread Nathan Chancellor
On Sun, Feb 21, 2021 at 03:07:17PM -0800, Linus Torvalds wrote: > On Thu, Feb 18, 2021 at 10:06 PM Dave Airlie wrote: > > > > Let me know if there are any issues, > > gcc was happy, and I obviously already pushed out my merge, but then > when I did my clang build afterwards, it reports: > >

Re: [PATCH v3] ARM: Implement SLS mitigation

2021-02-19 Thread Nathan Chancellor
04 > https://developer.arm.com/support/arm-security-updates/speculative-processor-vulnerability/downloads/straight-line-speculation > https://developer.arm.com/support/arm-security-updates/speculative-processor-vulnerability/frequently-asked-questions#SLS2 > > Suggested-by: Manoj Gupta >

[PATCH] drm/amd/pm/swsmu: Avoid using structure_size uninitialized in smu_cmn_init_soft_gpu_metrics

2021-02-18 Thread Nathan Chancellor
; ^ = 0 1 warning generated. Return in the default case, as the size of the header will not be known. Fixes: de4b7cd8cb87 ("drm/amd/pm/swsmu: unify the init soft gpu metrics function") Link: https://github.com/ClangBuiltLinux/linux/issues/1304 Signed-off-

[PATCH] Makefile: Remove # characters from compiler string

2021-02-16 Thread Nathan Chancellor
ux/linux/issues/1298 Reported-by: Michael Fuckner Signed-off-by: Nathan Chancellor --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index de1acaefe87e..9bfe1307447e 100644 --- a/Makefile +++ b/Makefile @@ -581,7 +581,7 @@ endif # Some architectures

[PATCH] drm/i915: Enable -Wuninitialized

2021-02-16 Thread Nathan Chancellor
ba9bf6 ("drm/i915/gt: Avoid uninitialized use of rpcurupei in frequency_show") References: 2034c2129bc4 ("drm/i915/display: Ensure that ret is always initialized in icl_combo_phy_verify_state") Reported-by: Arnd Bergmann Signed-off-by: Nathan Chancellor --- drivers/gpu/dr

Re: [PATCH 2/2] kbuild: check the minimum linker version in Kconfig

2021-02-16 Thread Nathan Chancellor
cconfig] Error 1 > make[1]: *** [Makefile:600: syncconfig] Error 2 > make: *** [Makefile:708: include/config/auto.conf] Error 2 > > Thanks to David Laight for suggesting shell script improvements. > > Signed-off-by: Masahiro Yamada Reviewed-by: Nathan Chancellor Tested-by

Re: [PATCH 1/2] kbuild: remove ld-version macro

2021-02-16 Thread Nathan Chancellor
On Tue, Feb 16, 2021 at 12:10:03PM +0900, Masahiro Yamada wrote: > There is no direct user of ld-version; you can use CONFIG_LD_VERSION > if needed. > > Signed-off-by: Masahiro Yamada Reviewed-by: Nathan Chancellor > --- > > scripts/Kbuild.include | 6 +- > 1 f

Re: [PATCH mips-next] vmlinux.lds.h: catch more UBSAN symbols into .data

2021-02-16 Thread Nathan Chancellor
On Tue, Feb 16, 2021 at 09:56:32AM -0800, 'Nick Desaulniers' via Clang Built Linux wrote: > On Tue, Feb 16, 2021 at 12:55 AM Alexander Lobakin wrote: > > > > LKP triggered lots of LD orphan warnings [0]: > > Thanks for the patch, just some questions. > > With which linker? Was there a

Re: [PATCH v1] clang_tools:gen_compile_commands: Change the default source directory

2021-02-14 Thread Nathan Chancellor
On Sun, Feb 14, 2021 at 07:49:22PM +0800, Stephen Zhang wrote: > Masahiro Yamada 于2021年2月13日周六 下午8:46写道: > > This is the steps I tested. > > > > > > masahiro@oscar:~/ref/linux$ make O=build defconfig all -j24 > > [ snip ] > > masahiro@oscar:~/ref/linux$ > >

Re: [PATCH] ARM: Implement Clang's SLS mitigation

2021-02-11 Thread Nathan Chancellor
Hi Jian, On Thu, Feb 11, 2021 at 09:14:58PM -0800, Jian Cai wrote: > This patch adds a config CONFIG_HARDEN_SLS_ALL that can be used to turn Drop "a config". > on -mharden-sls=all, which mitigates the straight-line speculation > vulnerability, or more commonly known as Spectre, Meldown.

[PATCH] qemu_fw_cfg: Make fw_cfg_rev_attr a proper kobj_attribute

2021-02-11 Thread Nathan Chancellor
rning. Update fw_cfg_showrev() accordingly. Fixes: 75f3e8e47f38 ("firmware: introduce sysfs driver for QEMU's fw_cfg device") Link: https://github.com/ClangBuiltLinux/linux/issues/1299 Signed-off-by: Nathan Chancellor --- drivers/firmware/qemu_fw_cfg.c | 8 +++- 1 file chan

Re: [PATCH v5 2/2] ARM: kprobes: rewrite test-arm.c in UAL

2021-02-11 Thread Nathan Chancellor
documentation/dui0473/c/writing-arm-assembly-language/assembly-language-changes-after-rvctv2-1 > Link: https://github.com/ClangBuiltLinux/linux/issues/1271 > Reported-by: Arnd Bergmann > Signed-off-by: Nick Desaulniers Reviewed-by: Nathan Chancellor > --- > arch/arm/probes/kprobes/t

Re: [PATCH] gen_compile_commands: prune some directories

2021-02-11 Thread Nathan Chancellor
ot support tools/", somebody > might try to get it back. Clarify this. > > Signed-off-by: Masahiro Yamada Sorry, I did not realize that gen_compile_commands.py did not intend to support. I was only looking at the history for the current location, not the former one of scripts/gen_compile_commands.

Re: [PATCH v1] clang_tools:gen_compile_commands: Change the default source directory

2021-02-10 Thread Nathan Chancellor
On Wed, Feb 10, 2021 at 08:15:27PM +0800, Stephen Zhang wrote: > Nathan Chancellor 于2021年2月10日周三 上午3:27写道: > > > Just as an FYI, your email was HTML, which means it won't hit LKML. > > > Thanks for pointing that out. The existence of a GFW makes it difficult for > me

Re: linux-next: build warning after merge of the rdma tree

2021-02-10 Thread Nathan Chancellor
On Wed, Feb 10, 2021 at 09:47:42AM -0400, Jason Gunthorpe wrote: > On Wed, Feb 10, 2021 at 09:11:49PM +1100, Stephen Rothwell wrote: > > Hi all, > > > > After merging the rdma tree, today's linux-next build (htmldocs) produced > > this warning: > > > > drivers/infiniband/core/device.c:859:

Re: [PATCH v1] clang_tools:gen_compile_commands: Change the default source directory

2021-02-09 Thread Nathan Chancellor
On Tue, Feb 09, 2021 at 09:56:20PM +0800, Stephen Zhang wrote: > Nathan Chancellor 于2021年2月9日周二 上午3:54写道: > > > On Mon, Feb 08, 2021 at 07:28:57PM +0800, Stephen Zhang wrote: > > > The default source directory is set equal to build directory which > > > specif

Re: [PATCH] i3c/master/mipi-i3c-hci: Fix position of __maybe_unused in i3c_hci_of_match

2021-02-08 Thread Nathan Chancellor
On Thu, Dec 31, 2020 at 06:42:23PM +0100, Alexandre Belloni wrote: > On Mon, 21 Dec 2020 19:59:31 -0700, Nathan Chancellor wrote: > > Clang warns: > > > > ../drivers/i3c/master/mipi-i3c-hci/core.c:780:21: warning: attribute > > declaration must precede defin

[PATCH v2] arm64: Make CPU_BIG_ENDIAN depend on ld.bfd or ld.lld 13.0.0+

2021-02-08 Thread Nathan Chancellor
d08feb7 Link: https://github.com/llvm/llvm-project/commit/eea34aae2e74e9b6fbdd5b95f479bc7f397bf387 Reported-by: Arnd Bergmann Signed-off-by: Nathan Chancellor --- v1 -> v2: * Conditionally disable CONFIG_CPU_BIG_ENDIAN for older versions of ld.lld now that proper support has been adde

Re: [PATCH v1] clang_tools:gen_compile_commands: Change the default source directory

2021-02-08 Thread Nathan Chancellor
On Mon, Feb 08, 2021 at 07:28:57PM +0800, Stephen Zhang wrote: > The default source directory is set equal to build directory which > specified by "-d".But it is designed to be set to the current working > directoy by default, as the help messge says.It makes a differece when > source directory

Re: [PATCH] arm64: Make CPU_BIG_ENDIAN depend on !LD_IS_LLD

2021-02-06 Thread Nathan Chancellor
On Mon, Feb 01, 2021 at 07:24:42PM -0700, Nathan Chancellor wrote: > Similar to commit 28187dc8ebd9 ("ARM: 9025/1: Kconfig: CPU_BIG_ENDIAN > depends on !LD_IS_LLD"), ld.lld does not support aarch64 big endian, > leading to the following build error when CONFIG_CPU_BIG_E

Re: [PATCH] memory: ti-emif-pm: Drop of_match_ptr from of_device_id table

2021-02-06 Thread Nathan Chancellor
to be how everyone is dealing with this warning: Reviewed-by: Nathan Chancellor > --- > drivers/memory/ti-emif-pm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/memory/ti-emif-pm.c b/drivers/memory/ti-emif-pm.c > index 6c747c1e98cb..179fec2da56d 10

Re: [PATCH] mm/mremap: fix BUILD_BUG_ON() error in get_extent

2021-02-05 Thread Nathan Chancellor
On Fri, Feb 05, 2021 at 01:02:23PM -0800, Andrew Morton wrote: > On Fri, 5 Feb 2021 12:00:05 -0700 Nathan Chancellor wrote: > > > > > > Reviewed-by: Nathan Chancellor > > > > > > > > Ping? It is pretty late into the 5.11 cycle and this is still b

Re: [PATCH v2] firmware_loader: Align .builtin_fw to 8

2021-02-05 Thread Nathan Chancellor
ues/1204 > Reported-by: kernel test robot > Signed-off-by: Fangrui Song > Acked-by: Arnd Bergmann Acked-by: Nathan Chancellor Andrew, is there any way to get this picked up with the following tags for 5.11? Tested-by: Douglas Anderson Reviewed-by: Nick Desaulniers Tested-by: Nick Desaulniers

Re: [PATCH] x86/efi: Remove EFI PGD build time checks

2021-02-05 Thread Nathan Chancellor
ip them out. > > See https://github.com/ClangBuiltLinux/linux/issues/256 for more info. > > Reported-by: Arnd Bergmann > Link: http://lkml.kernel.org/r/20210107223424.4135538-1-a...@kernel.org > Signed-off-by: Borislav Petkov Reviewed-by: Nathan Chancellor Tested-by: Nathan Chan

Re: [PATCH] mm/mremap: fix BUILD_BUG_ON() error in get_extent

2021-02-05 Thread Nathan Chancellor
On Wed, Feb 03, 2021 at 12:03:07PM -0800, Kees Cook wrote: > On Wed, Feb 03, 2021 at 11:48:40AM -0700, Nathan Chancellor wrote: > > On Tue, Jan 12, 2021 at 12:16:34PM -0700, Nathan Chancellor wrote: > > > On Wed, Dec 30, 2020 at 04:40:40PM +0100, Arnd Bergmann wrote: > >

Re: [PATCH] x86: efi: avoid BUILD_BUG_ON() for non-constant p4d_index

2021-02-05 Thread Nathan Chancellor
On Fri, Feb 05, 2021 at 10:27:54AM -0800, Nick Desaulniers wrote: > On Fri, Feb 5, 2021 at 2:35 AM Borislav Petkov wrote: > > > > On Wed, Feb 03, 2021 at 11:51:48AM -0700, Nathan Chancellor wrote: > > > x86_64 all{mod,yes}config with clang are going to ship broke

Re: [PATCH] staging: qlge/qlge_main: Use min_t instead of min

2021-02-04 Thread Nathan Chancellor
On Fri, Feb 05, 2021 at 03:24:51AM +0530, ameynarkhed...@gmail.com wrote: > From: Amey Narkhede > > Use min_t instead of min function in qlge/qlge_main.c > Fixes following checkpatch.pl warning: > WARNING: min() should probably be min_t(int, MAX_CPUS, num_online_cpus()) > > Signed-off-by: Amey

Re: [PATCH 1/2] crypto: octeontx2 - fix -Wpointer-bool-conversion warning

2021-02-04 Thread Nathan Chancellor
s: 64506017030d ("crypto: octeontx2 - add LF framework") > Signed-off-by: Arnd Bergmann Reviewed-by: Nathan Chancellor > --- > drivers/crypto/marvell/octeontx2/otx2_cptlf.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/crypto/marve

Re: [PATCH] net: hns3: avoid -Wpointer-bool-conversion warning

2021-02-04 Thread Nathan Chancellor
te to 'true' > [-Werror,-Wpointer-bool-conversion] > > This can never fail, so just remove the check. > > Fixes: 04987ca1b9b6 ("net: hns3: add debugfs support for tm nodes, priority > and qset info") > Signed-off-by: Arnd Bergmann Reviewed-by: Nathan Ch

Re: [PATCH] x86: efi: avoid BUILD_BUG_ON() for non-constant p4d_index

2021-02-04 Thread Nathan Chancellor
ing the assertions will do that as well. Feel free to carry forward Tested-by: Nathan Chancellor on a patch if you send it out. > diff --git a/arch/x86/include/asm/pgtable_64_types.h > b/arch/x86/include/asm/pgtable_64_types.h > index 91ac10654570..b6be19c09841 100644 > --- a/arch/

Re: next/master bisection: baseline.login on rk3288-rock2-square

2021-02-04 Thread Nathan Chancellor
On Thu, Feb 04, 2021 at 10:06:08AM -0800, 'Nick Desaulniers' via Clang Built Linux wrote: > On Thu, Feb 4, 2021 at 8:02 AM Ard Biesheuvel wrote: > > > > On Thu, 4 Feb 2021 at 16:53, Guillaume Tucker > > wrote: > > > > > > On 04/02/2021 15:42, Ard Biesheuvel wrote: > > > > On Thu, 4 Feb 2021 at

Re: [PATCH] x86: efi: avoid BUILD_BUG_ON() for non-constant p4d_index

2021-02-03 Thread Nathan Chancellor
On Wed, Jan 20, 2021 at 10:33:43AM +0100, Ard Biesheuvel wrote: > On Mon, 18 Jan 2021 at 22:42, Arvind Sankar wrote: > > > > On Mon, Jan 18, 2021 at 09:24:09PM +0100, Borislav Petkov wrote: > > > > > > As a matter of fact, it seems like the four assertions could be > > > > > > combined > > > > >

Re: [PATCH] mm/mremap: fix BUILD_BUG_ON() error in get_extent

2021-02-03 Thread Nathan Chancellor
On Tue, Jan 12, 2021 at 12:16:34PM -0700, Nathan Chancellor wrote: > On Wed, Dec 30, 2020 at 04:40:40PM +0100, Arnd Bergmann wrote: > > From: Arnd Bergmann > > > > clang cannt evaluate this function argument at compile time > > when the function is not inlined, whi

Re: [PATCH] kbuild: fix duplicated flags in DEBUG_CFLAGS

2021-02-03 Thread Nathan Chancellor
DEBUG_CFLAGS also when using Clang. > > Fixes: 121c5d08d53c ("kbuild: Only add -fno-var-tracking-assignments for old > GCC versions") > Reported-by: Sedat Dilek > Signed-off-by: Masahiro Yamada Reviewed-by: Nathan Chancellor > --- > > Makefile | 4 +++- > 1

Re: [PATCH] arm64: Make CPU_BIG_ENDIAN depend on !LD_IS_LLD

2021-02-02 Thread Nathan Chancellor
On Tue, Feb 02, 2021 at 09:04:34AM +0100, Arnd Bergmann wrote: > On Tue, Feb 2, 2021 at 3:25 AM Nathan Chancellor wrote: > > > > Similar to commit 28187dc8ebd9 ("ARM: 9025/1: Kconfig: CPU_BIG_ENDIAN > > depends on !LD_IS_LLD"), ld.lld does not support

[PATCH] arm64: Make CPU_BIG_ENDIAN depend on !LD_IS_LLD

2021-02-01 Thread Nathan Chancellor
ux/linux/issues/1288 Reported-by: Arnd Bergmann Signed-off-by: Nathan Chancellor --- arch/arm64/Kconfig | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig index f39568b28ec1..5ddf9fe7b4ee 100644 --- a/arch/arm64/Kconfig +++ b/arch/arm

Re: [PATCH] scripts: switch some more scripts explicitly to Python 3

2021-01-31 Thread Nathan Chancellor
On Mon, Feb 01, 2021 at 10:08:18AM +0900, Masahiro Yamada wrote: > For the same reason as commit 51839e29cb59 ("scripts: switch explicitly > to Python 3"), switch some more scripts, which I tested and confirmed > working on Python 3. > > Signed-off-by: Masahiro

Re: [PATCH v2] ext4: Enable code path when DX_DEBUG is set

2021-01-31 Thread Nathan Chancellor
On Mon, Feb 01, 2021 at 12:31:25AM +, Vinicius Tinti wrote: > By enabling -Wunreachable-code-aggressive on Clang the following code > paths are unreachable. > > This has been present since commit ac27a0ec112a ("[PATCH] ext4: initial > copy of files from ext3") and fs/ext3 had it present at

Re: aarch64-linux-gnu-ld: warning: orphan section `.eh_frame' from `drivers/media/platform/rockchip/rkisp1/rkisp1-capture.o' being placed in section `.eh_frame'

2021-01-30 Thread Nathan Chancellor
On Sun, Jan 31, 2021 at 03:52:15AM +0800, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: 0e9bcda5d286f4a26a5407bb38f55c55b453ecfb > commit: e6938cc1cb7763a363f62b78147f1f2fb972f49c media: rockchip: rkisp1: > destage

[PATCH v2] vmlinux.lds.h: Define SANTIZER_DISCARDS with CONFIG_GCOV_KERNEL=y

2021-01-30 Thread Nathan Chancellor
aulniers Signed-off-by: Nathan Chancellor --- v1 -> v2: * Keep configs and flag names alphabetized. * Drop mention of -ftest-coverage since it does not cause this issue per Fangrui. * Pick up review tags from Fangrui and Nick. include/asm-generic/vmlinux.lds.h | 9 + 1 file chan

Re: [PATCH v7 1/2] Kbuild: make DWARF version a choice

2021-01-30 Thread Nathan Chancellor
GCC since ~4.8 has defaulted to this DWARF version implicitly. > > Suggested-by: Arvind Sankar > Suggested-by: Fangrui Song > Suggested-by: Nathan Chancellor > Suggested-by: Masahiro Yamada > Signed-off-by: Nick Desaulniers One comment below: Reviewed-by: Nathan Chancellor

Re: [PATCH v7 2/2] Kbuild: implement support for DWARF v5

2021-01-29 Thread Nathan Chancellor
> Disabled for now if CONFIG_DEBUG_INFO_BTF is set; pahole doesn't yet > recognize the new additions to the DWARF debug info. Thanks to Sedat for > the report. > > Link: http://www.dwarfstd.org/doc/DWARF5.pdf > Reported-by: Sedat Dilek > Suggested-by: Arvind Sankar > Suggested-by:

Re: aarch64-linux-gnu-ld: warning: orphan section `.eh_frame' from `arch/arm64/kernel/ftrace.o' being placed in section `.eh_frame'

2021-01-29 Thread Nathan Chancellor
On Sat, Jan 30, 2021 at 09:21:49AM +0800, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: bec4c2968fce2f44ce62d05288a633cd99a722eb > commit: b3e5d80d0c48c0cc7bce56473672f4e6e1210910 arm64/build: Warn on orphan > section

Re: [PATCH] ext4: Remove unreachable code

2021-01-29 Thread Nathan Chancellor
ext3") and fs/ext3 had it present at the beginning of git history so it is safe to remove. > Signed-off-by: Vinicius Tinti Regardless of the commit message, I believe this is the right way to get rid of the warning: Reviewed-by: Nathan Chancellor > --- > fs/ext4/namei.c | 15 -

[PATCH v3] vmlinux.lds.h: Define SANTIZER_DISCARDS with CONFIG_GCOV_KERNEL=y

2021-01-29 Thread Nathan Chancellor
aulniers Signed-off-by: Nathan Chancellor --- v2 -> v3: * Fix double "unwanted" in comment section per Sedat. v1 -> v2: * Keep configs and flag names alphabetized. * Drop mention of -ftest-coverage since it does not cause this issue per Fangrui. * Pick up review tags from Fan

Re: [PATCH] Documentation/llvm: Add a section about supported architectures

2021-01-29 Thread Nathan Chancellor
On Wed, Jan 13, 2021 at 05:34:47PM -0700, Nathan Chancellor wrote: > The most common question around building the Linux kernel with clang is > "does it work?" and the answer has always been "it depends on your > architecture, configuration, and LLVM version" wit

[PATCH] vmlinux.lds.h: Define SANTIZER_DISCARDS with CONFIG_GCOV_KERNEL=y

2021-01-29 Thread Nathan Chancellor
for this issue. Take advantage of that section for this config as well so that there are no more orphan warnings. Link: https://bugs.llvm.org/show_bug.cgi?id=46478 Link: https://github.com/ClangBuiltLinux/linux/issues/1069 Reported-by: kernel test robot Signed-off-by: Nathan Chancellor --- include/asm-g

[PATCH v3] ubsan: Implement __ubsan_handle_alignment_assumption

2021-01-27 Thread Nathan Chancellor
/llvmorg-11.0.1/compiler-rt/lib/ubsan/ubsan_handlers.cpp#L151-L190 Acked-by: Kees Cook Signed-off-by: Nathan Chancellor --- v2 -> v3: * Add prototype right above definition to avoid introducing a warning with W=1. v1 -> v2: * Use __ffs instead of ffs because due to size of input (unsigne

[PATCH] MAINTAINERS/.mailmap: Use my @kernel.org address

2021-01-26 Thread Nathan Chancellor
Use my @kernel.org for all points of contact so that I am always accessible. Signed-off-by: Nathan Chancellor --- .mailmap| 1 + MAINTAINERS | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/.mailmap b/.mailmap index b1ab0129c7d6..b25a44ab5ba6 100644 --- a/.mailmap +++ b

Re: [PATCH 4.14 00/50] 4.14.217-rc1 review

2021-01-22 Thread Nathan Chancellor
On Fri, Jan 22, 2021 at 03:36:04PM +, Will Deacon wrote: > On Fri, Jan 22, 2021 at 08:43:18PM +0530, Naresh Kamboju wrote: > > On Fri, 22 Jan 2021 at 20:38, Greg Kroah-Hartman > > wrote: > > > > > > On Fri, Jan 22, 2021 at 08:32:46PM +0530, Naresh Kamboju wrote: > > > > On Fri, 22 Jan 2021 at

Re: arch/riscv/kernel/vdso/vdso-syms.S:2: Error: junk at end of line, first unrecognized character is `@'

2021-01-21 Thread Nathan Chancellor
On Fri, Jan 22, 2021 at 09:41:35AM +0800, kernel test robot wrote: > Hi Palmer, > > First bad commit (maybe != root cause): > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: 9f29bd8b2e7132b409178d1367dae1813017bd0e > commit:

[PATCH] mm/zswap: Add return value in zswap_frontswap_load

2021-01-21 Thread Nathan Chancellor
ot;mm/zswap: add the flag can_sleep_mapped", this path always returned 0. Restore that so we are not returning uninitialized memory. Link: https://github.com/ClangBuiltLinux/linux/issues/1263 Reported-by: kernelci.org bot Reported-by: kernel test robot Signed-off-by: Nathan Chancellor ---

Re: [PATCH] MAINTAINERS: adjust to clang-version.sh removal

2021-01-21 Thread Nathan Chancellor
On Thu, Jan 21, 2021 at 05:01:15PM +0100, Lukas Bulwahn wrote: > Commit 6c8ad4427f6e ("kbuild: check the minimum compiler version in > Kconfig") removed ./scripts/clang-version.sh and moved its content to > ./scripts/cc-version.sh. > > Since then, ./scripts/get_maintainer.pl --self-test=patterns

Re: [PATCH bpf-next v3] samples/bpf: Update build procedure for manually compiling LLVM and Clang

2021-01-21 Thread Nathan Chancellor
On Thu, Jan 21, 2021 at 12:08:31AM -0800, Andrii Nakryiko wrote: > On Wed, Jan 20, 2021 at 9:36 PM Nathan Chancellor > wrote: > > > > On Thu, Jan 21, 2021 at 01:27:35PM +0800, Tiezhu Yang wrote: > > > The current LLVM and Clang build procedure in samples/bpf/README.rs

Re: [PATCH bpf-next v3] samples/bpf: Update build procedure for manually compiling LLVM and Clang

2021-01-20 Thread Nathan Chancellor
IBS=OFF is the default option [2], so also change > Documentation/bpf/bpf_devel_QA.rst together. > > [1] https://clang.llvm.org/get_started.html > [2] https://www.llvm.org/docs/CMake.html > > Signed-off-by: Tiezhu Yang > Acked-by: Yonghong Song Reviewed-by: Nathan Chancellor Small c

Re: [PATCH v5 2/3] Kbuild: make DWARF version a choice

2021-01-20 Thread Nathan Chancellor
On Fri, Jan 15, 2021 at 01:06:15PM -0800, Nick Desaulniers wrote: > Modifies CONFIG_DEBUG_INFO_DWARF4 to be a member of a choice. Adds an > explicit CONFIG_DEBUG_INFO_DWARF2, which is the default. Does so in a > way that's forward compatible with existing configs, and makes adding > future

Re: [STABLE BACKPORT 4.4.y, 4.9.y and 4.14.y] compiler.h: Raise minimum version of GCC to 5.1 for arm64

2021-01-19 Thread Nathan Chancellor
ell King at the > > link below. > > > > Life is too short for buggy compilers, so raise the minimum GCC version > > required by arm64 to 5.1. > > > > Reported-by: Russell King > > Suggested-by: Arnd Bergmann > > Signed-off-by: Will Deacon > >

Re: [PATCH v4 1/2] arm: lib: xor-neon: remove unnecessary GCC < 4.6 warning

2021-01-19 Thread Nathan Chancellor
On Tue, Jan 19, 2021 at 03:17:23PM +0200, Adrian Ratiu wrote: > From: Nathan Chancellor > > Drop warning because kernel now requires GCC >= v4.9 after > commit 6ec4476ac825 ("Raise gcc version requirement to 4.9") > and clarify that -ftree-vectorize now alw

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

Re: [PATCH 5.4 40/62] spi: spi-geni-qcom: Fix geni_spi_isr() NULL dereference in timeout case

2021-01-16 Thread Nathan Chancellor
On Fri, Jan 15, 2021 at 01:28:02PM +0100, Greg Kroah-Hartman wrote: > From: Douglas Anderson > > commit 4aa1464acbe3697710279a4bd65cb4801ed30425 upstream. > > In commit 7ba9bdcb91f6 ("spi: spi-geni-qcom: Don't keep a local state > variable") we changed handle_fifo_timeout() so that we set >

Re: [PATCH 5.10 086/103] regmap: debugfs: Fix a memory leak when calling regmap_attach_dev

2021-01-15 Thread Nathan Chancellor
On Fri, Jan 15, 2021 at 09:18:19PM +0100, Pavel Machek wrote: > Hi! > > > From: Xiaolei Wang > > > > commit cffa4b2122f5f3e53cf3d529bbc74651f95856d5 upstream. > > > > After initializing the regmap through > > syscon_regmap_lookup_by_compatible, then regmap_attach_dev to the > > device, because

Re: [PATCH v3] module: Ignore _GLOBAL_OFFSET_TABLE_ when warning for undefined symbols

2021-01-15 Thread Nathan Chancellor
b.com/ClangBuiltLinux/linux/issues/1250 > Signed-off-by: Fangrui Song > Reviewed-by: Nick Desaulniers > Tested-by: Marco Elver > Cc: Reviewed-by: Nathan Chancellor > > --- > Changes in v2: > * Fix Marco's email address > * Add a function ignore_undef_symbol similar to

Re: [PATCH] mips: vdso: fix DWARF2 warning

2021-01-15 Thread Nathan Chancellor
On Fri, Jan 15, 2021 at 08:13:30PM +0100, Anders Roxell wrote: > When building mips tinyconifg the following warning show up > > make --silent --keep-going --jobs=8 > O=/home/anders/src/kernel/next/out/builddir ARCH=mips > CROSS_COMPILE=mips-linux-gnu- HOSTCC=clang CC=clang >

[PATCH] MIPS: VDSO: Use CLANG_FLAGS instead of filtering out '--target='

2021-01-15 Thread Nathan Chancellor
tps://github.com/ClangBuiltLinux/linux/issues/1256 Reported-by: Anders Roxell Signed-off-by: Nathan Chancellor --- arch/mips/vdso/Makefile | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/arch/mips/vdso/Makefile b/arch/mips/vdso/Makefile index 5810cc12bc1d..2131d3fd7333 100644

Re: [PATCH] ubifs: replay: Fix high stack usage, again

2021-01-15 Thread Nathan Chancellor
ifs: use crypto_shash_tfm_digest()") > Fixes: eb66eff6636d ("ubifs: replay: Fix high stack usage") > Signed-off-by: Arnd Bergmann Reviewed-by: Nathan Chancellor > --- > fs/ubifs/replay.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff

Re: [PATCH] x86: efi: avoid BUILD_BUG_ON() for non-constant p4d_index

2021-01-15 Thread Nathan Chancellor
On Fri, Jan 15, 2021 at 07:23:00PM +0100, Borislav Petkov wrote: > On Thu, Jan 07, 2021 at 11:34:15PM +0100, Arnd Bergmann wrote: > > From: Arnd Bergmann > > > > When 5-level page tables are enabled, clang triggers a BUILD_BUG_ON(): > > I have CONFIG_X86_5LEVEL=y, CONFIG_EFI=y and am using

Re: [PATCH] objtool: Don't fail on missing symbol table

2021-01-14 Thread Nathan Chancellor
e to only build that file when > CONFIG_PREEMPTION is enabled, but that will likely create confusion > if/when the thunks end up getting used by some other code again. > > Just ignore it and move on. > > Reported-by: Nathan Chancellor > Signed-off-by: Josh Poimboeuf

Re: drivers/iio/accel/adxl372.c:949:3: warning: converting the result of '<<' to a boolean always evaluates to true

2021-01-14 Thread Nathan Chancellor
On Fri, Jan 15, 2021 at 02:19:41AM +0800, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: 65f0d2414b7079556fbbcc070b3d1c9f9587606d > commit: b0fc6783d4ae58f796fb183c40dbc7fa3199d51d iio: accel: adxl372: Add > support for

Re: [PATCH v3] kbuild: check the minimum compiler version in Kconfig

2021-01-14 Thread Nathan Chancellor
e we have > although I am not sure if building the kernel with ICC is well-supported. > > [1] https://lkml.org/lkml/2021/1/10/250 > [2] https://lkml.org/lkml/2021/1/12/1708 > [3] https://lkml.org/lkml/2021/1/12/1533 > > Fixes: 87de84c9140e ("kbuild: remove cc-option test of -Werror=date-time") > Reported-by: Paul Gortmaker > Suggested-by: Linus Torvalds > Signed-off-by: Masahiro Yamada Reviewed-by: Nathan Chancellor Tested-by: Nathan Chancellor

[PATCH] MIPS: Compare __SYNC_loongson3_war against 0

2021-01-14 Thread Nathan Chancellor
a44 ("MIPS: cmpxchg: Omit redundant barriers for Loongson3") Reported-by: kernel test robot Signed-off-by: Nathan Chancellor --- arch/mips/include/asm/atomic.h | 2 +- arch/mips/include/asm/cmpxchg.h | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/mi

[PATCH v2] MAINTAINERS: Add compiler-clang.h to the Clang/LLVM section

2021-01-14 Thread Nathan Chancellor
The K: entry should ensure that Nick and I always get CC'd on patches that touch thid file but it is better to be explicit rather than implicit. Reviewed-by: Nick Desaulniers Signed-off-by: Nathan Chancellor --- v1 -> v2: * Omit scripts/clang-version.sh as Masahiro has proposed unify

Re: [PATCH v2] kbuild: check the minimum compiler version in Kconfig

2021-01-14 Thread Nathan Chancellor
error=date-time") > Reported-by: Paul Gortmaker > Suggested-by: Linus Torvalds > Signed-off-by: Masahiro Yamada I like this a lot, I think that erroring as early as possible when something is misconfigured is a good user experience. Reviewed-by: Nathan Chancellor Tested-by:

Re: [PATCH v2] kbuild: check the minimum compiler version in Kconfig

2021-01-14 Thread Nathan Chancellor
On Thu, Jan 14, 2021 at 06:20:15PM +0900, Masahiro Yamada wrote: > BTW, when I tried ICC a few years ago, > I could not build the kernel with it. Looking at the history behind include/linux/compiler-intel.h, the last time I see a change that actually references building a kernel with icc was

Re: [PATCH] Documentation/llvm: Add a section about supported architectures

2021-01-13 Thread Nathan Chancellor
On Wed, Jan 13, 2021 at 06:05:32PM -0800, Nick Desaulniers wrote: > On Wed, Jan 13, 2021 at 5:19 PM Nick Desaulniers > wrote: > > > > Patch looks fine, but `make -j htmldocs` seems to be taking forever > > for me so I can't render it. Is this a known issue? > > > > $ make -j htmldocs A clean

Re: [PATCH] bpf: Hoise pahole version checks into Kconfig

2021-01-13 Thread Nathan Chancellor
On Wed, Jan 13, 2021 at 02:38:27PM -0800, Andrii Nakryiko wrote: > Hm.. Just saw Linus proposing using $(error-if) in Kconfig for an > unrelated issue ([0]). If we can make this work, then it would catch > such issue early on, yet won't have any downsides of hiding > CONFIG_DEBUG_INFO_BTF if

[PATCH] Documentation/llvm: Add a section about supported architectures

2021-01-13 Thread Nathan Chancellor
ilt (with just clang or the LLVM utilities as well), and the level of support they receive. This will make it easier for people to try out building their kernel with LLVM and reporting issues that come about from it. Suggested-by: Miguel Ojeda Signed-off-by: Nathan Chancellor --- Documentation/

[PATCH] MAINTAINERS: Add a couple more files to the Clang/LLVM section

2021-01-13 Thread Nathan Chancellor
The K: entry should ensure that Nick and I always get CC'd on patches that touch these files but it is better to be explicit rather than implicit. Signed-off-by: Nathan Chancellor --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index cc1e6a5ee6e6

Re: [PATCH v4] pgo: add clang's Profile Guided Optimization infrastructure

2021-01-13 Thread Nathan Chancellor
Hi Bill, On Tue, Jan 12, 2021 at 10:19:58PM -0800, Bill Wendling wrote: > From: Sami Tolvanen > > Enable the use of clang's Profile-Guided Optimization[1]. To generate a > profile, the kernel is instrumented with PGO counters, a representative > workload is run, and the raw profile data is

Re: [PATCH v4 3/3] Kbuild: implement support for DWARF v5

2021-01-13 Thread Nathan Chancellor
between GNU readelf and llvm-readelf/llvm-dwarfdump to > validate the DWARF Version in the assembler feature detection script. > > Link: http://www.dwarfstd.org/doc/DWARF5.pdf > Suggested-by: Arvind Sankar > Suggested-by: Jakub Jelinek > Suggested-by: Masahiro Yamada >

Re: [PATCH v4 2/3] Kbuild: make DWARF version a choice

2021-01-13 Thread Nathan Chancellor
> future versions more straightforward. > > Suggested-by: Fangrui Song > Suggested-by: Masahiro Yamada > Signed-off-by: Nick Desaulniers Reviewed-by: Nathan Chancellor > --- > Makefile | 14 +- > lib/Kconfig.debug | 21 - > 2

Re: [PATCH v4 1/3] Remove $(cc-option,-gdwarf-4) dependency from CONFIG_DEBUG_INFO_DWARF4

2021-01-13 Thread Nathan Chancellor
So, the debug info > for *.S files might be improved if we want. > > In my understanding, the current code is intentional, not a bug. > > [1] > https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=31bf18645d98b4d3d7357353be840e320649a67d > > Signed-off-by: Masahiro Ya

Re: [PATCH] ubsan: Implement __ubsan_handle_alignment_assumption

2021-01-12 Thread Nathan Chancellor
On Wed, Jan 13, 2021 at 08:39:52AM +0800, kernel test robot wrote: > Hi Nathan, > > I love your patch! Perhaps something to improve: > > [auto build test WARNING on 7c53f6b671f4aba70ff15e1b05148b10d58c2837] > > url: > https://github.com/0day-ci/linux/commits

Re: [PATCH] [v2] ubsan: disable unsigned-overflow check for i386

2021-01-12 Thread Nathan Chancellor
fig builds") > Link: https://lore.kernel.org/lkml/20201230154749.746641-1-a...@kernel.org/ > Signed-off-by: Arnd Bergmann Reviewed-by: Nathan Chancellor > --- > v2: only turn it off for i386 as discussed > --- > lib/Kconfig.ubsan | 1 + > 1 file changed, 1 inserti

[PATCH v2] ubsan: Implement __ubsan_handle_alignment_assumption

2021-01-12 Thread Nathan Chancellor
/llvmorg-11.0.1/compiler-rt/lib/ubsan/ubsan_handlers.cpp#L151-L190 Acked-by: Kees Cook Signed-off-by: Nathan Chancellor --- v1 -> v2: * Use __ffs instead of ffs because due to size of input (unsigned long vs int) and we want a zero based index (Nick Desaulniers). * Pick up Kees's ack. lib/

Re: [PATCH] compiler.h: Raise minimum version of GCC to 5.1 for arm64

2021-01-12 Thread Nathan Chancellor
o > Cc: Catalin Marinas > Cc: Florian Weimer > Cc: Peter Zijlstra > Cc: Linus Torvalds > Cc: Nick Desaulniers > Reported-by: Russell King > Suggested-by: Arnd Bergmann > Link: https://lore.kernel.org/r/20210105154726.gd1...@shell.armlinux.org.uk > Signed-off-b

Re: [PATCH] ubsan: Implement __ubsan_handle_alignment_assumption

2021-01-12 Thread Nathan Chancellor
On Tue, Jan 12, 2021 at 01:53:30PM -0800, Nick Desaulniers wrote: > On Tue, Jan 12, 2021 at 1:37 PM Nathan Chancellor > wrote: > > > > > if real_ptr is an unsigned long, do we want to use `__ffs(real_ptr) + > > > 1` here rather than ffs which takes an int? It see

Re: upstream build error (12)

2021-01-12 Thread Nathan Chancellor
On Tue, Jan 12, 2021 at 12:40:58PM -0700, Nathan Chancellor wrote: > On Tue, Jan 12, 2021 at 11:20:27AM -0800, syzbot wrote: > > Hello, > > > > syzbot found the following issue on: > > > > HEAD commit:e609571b Merge tag 'nfs-for-5.11-2' of git://git.linux-n

<    1   2   3   4   5   6   7   8   9   10   >