[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v8 14/17] mptsas: remove unnecessary internal msi st
From: |
Cao jin |
Subject: |
[Qemu-devel] [PATCH v8 14/17] mptsas: remove unnecessary internal msi state flag |
Date: |
Fri, 10 Jun 2016 17:54:35 +0800 |
Internal flag msi_in_use in unnecessary, msi_uninit() could be called
directly, and msi_enabled() is enough to check device msi state.
cc: Markus Armbruster <address@hidden>
cc: Marcel Apfelbaum <address@hidden>
cc: Paolo Bonzini <address@hidden>
cc: Michael S. Tsirkin <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Signed-off-by: Cao jin <address@hidden>
---
hw/scsi/mptsas.c | 18 ++++++------------
hw/scsi/mptsas.h | 2 --
2 files changed, 6 insertions(+), 14 deletions(-)
diff --git a/hw/scsi/mptsas.c b/hw/scsi/mptsas.c
index 698be42..c1a0649 100644
--- a/hw/scsi/mptsas.c
+++ b/hw/scsi/mptsas.c
@@ -63,7 +63,7 @@ static void mptsas_update_interrupt(MPTSASState *s)
PCIDevice *pci = (PCIDevice *) s;
uint32_t state = s->intr_status & ~(s->intr_mask |
MPI_HIS_IOP_DOORBELL_STATUS);
- if (s->msi_in_use && msi_enabled(pci)) {
+ if (msi_enabled(pci)) {
if (state) {
trace_mptsas_irq_msi(s);
msi_notify(pci, 0);
@@ -1289,15 +1289,12 @@ static void mptsas_scsi_realize(PCIDevice *dev, Error
**errp)
error_append_hint(&err, "You have to use msi=auto (default) or "
"msi=off with this machine type.\n");
error_propagate(errp, err);
- s->msi_in_use = false;
return;
- } else if (ret) {
- /* With msi=auto, we fall back to MSI off silently */
- error_free(err);
- s->msi_in_use = false;
- } else {
- s->msi_in_use = true;
}
+ assert(!err || s->msi == ON_OFF_AUTO_AUTO);
+ /* With msi=auto, we fall back to MSI off silently */
+ error_free(err);
+
}
memory_region_init_io(&s->mmio_io, OBJECT(s), &mptsas_mmio_ops, s,
@@ -1337,9 +1334,7 @@ static void mptsas_scsi_uninit(PCIDevice *dev)
MPTSASState *s = MPT_SAS(dev);
qemu_bh_delete(s->request_bh);
- if (s->msi_in_use) {
- msi_uninit(dev);
- }
+ msi_uninit(dev);
}
static void mptsas_reset(DeviceState *dev)
@@ -1375,7 +1370,6 @@ static const VMStateDescription vmstate_mptsas = {
.post_load = mptsas_post_load,
.fields = (VMStateField[]) {
VMSTATE_PCI_DEVICE(dev, MPTSASState),
- VMSTATE_BOOL(msi_in_use, MPTSASState),
VMSTATE_UINT32(state, MPTSASState),
VMSTATE_UINT8(who_init, MPTSASState),
diff --git a/hw/scsi/mptsas.h b/hw/scsi/mptsas.h
index 0436a33..da014a3 100644
--- a/hw/scsi/mptsas.h
+++ b/hw/scsi/mptsas.h
@@ -31,8 +31,6 @@ struct MPTSASState {
OnOffAuto msi;
uint64_t sas_addr;
- bool msi_in_use;
-
/* Doorbell register */
uint32_t state;
uint8_t who_init;
--
2.1.0
- [Qemu-devel] [PATCH v8 03/17] change pvscsi_init_msi() type to void, (continued)
- [Qemu-devel] [PATCH v8 03/17] change pvscsi_init_msi() type to void, Cao jin, 2016/06/10
- [Qemu-devel] [PATCH v8 04/17] megasas: Fix check for msi_init() failure, Cao jin, 2016/06/10
- [Qemu-devel] [PATCH v8 09/17] megasas: change msi/msix property type, Cao jin, 2016/06/10
- [Qemu-devel] [PATCH v8 05/17] mptsas: change .realize function name, Cao jin, 2016/06/10
- [Qemu-devel] [PATCH v8 06/17] usb xhci: change msi/msix property type, Cao jin, 2016/06/10
- [Qemu-devel] [PATCH v8 02/17] fix some coding style problems, Cao jin, 2016/06/10
- [Qemu-devel] [PATCH v8 11/17] msi_init: change return value to 0 on success, Cao jin, 2016/06/10
- [Qemu-devel] [PATCH v8 07/17] intel-hda: change msi property type, Cao jin, 2016/06/10
- [Qemu-devel] [PATCH v8 01/17] pci core: assert ENOSPC when add capability, Cao jin, 2016/06/10
- [Qemu-devel] [PATCH v8 14/17] mptsas: remove unnecessary internal msi state flag,
Cao jin <=
- [Qemu-devel] [PATCH v8 12/17] pci: Convert msi_init() to Error and fix callers to check it, Cao jin, 2016/06/10
[Qemu-devel] [PATCH v8 08/17] mptsas: change msi property type, Cao jin, 2016/06/10
[Qemu-devel] [PATCH v8 10/17] pci bridge dev: change msi property type, Cao jin, 2016/06/10
[Qemu-devel] [PATCH v8 15/17] vmw_pvscsi: remove unnecessary internal msi state flag, Cao jin, 2016/06/10