[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[RFC PATCH 2/5] hw/block: Flash images can be smaller than the device
From: |
David Edmondson |
Subject: |
[RFC PATCH 2/5] hw/block: Flash images can be smaller than the device |
Date: |
Mon, 16 Nov 2020 10:42:13 +0000 |
When loading a flash image into a device, allow the image to be
smaller than the extent of the device.
Signed-off-by: David Edmondson <david.edmondson@oracle.com>
---
hw/block/block.c | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/hw/block/block.c b/hw/block/block.c
index 8b284e1f14..40262546bd 100644
--- a/hw/block/block.c
+++ b/hw/block/block.c
@@ -16,8 +16,8 @@
/*
* Read the entire contents of @blk into @buf.
- * @blk's contents must be @size bytes, and @size must be at most
- * BDRV_REQUEST_MAX_BYTES.
+ * @blk's contents must not be more than @size bytes, and must be at
+ * most BDRV_REQUEST_MAX_BYTES in length.
* On success, return true.
* On failure, store an error through @errp and return false.
* This function not intended for actual block devices, which read on
@@ -38,10 +38,10 @@ bool blk_check_size_and_read_all(BlockBackend *blk, void
*buf, hwaddr size,
name);
return false;
}
- if (blk_len != size) {
- error_setg(errp, "device requires %" HWADDR_PRIu " bytes, "
- "block backend %s provides %" PRIu64 " bytes",
- size, name, blk_len);
+ if (blk_len > size) {
+ error_setg(errp, "block backend %s is too large for device "
+ "(%" PRIu64 " > %" HWADDR_PRIu ")",
+ name, blk_len, size);
return false;
}
@@ -51,8 +51,8 @@ bool blk_check_size_and_read_all(BlockBackend *blk, void
*buf, hwaddr size,
* should probably rework the device to be more like an actual
* block device and read only on demand.
*/
- assert(size <= BDRV_REQUEST_MAX_BYTES);
- ret = blk_pread(blk, 0, buf, size);
+ assert(blk_len <= BDRV_REQUEST_MAX_BYTES);
+ ret = blk_pread(blk, 0, buf, blk_len);
if (ret < 0) {
error_setg_errno(errp, -ret, "can't read block backend %s",
name);
--
2.28.0
- [RFC PATCH 0/5] ARM: reduce the memory consumed when mapping UEFI flash images, David Edmondson, 2020/11/16
- [RFC PATCH 1/5] hw/block: blk_check_size_and_read_all should report backend name, David Edmondson, 2020/11/16
- [RFC PATCH 4/5] hw/arm: Flash image mapping follows image size, David Edmondson, 2020/11/16
- [RFC PATCH 5/5] hw/arm: Only minimise flash size on older machines, David Edmondson, 2020/11/16
- [RFC PATCH 3/5] hw/arm: Convert assertions about flash image size to error_report, David Edmondson, 2020/11/16
- [RFC PATCH 2/5] hw/block: Flash images can be smaller than the device,
David Edmondson <=
- Re: [RFC PATCH 0/5] ARM: reduce the memory consumed when mapping UEFI flash images, Philippe Mathieu-Daudé, 2020/11/16