|
 |
b38b0f |
From 707a777c2992e840d2c3dd4e1fbed5b0d6c682ec Mon Sep 17 00:00:00 2001
|
|
|
b38b0f |
From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= <philmd@redhat.com>
|
|
|
b38b0f |
Date: Tue, 23 Jul 2019 11:51:05 +0100
|
|
|
b38b0f |
Subject: [PATCH 01/14] hw/block/pflash_cfi01: Add missing DeviceReset()
|
|
|
b38b0f |
handler
|
|
|
b38b0f |
MIME-Version: 1.0
|
|
|
b38b0f |
Content-Type: text/plain; charset=UTF-8
|
|
|
b38b0f |
Content-Transfer-Encoding: 8bit
|
|
|
b38b0f |
|
|
|
b38b0f |
RH-Author: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
b38b0f |
Message-id: <20190723115105.31305-2-philmd@redhat.com>
|
|
|
b38b0f |
Patchwork-id: 89645
|
|
|
b38b0f |
O-Subject: [RHEL-8.1.0 qemu-kvm PATCH 1/1] hw/block/pflash_cfi01: Add missing DeviceReset() handler
|
|
|
b38b0f |
Bugzilla: 1707192
|
|
|
b38b0f |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
b38b0f |
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
b38b0f |
RH-Acked-by: John Snow <jsnow@redhat.com>
|
|
|
b38b0f |
|
|
|
b38b0f |
To avoid incoherent states when the machine resets (see bug report
|
|
|
b38b0f |
below), add the device reset callback.
|
|
|
b38b0f |
|
|
|
b38b0f |
A "system reset" sets the device state machine in READ_ARRAY mode
|
|
|
b38b0f |
and, after some delay, set the SR.7 READY bit.
|
|
|
b38b0f |
|
|
|
b38b0f |
Since we do not model timings, we set the SR.7 bit directly.
|
|
|
b38b0f |
|
|
|
b38b0f |
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1678713
|
|
|
b38b0f |
Reported-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
b38b0f |
Reviewed-by: John Snow <jsnow@redhat.com>
|
|
|
b38b0f |
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
|
|
|
b38b0f |
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
b38b0f |
Tested-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
b38b0f |
[Laszlo Ersek: Regression tested EDK2 OVMF IA32X64, ArmVirtQemu Aarch64
|
|
|
b38b0f |
https://lists.gnu.org/archive/html/qemu-devel/2019-07/msg04373.html]
|
|
|
b38b0f |
Message-Id: <20190718104837.13905-2-philmd@redhat.com>
|
|
|
b38b0f |
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
b38b0f |
(cherry picked from commit 3a283507c03474d285196620fca506bd1a89b198)
|
|
|
b38b0f |
[PMD: upstream commit e7b6274197c changed PFLASH_CFI01 <- CFI_PFLASH01,
|
|
|
b38b0f |
and upstream commit 1643406520f changed PFlashCFI01 <- pflash_t]
|
|
|
b38b0f |
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
b38b0f |
|
|
|
b38b0f |
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
|
|
|
b38b0f |
---
|
|
|
b38b0f |
hw/block/pflash_cfi01.c | 19 +++++++++++++++++++
|
|
|
b38b0f |
1 file changed, 19 insertions(+)
|
|
|
b38b0f |
|
|
|
b38b0f |
diff --git a/hw/block/pflash_cfi01.c b/hw/block/pflash_cfi01.c
|
|
|
b38b0f |
index 2e82840..1be351e 100644
|
|
|
b38b0f |
--- a/hw/block/pflash_cfi01.c
|
|
|
b38b0f |
+++ b/hw/block/pflash_cfi01.c
|
|
|
b38b0f |
@@ -876,6 +876,24 @@ static void pflash_cfi01_realize(DeviceState *dev, Error **errp)
|
|
|
b38b0f |
pfl->cfi_table[0x3f] = 0x01; /* Number of protection fields */
|
|
|
b38b0f |
}
|
|
|
b38b0f |
|
|
|
b38b0f |
+static void pflash_cfi01_system_reset(DeviceState *dev)
|
|
|
b38b0f |
+{
|
|
|
b38b0f |
+ pflash_t *pfl = CFI_PFLASH01(dev);
|
|
|
b38b0f |
+
|
|
|
b38b0f |
+ /*
|
|
|
b38b0f |
+ * The command 0x00 is not assigned by the CFI open standard,
|
|
|
b38b0f |
+ * but QEMU historically uses it for the READ_ARRAY command (0xff).
|
|
|
b38b0f |
+ */
|
|
|
b38b0f |
+ pfl->cmd = 0x00;
|
|
|
b38b0f |
+ pfl->wcycle = 0;
|
|
|
b38b0f |
+ memory_region_rom_device_set_romd(&pfl->mem, true);
|
|
|
b38b0f |
+ /*
|
|
|
b38b0f |
+ * The WSM ready timer occurs at most 150ns after system reset.
|
|
|
b38b0f |
+ * This model deliberately ignores this delay.
|
|
|
b38b0f |
+ */
|
|
|
b38b0f |
+ pfl->status = 0x80;
|
|
|
b38b0f |
+}
|
|
|
b38b0f |
+
|
|
|
b38b0f |
static Property pflash_cfi01_properties[] = {
|
|
|
b38b0f |
DEFINE_PROP_DRIVE("drive", struct pflash_t, blk),
|
|
|
b38b0f |
/* num-blocks is the number of blocks actually visible to the guest,
|
|
|
b38b0f |
@@ -920,6 +938,7 @@ static void pflash_cfi01_class_init(ObjectClass *klass, void *data)
|
|
|
b38b0f |
{
|
|
|
b38b0f |
DeviceClass *dc = DEVICE_CLASS(klass);
|
|
|
b38b0f |
|
|
|
b38b0f |
+ dc->reset = pflash_cfi01_system_reset;
|
|
|
b38b0f |
dc->realize = pflash_cfi01_realize;
|
|
|
b38b0f |
dc->props = pflash_cfi01_properties;
|
|
|
b38b0f |
dc->vmsd = &vmstate_pflash;
|
|
|
b38b0f |
--
|
|
|
b38b0f |
1.8.3.1
|
|
|
b38b0f |
|