Re: Call Trace and scary messages on kernel 2.6.27-rc5

2008-09-09 Thread Marc Zyngier
On Mon, 8 Sep 2008 21:47:29 -0300 Rogério Brito [EMAIL PROTECTED] wrote: Hi there. I just compiled a new 2.6.27-rc5 kernel for my standard Kurobox (an embedded NAS that has an MPC8241 CPU, if I'm not mistaken) and upon booting, I get these scary messages and Call Traces: [snip] This is a

[PATCH] Make auto-loading of therm_pm72 possible

2010-12-05 Thread Marc Zyngier
to be automatically loaded by udev on any semi-modern distribution. It fixes a major source of problem on G5 machines where the driver wasn't explicitely loaded by default, and the system would automatically shutdown under load. Tested on an Xserve G5. Signed-off-by: Marc Zyngier m...@misterjones.org

[PATCH] Make auto-loading of therm_pm72 possible

2011-01-03 Thread Marc Zyngier
to be automatically loaded by udev on any semi-modern distribution. It fixes a major source of problem on G5 machines where the driver wasn't explicitely loaded by default, and the system would automatically shutdown under load. Tested on an Xserve G5. Signed-off-by: Marc Zyngier m...@misterjones.org

Re: [PATCH 2/3] powerpc: PCI: Fix lookup of linux,pci-probe-only property

2015-08-14 Thread Marc Zyngier
Hi Bjorn, On 14/08/15 15:57, Bjorn Helgaas wrote: Hi Marc, On Fri, Aug 14, 2015 at 03:41:07PM +0100, Marc Zyngier wrote: When find_and_init_phbs() looks for the probe-only property, it seems to trust the firmware to be correctly written, and assumes that there is a parameter

[PATCH 1/3] PCI: pci-host-generic: Fix lookup of linux, pci-probe-only property

2015-08-14 Thread Marc Zyngier
on whatever the property pointer points to, which is likely to be junk. Instead, let's check for the validity of the property, and ignore it if the firmware couldn't make up its mind. Signed-off-by: Marc Zyngier marc.zyng...@arm.com --- drivers/pci/host/pci-host-generic.c | 15 ++- 1

[PATCH 0/3] PCI: arm64/powerpc: Fix parsing of linux,pci-probe-only

2015-08-14 Thread Marc Zyngier
, the bad property is removed from the Seatle DTS, because it is simply not necessary (it actually prevents me from using SR-IOV, which otherwise runs fine without the probe-only thing). Marc Zyngier (3): PCI: pci-host-generic: Fix lookup of linux,pci-probe-only property powerpc: PCI: Fix lookup

[PATCH 2/3] powerpc: PCI: Fix lookup of linux, pci-probe-only property

2015-08-14 Thread Marc Zyngier
-by: Marc Zyngier marc.zyng...@arm.com --- arch/powerpc/platforms/pseries/setup.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/arch/powerpc/platforms/pseries/setup.c b/arch/powerpc/platforms/pseries/setup.c index df6a704..6bdc1f9 100644 --- a/arch/powerpc

[PATCH 3/3] arm64: dts: Drop linux, pci-probe-only from the Seattle DTS

2015-08-14 Thread Marc Zyngier
-off-by: Marc Zyngier marc.zyng...@arm.com --- arch/arm64/boot/dts/amd/amd-overdrive.dts | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/boot/dts/amd/amd-overdrive.dts b/arch/arm64/boot/dts/amd/amd-overdrive.dts index 564a3f7..128fa94 100644 --- a/arch/arm64/boot/dts/amd/amd

[PATCH v2 3/4] powerpc: PCI: Fix lookup of linux, pci-probe-only property

2015-08-14 Thread Marc Zyngier
if the firmware couldn't make up its mind. Signed-off-by: Marc Zyngier marc.zyng...@arm.com --- arch/powerpc/platforms/pseries/setup.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/arch/powerpc/platforms/pseries/setup.c b/arch/powerpc/platforms/pseries/setup.c index

[PATCH v2 0/4] PCI: arm64/powerpc: Fix parsing of linux, pci-probe-only

2015-08-14 Thread Marc Zyngier
the probe-only thing). This has been tested on the offending Seattle board. * From v1: - Consolidate the parsing in of_pci.c (Bjorn) Marc Zyngier (4): of/pci: Add of_pci_check_probe_only to parse linux,pci-probe-only PCI: pci-host-generic: Fix lookup of linux,pci-probe-only property

[PATCH v2 1/4] of/pci: Add of_pci_check_probe_only to parse linux, pci-probe-only

2015-08-14 Thread Marc Zyngier
Both pci-host-generic and Pseries parse the linux,pci-probe-only to engage the PCI_PROBE_ONLY mode, and both have a subtle bug that can be triggered if the property has no parameter. Provide a generic implementation that can be used by both. Signed-off-by: Marc Zyngier marc.zyng...@arm.com

[PATCH v2 2/4] PCI: pci-host-generic: Fix lookup of linux, pci-probe-only property

2015-08-14 Thread Marc Zyngier
on whatever the property pointer points to, which is likely to be junk. Switch to the common of_pci.c implementation that doesn't suffer from this problem. Signed-off-by: Marc Zyngier marc.zyng...@arm.com --- drivers/pci/host/pci-host-generic.c | 9 + 1 file changed, 1 insertion(+), 8

[PATCH v2 4/4] arm64: dts: Drop linux, pci-probe-only from the Seattle DTS

2015-08-14 Thread Marc Zyngier
-off-by: Marc Zyngier marc.zyng...@arm.com --- arch/arm64/boot/dts/amd/amd-overdrive.dts | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/boot/dts/amd/amd-overdrive.dts b/arch/arm64/boot/dts/amd/amd-overdrive.dts index 564a3f7..128fa94 100644 --- a/arch/arm64/boot/dts/amd/amd

