From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44007) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wiecn-000500-JY for qemu-devel@nongnu.org; Fri, 09 May 2014 02:49:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Wiech-0002nK-Am for qemu-devel@nongnu.org; Fri, 09 May 2014 02:49:41 -0400 Received: from mx1.redhat.com ([209.132.183.28]:7441) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wiech-0002n6-2y for qemu-devel@nongnu.org; Fri, 09 May 2014 02:49:35 -0400 Message-ID: <1399618165.5213.42.camel@nilsson.home.kraxel.org> From: Gerd Hoffmann Date: Fri, 09 May 2014 08:49:25 +0200 In-Reply-To: <33183CC9F5247A488A2544077AF19020815E738B@SZXEMA503-MBS.china.huawei.com> References: <33183CC9F5247A488A2544077AF19020815E738B@SZXEMA503-MBS.china.huawei.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [Bug] cirrus_vga: qemu abort at booting when configure vgamem_mb <= 2 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Gonglei (Arei)" Cc: "qemu-devel@nongnu.org" On Fr, 2014-05-09 at 03:47 +0000, Gonglei (Arei) wrote: > Hi, Gerd > > The issue consequentially occur, I have tested various qemu versions, > including the current qemu.git. Missing sanity check. It's not valid. We mimic existing hardware here, and the cirrus card emulated has 4 MB video memory. So ideally we should just use that and be done with it. Problem is qemu has historically used 8 or 16 mb vga memory for cirrus, and simply changing it will break live migration. So cirrus should accept 4 MB (correct value), 8 MB and 16 MB (for backward compatibility) and reject everything else. Patches are welcome ;) If you want reduce the qemu memory footprint use stdvga instead which should handle memory sized from 1 MB to 256 MB just fine. cheers, Gerd