* [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter
@ 2019-05-24 7:58 Stefano Garzarella
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 1/3] qapi/block-core: update documentation of " Stefano Garzarella
` (4 more replies)
0 siblings, 5 replies; 7+ messages in thread
From: Stefano Garzarella @ 2019-05-24 7:58 UTC (permalink / raw)
To: qemu-devel
Cc: Kevin Wolf, integration, qemu-block, Markus Armbruster, Max Reitz
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(-)
--
2.20.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Qemu-devel] [PATCH v2 1/3] qapi/block-core: update documentation of preallocation parameter
2019-05-24 7:58 [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Stefano Garzarella
@ 2019-05-24 7:58 ` Stefano Garzarella
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 2/3] block/file-posix: update .help of BLOCK_OPT_PREALLOC option Stefano Garzarella
` (3 subsequent siblings)
4 siblings, 0 replies; 7+ messages in thread
From: Stefano Garzarella @ 2019-05-24 7:58 UTC (permalink / raw)
To: qemu-devel
Cc: Kevin Wolf, integration, qemu-block, Markus Armbruster, Max Reitz
Add default and available values in the documentation block of
each block device or protocol that supports the 'preallocation'
parameter during the image creation.
Suggested-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
---
qapi/block-core.json | 20 ++++++++++++++------
1 file changed, 14 insertions(+), 6 deletions(-)
diff --git a/qapi/block-core.json b/qapi/block-core.json
index 7ccbfff9d0..e2f7e2b89f 100644
--- a/qapi/block-core.json
+++ b/qapi/block-core.json
@@ -4103,7 +4103,10 @@
#
# @filename Filename for the new image file
# @size Size of the virtual disk in bytes
-# @preallocation Preallocation mode for the new image (default: off)
+# @preallocation Preallocation mode for the new image (default: off;
+# allowed values: off,
+# falloc (if defined CONFIG_POSIX_FALLOCATE),
+# full (if defined CONFIG_POSIX))
# @nocow Turn off copy-on-write (valid only on btrfs; default: off)
#
# Since: 2.12
@@ -4121,7 +4124,10 @@
#
# @location Where to store the new image file
# @size Size of the virtual disk in bytes
-# @preallocation Preallocation mode for the new image (default: off)
+# @preallocation Preallocation mode for the new image (default: off;
+# allowed values: off,
+# falloc (if defined CONFIG_GLUSTERFS_FALLOCATE),
+# full (if defined CONFIG_GLUSTERFS_ZEROFILL))
#
# Since: 2.12
##
@@ -4225,7 +4231,8 @@
# @backing-fmt Name of the block driver to use for the backing file
# @encrypt Encryption options if the image should be encrypted
# @cluster-size qcow2 cluster size in bytes (default: 65536)
-# @preallocation Preallocation mode for the new image (default: off)
+# @preallocation Preallocation mode for the new image (default: off;
+# allowed values: off, falloc, full, metadata)
# @lazy-refcounts True if refcounts may be updated lazily (default: off)
# @refcount-bits Width of reference counts in bits (default: 16)
#
@@ -4408,7 +4415,8 @@
# @location Where to store the new image file
# @size Size of the virtual disk in bytes
# @backing-file File name of a base image
-# @preallocation Preallocation mode (allowed values: off, full)
+# @preallocation Preallocation mode for the new image (default: off;
+# allowed values: off, full)
# @redundancy Redundancy of the image
# @object-size Object size of the image
#
@@ -4443,8 +4451,8 @@
#
# @file Node to create the image format on
# @size Size of the virtual disk in bytes
-# @preallocation Preallocation mode for the new image (allowed values: off,
-# metadata; default: off)
+# @preallocation Preallocation mode for the new image (default: off;
+# allowed values: off, metadata)
#
# Since: 2.12
##
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Qemu-devel] [PATCH v2 2/3] block/file-posix: update .help of BLOCK_OPT_PREALLOC option
2019-05-24 7:58 [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Stefano Garzarella
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 1/3] qapi/block-core: update documentation of " Stefano Garzarella
@ 2019-05-24 7:58 ` Stefano Garzarella
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 3/3] block/gluster: " Stefano Garzarella
` (2 subsequent siblings)
4 siblings, 0 replies; 7+ messages in thread
From: Stefano Garzarella @ 2019-05-24 7:58 UTC (permalink / raw)
To: qemu-devel
Cc: Kevin Wolf, integration, qemu-block, Markus Armbruster, Max Reitz
Show 'falloc' among the allowed values of 'preallocation'
option, only when it is supported (if defined CONFIG_POSIX_FALLOCATE)
Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
---
block/file-posix.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/block/file-posix.c b/block/file-posix.c
index d018429672..9632e3a87b 100644
--- a/block/file-posix.c
+++ b/block/file-posix.c
@@ -2751,7 +2751,11 @@ static QemuOptsList raw_create_opts = {
{
.name = BLOCK_OPT_PREALLOC,
.type = QEMU_OPT_STRING,
- .help = "Preallocation mode (allowed values: off, falloc, full)"
+ .help = "Preallocation mode (allowed values: off"
+#ifdef CONFIG_POSIX_FALLOCATE
+ ", falloc"
+#endif
+ ", full)"
},
{ /* end of list */ }
}
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Qemu-devel] [PATCH v2 3/3] block/gluster: update .help of BLOCK_OPT_PREALLOC option
2019-05-24 7:58 [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Stefano Garzarella
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 1/3] qapi/block-core: update documentation of " Stefano Garzarella
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 2/3] block/file-posix: update .help of BLOCK_OPT_PREALLOC option Stefano Garzarella
@ 2019-05-24 7:58 ` Stefano Garzarella
2019-05-28 6:15 ` [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Markus Armbruster
2019-05-29 14:14 ` Markus Armbruster
4 siblings, 0 replies; 7+ messages in thread
From: Stefano Garzarella @ 2019-05-24 7:58 UTC (permalink / raw)
To: qemu-devel
Cc: Kevin Wolf, integration, qemu-block, Markus Armbruster, Max Reitz
Add missing 'falloc' among the allowed values of 'preallocation'
option; show it and 'full' only when they are supported.
('falloc' is supported if defined CONFIG_GLUSTERFS_FALLOCATE,
'full' is supported if defined CONFIG_GLUSTERFS_ZEROFILL)
Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
---
block/gluster.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/block/gluster.c b/block/gluster.c
index e664ca4462..682fe49912 100644
--- a/block/gluster.c
+++ b/block/gluster.c
@@ -97,7 +97,14 @@ static QemuOptsList qemu_gluster_create_opts = {
{
.name = BLOCK_OPT_PREALLOC,
.type = QEMU_OPT_STRING,
- .help = "Preallocation mode (allowed values: off, full)"
+ .help = "Preallocation mode (allowed values: off"
+#ifdef CONFIG_GLUSTERFS_FALLOCATE
+ ", falloc"
+#endif
+#ifdef CONFIG_GLUSTERFS_ZEROFILL
+ ", full"
+#endif
+ ")"
},
{
.name = GLUSTER_OPT_DEBUG,
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter
2019-05-24 7:58 [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Stefano Garzarella
` (2 preceding siblings ...)
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 3/3] block/gluster: " Stefano Garzarella
@ 2019-05-28 6:15 ` Markus Armbruster
2019-05-29 12:49 ` Max Reitz
2019-05-29 14:14 ` Markus Armbruster
4 siblings, 1 reply; 7+ messages in thread
From: Markus Armbruster @ 2019-05-28 6:15 UTC (permalink / raw)
To: Kevin Wolf, Max Reitz
Cc: integration, qemu-devel, qemu-block, Stefano Garzarella
Kevin, Max, this series looks ready to me. Feel free to ask me to take
it through my tree.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter
2019-05-28 6:15 ` [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Markus Armbruster
@ 2019-05-29 12:49 ` Max Reitz
0 siblings, 0 replies; 7+ messages in thread
From: Max Reitz @ 2019-05-29 12:49 UTC (permalink / raw)
To: Markus Armbruster, Kevin Wolf
Cc: integration, qemu-devel, qemu-block, Stefano Garzarella
[-- Attachment #1: Type: text/plain, Size: 161 bytes --]
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*
:-)
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter
2019-05-24 7:58 [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Stefano Garzarella
` (3 preceding siblings ...)
2019-05-28 6:15 ` [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Markus Armbruster
@ 2019-05-29 14:14 ` Markus Armbruster
4 siblings, 0 replies; 7+ messages in thread
From: Markus Armbruster @ 2019-05-29 14:14 UTC (permalink / raw)
To: Stefano Garzarella
Cc: Kevin Wolf, integration, qemu-devel, qemu-block, Max Reitz
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!
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-05-29 14:15 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-24 7:58 [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Stefano Garzarella
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 1/3] qapi/block-core: update documentation of " Stefano Garzarella
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 2/3] block/file-posix: update .help of BLOCK_OPT_PREALLOC option Stefano Garzarella
2019-05-24 7:58 ` [Qemu-devel] [PATCH v2 3/3] block/gluster: " Stefano Garzarella
2019-05-28 6:15 ` [Qemu-devel] [PATCH v2 0/3] Update documentation and help related to the preallocation parameter Markus Armbruster
2019-05-29 12:49 ` Max Reitz
2019-05-29 14:14 ` Markus Armbruster
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.