[PATCH v4 2/4] PCI: pci-host-generic: Fix lookup of linux, pci-probe-only property

2015-09-04 Thread Marc Zyngier
on whatever the property pointer points to, which is likely to be junk. Switch to the common of_pci.c implementation that doesn't suffer from this problem. Signed-off-by: Marc Zyngier <marc.zyng...@arm.com> --- drivers/pci/host/pci-host-generic.c | 9 + 1 file changed, 1 insertion

[PATCH v4 1/4] of/pci: Add of_pci_check_probe_only to parse "linux, pci-probe-only"

2015-09-04 Thread Marc Zyngier
Both pci-host-generic and Pseries parse the "linux,pci-probe-only" property to engage the PCI_PROBE_ONLY mode, and both have a subtle bug that can be triggered if the property has no parameter. Provide a generic, safe implementation that can be used by both. Signed-off-by: Ma

[PATCH v4 4/4] arm64: dts: Drop linux, pci-probe-only from the Seattle DTS

2015-09-04 Thread Marc Zyngier
er. Acked-by: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com> Signed-off-by: Marc Zyngier <marc.zyng...@arm.com> --- arch/arm64/boot/dts/amd/amd-overdrive.dts | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/boot/dts/amd/amd-overdrive.dts b/arch/arm64/boot/dts/amd/a

[PATCH v4 3/4] powerpc: PCI: Fix lookup of linux, pci-probe-only property

2015-09-04 Thread Marc Zyngier
if the firmware couldn't make up its mind. Signed-off-by: Marc Zyngier <marc.zyng...@arm.com> --- arch/powerpc/platforms/pseries/setup.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/arch/powerpc/platforms/pseries/setup.c b/arch/powerpc/platforms/pseries/setup.c

[PATCH v4 0/4] PCI: arm64/powerpc: Fix parsing of linux, pci-probe-only

2015-09-04 Thread Marc Zyngier
afely read the property (Rob) - Add a log message to indicate when we enable probe-only (probably quite useful for debugging) * From v1: - Consolidate the parsing in of_pci.c (Bjorn) Marc Zyngier (4): of/pci: Add of_pci_check_probe_only to parse "linux,pci-probe-only" PCI: p

Re: [PATCH v2 1/4] of/pci: Add of_pci_check_probe_only to parse "linux, pci-probe-only"

2015-09-03 Thread Marc Zyngier
On 02/09/15 23:23, Bjorn Helgaas wrote: > On Fri, Aug 14, 2015 at 04:08:10PM -0500, Rob Herring wrote: >> On Fri, Aug 14, 2015 at 11:19 AM, Marc Zyngier <marc.zyng...@arm.com> wrote: >>> Both pci-host-generic and Pseries parse the "linux,pci-probe-only" >

[PATCH v3 0/4] PCI: arm64/powerpc: Fix parsing of linux, pci-probe-only

2015-09-03 Thread Marc Zyngier
he parsing in of_pci.c (Bjorn) Marc Zyngier (4): of/pci: Add of_pci_check_probe_only to parse "linux,pci-probe-only" PCI: pci-host-generic: Fix lookup of linux,pci-probe-only property powerpc: PCI: Fix lookup of linux,pci-probe-only property arm64: dts: Drop linux,pci-probe-only

[PATCH v3 4/4] arm64: dts: Drop linux, pci-probe-only from the Seattle DTS

2015-09-03 Thread Marc Zyngier
r. Signed-off-by: Marc Zyngier <marc.zyng...@arm.com> --- arch/arm64/boot/dts/amd/amd-overdrive.dts | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/boot/dts/amd/amd-overdrive.dts b/arch/arm64/boot/dts/amd/amd-overdrive.dts index 564a3f7..128fa94 100644 --- a/arch/arm64/

[PATCH v3 2/4] PCI: pci-host-generic: Fix lookup of linux, pci-probe-only property

2015-09-03 Thread Marc Zyngier
on whatever the property pointer points to, which is likely to be junk. Switch to the common of_pci.c implementation that doesn't suffer from this problem. Signed-off-by: Marc Zyngier <marc.zyng...@arm.com> --- drivers/pci/host/pci-host-generic.c | 9 + 1 file changed, 1 insertion

[PATCH v3 1/4] of/pci: Add of_pci_check_probe_only to parse "linux, pci-probe-only"

2015-09-03 Thread Marc Zyngier
Both pci-host-generic and Pseries parse the "linux,pci-probe-only" property to engage the PCI_PROBE_ONLY mode, and both have a subtle bug that can be triggered if the property has no parameter. Provide a generic, safe implementation that can be used by both. Signed-off-by: Ma

[PATCH v3 3/4] powerpc: PCI: Fix lookup of linux, pci-probe-only property

2015-09-03 Thread Marc Zyngier
if the firmware couldn't make up its mind. Signed-off-by: Marc Zyngier <marc.zyng...@arm.com> --- arch/powerpc/platforms/pseries/setup.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/arch/powerpc/platforms/pseries/setup.c b/arch/powerpc/platforms/pseries/setup.c

Re: [PATCH v4 0/4] PCI: arm64/powerpc: Fix parsing of linux, pci-probe-only

2015-09-17 Thread Marc Zyngier
On 17/09/15 16:30, Bjorn Helgaas wrote: > On Fri, Sep 04, 2015 at 05:50:07PM +0100, Marc Zyngier wrote: >> The pci-host-generic driver parses the linux,pci-probe-only property, >> and assumes that it will have a boolean parameter. >> >> Turns out that the Seattle DTS

