[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v11 00/28] IOMMU: Enable interrupt remapping for
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v11 00/28] IOMMU: Enable interrupt remapping for Intel IOMMU |
Date: |
Fri, 8 Jul 2016 18:36:16 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 |
On 08/07/2016 18:01, Radim Krčmář wrote:
> 2016-07-05 16:19+0800, Peter Xu:
>> This is v11 of Intel IR work. It is rebased to mst's branch
>> "tags/for_upstream", commit:
>>
>> "278a2a2 vmw_pvscsi: remove unnecessary internal msi state flag"
>>
>> This series mainly fixed several issues in v10 review comments, fixed
>> one bug with RHEL guests, added acked-by for Paolo, and a fresh new
>> rebase as mentioned above.
>>
>> To make it fast, I only did quick tests for this versiont. But at
>> least it should cover basic functions like: IOAPIC, MSI, multiple
>> vcpus, different guests (4.7 upstream and rhel 7.2), vhost, split/off
>> irqchips. More tests to be done.
>>
>> Meanwhile, there are several pending issues to be solved, which is
>> queued in my todo list and I'll continue the work after this series is
>> merged.
>>
>> Online branch:
>>
>> https://github.com/xzpeter/qemu vtd-intr-v11
>>
>> Please review, thanks.
>
> Testing found only one bug:
>
> The patchset doesn't work with if you have 16 APICs with IDs 0-15 and
> then some more, because KVM has hacked x2APIC support that translates
> logical interrupt with destination 0xff (first cluster of 16 VCPUs) into
> a broadcast (also works with lowest-priority) and then picks VCPU > 16
> as the destination, leading to messages like
>
> do_IRQ: 17.209 No irq handler for vector
>
> The fun part is that this is how KVM wanted to behave, when it allowed
> x2apic with unremapped ioapic. :(
> KVM could either finally remove x2apic without IR or add a way in which
> userspace could say whether an interrupt is x2APIC or not.
>
> QEMU cannot do anything to work around the bug, so I think it would be
> best to disable EIM for now.
What's the state of QEMU support for KVM_CAP_X2APIC_API? If it can make
it for hard freeze, it's not a showstopper for Peter's patch.
Thanks,
Paolo
- [Qemu-devel] [PATCH v11 20/28] intel_iommu: Add support for Extended Interrupt Mode, (continued)
- [Qemu-devel] [PATCH v11 20/28] intel_iommu: Add support for Extended Interrupt Mode, Peter Xu, 2016/07/05
- [Qemu-devel] [PATCH v11 21/28] intel_iommu: add SID validation for IR, Peter Xu, 2016/07/05
- [Qemu-devel] [PATCH v11 22/28] kvm-irqchip: simplify kvm_irqchip_add_msi_route, Peter Xu, 2016/07/05
- [Qemu-devel] [PATCH v11 23/28] kvm-irqchip: i386: add hook for add/remove virq, Peter Xu, 2016/07/05
- [Qemu-devel] [PATCH v11 24/28] kvm-irqchip: x86: add msi route notify fn, Peter Xu, 2016/07/05
- [Qemu-devel] [PATCH v11 25/28] kvm-irqchip: do explicit commit when update irq, Peter Xu, 2016/07/05
- [Qemu-devel] [PATCH v11 26/28] intel_iommu: support all masks in interrupt entry cache invalidation, Peter Xu, 2016/07/05
- [Qemu-devel] [PATCH v11 27/28] kvm-all: add trace events for kvm irqchip ops, Peter Xu, 2016/07/05
- [Qemu-devel] [PATCH v11 28/28] intel_iommu: disallow kernel-irqchip=on with IR, Peter Xu, 2016/07/05
- Re: [Qemu-devel] [PATCH v11 00/28] IOMMU: Enable interrupt remapping for Intel IOMMU, Radim Krčmář, 2016/07/08
- Re: [Qemu-devel] [PATCH v11 00/28] IOMMU: Enable interrupt remapping for Intel IOMMU,
Paolo Bonzini <=