mbox series

[v2,0/3] Update documentation and help related to the preallocation parameter

Message ID 20190524075848.23781-1-sgarzare@redhat.com (mailing list archive)
Headers show
Series Update documentation and help related to the preallocation parameter | expand

Message

Stefano Garzarella May 24, 2019, 7:58 a.m. UTC
Following Markus' advice, I updated the documentation of preallocation
parameter in qapi/block-core.json adding default and allowed values (patch 1).
I also updated the help related to BLOCK_OPT_PREALLOC in the QemuOptsList of
file-posix (patch 2) and gluster (patch 3).

v2:
 - Patch 1: removed references to "file-win32" and "file-posix" in the
            block comment [Markus]
 - Patch 3: reworded commit message [Markus]

Stefano Garzarella (3):
  qapi/block-core: update documentation of preallocation parameter
  block/file-posix: update .help of BLOCK_OPT_PREALLOC option
  block/gluster: update .help of BLOCK_OPT_PREALLOC option

 qapi/block-core.json | 20 ++++++++++++++------
 block/file-posix.c   |  6 +++++-
 block/gluster.c      |  9 ++++++++-
 3 files changed, 27 insertions(+), 8 deletions(-)

Comments

Markus Armbruster May 28, 2019, 6:15 a.m. UTC | #1
Kevin, Max, this series looks ready to me.  Feel free to ask me to take
it through my tree.
Max Reitz May 29, 2019, 12:49 p.m. UTC | #2
On 28.05.19 08:15, Markus Armbruster wrote:
> Kevin, Max, this series looks ready to me.  Feel free to ask me to take
> it through my tree.

*ask*

:-)
Markus Armbruster May 29, 2019, 2:14 p.m. UTC | #3
Stefano Garzarella <sgarzare@redhat.com> writes:

> Following Markus' advice, I updated the documentation of preallocation
> parameter in qapi/block-core.json adding default and allowed values (patch 1).
> I also updated the help related to BLOCK_OPT_PREALLOC in the QemuOptsList of
> file-posix (patch 2) and gluster (patch 3).

Queued.  Thanks!