Re: [PATCH 5/6] powerpc/64: Add support for out-of-line static calls

2022-09-16 Thread kernel test robot
to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Benjamin-Gray/Out-of-line-static-calls-for-powerpc64-ELF-V2/20220916-142951 base: https://git.kernel.org/pub/scm/linux/kernel/g

[PATCH] macintosh/ams: Adapt declaration of ams_i2c_remove() to earlier change

2022-09-16 Thread Uwe Kleine-König
Commit ed5c2f5fd10d ("i2c: Make remove callback return void") changed the prototype of ams_i2c_remove() but failed to adapt the declaration. Catch up and fix the declaration accordingly. Fixes: ed5c2f5fd10d ("i2c: Make remove callback return void") Reported-by: kernel test robot Signed-off-by: Uw

Re: [PATCH 1/6] powerpc/code-patching: Implement generic text patching function

2022-09-16 Thread kernel test robot
Hi Benjamin, Thank you for the patch! Yet something to improve: [auto build test ERROR on powerpc/next] [also build test ERROR on next-20220916] [cannot apply to linus/master v6.0-rc5] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest

[PATCH V2] tools/perf/tests: Fix perf probe error log check in skip_if_no_debuginfo

2022-09-16 Thread Athira Rajeev
The perf probe related tests like probe_vfs_getname.sh which is in "tools/perf/tests/shell" directory have dependency on debuginfo information in the kernel. Currently debuginfo check is handled by skip_if_no_debuginfo function in the file "lib/probe_vfs_getname.sh". skip_if_no_debuginfo function l

[PATCH v2 0/2] Remove unused macros from asm-offset

2022-09-16 Thread Disha Goel
The kvm code was refactored to convert some of kvm assembly routines to C. This includes commits which moved code path for the kvm guest entry/exit for p7/8 from aseembly to C. As part of the code changes, usage of some of the macros were removed. But definitions still exist in the assembly files.

[PATCH v2 1/2] powerpc/kvm: Remove unused macros from asm-offset

2022-09-16 Thread Disha Goel
The kvm code was refactored to convert some of kvm assembly routines to C. This includes commits which moved code path for the kvm guest entry/exit for p7/8 from aseembly to C. As part of the code changes, usage of some of the macros were removed. But definitions still exist in the assembly files.

[PATCH v2 2/2] powerpc/kvm: Remove unused references for MMCR3/SIER2/SIER3 registers

2022-09-16 Thread Disha Goel
From: Kajol Jain" Commit 57dc0eed73ca ("KVM: PPC: Book3S HV P9: Implement PMU save/restore in C") removed the PMU save/restore functions from assembly code and implemented these functions in C, for power9 and later platforms. After the code refactoring, Performance Monitoring Unit (PMU) register

Re: [PATCH 1/6] powerpc/code-patching: Implement generic text patching function

2022-09-16 Thread kernel test robot
Hi Benjamin, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on powerpc/next] [also build test WARNING on next-20220916] [cannot apply to linus/master v6.0-rc5] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we

Re: [PATCH V2] tools/perf/tests: Fix perf probe error log check in skip_if_no_debuginfo

2022-09-16 Thread Disha Goel
On 9/16/22 4:19 PM, Athira Rajeev wrote: The perf probe related tests like probe_vfs_getname.sh which is in "tools/perf/tests/shell" directory have dependency on debuginfo information in the kernel. Currently debuginfo check is handled by skip_if_no_debuginfo function in the file "lib/probe_vfs_

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-09-16 Thread Disha Goel
On 9/13/22 5:27 PM, Athira Rajeev wrote: perf stat includes option to specify aggr_mode to display per-socket, per-core, per-die, per-node counter details. Also there is option -A ( AGGR_NONE, -no-aggr ), where the counter values are displayed for each cpu along with "CPU" value in one field of

Re: [PATCH 1/3] powerpc/perf: Fix branch_filter support for multiple filters in powerpc

2022-09-16 Thread Disha Goel
On 9/13/22 5:25 PM, Athira Rajeev wrote: For PERF_SAMPLE_BRANCH_STACK sample type, different branch_sample_type ie branch filters are supported. The branch filters are requested via event attribute "branch_sample_type". Multiple branch filters can be passed in event attribute. Example: perf rec

Re: [PATCH V2] tools/perf/tests: Fix perf probe error log check in skip_if_no_debuginfo

2022-09-16 Thread kajoljain
On 9/16/22 16:19, Athira Rajeev wrote: > The perf probe related tests like probe_vfs_getname.sh which > is in "tools/perf/tests/shell" directory have dependency on > debuginfo information in the kernel. Currently debuginfo > check is handled by skip_if_no_debuginfo function in the > file "lib/pr

