From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anthony Liguori Subject: Re: [PATCH 0/8] Support for S3 ACPI state (suspend to memory) Date: Thu, 23 Oct 2008 13:18:45 -0500 Message-ID: <4900C005.3010709@codemonkey.ws> References: <20081023140459.10303.39562.stgit@gleb-debian.qumranet.com.qumranet.com> <4900A28B.6050509@codemonkey.ws> <20081023173748.GA13097@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: Gleb Natapov Return-path: Received: from yw-out-2324.google.com ([74.125.46.29]:27535 "EHLO yw-out-2324.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754319AbYJWSSu (ORCPT ); Thu, 23 Oct 2008 14:18:50 -0400 Received: by yw-out-2324.google.com with SMTP id 9so143160ywe.1 for ; Thu, 23 Oct 2008 11:18:49 -0700 (PDT) In-Reply-To: <20081023173748.GA13097@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Gleb Natapov wrote: > On Thu, Oct 23, 2008 at 11:12:59AM -0500, Anthony Liguori wrote: > >> Hi Gleb, >> >> Gleb Natapov wrote: >> >>> Hello, >>> >>> This patch series adds S3 (suspend to RAM) ACPI state. Most changes >>> are in the BIOS code and concern themselves with preventing BIOS from >>> using memory available to OS. >>> >>> >> I thought you initially sent this to qemu-devel. Any reason not to push >> this through QEMU instead of in KVM? >> >> > 90% of this patch touches BIOS code. It is easy to push QEMU bits > upstream, but BIOS part may be problematic. KVM BIOS is already > different from upstream. > I'm willing to expand the bios.diff that we have in QEMU as long as someone is submitting the patches to bochs. In this case, I don't think these changes are at all controversial. Regards, Anthony Liguori > -- > Gleb. >