kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Avi Kivity <avi@redhat.com>
Cc: Gleb Natapov <gleb@redhat.com>, kvm@vger.kernel.org
Subject: Re: [PATCH 0/8] Support for S3 ACPI state (suspend to memory)
Date: Thu, 23 Oct 2008 13:17:31 -0500	[thread overview]
Message-ID: <4900BFBB.6070300@codemonkey.ws> (raw)
In-Reply-To: <4900A879.3030505@redhat.com>

Avi Kivity wrote:
> Anthony Liguori wrote:
>>
>> I thought you initially sent this to qemu-devel.  Any reason not to 
>> push this through QEMU instead of in KVM?
>>
>
> Since it requires bios changes, it is basically unpushable to qemu.

Why?  As long as Gleb is willing to push those changes to Bochs, I'll 
stick them in the bios.diff for now.  The hard part is the dsdt changes 
but if the build is changed to run the dsdt through cpp first, we can 
just #ifdef BX_QEMU the qemu specific stuff.

>   It may also depend on existing kvm bios and acpi changes.

How much dependency is there here?

>
> If it is possible to do this via qemu, of course I much prefer that 
> route.

I looked through the patches.  They're quite reasonable so I'd like to 
push them into QEMU if it's possible.

Regards,

Anthony Liguori


  reply	other threads:[~2008-10-23 18:17 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-23 14:05 [PATCH 0/8] Support for S3 ACPI state (suspend to memory) Gleb Natapov
2008-10-23 14:05 ` [PATCH 1/8] Move PIC initialization out of line to save space in post code area Gleb Natapov
2008-10-23 14:05 ` [PATCH 2/8] Handle suspend in qemu Gleb Natapov
2008-10-23 14:05 ` [PATCH 3/8] Add S3 state to DSDT. Handle resume event in the BIOS Gleb Natapov
2008-10-26 14:58   ` Avi Kivity
2008-10-23 14:05 ` [PATCH 4/8] Disable init of SMM Gleb Natapov
2008-10-23 14:05 ` [PATCH 5/8] Execute rombios32 code from rom address 0xe0000 Gleb Natapov
2008-10-23 14:05 ` [PATCH 6/8] Don't use unreserved memory in BIOS Gleb Natapov
2008-10-23 14:05 ` [PATCH 7/8] Add DPMS support to cirrus vga bios Gleb Natapov
2008-10-23 14:05 ` [PATCH 8/8] Don't power down vga card on entering S3 state Gleb Natapov
2008-10-23 15:45   ` Anthony Liguori
2008-10-23 16:12 ` [PATCH 0/8] Support for S3 ACPI state (suspend to memory) Anthony Liguori
2008-10-23 16:38   ` Avi Kivity
2008-10-23 18:17     ` Anthony Liguori [this message]
2008-10-23 19:51       ` Avi Kivity
     [not found]   ` <20081023173748.GA13097@redhat.com>
2008-10-23 18:18     ` Anthony Liguori
     [not found]       ` <20081023182153.GA13492@redhat.com>
2008-10-23 18:41         ` Anthony Liguori
2008-10-23 22:04 ` Sebastian Herbszt

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=4900BFBB.6070300@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=avi@redhat.com \
    --cc=gleb@redhat.com \
    --cc=kvm@vger.kernel.org \
    /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).