diff mbox series

[4/4] hw/s390x: Deprecate the QMP @dump-skeys command

Message ID 20240530074544.25444-5-philmd@linaro.org
State New
Headers show
Series [1/4] hw/s390x: Introduce the @dump-s390-skeys QMP command | expand

Commit Message

Philippe Mathieu-Daudé May 30, 2024, 7:45 a.m. UTC
Prefer @dump-s390-skeys which is target agnostic.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 docs/about/deprecated.rst | 5 +++++
 qapi/misc-target.json     | 5 +++++
 2 files changed, 10 insertions(+)

Comments

Thomas Huth May 31, 2024, 4:49 a.m. UTC | #1
On 30/05/2024 09.45, Philippe Mathieu-Daudé wrote:
> Prefer @dump-s390-skeys which is target agnostic.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   docs/about/deprecated.rst | 5 +++++
>   qapi/misc-target.json     | 5 +++++
>   2 files changed, 10 insertions(+)
> 
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 40585ca7d5..3cb43085ba 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -148,6 +148,11 @@ accepted incorrect commands will return an error. Users should make sure that
>   all arguments passed to ``device_add`` are consistent with the documented
>   property types.
>   
> +``dump-skeys`` (since 9.1)
> +''''''''''''''''''''''''''
> +
> +Use the more generic ``dump-s390-skeys`` command.

FWIW, that's "more specific", not "more generic".

But as I said in my reply to the cover letter, we should maybe consider a 
more generic command instead, indeed.

  Thomas
diff mbox series

Patch

diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index 40585ca7d5..3cb43085ba 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -148,6 +148,11 @@  accepted incorrect commands will return an error. Users should make sure that
 all arguments passed to ``device_add`` are consistent with the documented
 property types.
 
+``dump-skeys`` (since 9.1)
+''''''''''''''''''''''''''
+
+Use the more generic ``dump-s390-skeys`` command.
+
 QEMU Machine Protocol (QMP) events
 ----------------------------------
 
diff --git a/qapi/misc-target.json b/qapi/misc-target.json
index 4e0a6492a9..e5109b1265 100644
--- a/qapi/misc-target.json
+++ b/qapi/misc-target.json
@@ -237,6 +237,10 @@ 
 #
 # @filename: the path to the file to dump to
 #
+# Features:
+#
+# @deprecated: This command is deprecated.  Use @dump-s390-skeys instead.
+#
 # Since: 2.5
 #
 # Example:
@@ -247,6 +251,7 @@ 
 ##
 { 'command': 'dump-skeys',
   'data': { 'filename': 'str' },
+  'features': ['deprecated'],
   'if': 'TARGET_S390X' }
 
 ##