Re: [PATCH 1/3] powerpc/perf: Fix branch_filter support for multiple filters in powerpc

2022-09-16 Thread kajoljain
On 9/16/22 17:02, Disha Goel wrote: > > On 9/13/22 5:25 PM, Athira Rajeev wrote: >> For PERF_SAMPLE_BRANCH_STACK sample type, different branch_sample_type >> ie branch filters are supported. The branch filters are requested via >> event attribute "branch_sample_type". Multiple branch filters ca

Re: [PATCH 2/3] tools/perf/tests: Fix branch stack sampling test to include sanity check for branch filter

2022-09-16 Thread kajoljain
On 9/13/22 17:25, Athira Rajeev wrote: > commit b55878c90ab9 ("perf test: Add test for branch stack sampling") > added test for branch stack sampling. There is a sanity check in the > beginning to skip the test if the hardware doesn't support branch stack > sampling. > > Snippet > <<>> > skip t

Re: [PATCH 3/3] tools/testing/selftests/powerpc: Update the bhrb filter sampling test to test for multiple branch filters

2022-09-16 Thread kajoljain
On 9/13/22 17:25, Athira Rajeev wrote: > For PERF_SAMPLE_BRANCH_STACK sample type, different branch_sample_type, > ie branch filters are supported. The testcase "bhrb_filter_map_test" > tests the valid and invalid filter maps in different powerpc platforms. > Update this testcase to include scen

[PATCH 2/2] powerpc/vmlinux.lds: Add an explicit symbol for the SRWX boundary

2022-09-16 Thread Michael Ellerman
Currently __init_begin is used as the boundary for strict RWX between executable/read-only text and data, and non-executable (after boot) code and data. But that's a little subtle, so add an explicit symbol to document that the SRWX boundary lies there, and add a comment making it clear that __ini

[PATCH 1/2] powerpc/vmlinux.lds: Ensure STRICT_ALIGN_SIZE is at least page aligned

2022-09-16 Thread Michael Ellerman
Add a check that STRICT_ALIGN_SIZE is aligned to at least PAGE_SIZE. That then makes the alignment to PAGE_SIZE immediately after the alignment to STRICT_ALIGN_SIZE redundant, so remove it. Signed-off-by: Michael Ellerman --- arch/powerpc/kernel/vmlinux.lds.S | 5 - 1 file changed, 4 insert

[PATCH] powerpc: Always select HAVE_EFFICIENT_UNALIGNED_ACCESS

2022-09-16 Thread Michael Ellerman
Currently powerpc selects HAVE_EFFICIENT_UNALIGNED_ACCESS in all cases but one. The exception is if the kernel is being built little endian and explicitly targetted for Power7. The combination of Power7 and little endian was never commercially supported, or widely used. It was only ever possible o

Re: [PATCH 1/2] powerpc/vmlinux.lds: Ensure STRICT_ALIGN_SIZE is at least page aligned

2022-09-16 Thread Christophe Leroy
Le 16/09/2022 à 15:14, Michael Ellerman a écrit : > Add a check that STRICT_ALIGN_SIZE is aligned to at least PAGE_SIZE. This cannot happen, the definitions in arch/powerpc/Kconfig don't allow that. > > That then makes the alignment to PAGE_SIZE immediately after the > alignment to STRICT_ALI

[PATCH][next] powerpc/xmon: Fix -Wswitch-unreachable warning in bpt_cmds

2022-09-16 Thread Gustavo A. R. Silva
When building with automatic stack variable initialization, GCC 12 complains about variables defined outside of switch case statements. Move the variable into the case that uses it, which silences the warning: arch/powerpc/xmon/xmon.c: In function ‘bpt_cmds’: arch/powerpc/xmon/xmon.c:1529:13: warn

Re: [PATCH][next] powerpc/xmon: Fix -Wswitch-unreachable warning in bpt_cmds

2022-09-16 Thread Kees Cook
On Fri, Sep 16, 2022 at 03:15:04PM +0100, Gustavo A. R. Silva wrote: > When building with automatic stack variable initialization, GCC 12 > complains about variables defined outside of switch case statements. > Move the variable into the case that uses it, which silences the warning: > > arch/powe

[powerpc:merge] BUILD SUCCESS 22c3e7df0ad978658f6b5fbea580bb4ffdb520b0

2022-09-16 Thread kernel test robot
-a004 i386 randconfig-a011 i386 randconfig-a015 s390 randconfig-r044-20220916 hexagon randconfig-r045-20220916 riscvrandconfig-r042-20220916 hexagon randconfig-r041-20220916 -- 0-DAY CI Kernel Test Ser

[powerpc:next] BUILD SUCCESS f88aabad33ea22be2ce1c60d8901942e4e2a9edb

