[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v5 4/7] target-arm/kvm: Pass requester ID to MSI r
From: |
Auger Eric |
Subject: |
Re: [Qemu-devel] [RFC v5 4/7] target-arm/kvm: Pass requester ID to MSI routing functions |
Date: |
Wed, 17 Aug 2016 18:05:05 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 |
Hi Peter,
On 12/08/2016 16:19, Peter Maydell wrote:
> On 2 August 2016 at 19:07, Eric Auger <address@hidden> wrote:
>> From: Pavel Fedin <address@hidden>
>>
>> Introduce global kvm_arm_msi_use_devid flag and pass device IDs in
>> kvm_arch_fixup_msi_route(). Device IDs are required by the ITS.
>>
>> Signed-off-by: Pavel Fedin <address@hidden>
>> Signed-off-by: Eric Auger <address@hidden>
>>
>> ---
>>
>> v3 -> v4:
>> - OR route->flags with KVM_MSI_VALID_DEVID
>> ---
>> target-arm/kvm.c | 6 ++++++
>> target-arm/kvm_arm.h | 3 +++
>> 2 files changed, 9 insertions(+)
>>
>> diff --git a/target-arm/kvm.c b/target-arm/kvm.c
>> index dbe393c..4675aa3 100644
>> --- a/target-arm/kvm.c
>> +++ b/target-arm/kvm.c
>> @@ -22,6 +22,7 @@
>> #include "cpu.h"
>> #include "internals.h"
>> #include "hw/arm/arm.h"
>> +#include "hw/pci/pci.h"
>> #include "exec/memattrs.h"
>> #include "hw/boards.h"
>> #include "qemu/log.h"
>> @@ -31,6 +32,7 @@ const KVMCapabilityInfo kvm_arch_required_capabilities[] =
>> {
>> };
>>
>> static bool cap_has_mp_state;
>> +bool kvm_arm_msi_use_devid;
>>
>> int kvm_arm_vcpu_init(CPUState *cs)
>> {
>> @@ -619,6 +621,10 @@ int kvm_arm_vgic_probe(void)
>> int kvm_arch_fixup_msi_route(struct kvm_irq_routing_entry *route,
>> uint64_t address, uint32_t data, PCIDevice
>> *dev)
>> {
>> + if (kvm_arm_msi_use_devid) {
>> + route->flags |= KVM_MSI_VALID_DEVID;
>> + route->u.msi.devid = pci_requester_id(dev);
>> + }
>> return 0;
>> }
>>
>> diff --git a/target-arm/kvm_arm.h b/target-arm/kvm_arm.h
>> index 633d088..befcfd1 100644
>> --- a/target-arm/kvm_arm.h
>> +++ b/target-arm/kvm_arm.h
>> @@ -121,6 +121,9 @@ bool write_kvmstate_to_list(ARMCPU *cpu);
>> void kvm_arm_reset_vcpu(ARMCPU *cpu);
>>
>> #ifdef CONFIG_KVM
>> +
>> +extern bool kvm_arm_msi_use_devid;
>> +
>
> The kernel documentation and most of the code for this flag
> isn't ARM specific, which suggests to me that we should put
> the QEMU support into the non-architecture-specific files too.
> That is, a flag and wrapper function like all the existing
> ones in include/sysemu/kvm.h, and then put the
> if (kvm_msi_use_devid()) {
> ...
> }
> in the functions in kvm-all.c that need it.
OK this makes sense to me too
Thank you for your time
Best Regards
Eric
>
> thanks
> -- PMM
>
- [Qemu-devel] [RFC v5 0/7] vITS support, Eric Auger, 2016/08/02
- [Qemu-devel] [RFC v5 2/7] target-arm: move gicv3_class_name from machine to kvm_arm.h, Eric Auger, 2016/08/02
- [Qemu-devel] [RFC v5 1/7] hw/intc/arm_gicv3_its: Implement ITS base class, Eric Auger, 2016/08/02
- [Qemu-devel] [RFC v5 3/7] linux-headers: update to 4.7-rc6 + ITS emulation and GSI routing, Eric Auger, 2016/08/02
- [Qemu-devel] [RFC v5 4/7] target-arm/kvm: Pass requester ID to MSI routing functions, Eric Auger, 2016/08/02
- [Qemu-devel] [RFC v5 5/7] hw/intc/arm_gicv3_its: Implement support for in-kernel ITS emulation, Eric Auger, 2016/08/02
- [Qemu-devel] [RFC v5 6/7] arm/virt: Add ITS to the virt board, Eric Auger, 2016/08/02
- [Qemu-devel] [RFC v5 7/7] hw/arm/virt-acpi-build: Add ITS description in ACPI MADT table, Eric Auger, 2016/08/02