Re: [Patch V6 2/6] irqchip: xilinx: Clean up irqdomain argument and read/write

2016-11-09 Thread Marc Zyngier
missing >> something. > > I wrote this patch/cleanup based on a review of driver by Marc when I moved > the > driver from arch/microblaze to drivers/irqchip > > "Marc Zyngier > > ... > > > arch/microblaze/kernel/intc.c | 196 > -

Re: [Patch v5 04/12] irqchip: xilinx: Add support for parent intc

2016-10-25 Thread Marc Zyngier
On 25/10/16 15:44, Sören Brinkmann wrote: > On Tue, 2016-10-25 at 12:49:33 +0200, Thomas Gleixner wrote: >> On Tue, 25 Oct 2016, Zubair Lutfullah Kakakhel wrote: >>> On 10/21/2016 10:48 AM, Marc Zyngier wrote: >>>> Shouldn't you return an error if irq is zero? >&

Re: [Patch v5 04/12] irqchip: xilinx: Add support for parent intc

2016-10-21 Thread Marc Zyngier
On 17/10/16 17:52, Zubair Lutfullah Kakakhel wrote: > The MIPS based xilfpga platform has the following IRQ structure > > Peripherals --> xilinx_intcontroller -> mips_cpu_int controller > > Add support for the driver to chain the irq handler > > Signed-off-by: Zubair Lutfullah Kakakhel

Re: [Patch v7 0/7] microblaze/PowerPC: Move irq-xilinx to irqchip

2016-11-22 Thread Marc Zyngier
On 14/11/16 12:13, Zubair Lutfullah Kakakhel wrote: > Hi, > > This patch series moves the Xilinx interrupt controller driver out > of arch/microblaze to drivers/irqchip and then cleans it up a bit. > And then removes another implementation of the driver in arch/powerpc. > > This effort results

Re: [Patch v7 6/7] irqchip: xilinx: Try to fall back if xlnx,kind-of-intr not provided

2016-11-21 Thread Marc Zyngier
On 21/11/16 14:05, Zubair Lutfullah Kakakhel wrote: > Hi, > > On 11/18/2016 01:29 PM, Thomas Gleixner wrote: >> On Mon, 14 Nov 2016, Zubair Lutfullah Kakakhel wrote: >> >>> The powerpc dts file does not have the xlnx,kind-of-intr property. >>> Instead of erroring out, give a warning instead. And

Re: [PATCH v6 2/4] irqchip/qeic: merge qeic init code from platforms to a common function

2016-12-16 Thread Marc Zyngier
On 16/12/16 08:43, Qiang Zhao wrote: > On 16/12/16 04:33, Marc Zyngier <marc.zyng...@arm.com> wrote: > >> -Original Message----- >> From: Marc Zyngier [mailto:marc.zyng...@arm.com] >> Sent: Friday, December 16, 2016 4:33 PM >> To: Qiang Zhao <qi

Re: [PATCH v6 2/4] irqchip/qeic: merge qeic init code from platforms to a common function

2016-12-16 Thread Marc Zyngier
On 28/09/16 04:25, Zhao Qiang wrote: > The codes of qe_ic init from a variety of platforms are redundant, > merge them to a common function and put it to irqchip/irq-qeic.c > > For non-p1021_mds mpc85xx_mds boards, use "qe_ic_init(np, 0, > qe_ic_cascade_low_mpic, qe_ic_cascade_high_mpic);"

Re: [Patch V7 4/4] irqchip/qeic: remove PPCisms for QEIC

2017-01-05 Thread Marc Zyngier
On 26/12/16 08:32, Zhao Qiang wrote: > QEIC was supported on PowerPC, and dependent on PPC, > Now it is supported on other platforms, so remove PPCisms. > > Signed-off-by: Zhao Qiang > --- > Changes for v6: > - new added > Changes for v7: > - fix warning > >

Re: [Patch V7 2/4] irqchip/qeic: merge qeic init code from platforms to a common function

2017-01-05 Thread Marc Zyngier
On 26/12/16 08:32, Zhao Qiang wrote: > The codes of qe_ic init from a variety of platforms are redundant, > merge them to a common function and put it to irqchip/irq-qeic.c > > For non-p1021_mds mpc85xx_mds boards, use "qe_ic_init(np, 0, > qe_ic_cascade_low_mpic, qe_ic_cascade_high_mpic);"

Re: [PATCH v9 4/4] irqchip/qeic: remove PPCisms for QEIC

2017-08-04 Thread Marc Zyngier
[Please add all the irqchip maintainers when posting irqchip patches...] On 03/08/17 04:38, Zhao Qiang wrote: > QEIC was supported on PowerPC, and dependent on PPC, > Now it is supported on other platforms, so remove PPCisms. > > Signed-off-by: Zhao Qiang > --- >

Re: [PATCH v10 0/4] this patchset is to remove PPCisms for QEIC

2017-11-02 Thread Marc Zyngier
On 01/11/17 17:09, Thomas Gleixner wrote: > On Wed, 1 Nov 2017, Qiang Zhao wrote: >> Michael Ellerman wrote >>> >>> Qiang Zhao writes: >>> Hi all, Could anybody review this patchset and take action on them? Thank you! >>> >>> Who maintains

Re: [RFC PATCH 03/23] genirq: Introduce IRQF_DELIVER_AS_NMI

2018-06-13 Thread Marc Zyngier
On 13/06/18 10:20, Thomas Gleixner wrote: > On Wed, 13 Jun 2018, Julien Thierry wrote: >> On 13/06/18 09:34, Peter Zijlstra wrote: >>> On Tue, Jun 12, 2018 at 05:57:23PM -0700, Ricardo Neri wrote: diff --git a/include/linux/interrupt.h b/include/linux/interrupt.h index 5426627..dbc5e02

