[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 02/31] qapi/qom: Drop deprecated 'props' from object-add
From: |
Kevin Wolf |
Subject: |
[PATCH v2 02/31] qapi/qom: Drop deprecated 'props' from object-add |
Date: |
Wed, 24 Feb 2021 14:52:26 +0100 |
The option has been deprecated in QEMU 5.0, remove it.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
---
qapi/qom.json | 6 +-----
docs/system/deprecated.rst | 5 -----
docs/system/removed-features.rst | 5 +++++
qom/qom-qmp-cmds.c | 21 ---------------------
4 files changed, 6 insertions(+), 31 deletions(-)
diff --git a/qapi/qom.json b/qapi/qom.json
index 0b0b92944b..96c91c1faf 100644
--- a/qapi/qom.json
+++ b/qapi/qom.json
@@ -211,10 +211,6 @@
#
# @id: the name of the new object
#
-# @props: a dictionary of properties to be passed to the backend. Deprecated
-# since 5.0, specify the properties on the top level instead. It is an
-# error to specify the same option both on the top level and in @props.
-#
# Additional arguments depend on qom-type and are passed to the backend
# unchanged.
#
@@ -232,7 +228,7 @@
#
##
{ 'command': 'object-add',
- 'data': {'qom-type': 'str', 'id': 'str', '*props': 'any'},
+ 'data': {'qom-type': 'str', 'id': 'str'},
'gen': false } # so we can get the additional arguments
##
diff --git a/docs/system/deprecated.rst b/docs/system/deprecated.rst
index 2fcac7861e..00b694e053 100644
--- a/docs/system/deprecated.rst
+++ b/docs/system/deprecated.rst
@@ -199,11 +199,6 @@ Use ``migrate-set-parameters`` and
``query-migrate-parameters`` instead.
Use arguments ``base-node`` and ``top-node`` instead.
-``object-add`` option ``props`` (since 5.0)
-'''''''''''''''''''''''''''''''''''''''''''
-
-Specify the properties for the object as top-level arguments instead.
-
``query-named-block-nodes`` and ``query-block`` result dirty-bitmaps[i].status
(since 4.0)
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
diff --git a/docs/system/removed-features.rst b/docs/system/removed-features.rst
index c8481cafbd..95f3fb2912 100644
--- a/docs/system/removed-features.rst
+++ b/docs/system/removed-features.rst
@@ -58,6 +58,11 @@ documentation of ``query-hotpluggable-cpus`` for additional
details.
Use ``blockdev-change-medium`` or ``change-vnc-password`` instead.
+``object-add`` option ``props`` (removed in 6.0)
+''''''''''''''''''''''''''''''''''''''''''''''''
+
+Specify the properties for the object as top-level arguments instead.
+
Human Monitor Protocol (HMP) commands
-------------------------------------
diff --git a/qom/qom-qmp-cmds.c b/qom/qom-qmp-cmds.c
index b40ac39f30..19fd5e117f 100644
--- a/qom/qom-qmp-cmds.c
+++ b/qom/qom-qmp-cmds.c
@@ -225,27 +225,6 @@ ObjectPropertyInfoList *qmp_qom_list_properties(const char
*typename,
void qmp_object_add(QDict *qdict, QObject **ret_data, Error **errp)
{
- QObject *props;
- QDict *pdict;
-
- props = qdict_get(qdict, "props");
- if (props) {
- pdict = qobject_to(QDict, props);
- if (!pdict) {
- error_setg(errp, QERR_INVALID_PARAMETER_TYPE, "props", "dict");
- return;
- }
- qobject_ref(pdict);
- qdict_del(qdict, "props");
- qdict_join(qdict, pdict, false);
- if (qdict_size(pdict) != 0) {
- error_setg(errp, "Option in 'props' conflicts with top level");
- qobject_unref(pdict);
- return;
- }
- qobject_unref(pdict);
- }
-
user_creatable_add_dict(qdict, false, errp);
}
--
2.29.2
- [PATCH v2 12/31] qapi/qom: Add ObjectOptions for can-*, (continued)
- [PATCH v2 12/31] qapi/qom: Add ObjectOptions for can-*, Kevin Wolf, 2021/02/24
- [PATCH v2 07/31] qapi/qom: Add ObjectOptions for memory-backend-*, Kevin Wolf, 2021/02/24
- [PATCH v2 19/31] qapi/qom: QAPIfy object-add, Kevin Wolf, 2021/02/24
- [PATCH v2 14/31] qapi/qom: Add ObjectOptions for filter-*, Kevin Wolf, 2021/02/24
- [PATCH v2 01/31] tests: Drop 'props' from object-add calls, Kevin Wolf, 2021/02/24
- [PATCH v2 02/31] qapi/qom: Drop deprecated 'props' from object-add,
Kevin Wolf <=
- [PATCH v2 03/31] qapi/qom: Add ObjectOptions for iothread, Kevin Wolf, 2021/02/24
- [PATCH v2 16/31] qapi/qom: Add ObjectOptions for confidential-guest-support, Kevin Wolf, 2021/02/24
- [PATCH v2 10/31] qapi/qom: Add ObjectOptions for secret*, deprecate 'loaded', Kevin Wolf, 2021/02/24
- [PATCH v2 05/31] qapi/qom: Add ObjectOptions for cryptodev-*, Kevin Wolf, 2021/02/24