Re: [PATCH v5 4/8] multifd: Add multifd-zlib-level parameter

[Markus Armbruster]

Juan Quintela <address@hidden> writes:

> It will indicate which level use for compression.
>
> Signed-off-by: Juan Quintela <address@hidden>

This is slightly confusing (there is no zlib compression), unless you
peek at the next patch (which adds zlib compression).

Three ways to make it less confusing:

* Squash the two commits

* Swap them: first add zlib compression with level hardcoded to 1, then
  make the level configurable.

* Have the first commit explain itself better.  Something like

    multifd: Add multifd-zlib-level parameter

    This parameter specifies zlib compression level.  The next patch
    will put it to use.

For QAPI:
Acked-by: Markus Armbruster <address@hidden>