[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 8/9] s390x/kvm: msi route fixup for non-pci
From: |
David Hildenbrand |
Subject: |
Re: [Qemu-devel] [PATCH v4 8/9] s390x/kvm: msi route fixup for non-pci |
Date: |
Fri, 4 Aug 2017 15:18:14 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 04.08.2017 13:29, Cornelia Huck wrote:
> If we don't provide pci, we cannot have a pci device for which we
> have to translate to adapter routes: just return -ENODEV.
>
> Signed-off-by: Cornelia Huck <address@hidden>
> ---
> target/s390x/kvm.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
> index 9de165d8b1..d8db1cbf6e 100644
> --- a/target/s390x/kvm.c
> +++ b/target/s390x/kvm.c
> @@ -2424,6 +2424,12 @@ int kvm_arch_fixup_msi_route(struct
> kvm_irq_routing_entry *route,
> uint32_t idx = data >> ZPCI_MSI_VEC_BITS;
> uint32_t vec = data & ZPCI_MSI_VEC_MASK;
>
> + if (!s390_has_feat(S390_FEAT_ZPCI)) {
> + /* How can we get here without pci enabled? */
> + g_assert(false);
> + return -ENODEV;
> + }
> +
/* How can we get here without pci enabled? */
g_assert(s390_has_feat(S390_FEAT_ZPCI));
?
> pbdev = s390_pci_find_dev_by_idx(s390_get_phb(), idx);
> if (!pbdev) {
> DPRINTF("add_msi_route no dev\n");
>
--
Thanks,
David
[Qemu-devel] [PATCH v4 9/9] s390x: refine pci dependencies, Cornelia Huck, 2017/08/04
[Qemu-devel] [PATCH v4 3/9] s390x: chsc nt2 events are pci-only, Cornelia Huck, 2017/08/04
[Qemu-devel] [PATCH v4 1/9] kvm: remove hard dependency on pci, Cornelia Huck, 2017/08/04