Re: [PATCH v3 3/4] PCI/AER: Disable AER interrupt on suspend

2023-04-20 Thread Kai-Heng Feng
On Thu, Apr 20, 2023 at 10:53 PM Sathyanarayanan Kuppuswamy wrote: > > > > On 4/20/23 5:59 AM, Kai-Heng Feng wrote: > > PCIe service that shares IRQ with PME may cause spurious wakeup on > > system suspend. > > > > PCIe Base Spec 5.0, section 5.2 "Link State Power Management" states > > that TLP

Re: [PATCH 01/22] powerpc, workqueue: Use alloc_ordered_workqueue() to create ordered workqueues

2023-04-20 Thread Michael Ellerman
Tejun Heo writes: > BACKGROUND > == > > When multiple work items are queued to a workqueue, their execution order > doesn't match the queueing order. They may get executed in any order and > simultaneously. When fully serialized execution - one by one in the queueing > order - is needed,

[powerpc:next] BUILD SUCCESS e4f42c1467e9233c35716eb2b4511fef990cdc23

2023-04-20 Thread kernel test robot
randconfig-r036-20230420 gcc arc allyesconfig gcc arc defconfig gcc arc randconfig-r006-20230417 gcc arc randconfig-r011-20230416 gcc arc randconfig-r011-20230420 gcc arc

[powerpc:merge] BUILD SUCCESS 5741d7d2a179be6418fbd85442d92b026477f5b8

2023-04-20 Thread kernel test robot
alpharandconfig-r036-20230420 gcc arc allyesconfig gcc arc buildonly-randconfig-r006-20230416 gcc arc defconfig gcc arc randconfig-r025-20230419 gcc arc randconfig

[PATCH 01/22] powerpc, workqueue: Use alloc_ordered_workqueue() to create ordered workqueues

2023-04-20 Thread Tejun Heo
BACKGROUND == When multiple work items are queued to a workqueue, their execution order doesn't match the queueing order. They may get executed in any order and simultaneously. When fully serialized execution - one by one in the queueing order - is needed, an ordered workqueue should be

Re: [PATCH 01/33] s390: Use _pt_s390_gaddr for gmap address tracking

2023-04-20 Thread Vishal Moola
On Wed, Apr 19, 2023 at 12:54 AM David Hildenbrand wrote: > > On 18.04.23 23:33, Vishal Moola wrote: > > On Tue, Apr 18, 2023 at 8:45 AM David Hildenbrand wrote: > >> > >> On 17.04.23 22:50, Vishal Moola (Oracle) wrote: > >>> s390 uses page->index to keep track of page tables for the guest

[PATCH 1/1] PCI: layerscape: Add the endpoint linkup notifier support

2023-04-20 Thread Frank Li
Layerscape has PME interrupt, which can be use as linkup notifer. Set CFG_READY bit when linkup detected. Signed-off-by: Xiaowei Bao Signed-off-by: Frank Li --- .../pci/controller/dwc/pci-layerscape-ep.c| 104 +- 1 file changed, 103 insertions(+), 1 deletion(-) diff --git

Re: [PATCH v2] powerpc/iommu: DMA address offset is incorrectly calculated with 2MB TCEs

