[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 1/5] m25p80: Improve error when the backend file size does not mat
From: |
Cédric Le Goater |
Subject: |
[PULL 1/5] m25p80: Improve error when the backend file size does not match the device |
Date: |
Tue, 7 Mar 2023 16:55:24 +0100 |
Currently, when a block backend is attached to a m25p80 device and the
associated file size does not match the flash model, QEMU complains
with the error message "failed to read the initial flash content".
This is confusing for the user.
Instead, use helper blk_check_size_and_read_all() introduced by commit
06f1521795 ("pflash: Require backend size to match device, improve
errors").
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Peter Delevoryas <peter@pjd.dev>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221115151000.2080833-1-clg@kaod.org>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
---
hw/block/m25p80.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c
index 802d2eb021..dc5ffbc4ff 100644
--- a/hw/block/m25p80.c
+++ b/hw/block/m25p80.c
@@ -24,6 +24,7 @@
#include "qemu/osdep.h"
#include "qemu/units.h"
#include "sysemu/block-backend.h"
+#include "hw/block/block.h"
#include "hw/qdev-properties.h"
#include "hw/qdev-properties-system.h"
#include "hw/ssi/ssi.h"
@@ -1615,8 +1616,7 @@ static void m25p80_realize(SSIPeripheral *ss, Error
**errp)
trace_m25p80_binding(s);
s->storage = blk_blockalign(s->blk, s->size);
- if (blk_pread(s->blk, 0, s->size, s->storage, 0) < 0) {
- error_setg(errp, "failed to read the initial flash content");
+ if (!blk_check_size_and_read_all(s->blk, s->storage, s->size, errp)) {
return;
}
} else {
--
2.39.2
- [PULL 0/5] aspeed queue, Cédric Le Goater, 2023/03/07
- [PULL 3/5] hw/arm/aspeed: Added TMP421 type sensor's support in yosemitev2, Cédric Le Goater, 2023/03/07
- [PULL 2/5] pflash: Fix blk_pread_nonzeroes(), Cédric Le Goater, 2023/03/07
- [PULL 4/5] hw/arm/aspeed: Added TMP421 type sensor's support in tiogapass, Cédric Le Goater, 2023/03/07
- [PULL 1/5] m25p80: Improve error when the backend file size does not match the device,
Cédric Le Goater <=
- [PULL 5/5] hw/arm/aspeed: Modified BMC FRU byte data in yosemitev2, Cédric Le Goater, 2023/03/07
- Re: [PULL 0/5] aspeed queue, Peter Maydell, 2023/03/09