[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/6] qemu-nbd: add compression flag support
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [PATCH 4/6] qemu-nbd: add compression flag support |
Date: |
Wed, 2 Oct 2019 10:29:50 +0000 |
01.10.2019 23:47, Eric Blake wrote:
> On 10/1/19 2:27 PM, Andrey Shinkevich wrote:
>> Added possibility to write compressed data by using the
>> blk_write_compressed. This action has the limitations of the format
>> driver. For example we can't write compressed data over other.
>>
>
>> +++ b/blockdev-nbd.c
>> @@ -191,7 +191,7 @@ void qmp_nbd_server_add(const char *device, bool
>> has_name, const char *name,
>> }
>> exp = nbd_export_new(bs, 0, len, name, NULL, bitmap, !writable,
>> !writable,
>> - NULL, false, on_eject_blk, errp);
>> + 0, NULL, false, on_eject_blk, errp);
>
> This is a lot of parameters. Should we be combining some of them into a
> struct, or even at least the booleans into a flags parameter?
>
>
>> +++ b/include/block/nbd.h
>> @@ -25,6 +25,10 @@
>> #include "crypto/tlscreds.h"
>> #include "qapi/error.h"
>> +enum {
>> + NBD_INTERNAL_FLAG_COMPRESS = 1 << 1, /* Use write compressed */
>> +};
>
> What happened to flag 1 << 0? What other flags do you anticipate adding?
Hmm, any way, creating flags parameter for only one flag seems useless.
I think, I'd prefer to cover all nbd_export_new parameters to a structure
with boolean fields, to avoid extra &/| arithmetic.
nbd_export_new is called from qmp_nbd_server_add and qemu-nbd main(), and
both places seems to benefit, if they will fill structure instead of local
variables.
--
Best regards,
Vladimir
- [PATCH 0/6] qcow2: advanced compression options, Andrey Shinkevich, 2019/10/01
- [PATCH 2/6] tests/qemu-iotests: add case of writing compressed data to multiple clusters., Andrey Shinkevich, 2019/10/01
- [PATCH 6/6] tests/qemu-iotests: add case for block-stream compress, Andrey Shinkevich, 2019/10/01
- [PATCH 1/6] qcow2: multiple clusters write compressed, Andrey Shinkevich, 2019/10/01
- [PATCH 4/6] qemu-nbd: add compression flag support, Andrey Shinkevich, 2019/10/01
- [PATCH 5/6] block-stream: add compress option, Andrey Shinkevich, 2019/10/01
- [PATCH 3/6] block: support compressed write for copy-on-read, Andrey Shinkevich, 2019/10/01
- Re: [PATCH 0/6] qcow2: advanced compression options, no-reply, 2019/10/01
- Re: [PATCH 0/6] qcow2: advanced compression options, no-reply, 2019/10/01