Re: [PATCH] Extract initrd free logic from arch-specific code.

2018-03-29 Thread Marc Zyngier
On Thu, 29 Mar 2018 16:58:27 +0100, Russell King - ARM Linux wrote: > > On Thu, Mar 29, 2018 at 05:43:47PM +0200, Geert Uytterhoeven wrote: > > On Thu, Mar 29, 2018 at 5:27 PM, Russell King - ARM Linux > > wrote: > > > On Thu, Mar 29, 2018 at 09:37:52AM +1100, Oliver

Re: [PATCH] Extract initrd free logic from arch-specific code.

2018-03-29 Thread Marc Zyngier
On Thu, 29 Mar 2018 18:32:47 +0100, Russell King - ARM Linux wrote: > > On Thu, Mar 29, 2018 at 05:53:14PM +0100, Marc Zyngier wrote: > > On Thu, 29 Mar 2018 16:58:27 +0100, > > Russell King - ARM Linux wrote: [...] > > > I'm not aware of there being an emulated

Re: [PATCH 10/10] soc: qcom: convert to devm_platform_ioremap_resource

2019-12-15 Thread Marc Zyngier
On Sat, 14 Dec 2019 17:54:47 + Yangtao Li wrote: > Use devm_platform_ioremap_resource() to simplify code. > > Signed-off-by: Yangtao Li > --- > drivers/soc/qcom/llcc-qcom.c| 7 +-- > drivers/soc/qcom/qcom-geni-se.c | 4 +--- > drivers/soc/qcom/qcom_aoss.c| 4 +--- >

Re: [PATCH 03/35] docs: fix broken references to text files

2020-04-09 Thread Marc Zyngier
> hwtracing/coresight/Kconfig > Signed-off-by: Mauro Carvalho Chehab > --- > Documentation/virt/kvm/arm/pvtime.rst| 2 +- > virt/kvm/arm/vgic/vgic-mmio-v3.c | 2 +- > virt/kvm/arm/vgic/vgic.h | 4 ++-- Acked-b

Re: [PATCH v2] KVM: Optimize kvm_arch_vcpu_ioctl_run function

2020-04-16 Thread Marc Zyngier
On 2020-04-16 08:03, Vitaly Kuznetsov wrote: Tianjia Zhang writes: In earlier versions of kvm, 'kvm_run' is an independent structure and is not included in the vcpu structure. At present, 'kvm_run' is already included in the vcpu structure, so the parameter 'kvm_run' is redundant. This patch

Re: [PATCH v2] KVM: Optimize kvm_arch_vcpu_ioctl_run function

2020-04-16 Thread Marc Zyngier
On 2020-04-16 09:45, Tianjia Zhang wrote: On 2020/4/16 16:28, Marc Zyngier wrote: [...] Overall, there is a large set of cleanups to be done when both the vcpu and the run structures are passed as parameters at the same time. Just grepping the tree for kvm_run is pretty instructive

Re: [PATCH v4 2/7] KVM: arm64: clean up redundant 'kvm_run' parameters

2020-05-05 Thread Marc Zyngier
Hi Tianjia, On 2020-04-27 05:35, Tianjia Zhang wrote: In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu' structure. For historical reasons, many kvm-related function parameters retain the 'kvm_run' and 'kvm_vcpu' parameters at the same time. This patch does a unified

Re: [RFC PATCH kernel 1/2] irq: Add reference counting to IRQ mappings

2020-10-27 Thread Marc Zyngier
Hi Alexey, On 2020-10-27 09:06, Alexey Kardashevskiy wrote: PCI devices share 4 legacy INTx interrupts from the same PCI host bridge. Device drivers map/unmap hardware interrupts via irq_create_mapping()/ irq_dispose_mapping(). The problem with that these interrupts are shared and when

Re: [PATCH v3 2/2] powerpc/pseries: pass MSI affinity to irq_create_mapping()

2020-11-25 Thread Marc Zyngier
On 2020-11-25 16:24, Laurent Vivier wrote: On 25/11/2020 17:05, Denis Kirjanov wrote: On 11/25/20, Laurent Vivier wrote: With virtio multiqueue, normally each queue IRQ is mapped to a CPU. But since commit 0d9f0a52c8b9f ("virtio_scsi: use virtio IRQ affinity") this is broken on pseries.

Re: [PATCH v2 1/2] genirq: add an irq_create_mapping_affinity() function

2020-11-25 Thread Marc Zyngier
On 2020-11-25 14:09, Laurent Vivier wrote: On 25/11/2020 14:20, Thomas Gleixner wrote: Laurent, On Wed, Nov 25 2020 at 12:16, Laurent Vivier wrote: The proper subsystem prefix is: 'genirq/irqdomain:' and the first letter after the colon wants to be uppercase. Ok. This function adds an

Re: [PATCH 15/31] KVM: PPC: Book3S HV: XIVE: Fix mapping of passthrough interrupts

2021-05-15 Thread Marc Zyngier
On Fri, 14 May 2021 21:51:51 +0100, Thomas Gleixner wrote: > > On Fri, Apr 30 2021 at 10:03, Cédric Le Goater wrote: > > CC: +Marc Thanks Thomas. > > > PCI MSI interrupt numbers are now mapped in a PCI-MSI domain but the > > underlying calls handling the passthrough of the interrupt in the >

Re: [PATCH 2/6] KVM: mmu: also return page from gfn_to_pfn

2021-06-24 Thread Marc Zyngier
Hi David, On Thu, 24 Jun 2021 04:57:45 +0100, David Stevens wrote: > > From: David Stevens > > Return a struct kvm_pfn_page containing both a pfn and an optional > struct page from the gfn_to_pfn family of functions. This differentiates > the gup and follow_fault_pfn cases, which allows

