qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Igor Mammedov <imammedo@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Phil Dennis-Jordan <phil@philjordan.eu>,
	qemu-devel@nongnu.org, Paolo Bonzini <pbonzini@redhat.com>,
	Richard Henderson <rth@twiddle.net>,
	Eduardo Habkost <ehabkost@redhat.com>
Subject: Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt
Date: Wed, 18 Jan 2017 18:19:18 +0100	[thread overview]
Message-ID: <20170118181918.783cb7bd@nial.brq.redhat.com> (raw)
In-Reply-To: <20170118175335-mutt-send-email-mst@kernel.org>

On Wed, 18 Jan 2017 18:30:59 +0200
"Michael S. Tsirkin" <mst@redhat.com> wrote:

> On Wed, Jan 18, 2017 at 12:45:54PM +0100, Phil Dennis-Jordan wrote:
[...]

> > I suspect more might be involved in enabling ACPI 2.0, and it should probably be an option so as to avoid regressions. I don't know what the best approach would be for this, so comments welcome. Should adding the reset register to the FADT also be configurable?  
> 
> I would say an option to make FADT use rev 3 format would be a good
> idea.
> 
> I'd make it the default if XP survives.
if XP and legacy linux survive,
I'd skip adding option as probably there won't be any users,
in unlikely case such user surfaces we always can add option later
but we can't do it other way around (i.e. take it away).

  reply	other threads:[~2017-01-18 17:19 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-18 11:45 [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt Phil Dennis-Jordan
2017-01-18 16:30 ` Michael S. Tsirkin
2017-01-18 17:19   ` Igor Mammedov [this message]
2017-01-31 14:31     ` Phil Dennis-Jordan
2017-01-31 14:58       ` Michael S. Tsirkin
2017-01-31 15:41         ` Igor Mammedov
2017-01-31 16:04           ` Phil Dennis-Jordan
2017-01-31 16:17             ` Igor Mammedov
2017-02-02 16:12             ` Michael S. Tsirkin
2017-01-31 16:28         ` Laszlo Ersek
2017-01-31 18:17           ` Michael S. Tsirkin
2017-01-31 19:08             ` Laszlo Ersek
2017-02-06 16:44               ` Phil Dennis-Jordan
2017-02-07  0:09                 ` Laszlo Ersek
2017-02-01 11:37             ` Igor Mammedov
2017-02-01 12:52               ` Laszlo Ersek
2017-02-01 13:03                 ` Laszlo Ersek
2017-02-01 15:16                   ` Igor Mammedov
2017-02-01 16:03                     ` Laszlo Ersek
2017-02-01 16:17                       ` Michael S. Tsirkin
2017-02-01 16:27                         ` Laszlo Ersek
2017-02-01 14:49               ` Michael S. Tsirkin
2017-02-06 16:30           ` Phil Dennis-Jordan
2017-02-07 19:54             ` Laszlo Ersek
2017-02-07 21:02               ` Phil Dennis-Jordan
2017-02-08  0:52                 ` Laszlo Ersek
2017-01-19 18:09   ` Phil Dennis-Jordan
2017-01-23 11:12     ` Igor Mammedov
2017-01-26 13:43       ` Phil Dennis-Jordan
2017-01-27 13:57         ` Igor Mammedov
2017-01-27 16:05           ` Paolo Bonzini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170118181918.783cb7bd@nial.brq.redhat.com \
    --to=imammedo@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=phil@philjordan.eu \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).