qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Deprecate the "-no-acpi" command line switch


From: Thomas Huth
Subject: Re: [PATCH] Deprecate the "-no-acpi" command line switch
Date: Fri, 24 Feb 2023 09:37:16 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.13.0

On 24/02/2023 09.20, Sunil V L wrote:
On Fri, Feb 24, 2023 at 08:59:41AM +0100, Thomas Huth wrote:
On 24/02/2023 08.56, Sunil V L wrote:
On Fri, Feb 24, 2023 at 08:34:41AM +0100, Thomas Huth wrote:
Similar to "-no-hpet", the "-no-acpi" switch is a legacy command
line option that should be replaced with the "acpi" machine parameter
nowadays.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
   docs/about/deprecated.rst | 6 ++++++
   softmmu/vl.c              | 1 +
   2 files changed, 7 insertions(+)

diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index ee95bcb1a6..15084f7bea 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -99,6 +99,12 @@ form is preferred.
   The HPET setting has been turned into a machine property.
   Use ``-machine hpet=off`` instead.
+``-no-acpi`` (since 8.0)
+''''''''''''''''''''''''
+
+The ``-no-acpi`` setting has been turned into a machine property.
+Use ``-machine acpi=off`` instead.
+
   ``-accel hax`` (since 8.0)
   ''''''''''''''''''''''''''
diff --git a/softmmu/vl.c b/softmmu/vl.c
index 459588aa7d..07d5215325 100644
--- a/softmmu/vl.c
+++ b/softmmu/vl.c
@@ -3271,6 +3271,7 @@ void qemu_init(int argc, char **argv)
                   vnc_parse(optarg);
                   break;
               case QEMU_OPTION_no_acpi:
+                warn_report("-no-hpet is deprecated, use '-machine acpi=off' 
instead");

"no-acpi" is deprecated?

That's at least my plan. The patch is under discussion, but at least in my
opinion, this option should go away in the long run, yes.

Sorry for not being clear. It is mentioned as -no-hpet is deprecated. It
should be -no-acpi.

D'oh, stupid copy-n-paste bug! Thanks, I'll send a v2 :-)

 Thomas




reply via email to

[Prev in Thread] Current Thread [Next in Thread]