2022-09-16 Thread kernel test robot
a011 arm64allyesconfig x86_64randconfig-a015 sh allmodconfig ia64 allmodconfig x86_64randconfig-c001 i386 randconfig-c001 arm randconfig-c00

[powerpc:next-test] BUILD SUCCESS 3d7a198cfdb47405cfb4a3ea523876569fe341e6

2022-09-16 Thread kernel test robot
allmodconfig x86_64randconfig-c001 i386 randconfig-c001 arm randconfig-c002-20220916 csky allnoconfig alpha allnoconfig arc

Re: [PATCH v2 6/7] powerpc/64/build: merge .got and .toc input sections

2022-09-16 Thread Christophe Leroy
Le 16/09/2022 à 06:07, Nicholas Piggin a écrit : > Follow the binutils ld internal linker script and merge .got and .toc > input sections in the .got output section. > > Signed-off-by: Nicholas Piggin > --- > arch/powerpc/kernel/vmlinux.lds.S | 3 +-- > 1 file changed, 1 insertion(+), 2 dele

Re: [PATCH] macintosh/ams: Adapt declaration of ams_i2c_remove() to earlier change

2022-09-16 Thread Wolfram Sang
> I don't know how to proceed with this fix. Squashing into the broken > commit is out of the game as the commit is on a stable branch that is > already merged in a few trees. Maybe let it go in via the i2c tree? I think it would be simplest if I put it on top of my for-next branch. The other opt

[linux-next:master] BUILD REGRESSION d5538ab91d3a9a237805be6f8c6c272af2987995

2022-09-16 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: d5538ab91d3a9a237805be6f8c6c272af2987995 Add linux-next specific files for 20220916 Error/Warning reports: https://lore.kernel.org/linux-media/202209162214.lzfxhn2b-...@intel.com https

Re: [PATCH] Revert "powerpc/rtas: Implement reentrant rtas call"

2022-09-16 Thread Nathan Lynch
"Nicholas Piggin" writes: > On Wed Sep 14, 2022 at 3:39 AM AEST, Leonardo Brás wrote: >> On Mon, 2022-09-12 at 14:58 -0500, Nathan Lynch wrote: >> > Leonardo Brás writes: >> > > On Fri, 2022-09-09 at 09:04 -0500, Nathan Lynch wrote: >> > > > Leonardo Brás writes: >> > > > > On Wed, 2022-09-07 at

Re: [PATCH 2/2] Discard .note.gnu.property sections in generic NOTES

2022-09-16 Thread Omar Sandoval
On Tue, Apr 28, 2020 at 06:21:05AM -0700, H.J. Lu wrote: > With the command-line option, -mx86-used-note=yes, the x86 assembler > in binutils 2.32 and above generates a program property note in a note > section, .note.gnu.property, to encode used x86 ISAs and features. But > kernel linker script o

Re: [PATCH 2/2] Discard .note.gnu.property sections in generic NOTES

2022-09-16 Thread Christophe Leroy
Le 16/09/2022 à 21:40, Omar Sandoval a écrit : > [Vous ne recevez pas souvent de courriers de osan...@osandov.com. D?couvrez > pourquoi ceci est important ? https://aka.ms/LearnAboutSenderIdentification ] > > On Tue, Apr 28, 2020 at 06:21:05AM -0700, H.J. Lu wrote: >> With the command-line opti

Re: [PATCH] macintosh/ams: Adapt declaration of ams_i2c_remove() to earlier change

2022-09-16 Thread Michael Ellerman
Wolfram Sang writes: >> I don't know how to proceed with this fix. Squashing into the broken >> commit is out of the game as the commit is on a stable branch that is >> already merged in a few trees. Maybe let it go in via the i2c tree? > > I think it would be simplest if I put it on top of my for

Re: [PATCH v2 6/7] powerpc/64/build: merge .got and .toc input sections

2022-09-16 Thread Michael Ellerman
Christophe Leroy writes: > Le 16/09/2022 à 06:07, Nicholas Piggin a écrit : >> Follow the binutils ld internal linker script and merge .got and .toc >> input sections in the .got output section. >> >> Signed-off-by: Nicholas Piggin >> --- >> arch/powerpc/kernel/vmlinux.lds.S | 3 +-- >> 1 fil

Re: [PATCH 1/2] powerpc/vmlinux.lds: Ensure STRICT_ALIGN_SIZE is at least page aligned

2022-09-16 Thread Michael Ellerman
Christophe Leroy writes: > Le 16/09/2022 à 15:14, Michael Ellerman a écrit : >> Add a check that STRICT_ALIGN_SIZE is aligned to at least PAGE_SIZE. > > This cannot happen, the definitions in arch/powerpc/Kconfig don't allow > that. It can't happen yet :) There's enough combinations of DATA_SHI