2023-04-20 Thread Gaurav Batra
Hello Michael, I was looking into the Bug: 199106 (https://bugzilla.linux.ibm.com/show_bug.cgi?id=199106). In the Bug, Mellanox driver was timing out when enabling SRIOV device. I tested, Alexey's patch and it fixes the issue with Mellanox driver. The down side to Alexey's fix is that

Re: [PATCH v9 0/6] Introduce 64b relocatable kernel

2023-04-20 Thread patchwork-bot+linux-riscv
Hello: This series was applied to riscv/linux.git (for-next) by Palmer Dabbelt : On Wed, 29 Mar 2023 06:53:23 +0200 you wrote: > After multiple attempts, this patchset is now based on the fact that the > 64b kernel mapping was moved outside the linear mapping. > > The first patch allows to

Re: [PATCH v2] powerpc/iommu: DMA address offset is incorrectly calculated with 2MB TCEs

2023-04-20 Thread Michael Ellerman
Gaurav Batra writes: > When DMA window is backed by 2MB TCEs, the DMA address for the mapped > page should be the offset of the page relative to the 2MB TCE. The code > was incorrectly setting the DMA address to the beginning of the TCE > range. > > Mellanox driver is reporting timeout trying to

Re: [PATCH 10/11] iommu: Split iommu_group_add_device()

2023-04-20 Thread Jason Gunthorpe
On Thu, Apr 20, 2023 at 12:25:11PM +0800, Baolu Lu wrote: > On 4/20/23 12:11 AM, Jason Gunthorpe wrote: > > @@ -451,16 +454,17 @@ static int __iommu_probe_device(struct device *dev, > > struct list_head *group_list > > goto out_unlock; > > group = dev->iommu_group; > > - ret =

Re: [PATCH v3 3/4] PCI/AER: Disable AER interrupt on suspend

2023-04-20 Thread Sathyanarayanan Kuppuswamy
On 4/20/23 5:59 AM, Kai-Heng Feng wrote: > PCIe service that shares IRQ with PME may cause spurious wakeup on > system suspend. > > PCIe Base Spec 5.0, section 5.2 "Link State Power Management" states > that TLP and DLLP transmission is disabled for a Link in L2/L3 Ready > (D3hot), L2 (D3cold

Re: [PATCH v3 2/4] PCI/AER: Factor out interrupt toggling into helpers

2023-04-20 Thread Sathyanarayanan Kuppuswamy
On 4/20/23 5:59 AM, Kai-Heng Feng wrote: > There are many places that enable and disable AER interrput, so move > them into helpers. Add "No functional changes intended" > > Signed-off-by: Kai-Heng Feng > --- Looks good to me. Reviewed-by: Kuppuswamy Sathyanarayanan > v3: > - Correct

Re: [RFC PATCH] KVM: PPC: Update MAINTAINERS

2023-04-20 Thread Michael Ellerman
Nicholas Piggin writes: > Michael is maintaining KVM PPC with the powerpc tree at the moment, > just doesn't necessarily have time to be across all of KVM. But I > think that's okay, from mechanics of how patches flow upstream he is > maintainer. And it probably makes a bit more sense to people

Re: [PATCH v3 2/4] PCI/AER: Factor out interrupt toggling into helpers

2023-04-20 Thread Mika Westerberg
On Thu, Apr 20, 2023 at 08:59:38PM +0800, Kai-Heng Feng wrote: > There are many places that enable and disable AER interrput, so move > them into helpers. > > Signed-off-by: Kai-Heng Feng Reviewed-by: Mika Westerberg

[PATCH v3 4/4] PCI/DPC: Disable DPC interrupt during suspend

2023-04-20 Thread Kai-Heng Feng
PCIe service that shares IRQ with PME may cause spurious wakeup on system suspend. Since AER is conditionally disabled in previous patch, also apply the same logic to disable DPC which depends on AER to work. PCIe Base Spec 5.0, section 5.2 "Link State Power Management" states that TLP and DLLP

[PATCH v3 3/4] PCI/AER: Disable AER interrupt on suspend

2023-04-20 Thread Kai-Heng Feng
PCIe service that shares IRQ with PME may cause spurious wakeup on system suspend. PCIe Base Spec 5.0, section 5.2 "Link State Power Management" states that TLP and DLLP transmission is disabled for a Link in L2/L3 Ready (D3hot), L2 (D3cold with aux power) and L3 (D3cold), so we don't lose much

[PATCH v3 2/4] PCI/AER: Factor out interrupt toggling into helpers

2023-04-20 Thread Kai-Heng Feng
There are many places that enable and disable AER interrput, so move them into helpers. Signed-off-by: Kai-Heng Feng --- v3: - Correct subject. v2: - New patch. drivers/pci/pcie/aer.c | 45 +- 1 file changed, 27 insertions(+), 18 deletions(-) diff

Re: next: powerpc: gpio_mdio.c:(.text+0x13c): undefined reference to `__of_mdiobus_register'

2023-04-20 Thread Arnd Bergmann
On Thu, Apr 20, 2023, at 12:57, Naresh Kamboju wrote: > Following build failures noticed on Linux next-20230419 for powerpc. > > Regressions found on powerpc: > - build/gcc-8-defconfig > - build/clang-16-defconfig > - build/gcc-12-defconfig > - build/clang-nightly-defconfig > > > Reported-by:

[PATCH v8 00/10] arm64: Add framework to turn an IPI as NMI

2023-04-20 Thread Douglas Anderson
This is an attempt to resurrect Sumit's old patch series [1] that allowed us to use the arm64 pseudo-NMI to get backtraces of CPUs and also to round up CPUs in kdb/kgdb. The last post from Sumit that I could find was v7, so I called this series v8. I haven't copied all of his old changelongs here,

Re: [PATCH 4/33] mm: add utility functions for ptdesc

2023-04-20 Thread Vernon Yang
On Mon, Apr 17, 2023 at 01:50:19PM -0700, Vishal Moola wrote: > Introduce utility functions setting the foundation for ptdescs. These > will also assist in the splitting out of ptdesc from struct page. > > ptdesc_alloc() is defined to allocate new ptdesc pages as compound > pages. This is to

next: powerpc: gpio_mdio.c:(.text+0x13c): undefined reference to `__of_mdiobus_register'

2023-04-20 Thread Naresh Kamboju
Following build failures noticed on Linux next-20230419 for powerpc. Regressions found on powerpc: - build/gcc-8-defconfig - build/clang-16-defconfig - build/gcc-12-defconfig - build/clang-nightly-defconfig Reported-by: Linux Kernel Functional Testing Build log:

Re: [PATCH v2 4/4] PCI/DPC: Disable DPC interrupt during suspend

2023-04-20 Thread Mika Westerberg
On Thu, Apr 20, 2023 at 09:58:30AM +0800, Kai-Heng Feng wrote: > PCIe service that shares IRQ with PME may cause spurious wakeup on > system suspend. > > Since AER is conditionally disabled in previous patch, also apply the > same logic to disable DPC which depends on AER to work. > > PCIe Base

Re: [PATCH v2 3/4] PCI/AER: Disable AER interrupt on suspend

2023-04-20 Thread Mika Westerberg
On Thu, Apr 20, 2023 at 09:58:29AM +0800, Kai-Heng Feng wrote: > PCIe service that shares IRQ with PME may cause spurious wakeup on > system suspend. > > PCIe Base Spec 5.0, section 5.2 "Link State Power Management" states > that TLP and DLLP transmission is disabled for a Link in L2/L3 Ready >

Re: [PATCH v2 2/4] PCI/AER: Factor out interrput toggling into helpers

2023-04-20 Thread Mika Westerberg
Typo in $subject: interrput -> interrupt On Thu, Apr 20, 2023 at 09:58:28AM +0800, Kai-Heng Feng wrote: > There are many places that enable and disable AER interrput, so move > them into helpers. > > Signed-off-by: Kai-Heng Feng > --- > v2: > - New patch. > > drivers/pci/pcie/aer.c | 45

Re: [PATCH v2 0/2] Remove POWER10_CPU dependency and move PPC_MODULE_FEATURE_P10.

2023-04-20 Thread Herbert Xu
On Thu, Apr 13, 2023 at 03:46:23PM -0400, Danny Tsen wrote: > Remove Power10 dependency in Kconfig and detect Power10 feature at runtime. > Move PPC_MODULE_FEATURE_P10 definition to be in > arch/powerpc/include/asm/cpufeature.h. > > Signed-off-by: Danny Tsen > > Danny Tsen (2): > Kconfig:

Re: [PATCH] powerpc/rtas: Replace one-element arrays with flexible arrays

2023-04-20 Thread Andrew Donnellan
On Fri, 2023-01-27 at 07:10 -0600, Nathan Lynch wrote: > > > > I see at least one place that consults the size of one of these > > > > structs, > > > > in get_pseries_errorlog(): > > > > > > > > /* Check that we understand the format */ > > > > if (ext_log_length < sizeof(struct >