Re: [PATCH 3/6] KVM: x86/mmu: avoid struct page in MMU

2021-06-24 Thread Marc Zyngier
On Thu, 24 Jun 2021 09:58:00 +0100, Nicholas Piggin wrote: > > Excerpts from David Stevens's message of June 24, 2021 1:57 pm: > > From: David Stevens > > out_unlock: > > if (is_tdp_mmu_root(vcpu->kvm, vcpu->arch.mmu->root_hpa)) > > read_unlock(>kvm->mmu_lock); > > else > >

Re: [PATCH 4/6] KVM: arm64/mmu: avoid struct page in MMU

2021-06-24 Thread Marc Zyngier
On Thu, 24 Jun 2021 04:57:47 +0100, David Stevens wrote: > > From: David Stevens > > Avoid converting pfns returned by follow_fault_pfn to struct pages to > transiently take a reference. The reference was originally taken to > match the reference taken by gup. However, pfns returned by >

Re: [PATCH 2/9] ARM: PXA: Kill use of irq_create_strict_mappings()

2021-04-27 Thread Marc Zyngier
Hi Guenter, Thanks for the heads up. On Mon, 26 Apr 2021 23:39:42 +0100, Guenter Roeck wrote: > > On Tue, Apr 06, 2021 at 10:35:50AM +0100, Marc Zyngier wrote: > > irq_create_strict_mappings() is a poor way to allow the use of > > a linear IRQ domain as a legacy one. Let'

[PATCH 1/9] irqdomain: Reimplement irq_linear_revmap() with irq_find_mapping()

2021-04-06 Thread Marc Zyngier
meaningful difference compared to the cost of taking an interrupt. Reimplement irq_linear_revmap() with irq_find_mapping() in order to preserve source code compatibility, and rename the internal field for a measure. Signed-off-by: Marc Zyngier --- include/linux/irqdomain.h | 22

[PATCH 0/9] Cleaning up some of the irqdomain features

2021-04-06 Thread Marc Zyngier
in the way irqdomains work. M. Marc Zyngier (9): irqdomain: Reimplement irq_linear_revmap() with irq_find_mapping() ARM: PXA: Kill use of irq_create_strict_mappings() irqchip/jcore-aic: Kill use of irq_create_strict_mappings() sh: intc: Drop the use of irq_create_identity_mapping

[PATCH 4/9] sh: intc: Drop the use of irq_create_identity_mapping()

2021-04-06 Thread Marc Zyngier
Instead of playing games with using irq_create_identity_mapping() and irq_domain_associate(), drop the use of the former and only use the latter, together with the allocation of the irq_desc as needed. It doesn't make the code less awful, but at least the intent is clearer. Signed-off-by: Marc

[PATCH 6/9] mips: netlogic: Use irq_domain_simple_ops for XLP PIC

2021-04-06 Thread Marc Zyngier
Use the generic irq_domain_simple_ops structure instead of a home-grown one. Signed-off-by: Marc Zyngier --- arch/mips/netlogic/common/irq.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/arch/mips/netlogic/common/irq.c b/arch/mips/netlogic/common/irq.c index

[PATCH 2/9] ARM: PXA: Kill use of irq_create_strict_mappings()

2021-04-06 Thread Marc Zyngier
irq_create_strict_mappings() is a poor way to allow the use of a linear IRQ domain as a legacy one. Let's be upfront about it and use a legacy domain when appropriate. Signed-off-by: Marc Zyngier --- arch/arm/mach-pxa/pxa_cplds_irqs.c | 24 +++- 1 file changed, 11 insertions

[PATCH 9/9] irqdomain: Kill irq_domain_add_legacy_isa

2021-04-06 Thread Marc Zyngier
This helper doesn't have a user anymore, let's remove it. Signed-off-by: Marc Zyngier --- Documentation/core-api/irq/irq-domain.rst | 1 - include/linux/irqdomain.h | 11 --- 2 files changed, 12 deletions(-) diff --git a/Documentation/core-api/irq/irq-domain.rst b

[PATCH 5/9] irqdomain: Kill irq_create_strict_mappings()/irq_create_identity_mapping()

2021-04-06 Thread Marc Zyngier
No user of these APIs are left, remove them. Signed-off-by: Marc Zyngier --- include/linux/irqdomain.h | 9 - kernel/irq/irqdomain.c| 35 --- 2 files changed, 44 deletions(-) diff --git a/include/linux/irqdomain.h b/include/linux/irqdomain.h index

[PATCH 8/9] powerpc: Convert irq_domain_add_legacy_isa use to irq_domain_add_legacy

2021-04-06 Thread Marc Zyngier
NR_IRQS_LEGACY. The dependency cannot be broken yet as there is a lot of PPC-related code that depends on it, but that's the first step towards it. A followup patch will remove irq_domain_add_legacy_isa. Signed-off-by: Marc Zyngier --- arch/powerpc/include/asm/irq.h | 4 ++-- arch/powerpc

[PATCH 7/9] irqdomain: Drop references to recusive irqdomain setup

2021-04-06 Thread Marc Zyngier
It was never completely implemented, and was removed a long time ago. Adjust the documentation to reflect this. Signed-off-by: Marc Zyngier --- kernel/irq/irqdomain.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c

[PATCH 3/9] irqchip/jcore-aic: Kill use of irq_create_strict_mappings()

2021-04-06 Thread Marc Zyngier
irq_create_strict_mappings() is a poor way to allow the use of a linear IRQ domain as a legacy one. Let's be upfront about it. Signed-off-by: Marc Zyngier --- drivers/irqchip/irq-jcore-aic.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/irqchip/irq-jcore-aic.c

