From mboxrd@z Thu Jan  1 00:00:00 1970
From: Markus Armbruster <armbru@redhat.com>
Subject: Re: [Qemu-devel] [PATCH uq/master] kvmvapic: Disable if there is insufficient memory
Date: Tue, 14 Aug 2012 13:40:58 +0200
Message-ID: <87txw5itv9.fsf@blackfin.pond.sub.org>
References: <502A32BB.2000607@siemens.com>
Mime-Version: 1.0
Content-Type: text/plain
Cc: Avi Kivity <avi@redhat.com>, Marcelo Tosatti <mtosatti@redhat.com>,
	qemu-devel <qemu-devel@nongnu.org>, kvm <kvm@vger.kernel.org>
To: Jan Kiszka <jan.kiszka@siemens.com>
Return-path: <kvm-owner@vger.kernel.org>
Received: from mx1.redhat.com ([209.132.183.28]:32468 "EHLO mx1.redhat.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1756066Ab2HNLlD (ORCPT <rfc822;kvm@vger.kernel.org>);
	Tue, 14 Aug 2012 07:41:03 -0400
In-Reply-To: <502A32BB.2000607@siemens.com> (Jan Kiszka's message of "Tue, 14
	Aug 2012 13:12:59 +0200")
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

Jan Kiszka <jan.kiszka@siemens.com> writes:

> We need at least 1M of RAM to map the option ROM. Otherwise, we will
> corrupt host memory or even crash.

Let's put a reproducer in the commit message, if it's not too much
trouble.  Here's mine:

    $ qemu-system-x86_64 -nodefaults --enable-kvm -vnc :0 -m 640k
    Segmentation fault (core dumped)

> Reported-by: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>

Tested-by: Markus Armbruster <armbru@redhat.com>