[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 06/18] qapi/qom: Add ObjectOptions for rng-*, deprecate 'opened'
From: |
Kevin Wolf |
Subject: |
[PATCH 06/18] qapi/qom: Add ObjectOptions for rng-*, deprecate 'opened' |
Date: |
Mon, 30 Nov 2020 13:25:26 +0100 |
This adds a QAPI schema for the properties of the rng-* objects.
The 'opened' property doesn't seem to make sense as an external
interface: It is automatically set to true in ucc->complete, and
explicitly setting it to true earlier just means that trying to set
additional options will result in an error. After the property has once
been set to true (i.e. when the object construction has completed), it
can never be reset to false. In other words, the 'opened' property is
useless. Mark it as deprecated in the schema from the start.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
---
qapi/qom.json | 56 ++++++++++++++++++++++++++++++++++++--
docs/system/deprecated.rst | 9 ++++++
2 files changed, 63 insertions(+), 2 deletions(-)
diff --git a/qapi/qom.json b/qapi/qom.json
index f8a1da43ad..f3d1a55cb8 100644
--- a/qapi/qom.json
+++ b/qapi/qom.json
@@ -368,6 +368,52 @@
'*hugetlbsize': 'size',
'*seal': 'bool' } }
+##
+# @RngProperties:
+#
+# Properties for objects of classes derived from rng.
+#
+# @opened: if true, the device is opened immediately when applying this option
+# and will probably fail when processing the next option. Don't use;
+# only provided for compatibility. (default: false)
+#
+# Features:
+# @deprecated: Member @opened is deprecated. Setting true doesn't make sense,
+# and false is already the default.
+#
+# Since: 6.0
+##
+{ 'struct': 'RngProperties',
+ 'data': { '*opened': { 'type': 'bool', 'features': ['deprecated'] } } }
+
+##
+# @RngEgdProperties:
+#
+# Properties for rng-egd objects.
+#
+# @chardev: the name of a character device backend that provides the connection
+# to the RNG daemon
+#
+# Since: 6.0
+##
+{ 'struct': 'RngEgdProperties',
+ 'base': 'RngProperties',
+ 'data': { 'chardev': 'str' } }
+
+##
+# @RngRandomProperties:
+#
+# Properties for rng-random objects.
+#
+# @filename: the filename of the device on the host to obtain entropy from
+# (default: "/dev/urandom")
+#
+# Since: 6.0
+##
+{ 'struct': 'RngRandomProperties',
+ 'base': 'RngProperties',
+ 'data': { '*filename': 'str' } }
+
##
# @ObjectType:
#
@@ -386,7 +432,10 @@
'iothread',
'memory-backend-file',
'memory-backend-memfd',
- 'memory-backend-ram'
+ 'memory-backend-ram',
+ 'rng-builtin',
+ 'rng-egd',
+ 'rng-random'
] }
##
@@ -416,7 +465,10 @@
'iothread': 'IothreadProperties',
'memory-backend-file': 'MemoryBackendFileProperties',
'memory-backend-memfd': 'MemoryBackendMemfdProperties',
- 'memory-backend-ram': 'MemoryBackendProperties'
+ 'memory-backend-ram': 'MemoryBackendProperties',
+ 'rng-builtin': 'RngProperties',
+ 'rng-egd': 'RngEgdProperties',
+ 'rng-random': 'RngRandomProperties'
} }
##
diff --git a/docs/system/deprecated.rst b/docs/system/deprecated.rst
index 565389697e..30a694a39a 100644
--- a/docs/system/deprecated.rst
+++ b/docs/system/deprecated.rst
@@ -146,6 +146,15 @@ Drives with interface types other than ``if=none`` are for
onboard
devices. It is possible to use drives the board doesn't pick up with
-device. This usage is now deprecated. Use ``if=none`` instead.
+``opened```property of ``rng-*```objects (since 6.0.0)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''
+
+The only effect of specifying ``opened=on`` in the command line or QMP
+``object-add`` is that the device is opened immediately, possibly before all
+other options have been processed. This will either have no effect (if
+``opened`` was the last option) or cause errors. The property is therefore
+useless and should not be specified.
+
QEMU Machine Protocol (QMP) commands
------------------------------------
--
2.28.0
- [PATCH 00/18] qapi/qom: QAPIfy object-add, Kevin Wolf, 2020/11/30
- [PATCH 01/18] qapi/qom: Add ObjectOptions for iothread, Kevin Wolf, 2020/11/30
- [PATCH 02/18] qapi/qom: Add ObjectOptions for authz-*, Kevin Wolf, 2020/11/30
- [PATCH 03/18] qapi/qom: Add ObjectOptions for cryptodev-*, Kevin Wolf, 2020/11/30
- [PATCH 06/18] qapi/qom: Add ObjectOptions for rng-*, deprecate 'opened',
Kevin Wolf <=
- [PATCH 04/18] qapi/qom: Add ObjectOptions for dbus-vmstate, Kevin Wolf, 2020/11/30
- [PATCH 05/18] qapi/qom: Add ObjectOptions for memory-backend-*, Kevin Wolf, 2020/11/30
- [PATCH 08/18] qapi/qom: Add ObjectOptions for secret*, deprecate 'loaded', Kevin Wolf, 2020/11/30
- [PATCH 07/18] qapi/qom: Add ObjectOptions for throttle-group, Kevin Wolf, 2020/11/30
- [PATCH 09/18] qapi/qom: Add ObjectOptions for tls-*, deprecate 'loaded', Kevin Wolf, 2020/11/30
- [PATCH 10/18] qapi/qom: Add ObjectOptions for can-*, Kevin Wolf, 2020/11/30
- [PATCH 11/18] qapi/qom: Add ObjectOptions for colo-compare, Kevin Wolf, 2020/11/30
- [PATCH 12/18] qapi/qom: Add ObjectOptions for filter-*, Kevin Wolf, 2020/11/30
- [PATCH 13/18] qapi/qom: Add ObjectOptions for pr-manager-helper, Kevin Wolf, 2020/11/30
- [PATCH 14/18] qapi/qom: Add ObjectOptions for sev-guest, Kevin Wolf, 2020/11/30