Re: [PATCH 1/9] irqdomain: Reimplement irq_linear_revmap() with irq_find_mapping()

2021-04-06 Thread Marc Zyngier
Christophe, On Tue, 06 Apr 2021 12:21:33 +0100, Christophe Leroy wrote: > > > > Le 06/04/2021 à 11:35, Marc Zyngier a écrit : > > irq_linear_revmap() is supposed to be a fast path for domain > > lookups, but it only exposes low-level details of the irqdomain > >

Re: [PATCH 4/9] sh: intc: Drop the use of irq_create_identity_mapping()

2021-04-06 Thread Marc Zyngier
On Tue, 06 Apr 2021 11:32:13 +0100, Geert Uytterhoeven wrote: > > Hi Marc, > > On Tue, Apr 6, 2021 at 11:44 AM Marc Zyngier wrote: > > Instead of playing games with using irq_create_identity_mapping() > > and irq_domain_associate(), drop the use of the former and

Re: [GIT PULL] Please pull powerpc/linux.git powerpc-5.15-1 tag

2021-09-03 Thread Marc Zyngier
Hi Michael, On Fri, 03 Sep 2021 14:36:57 +0100, Michael Ellerman wrote: > > Hi Linus, > > Please pull powerpc updates for 5.15. > > A bit of a small batch this time. > > There was one conflict against my own fixes branch, and the resolution was a > little bit > messy, so I just did a merge

Re: [PATCH v5 00/14] PCI: Add support for Apple M1

2021-10-05 Thread Marc Zyngier
On Mon, 04 Oct 2021 21:42:45 +0100, Linus Walleij wrote: > > On Mon, Oct 4, 2021 at 9:52 PM Rob Herring wrote: > > > FYI, I pushed patches 1-3 to kernelCI and didn't see any regressions. > > I am a bit worried about changes to the DT interrupt parsing and > > ancient platforms (such as

Re: [PATCH v3 07/12] KVM: arm64: Use Makefile.kvm for common files

2021-11-17 Thread Marc Zyngier
m.o mmu.o mmio.o psci.o perf.o hypercalls.o pvtime.o \ >inject_fault.o va_layout.o handle_exit.o \ >guest.o debug.o reset.o sys_regs.o \ >vgic-sys-reg-v3.o fpsimd.o pmu.o \ Acked-by: Marc Zyngier M. -- Without deviation from the norm, progress is not possible.

Re: [PATCH v3 02/12] KVM: Add Makefile.kvm for common files, use it for x86

2021-11-17 Thread Marc Zyngier
m-y := $(KVM)/kvm_main.o $(KVM)/eventfd.o $(KVM)/binary_stats.o > +kvm-$(CONFIG_KVM_VFIO) += $(KVM)/vfio.o > +kvm-$(CONFIG_KVM_MMIO) += $(KVM)/coalesced_mmio.o > +kvm-$(CONFIG_KVM_ASYNC_PF) += $(KVM)/async_pf.o > +kvm-$(CONFIG_HAVE_KVM_IRQ_ROUTING) += $(KVM)/irqchip.o > +kvm-$(CONFIG_HAVE_KVM_DIRTY_RING) += $(KVM)/dirty_ring.o Acked-by: Marc Zyngier M. -- Without deviation from the norm, progress is not possible.

Re: [PATCH v3 08/12] KVM: Propagate vcpu explicitly to mark_page_dirty_in_slot()

2021-11-17 Thread Marc Zyngier
On Wed, 17 Nov 2021 17:39:59 +, David Woodhouse wrote: > > From: David Woodhouse > > The kvm_dirty_ring_get() function uses kvm_get_running_vcpu() to work out > which dirty ring to use, but there are some use cases where that doesn't > work. > > There's one in setting the Xen shared info

Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates

2021-11-11 Thread Marc Zyngier
On Wed, 10 Nov 2021 18:07:24 +, Christian Zigotzky wrote: > > On 09 November 2021 at 03:45 pm, Christian Zigotzky wrote: > > Hello, > > > > The Nemo board [1] doesn't recognize any ATA disks with the > pci-v5.16 updates [2]. > > > > Error messages: > > > > ata4.00: gc timeout cmd 0xec > >

Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates

2021-11-10 Thread Marc Zyngier
ATA disks without any problems. > > > > I created a patch for an easy reverting the bad commit [1]. With this patch > > we can do further our kernel tests. > > > > Could you please check the first bad commit [2]? > > > > Thanks, > > Chr

Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates

2021-11-12 Thread Marc Zyngier
On Fri, 12 Nov 2021 09:40:30 +, Christian Zigotzky wrote: > > On 11 November 2021 at 06:39 pm, Marc Zyngier wrote: > > On Wed, 10 Nov 2021 18:07:24 +, > > Christian Zigotzky wrote: > >> On 09 November 2021 at 03:45 pm, Christian Zigotzky wrote: > >>

Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates

2021-11-12 Thread Marc Zyngier
On Fri, 12 Nov 2021 14:15:18 +, Christian Zigotzky wrote: > > On 12 November 2021 at 02:41 pm, Marc Zyngier wrote: > > On Fri, 12 Nov 2021 09:40:30 +, > > Christian Zigotzky wrote: > >> On 11 November 2021 at 06:39 pm, Marc Zyngier wrote: > >>

Re: [PATCH 16/39] irqdomain: Make normal and nomap irqdomains exclusive

2021-11-16 Thread Marc Zyngier
On Mon, 15 Nov 2021 19:05:17 +, Cédric Le Goater wrote: > > Hello Mark, s/k/c/ > > On 5/20/21 18:37, Marc Zyngier wrote: > > Direct mappings are completely exclusive of normal mappings, meaning > > that we can refactor the code slightl

