[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 15/17] s390x/pci: fix stpcifc_service_call
From: |
Cornelia Huck |
Subject: |
[Qemu-devel] [PATCH 15/17] s390x/pci: fix stpcifc_service_call |
Date: |
Fri, 24 Jun 2016 15:29:04 +0200 |
From: Yi Min Zhao <address@hidden>
Firstly the function misses dmaas checking. This patch adds it.
Secondly the function uses s390_pci_find_dev_by_fh() to look up the
zpci device. This may fail if the guest provides a valid and disabled
fh but fh of the associated zpci device is enabled. Thus we use
s390_pci_find_dev_by_idx() instead.
Signed-off-by: Yi Min Zhao <address@hidden>
Reviewed-by: Pierre Morel <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>
---
hw/s390x/s390-pci-inst.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/hw/s390x/s390-pci-inst.c b/hw/s390x/s390-pci-inst.c
index 70db835..37572df 100644
--- a/hw/s390x/s390-pci-inst.c
+++ b/hw/s390x/s390-pci-inst.c
@@ -944,6 +944,7 @@ int mpcifc_service_call(S390CPU *cpu, uint8_t r1, uint64_t
fiba, uint8_t ar)
int stpcifc_service_call(S390CPU *cpu, uint8_t r1, uint64_t fiba, uint8_t ar)
{
CPUS390XState *env = &cpu->env;
+ uint8_t dmaas;
uint32_t fh;
ZpciFib fib;
S390PCIBusDevice *pbdev;
@@ -956,13 +957,20 @@ int stpcifc_service_call(S390CPU *cpu, uint8_t r1,
uint64_t fiba, uint8_t ar)
}
fh = env->regs[r1] >> 32;
+ dmaas = (env->regs[r1] >> 16) & 0xff;
+
+ if (dmaas) {
+ setcc(cpu, ZPCI_PCI_LS_ERR);
+ s390_set_status_code(env, r1, ZPCI_STPCIFC_ST_INVAL_DMAAS);
+ return 0;
+ }
if (fiba & 0x7) {
program_interrupt(env, PGM_SPECIFICATION, 6);
return 0;
}
- pbdev = s390_pci_find_dev_by_fh(fh);
+ pbdev = s390_pci_find_dev_by_idx(fh & FH_MASK_INDEX);
if (!pbdev) {
setcc(cpu, ZPCI_PCI_LS_INVAL_HANDLE);
return 0;
--
2.9.0
- [Qemu-devel] [PATCH 00/17] s390x: the big pci update, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 05/17] s390x/pci: refactor s390_pci_find_dev_by_fh, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 16/17] s390x/pci: replace fid with idx in msg data of msix, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 17/17] s390x/pci: make hot-unplug handler smoother, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 08/17] s390x/pci: introduce S390PCIIOMMU, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 15/17] s390x/pci: fix stpcifc_service_call,
Cornelia Huck <=
- [Qemu-devel] [PATCH 04/17] s390x/pci: unify FH_ macros, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 03/17] s390x/pci: write fid in CLP_QUERY_PCI_FN, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 10/17] s390x/pci: enable uid-checking, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 12/17] s390x/pci: add checkings in CLP_SET_PCI_FN, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 06/17] s390x/pci: enforce zPCI state checking, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 01/17] s390x/pci: fix failures of dma map/unmap, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 02/17] s390x/pci: acceleration for getting S390pciState, Cornelia Huck, 2016/06/24
- [Qemu-devel] [PATCH 09/17] s390x/pci: introduce S390PCIBusDevice qdev, Cornelia Huck, 2016/06/24