On 08/06/2018 07:37 AM, Kevin Wolf wrote:
blockdev-add fails if an invalid node name is given, so we should
document what a valid node name even is.

Reported-by: Cong Li <c...@redhat.com>
Signed-off-by: Kevin Wolf <kw...@redhat.com>
---
  qapi/block-core.json | 3 +++
  1 file changed, 3 insertions(+)

Reviewed-by: Eric Blake <ebl...@redhat.com>


diff --git a/qapi/block-core.json b/qapi/block-core.json
index 62a92fa4f4..fe565ebfaf 100644
--- a/qapi/block-core.json
+++ b/qapi/block-core.json
@@ -3563,6 +3563,9 @@
  # @driver:        block driver name
  # @node-name:     the node name of the new node (Since 2.0).
  #                 This option is required on the top level of blockdev-add.
+#                 Valid node names start with an alphabetic character and may
+#                 contain only alphanumeric characters, '-', '.' and '_'. Their
+#                 maximum length is 31 characters.
  # @discard:       discard-related options (default: ignore)
  # @cache:         cache-related options
  # @read-only:     whether the block device should be read-only (default: 
false).


--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Reply via email to