Am 11.05.2015 um 16:40 hat Max Reitz geschrieben: > On 08.05.2015 19:21, Kevin Wolf wrote: > >Signed-off-by: Kevin Wolf <kw...@redhat.com> > >--- > > blockdev.c | 24 ++++++++++++------------ > > include/block/block.h | 8 ++++++++ > > 2 files changed, 20 insertions(+), 12 deletions(-) > > Any reason for not making it part of the BLOCK_OPT_* macros in block_int.h?
Those aren't bdrv_open(), but bdrv_create() options, which should be abundantly clear from the fact that they start in BLOCK instead of BDRV. *cough* I guess I can move the new options to block_int.h indeed, but when it comes to renaming to make the difference clearer, I think I'd prefer renaming the create options. Any opinions on that? Kevin