Re: [PATCH] powerpc/qspinlock: Fix 32-bit build

2022-12-12 Thread Michael Ellerman
On Thu, 8 Dec 2022 22:32:25 +1000, Nicholas Piggin wrote: > Some 32-bit configurations don't pull in the spin_begin/end/relax > definitions. Fix is to restore a lost include. > > Applied to powerpc/next. [1/1] powerpc/qspinlock: Fix 32-bit build

Re: [PATCH v1 5/5] powerpc/64e: Fix build failure with GCC 12 (unrecognized opcode: `wrteei')

2022-12-12 Thread Michael Ellerman
Christophe Leroy writes: > Le 11/12/2022 à 18:32, Pali Rohár a écrit : >> On Monday 11 July 2022 16:19:33 Christophe Leroy wrote: >>> With GCC 12, corenet64_smp_defconfig leads to the following build errors: >>> >>>CC arch/powerpc/kernel/irq.o >>> {standard input}: Assembler messages:

Re: [PATCH v6 00/16] objtool: Enable and implement --mcount option on powerpc

2022-12-08 Thread Michael Ellerman
On Mon, 14 Nov 2022 23:27:38 +0530, Sathvika Vasireddy wrote: > This patchset enables and implements objtool --mcount > option on powerpc. This applies atop powerpc/merge branch. > > Changelog: > > > v6: > > [...] Applied to powerpc/next (and topic/objtool). [01/16] powerpc: Fix

Re: [PATCH] powerpc/64s: Add missing declaration for machine_check_early_boot()

2022-12-08 Thread Michael Ellerman
On Sat, 26 Nov 2022 00:25:21 +1100, Michael Ellerman wrote: > There's no declaration for machine_check_early_boot(), which leads to a > build failure with W=1. Add one. > > Applied to powerpc/fixes. [1/1] powerpc/64s: Add missing declaration for machine_check_early_boot()

Re: [PATCH v2] powerpc/bpf/32: Fix Oops on tail call tests

2022-12-08 Thread Michael Ellerman
On Thu, 24 Nov 2022 09:37:27 +0100, Christophe Leroy wrote: > test_bpf tail call tests end up as: > > test_bpf: #0 Tail call leaf jited:1 85 PASS > test_bpf: #1 Tail call 2 jited:1 111 PASS > test_bpf: #2 Tail call 3 jited:1 145 PASS > test_bpf: #3 Tail call 4 jited:1 170 PASS >

Re: [PATCH v2] powerpc/83xx/mpc832x_rdb: call platform_device_put() in error case in of_fsl_spi_probe()

2022-12-08 Thread Michael Ellerman
On Sat, 29 Oct 2022 19:16:26 +0800, Yang Yingliang wrote: > If platform_device_add() is not called or failed, it can not call > platform_device_del() to clean up memory, it should call > platform_device_put() in error case. > > Applied to powerpc/next. [1/1] powerpc/83xx/mpc832x_rdb: call

Re: [PATCH] selftests: powerpc: Use "grep -E" instead of "egrep"

2022-12-08 Thread Michael Ellerman
On Thu, 1 Dec 2022 10:49:57 +0800, Tiezhu Yang wrote: > The latest version of grep claims the egrep is now obsolete so the build > now contains warnings that look like: > egrep: warning: egrep is obsolescent; using grep -E > fix this using "grep -E" instead. > > sed -i "s/egrep/grep -E/g"

Re: [PATCH] KVM: PPC: Use the arg->size directly for kvm_vm_ioctl_create_spapr_tce

2022-12-08 Thread Michael Ellerman
On Sun, 3 Jul 2022 13:29:32 -0400, Deming Wang wrote: > Use arg->size directly may be better for code readability.Because, the > variable of size has not been found for special purpose at present. > Also,We can reduce the use of a variable. > > Applied to powerpc/next. [1/1] KVM: PPC: Use the

Re: [PATCH v5 1/7] powerpc/64: Add INTERRUPT_SANITIZE_REGISTERS Kconfig

2022-12-08 Thread Michael Ellerman
On Thu, 1 Dec 2022 18:10:13 +1100, Rohan McLure wrote: > Add Kconfig option for enabling clearing of registers on arrival in an > interrupt handler. This reduces the speculation influence of registers > on kernel internals. The option will be consumed by 64-bit systems that > feature speculation

Re: [PATCH] powerpc/fsl-pci: Choose PCI host bridge with alias pci0 as the primary

2022-12-08 Thread Michael Ellerman
On Sat, 20 Aug 2022 14:33:27 +0200, Pali Rohár wrote: > If there's no PCI host bridge with ISA then check for PCI host bridge with > alias "pci0" (first PCI host bridge) and if it exists then choose it as the > primary PCI host bridge. > > This makes choice of primary PCI host bridge more stable

Re: [PATCH] powerpc: dts: turris1x.dts: Add channel labels for temperature sensor

2022-12-08 Thread Michael Ellerman
On Fri, 30 Sep 2022 14:39:01 +0200, Pali Rohár wrote: > Channel 0 of SA56004ED chip refers to internal SA56004ED chip sensor (chip > itself is located on the board) and channel 1 of SA56004ED chip refers to > external sensor which is connected to temperature diode of the P2020 CPU. > > Applied

Re: [PATCH v6 0/4] Option to build big-endian with ELFv2 ABI

2022-12-08 Thread Michael Ellerman
On Mon, 28 Nov 2022 14:15:35 +1000, Nicholas Piggin wrote: > This is hopefully the final attempt. Luis was happy for the module > patch to go via the powerpc tree, so I've put the the ELFv2 for big > endian build patches into the series. Hopefully we can deprecate > the ELFv1 ABI > > Since v5, I

Re: (subset) [PATCH v3 00/17] powerpc: alternate queued spinlock implementation

2022-12-08 Thread Michael Ellerman
On Sat, 26 Nov 2022 19:59:15 +1000, Nicholas Piggin wrote: > This replaces the generic queued spinlock code (like s390 does) with > our own implementation. There is an extra shim patch 1a to get the > series to apply. > > Generic PV qspinlock code is causing latency / starvation regressions on >

Re: [PATCH] powerpc/64s/hash: add stress_hpt kernel boot option to increase hash faults

2022-12-08 Thread Michael Ellerman
On Mon, 24 Oct 2022 13:01:50 +1000, Nicholas Piggin wrote: > This option increases the number of hash misses by limiting the number > of kernel HPT entries, by keeping a per-CPU record of the last kernel > HPTEs installed, and removing that from the hash table on the next hash > insertion. A timer

Re: [PATCH 00/17] powerpc: Remove STACK_FRAME_OVERHEAD

2022-12-08 Thread Michael Ellerman
On Sun, 27 Nov 2022 22:49:25 +1000, Nicholas Piggin wrote: > Since RFC: > - Fix a compile bug. > - Fix BookE KVM properly. Hopefully -- I don't have a BookE > KVM environment to test. Can QEMU do it? Is it still tested? > - Drop the last two patches that changed the stack layout, they > can be

Re: [PATCH 0/3] selftests/powerpc: ptrace/perf_hwbreak test fixes

2022-12-08 Thread Michael Ellerman
On Tue, 22 Nov 2022 12:10:51 +0530, Naveen N. Rao wrote: > This series fixes two issues with the perf_hwbreak test: > - Test fails on systems with large number of cpus due to hitting the > resource limit for maximum number of open file descriptors. > - Test fails on systems running with lower

Re: [PATCH 00/13] RTAS maintenance

2022-12-08 Thread Michael Ellerman
On Fri, 18 Nov 2022 09:07:38 -0600, Nathan Lynch wrote: > A collection of loosely-related RTAS code changes, most notably: > > * Fixing misuses of rtas_token() for non-function properties. > * The stronger validation for sys_rtas() offered by the > PPC_RTAS_FILTER config option is always

Re: [PATCH] powerpc/ftrace: fix syscall tracing on PPC64_ELF_ABI_V1

2022-12-08 Thread Michael Ellerman
On Thu, 1 Dec 2022 11:14:42 -0500, Michael Jeanson wrote: > In v5.7 the powerpc syscall entry/exit logic was rewritten in C, on > PPC64_ELF_ABI_V1 this resulted in the symbols in the syscall table > changing from their dot prefixed variant to the non-prefixed ones. > > Since ftrace prefixes a dot

Re: (subset) [PATCH 1/3] firmware_loader: remove #include

2022-12-08 Thread Michael Ellerman
On Sat, 26 Nov 2022 06:09:59 +0100, Thomas Weißschuh wrote: > utsrelease.h is potentially generated on each build. > By removing this unused include we can get rid of some spurious > recompilations. > > Applied to powerpc/next. [2/3] powerpc/book3e: remove #include

Re: [PATCH] selftests/powerpc: Fix resource leaks

2022-12-08 Thread Michael Ellerman
On Mon, 5 Dec 2022 12:44:27 +0400, Miaoqian Lin wrote: > In check_all_cpu_dscr_defaults, opendir() opens the directory stream. > Add missing closedir() in the error path to release it. > > In check_cpu_dscr_default, open() creates an open file descriptor. > Add missing close() in the error path

Re: [PATCH v2 0/2] Consider the size of the added CPU nodes in the kexec FDT

2022-12-08 Thread Michael Ellerman
On Thu, 10 Nov 2022 19:06:17 +0100, Laurent Dufour wrote: > When adding CPUs to an already big system (test show it seems to start with > more than 256 CPUs), the kernel is showing error messages when building the > FDT for the kexec kernel (kdump or kexec). > > It's worth to mention that the

Re: [PATCH] pseries/mobility: reset the RCU watchdogs after a LPM

2022-12-08 Thread Michael Ellerman
On Fri, 25 Nov 2022 18:32:04 +0100, Laurent Dufour wrote: > The RCU watchdog timer should be reset when restarting the CPU after a Live > Partition Mobility operation. > > Applied to powerpc/next. [1/1] pseries/mobility: reset the RCU watchdogs after a LPM

Re: [PATCH] powerpc/pseries: unregister VPA when hot unplugging a CPU

2022-12-08 Thread Michael Ellerman
On Mon, 14 Nov 2022 17:01:50 +0100, Laurent Dufour wrote: > The VPA should unregister when offlining a CPU. Otherwise there could be a > short window where 2 CPUs could share the same VPA. > > This happens because the hypervisor is still keeping the VPA attached to > the vCPU even if it became

Re: [PATCH v3] powerpc/hv-gpci: Fix hv_gpci event list

2022-12-08 Thread Michael Ellerman
On Wed, 30 Nov 2022 23:15:13 +0530, Kajol Jain wrote: > Based on getPerfCountInfo v1.018 documentation, some of the > hv_gpci events were deprecated for platform firmware that > supports counter_info_version 0x8 or above. > > Fix the hv_gpci event list by adding a new attribute group > called

Re: [PATCH] KVM: Fix spelling mistake

2022-12-08 Thread Michael Ellerman
On Thu, 23 Jun 2022 18:20:31 +0800, Zhang Jiaming wrote: > Change 'subsquent' to 'subsequent'. > Change 'accross' to 'across'. > > Applied to powerpc/next. [1/1] KVM: Fix spelling mistake https://git.kernel.org/powerpc/c/392a58f1eaab0c90b80d7ba4a03dbf6eaaeabe60 cheers

Re: [PATCH] cxl: Remove unnecessary cxl_pci_window_alignment()

2022-12-08 Thread Michael Ellerman
On Mon, 5 Dec 2022 16:32:31 -0600, Bjorn Helgaas wrote: > From: Bjorn Helgaas > > cxl_pci_window_alignment() is referenced only via the struct > pci_controller_ops.window_alignment function pointer, and only in the > powerpc implementation of pcibios_window_alignment(). > >

Re: [PATCH] powerpc/ps3: mark ps3_system_bus_type static

2022-12-08 Thread Michael Ellerman
On Tue, 22 Nov 2022 08:22:25 +0100, Christoph Hellwig wrote: > ps3_system_bus_type is only used inside of system-bus.c, so remove > the external declaration and the very outdated comment next to it. > > Applied to powerpc/next. [1/1] powerpc/ps3: mark ps3_system_bus_type static

Re: (subset) [PATCH 00/11] Fix pca954x i2c-mux node names

2022-12-08 Thread Michael Ellerman
On Fri, 2 Dec 2022 17:49:15 +0100, Geert Uytterhoeven wrote: > Hi all, > > According to the I2C bus multiplexer/switch DT bindings, i2c-mux nodes > should be named "i2c-mux" (or something similar). > This patch series renames nodes for pca954x i2c-muxes that are flagged > by > > [...]

Re: [PATCH v2 1/5] powerpc/code-patching: Remove #ifdef CONFIG_STRICT_KERNEL_RWX

2022-12-08 Thread Michael Ellerman
On Fri, 2 Dec 2022 09:31:39 +0100, Christophe Leroy wrote: > No need to have one implementation of patch_instruction() for > CONFIG_STRICT_KERNEL_RWX and one for !CONFIG_STRICT_KERNEL_RWX. > > In patch_instruction(), call raw_patch_instruction() when > !CONFIG_STRICT_KERNEL_RWX. > > In

Re: [PATCH] KVM: PPC: Book3S HV: Use the bitmap API to allocate bitmaps

2022-12-08 Thread Michael Ellerman
On Sat, 9 Jul 2022 17:56:43 +0200, Christophe JAILLET wrote: > Use bitmap_zalloc()/bitmap_free() instead of hand-writing them. > > It is less verbose and it improves the semantic. > > Applied to powerpc/next. [1/1] KVM: PPC: Book3S HV: Use the bitmap API to allocate bitmaps

Re: [PATCH] KVM: PPC: use __func__ to get funcion's name in an output message

2022-12-08 Thread Michael Ellerman
On Fri, 17 Jun 2022 23:50:19 +0800 (GMT+08:00), XueBing Chen wrote: > Prefer using '"%s...", __func__' to get current function's name in > an output message. > > Applied to powerpc/next. [1/1] KVM: PPC: use __func__ to get funcion's name in an output message

Re: [PATCH v2] powerpc/cpuidle: Set CPUIDLE_FLAG_POLLING for snooze state

2022-12-08 Thread Michael Ellerman
On Mon, 14 Nov 2022 20:26:11 +0530, Aboorva Devarajan wrote: > During the comparative study of cpuidle governors, it is noticed that the > menu governor does not select CEDE state in some scenarios even though when > the sleep duration of the CPU exceeds the target residency of the CEDE idle >

Re: (subset) [PATCH v3 0/7] Expand selftest utils

2022-12-08 Thread Michael Ellerman
On Mon, 28 Nov 2022 15:19:41 +1100, Benjamin Gray wrote: > Started this when writing tests for a feature I'm working on, needing a way to > read/write numbers to system files. After writing some utils to safely handle > file IO and parsing, I realised I'd made the ~6th file read/write >

Re: [PATCH v10 0/9] powerpc/code-patching: Use temporary mm for Radix MMU

2022-12-08 Thread Michael Ellerman
On Wed, 9 Nov 2022 15:51:03 +1100, Benjamin Gray wrote: > This is a revision of Chris and Jordan's series to introduce a per-cpu > temporary > mm to be used for patching with strict rwx on radix mmus. > > v10: * Don't set poking_init_enabled on boot CPU init failure > * Remove extern from

Re: [PATCH] powerpc: qspinlock: Use asm-generic definition for queued_spin_lock

2022-12-07 Thread Michael Ellerman
Rohan McLure writes: > asm-generic/qspinlock.h provides an identical implementation of > queued_spin_lock. Remove the variant in asm/qspinlock.h. This code has changed recently, so this patch no longer applies. See 9f61521c7a28 ("powerpc/qspinlock: powerpc qspinlock implementation") in

Re: [PATCH] powerpc/ftrace: fix syscall tracing on PPC64_ELF_ABI_V1

2022-12-06 Thread Michael Ellerman
Mathieu Desnoyers writes: > On 2022-12-05 17:50, Michael Ellerman wrote: >> Michael Jeanson writes: >>> On 2022-12-05 15:11, Michael Jeanson wrote: >>>>>>> Michael Jeanson writes: >>>>>>>> In v5.7 the powerpc syscall en

Re: [PATCH v5 5/5] powerpc: mm: support page table check

2022-12-06 Thread Michael Ellerman
Rohan McLure writes: > On creation and clearing of a page table mapping, instrument such calls > by invoking page_table_check_pte_set and page_table_check_pte_clear > respectively. These calls serve as a sanity check against illegal > mappings. > > Enable ARCH_SUPPORTS_PAGE_TABLE_CHECK for all

Re: [PATCH v5 2/5] powerpc: mm: Implement p{m,u,4}d_leaf on all platforms

2022-12-06 Thread Michael Ellerman
Rohan McLure writes: > The check that a higher-level entry in multi-level pages contains a page > translation entry (pte) is performed by p{m,u,4}d_leaf stubs, which may > be specialised for each choice of mmu. In a prior commit, we replace > uses to the catch-all stubs, p{m,u,4}d_is_leaf with

Re: [PATCH] powerpc/ftrace: fix syscall tracing on PPC64_ELF_ABI_V1

2022-12-05 Thread Michael Ellerman
Michael Jeanson writes: > On 2022-12-05 15:11, Michael Jeanson wrote: > Michael Jeanson writes: >> In v5.7 the powerpc syscall entry/exit logic was rewritten in C, on >> PPC64_ELF_ABI_V1 this resulted in the symbols in the syscall table >> changing from their dot prefixed variant

Re: [PATCH] powerpc/ftrace: fix syscall tracing on PPC64_ELF_ABI_V1

2022-12-04 Thread Michael Ellerman
+ > Cc: Steven Rostedt > Cc: Masami Hiramatsu > Cc: Mark Rutland > Cc: Michael Ellerman > Cc: Nicholas Piggin > Cc: Christophe Leroy > Cc: Michal Suchanek > Cc: linuxppc-dev@lists.ozlabs.org > Cc: linux-ker...@vger.kernel.org > Signed-off-by: Michael Jeanson

[GIT PULL] Please pull powerpc/linux.git powerpc-6.1-6 tag

2022-12-04 Thread Michael Ellerman
. - -- Christophe Leroy (1): powerpc/bpf/32: Fix Oops on tail call tests Michael Ellerman (1): powerpc/64s: Add missing declaration for machine_check_early_boot() arch/powerpc/include/asm/interrupt.h | 1 + arch/powerpc/net/bpf_jit_comp32.c| 52 2 files changed, 22

Re: [PATCH v3 3/7] selftests/powerpc: Add generic read/write file util

2022-12-02 Thread Michael Ellerman
Benjamin Gray writes: > File read/write is reimplemented in about 5 different ways in the > various PowerPC selftests. This indicates it should be a common util. > > Add a common read_file / write_file implementation and convert users > to it where (easily) possible. > > Signed-off-by: Benjamin

Re: linux-next: build failure after merge of the tip tree

2022-12-02 Thread Michael Ellerman
Stephen Rothwell writes: > Hi all, > > After merging the tip tree, today's linux-next build (powerpc > ppc64_defconfig) failed like this: > > arch/powerpc/lib/code-patching.c: In function 'text_area_cpu_up_mm': > arch/powerpc/lib/code-patching.c:157:14: error: implicit declaration of > function

Re: linux-next: boot failure after merge of the powerpc tree

2022-12-02 Thread Michael Ellerman
Stephen Rothwell writes: > Hi all, > > After merging all the trees, today's linux-next qemu run (powerpc > pseries_le_defconfig with kvm) crashed like this: > > Memory: 2029504K/2097152K available (14592K kernel code, 2944K rwdata, 18176K > rodata, 5120K init, 1468K bss, 67648K reserved, 0K

Re: [powerpc:next] BUILD REGRESSION 3604661f206238a0295df53e38b7986a91c00cc4

2022-12-01 Thread Michael Ellerman
kernel test robot writes: > tree/branch: > https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next > branch HEAD: 3604661f206238a0295df53e38b7986a91c00cc4 Merge branch > 'topic/qspinlock' into next > > Error/Warning reports: > >

Re: [PATCH 0/5] remove label = "cpu" from DSA dt-binding

2022-12-01 Thread Michael Ellerman
Arınç ÜNAL writes: > On 30.11.2022 18:55, Andrew Lunn wrote: >> On Wed, Nov 30, 2022 at 05:10:35PM +0300, Arınç ÜNAL wrote: >>> Hello folks, >>> >>> With this patch series, we're completely getting rid of 'label = "cpu";' >>> which is not used by the DSA dt-binding at all. >>> >>> Information for

Re: [PATCH 5/5] powerpc: dts: remove label = "cpu" from DSA dt-binding

2022-12-01 Thread Michael Ellerman
Arınç ÜNAL writes: > This is not used by the DSA dt-binding, so remove it from all devicetrees. > > Signed-off-by: Arınç ÜNAL > --- > arch/powerpc/boot/dts/turris1x.dts | 2 -- > 1 file changed, 2 deletions(-) Adding Pali to Cc. These were only recently updated in commit: 8bf056f57f1d

Re: [PATCH v2 26/50] KVM: PPC: Move processor compatibility check to module init

2022-11-30 Thread Michael Ellerman
24: note: each undeclared identifier is reported only once for each function it appears in It needs the delta below to compile. With that: Acked-by: Michael Ellerman (powerpc) cheers diff --git a/arch/powerpc/kvm/e500mc.c b/arch/powerpc/kvm/e500mc.c index 795667f7ebf

Re: [syzbot] WARNING in btrfs_free_reserved_data_space_noquota

2022-11-30 Thread Michael Ellerman
syzbot writes: > Hello, > > syzbot found the following issue on: > > HEAD commit:b7b275e60bcd Linux 6.1-rc7 > git tree: upstream > console+strace: https://syzkaller.appspot.com/x/log.txt?x=158a7b7388 > kernel config: https://syzkaller.appspot.com/x/.config?x=2325e409a9a893e1 >

Re: [PATCH] macintosh/windfarm_pid: Add header file macro definition

2022-11-30 Thread Michael Ellerman
On Thu, 7 Jul 2022 09:59:49 +0800, Li zeming wrote: > I think the header file could avoid redefinition errors. > at compile time by adding macro definitions. > > Applied to powerpc/next. [1/1] macintosh/windfarm_pid: Add header file macro definition

Re: [PATCH] macintosh/ams/ams: Add header file macro definition

2022-11-30 Thread Michael Ellerman
On Thu, 7 Jul 2022 09:53:52 +0800, Li zeming wrote: > Add header file macro definition. > > Applied to powerpc/next. [1/1] macintosh/ams/ams: Add header file macro definition https://git.kernel.org/powerpc/c/2dfcace75e1e1dfbd89af63fce1bfe8aebe38427 cheers

Re: [PATCH v2] ocxl: fix pci device refcount leak when calling get_function_0()

2022-11-30 Thread Michael Ellerman
On Mon, 21 Nov 2022 23:43:39 +0800, Yang Yingliang wrote: > get_function_0() calls pci_get_domain_bus_and_slot(), as comment > says, it returns a pci device with refcount increment, so after > using it, pci_dev_put() needs be called. > > Get the device reference when get_function_0() is not

Re: [PATCH v2] macintosh: fix possible memory leak in macio_add_one_device()

2022-11-30 Thread Michael Ellerman
On Fri, 4 Nov 2022 11:25:51 +0800, Yang Yingliang wrote: > Afer commit 1fa5ae857bb1 ("driver core: get rid of struct device's > bus_id string array"), the name of device is allocated dynamically, > it need be freed when of_device_register() fails, call put_device() > to give up the reference that

Re: [PATCH] powerpc/xive: add missing iounmap() in error path in xive_spapr_populate_irq_data()

2022-11-30 Thread Michael Ellerman
On Mon, 17 Oct 2022 11:23:33 +0800, Yang Yingliang wrote: > If remapping 'data->trig_page' fails, the 'data->eoi_mmio' need be unmapped > before returning from xive_spapr_populate_irq_data(). > > Applied to powerpc/next. [1/1] powerpc/xive: add missing iounmap() in error path in

Re: [PATCH -next] macintosh: Switch to use for_each_child_of_node() macro

2022-11-30 Thread Michael Ellerman
On Fri, 16 Sep 2022 22:16:38 +0800, Yang Yingliang wrote: > Use for_each_child_of_node() macro instead of open coding it. > No functional change. > > Applied to powerpc/next. [1/1] macintosh: Switch to use for_each_child_of_node() macro

Re: [PATCH] misc: ocxl: fix possible name leak in ocxl_file_register_afu()

2022-11-30 Thread Michael Ellerman
On Fri, 11 Nov 2022 22:59:29 +0800, Yang Yingliang wrote: > If device_register() returns error in ocxl_file_register_afu(), > the name allocated by dev_set_name() need be freed. As comment > of device_register() says, it should use put_device() to give > up the reference in the error path. So fix

Re: [PATCH 1/2] cxl: fix possible null-ptr-deref in cxl_guest_init_afu|adapter()

2022-11-30 Thread Michael Ellerman
On Fri, 11 Nov 2022 22:54:39 +0800, Yang Yingliang wrote: > If device_register() fails in cxl_register_afu|adapter(), the device > is not added, device_unregister() can not be called in the error path, > otherwise it will cause a null-ptr-deref because of removing not added > device. > > As

Re: [PATCH] powerpc: Use "grep -E" instead of "egrep"

2022-11-30 Thread Michael Ellerman
On Fri, 18 Nov 2022 17:40:29 +0800, Tiezhu Yang wrote: > The latest version of grep claims the egrep is now obsolete so the build > now contains warnings that look like: > egrep: warning: egrep is obsolescent; using grep -E > fix this up by moving the related file to use "grep -E" instead. >

Re: [PATCH] powerpc: pasemi: add __init/__exit annotations to module init/exit funcs

2022-11-30 Thread Michael Ellerman
On Sun, 11 Sep 2022 16:43:44 +0800, Xiu Jianfeng wrote: > Add missing __init/__exit annotations to module init/exit funcs. > > Applied to powerpc/next. [1/1] powerpc: pasemi: add __init/__exit annotations to module init/exit funcs

Re: [PATCH] powerpc/xive: remove unused parameter

2022-11-30 Thread Michael Ellerman
On Wed, 13 Apr 2022 06:55:07 -0400, Deming wrote: > From: Deming Wang > > the parameter xc balongng to xive_cleanup_single_escalation is unused,so we > need to remove it. > > Applied to powerpc/next. [1/1] powerpc/xive: remove unused parameter

Re: [PATCH] powerpc: Fix formatting problems to make code look more beautiful

2022-11-30 Thread Michael Ellerman
On Fri, 1 Jul 2022 05:45:53 -0400, Deming Wang wrote: > Operators should be separated by spaces in tce_buildmulti_pSeriesLP > > Applied to powerpc/next. [1/1] powerpc: Fix formatting problems to make code look more beautiful

Re: (subset) [PATCH 000/606] i2c: Complete conversion to i2c_probe_new

2022-11-30 Thread Michael Ellerman
On Fri, 18 Nov 2022 23:35:34 +0100, Uwe Kleine-König wrote: > since commit b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new() > call-back type") from 2016 there is a "temporary" alternative probe > callback for i2c drivers. > > This series completes all drivers to this new callback (unless I

Re: [PATCH] mm: check the return value of ioremap() in macio_init()

2022-11-30 Thread Michael Ellerman
On Tue, 2 Aug 2022 15:41:48 +0800, studentxs...@163.com wrote: > From: Xie Shaowen > > The function ioremap() in macio_init() can fail, so > its return value should be checked. > > Applied to powerpc/next. [1/1] mm: check the return value of ioremap() in macio_init()

Re: [PATCH v2] macintosh: via-pmu-backlight: Use backlight helper

2022-11-30 Thread Michael Ellerman
On Thu, 16 Jun 2022 19:04:24 +0200, Stephen Kitt wrote: > backlight_properties.fb_blank is deprecated. The states it represents > are handled by other properties; but instead of accessing those > properties directly, drivers should use the helpers provided by > backlight.h. > > Instead of

Re: [PATCH] powerpc: suppress some linker warnings in recent linker versions

2022-11-30 Thread Michael Ellerman
On Mon, 10 Oct 2022 16:57:21 +1100, Stephen Rothwell wrote: > This is a follow on from commit > > 0d362be5b142 ("Makefile: link with -z noexecstack --no-warn-rwx-segments") > > for arch/powerpc/boot to address wanrings like: > > ld: warning: opal-calls.o: missing .note.GNU-stack section

Re: [PATCH] powerpc/8xx: Fix warning in hw_breakpoint_handler()

2022-11-30 Thread Michael Ellerman
On Mon, 24 Oct 2022 15:13:46 +1100, Russell Currey wrote: > In hw_breakpoint_handler(), ea is set by wp_get_instr_detail() except > for 8xx, leading the variable to be passed uninitialised to > wp_check_constraints(). This is safe as wp_check_constraints() returns > early without using ea, so

Re: [PATCH -next v2] powerpc/mpic_msgr: fix cast removes address space of expression warnings

2022-11-30 Thread Michael Ellerman
On Wed, 19 Oct 2022 14:34:14 +0800, ruanjinjie wrote: > When build Linux kernel, encounter the following warnings: > > ./arch/powerpc/sysdev/mpic_msgr.c:230:38: warning: cast removes address space > '__iomem' of expression > ./arch/powerpc/sysdev/mpic_msgr.c:230:27: warning: incorrect type in >

Re: [PATCH] powerpc/mpc52xx_lpbfifo: fix all kernel-doc warnings

2022-11-30 Thread Michael Ellerman
On Wed, 23 Nov 2022 22:19:18 -0800, Randy Dunlap wrote: > Fix multiple kernel-doc warnings in mpc52xx_lpbfifo.c: > > arch/powerpc/platforms/52xx/mpc52xx_lpbfifo.c:377: warning: expecting > prototype for mpc52xx_lpbfifo_bcom_poll(). Prototype was for > mpc52xx_lpbfifo_poll() instead > >

Re: [PATCH v3] powerpc: add compile-time support for lbarx, lharx

2022-11-30 Thread Michael Ellerman
On Fri, 9 Sep 2022 15:23:12 +1000, Nicholas Piggin wrote: > ISA v2.06 (POWER7 and up) as well as e6500 support lbarx and lharx. > Add a compile option that allows code to use it, and add support in > cmpxchg and xchg 8 and 16 bit values without shifting and masking. > > Applied to powerpc/next.

Re: [PATCH] powerpc: remove the last remnants of cputime_t

2022-11-30 Thread Michael Ellerman
On Thu, 6 Oct 2022 20:56:53 +1000, Nicholas Piggin wrote: > cputime_t was a core kernel type, removed by commits > ed5c8c854f2b..b672592f0221. As explained in commit b672592f0221 > ("sched/cputime: Remove generic asm headers"), the final cleanup > is for the arch to provide cputime_to_nsec[s]().

Re: [PATCH] docs: powerpc: add POWER9 and POWER10 to CPU families

2022-11-30 Thread Michael Ellerman
On Wed, 10 Aug 2022 04:03:21 +, Nicholas Miehlbradt wrote: > Add POWER9 and POWER10 to CPU families and list Radix MMU. > > Applied to powerpc/next. [1/1] docs: powerpc: add POWER9 and POWER10 to CPU families https://git.kernel.org/powerpc/c/addebe1cfa71eb29caa9d5c6bc719171e4e76414

Re: [PATCH 0/6] powerpc/pseries - bugfixes/cleanups for PLPKS driver

2022-11-30 Thread Michael Ellerman
On Sun, 6 Nov 2022 15:58:33 -0500, Nayna Jain wrote: > This patchset fixes some bugs and does some cleanups. > > Nayna Jain (6): > powerpc/pseries: fix the object owners enum value in plpks driver > powerpc/pseries: Fix the H_CALL error code in PLPKS driver > powerpc/pseries: Return -EIO

Re: [PATCH 1/2] powerpc: Make instruction dump work with scripts/decodecode

2022-11-30 Thread Michael Ellerman
On Thu, 6 Oct 2022 14:20:18 +1100, Michael Ellerman wrote: > Matt reported that scripts/decodecode doesn't work for the instruction > dump in the powerpc oops output. Although there are scripts around that > can decode it, it would be preferable if the standard in-tree script > work

Re: [PATCH] powerpc/ftrace: Ignore weak functions

2022-11-30 Thread Michael Ellerman
On Tue, 9 Aug 2022 16:24:25 +0530, Naveen N. Rao wrote: > Extend commit b39181f7c6907d ("ftrace: Add FTRACE_MCOUNT_MAX_OFFSET to > avoid adding weak function") to ppc32 and ppc64 -mprofile-kernel by > defining FTRACE_MCOUNT_MAX_OFFSET. > > For ppc64 -mprofile-kernel ABI, we can have two

Re: (subset) [PATCH 0/5] powerpc/kprobes: preempt related changes and cleanups

2022-11-30 Thread Michael Ellerman
On Thu, 20 Oct 2022 22:58:56 +0530, Naveen N. Rao wrote: > This series attempts to address some of the concerns raised in > https://github.com/linuxppc/issues/issues/440 > > The last two patches are minor cleanups in related kprobes code. > > - Naveen > > [...] Applied to powerpc/next. [1/5]

Re: [PATCH] powerpc/pseries/eeh: Fix some kernel-doc warnings

2022-11-30 Thread Michael Ellerman
On Mon, 31 Oct 2022 02:37:06 -0400, Bo Liu wrote: > Fixes the following W=1 kernel build warning(s): > arch/powerpc/platforms/pseries/eeh_pseries.c:163: warning: Function > parameter or member 'config_addr' not described in 'pseries_eeh_phb_reset' >

Re: [PATCH] cxl: Fix refcount leak in cxl_calc_capp_routing

2022-11-30 Thread Michael Ellerman
On Sun, 5 Jun 2022 10:00:38 +0400, Miaoqian Lin wrote: > of_get_next_parent() returns a node pointer with refcount incremented, > we should use of_node_put() on it when not need anymore. > This function only calls of_node_put() in normal path, > missing it in the error path. > Add missing

Re: [PATCH] cxl: fix typo in comment

2022-11-30 Thread Michael Ellerman
On Sat, 21 May 2022 13:11:32 +0200, Julia Lawall wrote: > Spelling mistake (triple letters) in comment. > Detected with the help of Coccinelle. > > Applied to powerpc/next. [1/1] cxl: fix typo in comment https://git.kernel.org/powerpc/c/37195edebf479b94f1e20c2a83a29e4beebe7ff5 cheers

Re: [PATCH v2] powerpc/microwatt: Add litesd

2022-11-30 Thread Michael Ellerman
On Fri, 30 Sep 2022 16:20:12 +0930, Joel Stanley wrote: > This is the register layout of the litesd peripheral for the fusesoc > based Microwatt SoC. > > It requires a description of the system clock, which is hardcoded to > 100MHz. > > > [...] Applied to powerpc/next. [1/1]

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

2022-11-30 Thread Michael Ellerman
On Fri, 16 Sep 2022 15:15:04 +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: > >

Re: [PATCH v2] macintosh/via-pmu: Avoid compiler warnings when CONFIG_PROC_FS is disabled

2022-11-30 Thread Michael Ellerman
On Mon, 21 Mar 2022 20:28:00 +1100, Finn Thain wrote: > drivers/macintosh/via-pmu.c:897:12: warning: 'pmu_battery_proc_show' defined > but not used [-Wunused-function] > static int pmu_battery_proc_show(struct seq_file *m, void *v) > ^ >

Re: [PATCH] powerpc/warp: switch to using gpiod API

2022-11-30 Thread Michael Ellerman
On Mon, 26 Sep 2022 23:03:25 -0700, Dmitry Torokhov wrote: > This switches PIKA Warp away from legacy gpio API and to newer gpiod > API, so that we can eventually deprecate the former. > > Because LEDs are normally driven by leds-gpio driver, but the > platform code also wants to access the LEDs

Re: [PATCH] powerpc/sgy_cts1000: convert to using gpiod API and facelift

2022-11-30 Thread Michael Ellerman
On Tue, 27 Sep 2022 12:23:58 -0700, Dmitry Torokhov wrote: > This patch converts the driver to newer gpiod API, and away from > OF-specific legacy gpio API that we want to stop using. > > While at it, let's address a few more issues: > > - switch to using dev_info()/pr_info() and friends > -

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

2022-11-30 Thread Michael Ellerman
On Fri, 16 Sep 2022 16:27:34 +0530, Disha Goel wrote: > 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

Re: [PATCH] selftests/powerpc: Remove repeated word in comments

2022-11-30 Thread Michael Ellerman
On Sat, 29 Oct 2022 05:46:43 -0400, Shaomin Deng wrote: > Remove the repeated word "not" in comments. > > Applied to powerpc/next. [1/1] selftests/powerpc: Remove repeated word in comments https://git.kernel.org/powerpc/c/ad8284ead833379fc57d90e50dbae1352b116c2b cheers

Re: [PATCH] drivers/ps3: Fix double word in comments

2022-11-30 Thread Michael Ellerman
On Sun, 4 Sep 2022 11:51:02 -0400, Shaomin Deng wrote: > Drop the repeated word "when" in comments. > > Applied to powerpc/next. [1/1] drivers/ps3: Fix double word in comments https://git.kernel.org/powerpc/c/ff8fae94e26f5cd2779ceda0ee6d714a10501abd cheers

Re: [PATCH] selftests/powerpc: Fix spelling mistake "mmaping" -> "mmapping"

2022-11-30 Thread Michael Ellerman
On Fri, 21 Oct 2022 09:45:45 +0100, Colin Ian King wrote: > There is a spelling mistake in a perror message. Fix it. > > Applied to powerpc/next. [1/1] selftests/powerpc: Fix spelling mistake "mmaping" -> "mmapping"

Re: [PATCH] powerpc: Remove find_current_mm_pte()

2022-11-30 Thread Michael Ellerman
On Wed, 9 Mar 2022 10:29:50 +0100, Christophe Leroy wrote: > Last usage of find_current_mm_pte() was removed by > commit 15759cb054ef ("powerpc/perf/callchain: Use > __get_user_pages_fast in read_user_stack_slow") > > Remove it. > > > [...] Applied to powerpc/next. [1/1] powerpc: Remove

Re: [PATCH] powerpc/8xx: Simplify pte_update() with 16k pages

2022-11-30 Thread Michael Ellerman
On Wed, 28 Sep 2022 08:29:00 +0200, Christophe Leroy wrote: > While looking at code generated for code patching, I saw that > pte_clear generated: > > 2d8: 38 a0 00 00 li r5,0 > 2dc: 38 e0 10 00 li r7,4096 > 2e0: 39 00 20 00 li r8,8192 > 2e4: 39 40 30 00 li

Re: [PATCH] powerpc/8xx: Reverse order entries are written by __set_pte_at()

2022-11-30 Thread Michael Ellerman
On Wed, 28 Sep 2022 08:29:22 +0200, Christophe Leroy wrote: > At the time being, with 16k pages __set_pte_at() writes table entries > in reverse order: > > 294: 91 49 00 0c stw r10,12(r9) > 298: 91 49 00 08 stw r10,8(r9) > 29c: 91 49 00 04 stw r10,4(r9) > 2a0: 91 49 00

Re: [PATCH] powerpc/sysdev: Remove some duplicate prefix in some messages

2022-11-30 Thread Michael Ellerman
On Sun, 9 Oct 2022 12:49:50 +0200, Christophe JAILLET wrote: > At the beginning of the file, we have: >#define pr_fmt(fmt) "xive: " fmt > > So, there is no need to duplicate "XIVE:" in debug and error messages. > > For the records, these useless prefix have been added in commit >

Re: [PATCH] powerpc: platforms: 52xx: Fix a resource leak in an error handling path

2022-11-30 Thread Michael Ellerman
On Sat, 29 Jan 2022 08:16:04 +0100, Christophe JAILLET wrote: > The error handling path of mpc52xx_lpbfifo_probe() and a request_irq() is > not balanced by a corresponding free_irq(). > > Add the missing call, as already done in the remove function. > > Applied to powerpc/next. [1/1] powerpc:

Re: [PATCH v3 -next] powerpc/powermac: Fix symbol not declared warnings

2022-11-30 Thread Michael Ellerman
On Thu, 3 Nov 2022 15:01:22 +0800, Chen Lifu wrote: > 1. ppc_override_l2cr and ppc_override_l2cr_value are only used in > l2cr_init() function, remove them and used *l2cr directly. > 2. has_l2cache is not used outside of the file, so mark it static and > do not initialise statics to 0. >

Re: [PATCH] macintosh: windfarm_pm91: Fix warning comparing pointer to 0

2022-11-30 Thread Michael Ellerman
On Thu, 17 Mar 2022 10:42:33 +0800, Haowen Bai wrote: > Avoid pointer type value compared with 0 to make code clear. > > Applied to powerpc/next. [1/1] macintosh: windfarm_pm91: Fix warning comparing pointer to 0 https://git.kernel.org/powerpc/c/88316944c3b3aa3ce3249c51689ef1621049df9d

Re: [PATCH] macintosh: windfarm_pm81: Fix warning comparing pointer to 0

2022-11-30 Thread Michael Ellerman
On Thu, 17 Mar 2022 10:32:39 +0800, Haowen Bai wrote: > Avoid pointer type value compared with 0 to make code clear. > > Applied to powerpc/next. [1/1] macintosh: windfarm_pm81: Fix warning comparing pointer to 0 https://git.kernel.org/powerpc/c/e0acfdd13474815696595206e11169736b4bca9d

Re: [PATCH] macintosh: adb: Fix warning comparing pointer to 0

2022-11-30 Thread Michael Ellerman
On Thu, 17 Mar 2022 10:35:54 +0800, Haowen Bai wrote: > Avoid pointer type value compared with 0 to make code clear. > > Applied to powerpc/next. [1/1] macintosh: adb: Fix warning comparing pointer to 0 https://git.kernel.org/powerpc/c/2f59562c140d3119328f869126e8e593a99a392f cheers

Re: [PATCH] macintosh: windfarm_pm121: Fix warning comparing pointer to 0

2022-11-30 Thread Michael Ellerman
On Thu, 17 Mar 2022 10:57:02 +0800, Haowen Bai wrote: > Avoid pointer type value compared with 0 to make code clear. > > Applied to powerpc/next. [1/1] macintosh: windfarm_pm121: Fix warning comparing pointer to 0 https://git.kernel.org/powerpc/c/a823307bf0a3b79b27eea916bf6499ba4377cdf9

Re: [PATCH] macintosh: macio-adb: Fix warning comparing pointer to 0

2022-11-30 Thread Michael Ellerman
On Thu, 17 Mar 2022 17:24:49 +0800, Haowen Bai wrote: > Avoid pointer type value compared with 0 to make code clear. > > Applied to powerpc/next. [1/1] macintosh: macio-adb: Fix warning comparing pointer to 0 https://git.kernel.org/powerpc/c/fc21ed8f26d980428f9b4e08e0fb72c7f7ffc9b8

Re: [PATCH 13/13] powerpc/rtas: place tracepoints in do_enter_rtas()

2022-11-29 Thread Michael Ellerman
Nathan Lynch writes: > Michael Ellerman writes: >> Nathan Lynch writes: >>> "Nicholas Piggin" writes: >>>> On Sat Nov 19, 2022 at 1:07 AM AEST, Nathan Lynch wrote: >>>>> Call the just-added rtas tracepoints in do_enter_rtas(), taking care

Re: [PATCH] powerpc: dts: turris1x.dts: Add channel labels for temperature sensor

2022-11-28 Thread Michael Ellerman
Pali Rohár writes: > Michael, could you take this patch? Yep. With these dts patches it always helps if you tell me that it passes the DT schema checks, so that I don't get yelled at by the DT people :) cheers > On Sunday 09 October 2022 14:05:06 Pali Rohár wrote: >> On Friday 30 September

<    8   9   10   11   12   13   14   15   16   17   >