This option tells whether a hard disk should be partitioned or not. It defaults to true and have the prime effect of preventing a master boot record (MBR) to be initialized.
This is useful as some operating system (QNX, Rtems) don't recognized FAT mounted disks (especially SD cards) if a MBR is present. Signed-off-by: Clément Chigot <[email protected]> --- block/vvfat.c | 21 +++++++++++++++++++-- qapi/block-core.json | 10 +++++++--- 2 files changed, 26 insertions(+), 5 deletions(-) diff --git a/block/vvfat.c b/block/vvfat.c index 814796d918..de6031db98 100644 --- a/block/vvfat.c +++ b/block/vvfat.c @@ -306,7 +306,8 @@ typedef struct BDRVVVFATState { array_t fat,directory,mapping; char volume_label[11]; - uint32_t offset_to_bootsector; /* 0 for floppy, 0x3f for disk */ + /* 0x3f for partitioned disk, 0x0 otherwise */ + uint32_t offset_to_bootsector; unsigned int cluster_size; unsigned int sectors_per_cluster; @@ -1082,6 +1083,12 @@ static QemuOptsList runtime_opts = { .type = QEMU_OPT_BOOL, .help = "Make the image writable", }, + { + .name = "partitioned", + .type = QEMU_OPT_BOOL, + .def_value_str = "true", + .help = "Do not add a Master Boot Record on this disk", + }, { /* end of list */ } }, }; @@ -1092,6 +1099,7 @@ static void vvfat_parse_filename(const char *filename, QDict *options, int fat_type = 0; bool floppy = false; bool rw = false; + bool partitioned = true; int i; if (!strstart(filename, "fat:", NULL)) { @@ -1116,6 +1124,10 @@ static void vvfat_parse_filename(const char *filename, QDict *options, rw = true; } + if (strstr(filename, ":unpartitioned:")) { + partitioned = false; + } + /* Get the directory name without options */ i = strrchr(filename, ':') - filename; assert(i >= 3); @@ -1131,6 +1143,7 @@ static void vvfat_parse_filename(const char *filename, QDict *options, qdict_put_int(options, "fat-type", fat_type); qdict_put_bool(options, "floppy", floppy); qdict_put_bool(options, "rw", rw); + qdict_put_bool(options, "partitioned", partitioned); } static int vvfat_open(BlockDriverState *bs, QDict *options, int flags, @@ -1196,7 +1209,10 @@ static int vvfat_open(BlockDriverState *bs, QDict *options, int flags, if (!s->fat_type) { s->fat_type = 16; } - s->offset_to_bootsector = 0x3f; + /* Reserver space for MBR */ + if (qemu_opt_get_bool(opts, "partitioned", true)) { + s->offset_to_bootsector = 0x3f; + } cyls = s->fat_type == 12 ? 64 : 1024; heads = 16; secs = 63; @@ -3246,6 +3262,7 @@ static const char *const vvfat_strong_runtime_opts[] = { "floppy", "label", "rw", + "partitioned", NULL }; diff --git a/qapi/block-core.json b/qapi/block-core.json index b82af74256..8a479ba090 100644 --- a/qapi/block-core.json +++ b/qapi/block-core.json @@ -3464,8 +3464,8 @@ # # @fat-type: FAT type: 12, 16 or 32 # -# @floppy: whether to export a floppy image (true) or partitioned hard -# disk (false; default) +# @floppy: whether to export a floppy image (true) or hard disk +# (false; default) # # @label: set the volume label, limited to 11 bytes. FAT16 and FAT32 # traditionally have some restrictions on labels, which are @@ -3474,11 +3474,15 @@ # # @rw: whether to allow write operations (default: false) # +# @partitioned: whether a hard disk will be partitioned +# (default: true) +# (since 10.2) +# # Since: 2.9 ## { 'struct': 'BlockdevOptionsVVFAT', 'data': { 'dir': 'str', '*fat-type': 'int', '*floppy': 'bool', - '*label': 'str', '*rw': 'bool' } } + '*label': 'str', '*rw': 'bool', '*partitioned': 'bool' } } ## # @BlockdevOptionsGenericFormat: -- 2.43.0
