[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 6/6] megasas: undo the overwrites of user configurat
From: |
Cao jin |
Subject: |
[Qemu-devel] [PATCH 6/6] megasas: undo the overwrites of user configuration |
Date: |
Wed, 17 Aug 2016 22:39:07 +0800 |
Commit afea4e14 seems forgetting to undo the overwrites, which is
unsuitable.
cc: Hannes Reinecke <address@hidden>
cc: Paolo Bonzini <address@hidden>
cc: Markus Armbruster <address@hidden>
cc: Marcel Apfelbaum <address@hidden>
cc: Michael S. Tsirkin <address@hidden>
Signed-off-by: Cao jin <address@hidden>
---
hw/scsi/megasas.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/hw/scsi/megasas.c b/hw/scsi/megasas.c
index bde408d..6f73217 100644
--- a/hw/scsi/megasas.c
+++ b/hw/scsi/megasas.c
@@ -2333,11 +2333,10 @@ static void megasas_scsi_realize(PCIDevice *dev, Error
**errp)
"msi=off with this machine type.\n");
error_propagate(errp, err);
return;
- } else if (ret) {
- /* With msi=auto, we fall back to MSI off silently */
- s->msi = ON_OFF_AUTO_OFF;
- error_free(err);
}
+ assert(!err || s->msix == 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), &megasas_mmio_ops, s,
--
2.1.0
- Re: [Qemu-devel] [PATCH 4/6] e1000e: drop unnecessary funtions, (continued)
[Qemu-devel] [PATCH 5/6] megasas: remove unnecessary megasas_use_msix(), Cao jin, 2016/08/17
[Qemu-devel] [PATCH 6/6] megasas: undo the overwrites of user configuration,
Cao jin <=
[Qemu-devel] [PATCH 1/6] msix_init: assert programming error, Cao jin, 2016/08/17
[Qemu-devel] [PATCH 2/6] pci: Convert msix_init() to Error and fix callers to check it, Cao jin, 2016/08/17