Re: [PATCH 0/5] KVM: Turn the vcpu array into an xarray

2021-11-16 Thread Marc Zyngier
On Tue, 16 Nov 2021 15:03:40 +, Paolo Bonzini wrote: > > On 11/5/21 20:20, Marc Zyngier wrote: > > The kvm structure is pretty large. A large portion of it is the vcpu > > array, which is 4kB on x86_64 and arm64 as they deal with 512 vcpu > > VMs. Of course, hardly

[PATCH v2 4/7] KVM: x86: Use kvm_get_vcpu() instead of open-coded access

2021-11-16 Thread Marc Zyngier
As we are about to change the way vcpus are allocated, mandate the use of kvm_get_vcpu() instead of open-coding the access. Signed-off-by: Marc Zyngier --- arch/x86/kvm/vmx/posted_intr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kvm/vmx/posted_intr.c b/arch

[PATCH v2 3/7] KVM: s390: Use kvm_get_vcpu() instead of open-coded access

2021-11-16 Thread Marc Zyngier
As we are about to change the way vcpus are allocated, mandate the use of kvm_get_vcpu() instead of open-coding the access. Reviewed-by: Claudio Imbrenda Signed-off-by: Marc Zyngier --- arch/s390/kvm/kvm-s390.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/arch

[PATCH v2 0/7] KVM: Turn the vcpu array into an xarray

2021-11-16 Thread Marc Zyngier
into an xarray, which results in a net code deletion after a bit of cleanup. * From v1: - Rebased on v5.16-rc1 - Dropped the dubious locking on teardown - Converted kvm_for_each_vcpu() to xa_for_each_range(), together with an invasive change converting the index to an unsigned long Marc

[PATCH v2 1/7] KVM: Move wiping of the kvm->vcpus array to common code

2021-11-16 Thread Marc Zyngier
her changes. The locking is dropped altogether, as this should only be called when there is no further references on the kvm structure. Reviewed-by: Claudio Imbrenda Signed-off-by: Marc Zyngier --- arch/arm64/kvm/arm.c | 10 +- arch/mips/kvm/mips.c |

[PATCH v2 2/7] KVM: mips: Use kvm_get_vcpu() instead of open-coded access

2021-11-16 Thread Marc Zyngier
As we are about to change the way vcpus are allocated, mandate the use of kvm_get_vcpu() instead of open-coding the access. Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Marc Zyngier --- arch/mips/kvm/loongson_ipi.c | 4 ++-- arch/mips/kvm/mips.c | 2 +- 2 files changed, 3

[PATCH v2 6/7] KVM: Use 'unsigned long' as kvm_for_each_vcpu()'s index

2021-11-16 Thread Marc Zyngier
Everywhere we use kvm_for_each_vpcu(), we use an int as the vcpu index. Unfortunately, we're about to move rework the iterator, which requires this to be upgrade to an unsigned long. Let's bite the bullet and repaint all of it in one go. Signed-off-by: Marc Zyngier --- arch/arm64/kvm

[PATCH v2 5/7] KVM: Convert the kvm->vcpus array to a xarray

2021-11-16 Thread Marc Zyngier
this memory, let's use an xarray instead, which gives us almost the same flexibility as a normal array, but with a reduced memory usage with smaller VMs. Signed-off-by: Marc Zyngier --- include/linux/kvm_host.h | 5 +++-- virt/kvm/kvm_main.c | 15 +-- 2 files changed, 12

[PATCH v2 7/7] KVM: Convert kvm_for_each_vcpu() to using xa_for_each_range()

2021-11-16 Thread Marc Zyngier
Now that the vcpu array is backed by an xarray, use the optimised iterator that matches the underlying data structure. Suggested-by: Sean Christopherson Signed-off-by: Marc Zyngier --- include/linux/kvm_host.h | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/include

Re: [PATCH] powerpc/xive: Change IRQ domain to a tree domain

2021-11-16 Thread Marc Zyngier
driver to use a 'tree' domain type instead. > > [1] For instance, a linux KVM guest with virtio-rng and virtio-balloon > devices. > > Cc: Marc Zyngier > Cc: sta...@vger.kernel.org # v5.14+ > Fixes: 4f86a06e2d6e ("irqdomain: Make normal and nomap irqdomains exclus

Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates

2021-11-10 Thread Marc Zyngier
On Thu, 11 Nov 2021 05:24:52 +, Christian Zigotzky wrote: > > On 10 November 2021 at 08:09 pm, Marc Zyngier wrote: > > HI all, > > > > On Wed, 10 Nov 2021 18:41:06 +, > > Bjorn Helgaas wrote: > >> On Wed, Nov 10, 2021 at 07:07:24PM +0100, Christi

Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates

2021-11-11 Thread Marc Zyngier
On Thu, 11 Nov 2021 10:44:30 +, Christian Zigotzky wrote: > > On 11 November 2021 at 11:20 am, Marc Zyngier wrote: > > On Thu, 11 Nov 2021 07:47:08 +, > > Christian Zigotzky wrote: > >> On 11 November 2021 at 08:13 am, Marc Zyngier wrote: > >>

Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates

2021-11-11 Thread Marc Zyngier
On Thu, 11 Nov 2021 07:47:08 +, Christian Zigotzky wrote: > > On 11 November 2021 at 08:13 am, Marc Zyngier wrote: > > On Thu, 11 Nov 2021 05:24:52 +, > > Christian Zigotzky wrote: > >> Hello Marc, > >> > >> Here you are: > >> ht

[PATCH 1/5] KVM: Move wiping of the kvm->vcpus array to common code

