Re: [PATCH v1 08/22] powerpc/ftrace: Make __ftrace_make_{nop/call}() common to PPC32 and PPC64

2022-04-17 Thread Naveen N. Rao
Christophe Leroy wrote: Since c93d4f6ecf4b ("powerpc/ftrace: Add module_trampoline_target() for PPC32"), __ftrace_make_nop() for PPC32 is very similar to the one for PPC64. Same for __ftrace_make_call(). Make them common. Signed-off-by: Christophe Leroy --- arch/powerpc/kernel/trace/ftrace.c

Re: [PATCH v1 07/22] powerpc/ftrace: Use patch_instruction() return directly

2022-04-17 Thread Naveen N. Rao
Christophe Leroy wrote: Instead of returning -EPERM when patch_instruction() fails, just return what patch_instruction returns. That simplifies ftrace_modify_code(): 0: 94 21 ff c0 stwur1,-64(r1) 4: 93 e1 00 3c stw r31,60(r1) 8: 7c 7f 1b 79

Re: [PATCH v1 06/22] powerpc/ftrace: Inline ftrace_modify_code()

2022-04-17 Thread Naveen N. Rao
Christophe Leroy wrote: Inlining ftrace_modify_code(), it increases a bit the size of ftrace code but brings 5% improvment on ftrace activation. Signed-off-by: Christophe Leroy --- arch/powerpc/kernel/trace/ftrace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/power

[RFC PATCH] ndtest: Make ndtest a module on its own

2022-04-17 Thread Shivaprasad G Bhat
Today ndtest module is compiled as nfit_test.ko depending on if the CONFIG_ACPI_NFIT defined or not. It is more advantageous to make ndtest a module on its own so that the unit tests can be run serially on the same host without a need for recompilation of sources. The patch modifies the Kbuild fi

[PATCH] powerpc/papr_scm: Move duplicate definitions to common header files

2022-04-17 Thread Shivaprasad G Bhat
papr_scm and ndtest share common PDSM payload structs like nd_papr_pdsm_health. Presently these structs are duplicated across papr_pdsm.h and ndtest.h header files. Since 'ndtest' is essentially arch independent and can run on platforms other than PPC64, a way needs to be deviced to avoid redundanc

[PATCH] ndtest: Cleanup all of blk namespace specific code

2022-04-17 Thread Shivaprasad G Bhat
With the nd_namespace_blk and nd_blk_region infrastructures being removed, the ndtest still has some references to the old code. So the compilation fails as below, ../tools/testing/nvdimm/test/ndtest.c:204:25: error: ‘ND_DEVICE_NAMESPACE_BLK’ undeclared here (not in a function); did you mean ‘ND_

Re: [PATCH v4 2/2] PCI/DPC: Disable DPC service when link is in L2/L3 ready, L2 and L3 state

2022-04-17 Thread Sathyanarayanan Kuppuswamy
On 4/8/22 8:31 AM, Kai-Heng Feng wrote: On Intel Alder Lake platforms, Thunderbolt entering D3cold can cause some errors reported by AER: [ 30.100211] pcieport :00:1d.0: AER: Uncorrected (Non-Fatal) error received: :00:1d.0 [ 30.100251] pcieport :00:1d.0: PCIe Bus Error: sever

[PATCH] macintosh: macio_asic: fix resource_size.cocci warnings

2022-04-17 Thread Yihao Han
drivers/macintosh/macio_asic.c:219:26-29: WARNING: Suspicious code. resource_size is maybe missing with res drivers/macintosh/macio_asic.c:221:26-29: WARNING: Suspicious code. resource_size is maybe missing with res Use resource_size function on resource object instead of explicit computation. Ge