[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/8] block: Integrate bdrv_file_open() into b
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/8] block: Integrate bdrv_file_open() into bdrv_open() |
Date: |
Mon, 17 Feb 2014 13:42:43 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 15.02.2014 um 04:30 hat Max Reitz geschrieben:
> bdrv_file_open() is now nearly a subset of bdrv_open(), except for the
> fact that bdrv_file_open() is for protocols and bdrv_open() for block
> drivers. It is possible to use bdrv_file_open() with a block driver, but
> in that case that block driver must be explicitly specified.
>
> Due to these great similarities, bdrv_file_open() can be integrated and
> made a special case of bdrv_open(). If the flag BDRV_O_PROTOCOL is
> specified, bdrv_open() will now do what bdrv_file_open() used to do:
> Auto-detecting a protocol instead of a block driver.
>
> This series implements this and changes all calls to bdrv_file_open() to
> bdrv_open() calls with BDRV_O_PROTOCOL specified.
>
> Note that this flag cannot be discerned automatically since it is
> impossible for bdrv_open() to know by itself whether a given file should
> be opened with or without the format layer involved: Both are valid
> alternatives. Therefore, it still has to be specified by the user.
This series conflicts with Benoît's patches that have been merged into
master. When rebasing, please be careful with the code motion patch so
that you don't accidentally revert Benoît's changes. (It's an easy
conflict to resolve, but not trivial enough for me to do it while
applying the patch, with no additional review.)
Kevin
- [Qemu-devel] [PATCH v3 1/8] block: Change BDS parameter of bdrv_open() to **, (continued)
- [Qemu-devel] [PATCH v3 1/8] block: Change BDS parameter of bdrv_open() to **, Max Reitz, 2014/02/14
- [Qemu-devel] [PATCH v3 2/8] block: Add reference parameter to bdrv_open(), Max Reitz, 2014/02/14
- [Qemu-devel] [PATCH v3 3/8] block: Make bdrv_file_open() static, Max Reitz, 2014/02/14
- [Qemu-devel] [PATCH v3 4/8] block: Reuse reference handling from bdrv_open(), Max Reitz, 2014/02/14
- [Qemu-devel] [PATCH v3 5/8] block: Remove bdrv_new() from bdrv_file_open(), Max Reitz, 2014/02/14
- [Qemu-devel] [PATCH v3 6/8] block: Handle bs->options in bdrv_open() only, Max Reitz, 2014/02/14
- [Qemu-devel] [PATCH v3 7/8] block: Reuse success path from bdrv_open(), Max Reitz, 2014/02/14
- [Qemu-devel] [PATCH v3 8/8] block: Remove bdrv_open_image()'s force_raw option, Max Reitz, 2014/02/14
- Re: [Qemu-devel] [PATCH v3 0/8] block: Integrate bdrv_file_open() into bdrv_open(),
Kevin Wolf <=