]> git.proxmox.com Git - qemu.git/commit - blockdev.c
block: Allow omitting the file name when using driver-specific options
authorKevin Wolf <kwolf@redhat.com>
Mon, 18 Mar 2013 15:40:51 +0000 (16:40 +0100)
committerKevin Wolf <kwolf@redhat.com>
Fri, 22 Mar 2013 16:51:32 +0000 (17:51 +0100)
commitc2ad1b0c465a9ea8375eaff14bbd85705c673f73
tree705456a5885c9ac7223fa2b75158e09adbc1d38f
parentf5866fa438bff586f215c137dc71edb4e0770536
block: Allow omitting the file name when using driver-specific options

After this patch, using -drive with an empty file name continues to open
the file if driver-specific options are used. If no driver-specific
options are specified, the semantics stay as it was: It defines a drive
without an inserted medium.

In order to achieve this, bdrv_open() must be made safe to work with a
NULL filename parameter. The assumption that is made is that only block
drivers which implement bdrv_parse_filename() support using driver
specific options and could therefore work without a filename. These
drivers must make sure to cope with NULL in their implementation of
.bdrv_open() (this is only NBD for now). For all other drivers, the
block layer code will make sure to error out before calling into their
code - they can't possibly work without a filename.

Now an NBD connection can be opened like this:

  qemu-system-x86_64 -drive file.driver=nbd,file.port=1234,file.host=::1

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
block.c
blockdev.c
include/block/block_int.h