2021-11-05 Thread Marc Zyngier
her changes. Signed-off-by: Marc Zyngier --- arch/arm64/kvm/arm.c | 10 +- arch/mips/kvm/mips.c | 21 + arch/powerpc/kvm/powerpc.c | 10 +- arch/riscv/kvm/vm.c| 10 +- arch/s390/kvm/kvm-s390.c | 18 +- arch/x86/kvm/

[PATCH 5/5] KVM: Convert the kvm->vcpus array to a xarray

2021-11-05 Thread Marc Zyngier
an xarray instead, which gives us almost the same flexibility as a normal array, but with a reduced memory usage with smaller VMs. Signed-off-by: Marc Zyngier --- include/linux/kvm_host.h | 5 +++-- virt/kvm/kvm_main.c | 15 +-- 2 files changed, 12 insertions(+), 8 deletions(-) diff

[PATCH 3/5] KVM: s390: Use kvm_get_vcpu() instead of open-coded access

2021-11-05 Thread Marc Zyngier
As we are about to change the way vcpus are allocated, mandate the use of kvm_get_vcpu() instead of open-coding the access. Signed-off-by: Marc Zyngier --- arch/s390/kvm/kvm-s390.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390

[PATCH 4/5] KVM: x86: Use kvm_get_vcpu() instead of open-coded access

2021-11-05 Thread Marc Zyngier
As we are about to change the way vcpus are allocated, mandate the use of kvm_get_vcpu() instead of open-coding the access. Signed-off-by: Marc Zyngier --- arch/x86/kvm/vmx/posted_intr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kvm/vmx/posted_intr.c b/arch

[PATCH 0/5] KVM: Turn the vcpu array into an xarray

2021-11-05 Thread Marc Zyngier
into an xarray, which results in a net code deletion after a bit of cleanup. This series is on top of the current linux/master as it touches the RISC-V implementation. Only tested on arm64. Marc Zyngier (5): KVM: Move wiping of the kvm->vcpus array to common code KVM: mips: Use kvm_get_v

[PATCH 2/5] KVM: mips: Use kvm_get_vcpu() instead of open-coded access

2021-11-05 Thread Marc Zyngier
As we are about to change the way vcpus are allocated, mandate the use of kvm_get_vcpu() instead of open-coding the access. Signed-off-by: Marc Zyngier --- arch/mips/kvm/loongson_ipi.c | 4 ++-- arch/mips/kvm/mips.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [PATCH 5/5] KVM: Convert the kvm->vcpus array to a xarray

2021-11-08 Thread Marc Zyngier
On 2021-11-06 11:48, Marc Zyngier wrote: On Fri, 05 Nov 2021 20:21:36 +, Sean Christopherson wrote: On Fri, Nov 05, 2021, Marc Zyngier wrote: > At least on arm64 and x86, the vcpus array is pretty huge (512 entries), > and is mostly empty in most cases (running 512 vc

Re: [PATCH 1/5] KVM: Move wiping of the kvm->vcpus array to common code

2021-11-06 Thread Marc Zyngier
On Fri, 05 Nov 2021 20:12:12 +, Sean Christopherson wrote: > > On Fri, Nov 05, 2021, Marc Zyngier wrote: > > All architectures have similar loops iterating over the vcpus, > > freeing one vcpu at a time, and eventually wiping the reference > > off the vc

Re: [PATCH 5/5] KVM: Convert the kvm->vcpus array to a xarray

2021-11-06 Thread Marc Zyngier
On Fri, 05 Nov 2021 20:21:36 +, Sean Christopherson wrote: > > On Fri, Nov 05, 2021, Marc Zyngier wrote: > > At least on arm64 and x86, the vcpus array is pretty huge (512 entries), > > and is mostly empty in most cases (running 512 vcpu VMs is not that > > common).

Re: [PATCH 0/2] Fix arm64 boot regression in 5.14

2021-07-20 Thread Marc Zyngier
: Benjamin Herrenschmidt Cc: Christophe Leroy Cc: Paul Mackerras Cc: Jonathan Marek Cc: Catalin Marinas Cc: Andrew Morton Cc: Nicholas Piggin Cc: Mark Rutland Cc: Geert Uytterhoeven Cc: Marc Zyngier Cc: linuxppc-dev@lists.ozlabs.org Cc: linux-arm-ker...@lists.infradead.org --->8 Jona

Re: [PATCH] powerpc/sysdev: Use of_device_get_match_data()

2022-03-04 Thread Marc Zyngier
On Fri, 04 Mar 2022 13:10:19 +, Christophe Leroy wrote: > > > > Le 04/03/2022 à 02:18, cgel@gmail.com a écrit : > > From: Minghao Chi (CGEL ZTE) > > > > Use of_device_get_match_data() to simplify the code. > > > > Reported-by: Zeal Robot > > Signed-off-by: Minghao Chi (CGEL ZTE) >

Re: [PATCH v6 32/33] arm64: irq-gic: Replace unreachable() with -EINVAL

2022-06-23 Thread Marc Zyngier
On 2022-06-23 02:49, Chen Zhongjin wrote: Using unreachable() at default of switch generates an extra branch at end of the function, and compiler won't generate a ret to close this branch because it knows it's unreachable. If there's no instruction in this branch, compiler will generate a NOP,

Re: [PATCH 23/36] arm64,smp: Remove trace_.*_rcuidle() usage

2022-06-15 Thread Marc Zyngier
} > > > > if ((unsigned)ipinr < NR_IPI) > > - trace_ipi_exit_rcuidle(ipi_types[ipinr]); > > + trace_ipi_exit(ipi_types[ipinr]); > > } > > > > static irqreturn_t ipi_handler(int irq, void *data) Acked-by: Marc Zyngier M. -- Without deviation from the norm, progress is not possible.

  1   2   >