On Tue, 2020-11-10 at 16:54 +0100, Thomas Gleixner wrote: > On Tue, Nov 10 2020 at 08:55, Tom Lendacky wrote: > > On 11/10/20 8:34 AM, Thomas Gleixner wrote: > > I was about to send the dmesg output when I saw this. A quick test > > with > > this change resolves the boot issue, thanks! > > /me feels stupid > > > I'm still seeing the warning at arch/x86/kernel/apic/apic.c:2527, > > but I'll > > start a separate thread on that. > > Can you please provide the backtrace?
We'll want something like this, to stop AMD IOMMU from registering its
irqdomain even when it shouldn't. But that's the wrong way round; that
would give you remapped MSIs when it should be using the
x86_vector_domain for them. And you have the converse, it seems.
--- a/drivers/iommu/amd/init.c
+++ b/drivers/iommu/amd/init.c
@@ -1601,9 +1601,11 @@ static int __init init_iommu_one(struct amd_iommu
*iommu, struct ivhd_header *h)
if (ret)
return ret;
- ret = amd_iommu_create_irq_domain(iommu);
- if (ret)
- return ret;
+ if (amd_iommu_irq_remap) {
+ ret = amd_iommu_create_irq_domain(iommu);
+ if (ret)
+ return ret;
+ }
/*
* Make sure IOMMU is not considered to translate itself. The IVRS
smime.p7s
Description: S/MIME cryptographic signature
