From patchwork Tue Dec  8 14:34:18 2009
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Subject: block/vdi: allow disk sizes not multiple of block size
Date: Tue, 08 Dec 2009 04:34:18 -0000
From: =?utf-8?q?Fran=C3=A7ois_Revol?= <revol@free.fr>
X-Patchwork-Id: 40637
Message-Id: <630568266-BeMail@laptop>
To: qemu-devel@nongnu.org

The disk image I created from my old laptop disk with VBoxManage 
internalcommand converthd obviously was not a multiple of 1MB as when 
created from scratch. This fixes QEMU refusing it. We still require the 
size to be a multiple of sector size though.
It then boots correctly.

Signed-off-by:


Allow opening VDI images with size not multiple of 1MB (as when converted from a raw disk).

Signed-off-by: François Revol <revol@free.fr>

diff --git a/block/vdi.c b/block/vdi.c
index 45aa81c..c91961a 100644
--- a/block/vdi.c
+++ b/block/vdi.c
@@ -411,14 +411,17 @@ static int vdi_open(BlockDriverState *bs, const char *filename, int flags)
         /* We only support data blocks which start on a sector boundary. */
         logout("unsupported data offset 0x%x B\n", header.offset_data);
         goto fail;
+    } else if (header.disk_size % SECTOR_SIZE != 0) {
+        logout("unsupported disk size %" PRIu64 " B\n", header.disk_size);
+        goto fail;
     } else if (header.sector_size != SECTOR_SIZE) {
         logout("unsupported sector size %u B\n", header.sector_size);
         goto fail;
     } else if (header.block_size != 1 * MiB) {
         logout("unsupported block size %u B\n", header.block_size);
         goto fail;
-    } else if (header.disk_size !=
-               (uint64_t)header.blocks_in_image * header.block_size) {
+    } else if ((header.disk_size + header.block_size - 1) / header.block_size !=
+               (uint64_t)header.blocks_in_image) {
         logout("unexpected block number %u B\n", header.blocks_in_image);
         goto fail;
     } else if (!uuid_is_null(header.uuid_link)) {