qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: peter.maydell@linaro.org
Cc: qemu-devel@nongnu.org, rth@twiddle.net, agraf@suse.de,
	thuth@redhat.com, borntraeger@de.ibm.com, david@redhat.com,
	Yi Min Zhao <zyimin@linux.vnet.ibm.com>,
	Cornelia Huck <cohuck@redhat.com>
Subject: [Qemu-devel] [PULL v2 04/38] s390x/pci: fixup ind_offset of msix routing entry
Date: Tue, 19 Sep 2017 18:43:03 +0200	[thread overview]
Message-ID: <20170919164337.18555-5-cohuck@redhat.com> (raw)
In-Reply-To: <20170919164337.18555-1-cohuck@redhat.com>

From: Yi Min Zhao <zyimin@linux.vnet.ibm.com>

The guest uses the mpcifc instruction to register the aibvo of a zpci
device, which is the starting offset of indicators in the indicator
area and thus remains constant. Each msix vector is an offset from the
aibvo. When we map a msix route to an adapter route, we should not
modify the starting offset, but instead add the vector to the starting
offset to get the absolute offset in the specific route.

Signed-off-by: Yi Min Zhao <zyimin@linux.vnet.ibm.com>
Message-Id: <1504606380-49341-3-git-send-email-zyimin@linux.vnet.ibm.com>
Signed-off-by: Cornelia Huck <cohuck@redhat.com>
---
 target/s390x/kvm.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
index 1130a391d4..375369fd1c 100644
--- a/target/s390x/kvm.c
+++ b/target/s390x/kvm.c
@@ -2436,14 +2436,12 @@ int kvm_arch_fixup_msi_route(struct kvm_irq_routing_entry *route,
         return -ENODEV;
     }
 
-    pbdev->routes.adapter.ind_offset = vec;
-
     route->type = KVM_IRQ_ROUTING_S390_ADAPTER;
     route->flags = 0;
     route->u.adapter.summary_addr = pbdev->routes.adapter.summary_addr;
     route->u.adapter.ind_addr = pbdev->routes.adapter.ind_addr;
     route->u.adapter.summary_offset = pbdev->routes.adapter.summary_offset;
-    route->u.adapter.ind_offset = pbdev->routes.adapter.ind_offset;
+    route->u.adapter.ind_offset = pbdev->routes.adapter.ind_offset + vec;
     route->u.adapter.adapter_id = pbdev->routes.adapter.adapter_id;
     return 0;
 }
-- 
2.13.5

  parent reply	other threads:[~2017-09-19 16:44 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-19 16:42 [Qemu-devel] [PULL v2 00/38] various s390x patches (+some fixes) Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 01/38] s390x/css: fix cc handling for XSCH Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 02/38] tests: Enable the drive_del test also on s390x Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 03/38] s390x/pci: remove idx from msix msg data Cornelia Huck
2017-09-19 16:43 ` Cornelia Huck [this message]
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 05/38] s390x/pci: add iommu replay callback Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 06/38] hw/misc/ivshmem: Fix ivshmem_recv_msg() to also work on big endian systems Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 07/38] s390x/css: drop data-check in interpretation Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 08/38] s390x/css: fix NULL handling for CCW addresses Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 09/38] s390x/css: remove unused error handling branch Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 10/38] s390: set DHCP client architecure id for netboot Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 11/38] exec, dump, i386, ppc, s390x: don't include exec/cpu-all.h explicitly Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 12/38] cpu: drop old comments describing members Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 13/38] s390x: get rid of s390-virtio.c Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 14/38] s390x: rename s390-virtio.h to s390-virtio-hcall.h Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 15/38] s390x: move s390_virtio_hypercall() " Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 16/38] s390x: move subsystem_reset() to s390-virtio-ccw.h Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 17/38] target/s390x: move some s390x typedefs to cpu-qom.h Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 18/38] s390x: move sclp_service_call() to sclp.h Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 19/38] target/s390x: use trigger_pgm_exception() in s390_cpu_handle_mmu_fault() Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 20/38] target/s390x: use program_interrupt() in per_check_exception() Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 21/38] s390x: allow only 1 CPU with TCG Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 22/38] target/s390x: set cpu->id for linux user when realizing Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 23/38] target/s390x: use "core-id" for cpu number/address/id handling Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 24/38] target/s390x: rename next_cpu_id to next_core_id Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 25/38] s390x: print CPU definitions in sorted order Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 26/38] s390x: allow cpu hotplug via device_add Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 27/38] s390x: CPU hot unplug via device_del cannot work for now Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 28/38] s390x: implement query-hotpluggable-cpus Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 29/38] s390x: get rid of cpu_states and use possible_cpus instead Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 30/38] s390x: get rid of cpu_s390x_create() Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 31/38] s390x: generate sclp cpu information from possible_cpus Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 32/38] s390x: allow CPU hotplug in random core-id order Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 33/38] virtio-ccw: remove stale comments on endianness Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 34/38] configure: Allow --enable-seccomp on s390x, too Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 35/38] s390x/ccw: create s390 phb for compat reasons as well Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 36/38] virtio-gpu: Handle endian conversion Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 37/38] virtio-ccw: Create a virtio gpu device for the ccw bus Cornelia Huck
2017-09-19 16:43 ` [Qemu-devel] [PULL v2 38/38] MAINTAINERS/s390x: add terminal3270.c Cornelia Huck
2017-09-19 17:03 ` [Qemu-devel] [PULL v2 00/38] various s390x patches (+some fixes) Christian Borntraeger
2017-09-20  7:20   ` Cornelia Huck
2017-09-19 18:23 ` Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170919164337.18555-5-cohuck@redhat.com \
    --to=cohuck@redhat.com \
    --cc=agraf@suse.de \
    --cc=borntraeger@de.ibm.com \
    --cc=david@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    --cc=thuth@redhat.com \
    --cc=zyimin@linux.vnet.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).