From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:55007) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rwtek-0005kR-Pl for qemu-devel@nongnu.org; Mon, 13 Feb 2012 06:01:20 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Rwtej-0007go-KM for qemu-devel@nongnu.org; Mon, 13 Feb 2012 06:01:14 -0500 Received: from mx1.redhat.com ([209.132.183.28]:6185) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rwtej-0007gW-CN for qemu-devel@nongnu.org; Mon, 13 Feb 2012 06:01:13 -0500 Date: Mon, 13 Feb 2012 13:01:07 +0200 From: Gleb Natapov Message-ID: <20120213110107.GX18866@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Subject: [Qemu-devel] [PATCH] Running vgabios during resume from S3 on QEMU by default List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: aliguori@us.ibm.com, seabios@seabios.org, kraxel@redhat.com, kevin@koconnor.net Run vgabios during resume from S3 by default on QEMU. QEMU still able to modify SeaBIOS behavior if it wishes so by providing etc/s3-resume-vga-init file. With QEMU emulated vga cards this behaviour is desirable otherwise console becomes unusable with Linux guests after resume. Since we control vgabios source we can be sure that running it on resume from S3 is safe. Signed-off-by: Gleb Natapov --- Older versions of SeaBIOS had a runtime configure option (disabled by default) to enable this behaviour and RHEL always enabled it, so this code path is well tested. But QEMU upstream always compiled SeaBIOS with default options and hence this patch will modify upstream behaviour. Are there any objections to this change from QEMU side? diff --git a/src/optionroms.c b/src/optionroms.c index 27cfffd..06db1c1 100644 --- a/src/optionroms.c +++ b/src/optionroms.c @@ -423,7 +423,7 @@ vga_setup(void) // Load some config settings that impact VGA. EnforceChecksum = romfile_loadint("etc/optionroms-checksum", 1); - S3ResumeVgaInit = romfile_loadint("etc/s3-resume-vga-init", 0); + S3ResumeVgaInit = romfile_loadint("etc/s3-resume-vga-init", !CONFIG_COREBOOT); ScreenAndDebug = romfile_loadint("etc/screen-and-debug", 1); if (CONFIG_OPTIONROMS_DEPLOYED) { -- Gleb.