From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1MVVE7-0008F5-Ch
	for qemu-devel@nongnu.org; Mon, 27 Jul 2009 14:47:11 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1MVVE2-0008EJ-LK
	for qemu-devel@nongnu.org; Mon, 27 Jul 2009 14:47:10 -0400
Received: from [199.232.76.173] (port=46377 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1MVVE2-0008EG-C8
	for qemu-devel@nongnu.org; Mon, 27 Jul 2009 14:47:06 -0400
Received: from e36.co.us.ibm.com ([32.97.110.154]:54858)
	by monty-python.gnu.org with esmtps
	(TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60)
	(envelope-from <eak@us.ibm.com>) id 1MVVE1-00045k-T9
	for qemu-devel@nongnu.org; Mon, 27 Jul 2009 14:47:06 -0400
Received: from d03relay04.boulder.ibm.com (d03relay04.boulder.ibm.com
	[9.17.195.106])
	by e36.co.us.ibm.com (8.14.3/8.13.1) with ESMTP id n6RIjMvZ012911
	for <qemu-devel@nongnu.org>; Mon, 27 Jul 2009 12:45:22 -0600
Received: from d03av01.boulder.ibm.com (d03av01.boulder.ibm.com [9.17.195.167])
	by d03relay04.boulder.ibm.com (8.13.8/8.13.8/NCO v9.2) with ESMTP id
	n6RIl0vF213282
	for <qemu-devel@nongnu.org>; Mon, 27 Jul 2009 12:47:00 -0600
Received: from d03av01.boulder.ibm.com (loopback [127.0.0.1])
	by d03av01.boulder.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id
	n6RIl0B6002646
	for <qemu-devel@nongnu.org>; Mon, 27 Jul 2009 12:47:00 -0600
Message-ID: <4A6DF629.1030802@us.ibm.com>
Date: Mon, 27 Jul 2009 14:47:05 -0400
From: Beth Kon <eak@us.ibm.com>
MIME-Version: 1.0
References: <4A69B200.3030500@sgi.com>
	<20090725015649.GA15207@morn.localdomain>
	<4A6D562C.3010103@sgi.com>
In-Reply-To: <4A6D562C.3010103@sgi.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: [Qemu-devel] Re: [PATCH] Seabios irq override support.
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Jes Sorensen <jes@sgi.com>
Cc: Kevin O'Connor <kevin@koconnor.net>, qemu-devel <qemu-devel@nongnu.org>

Jes Sorensen wrote:
> On 07/25/2009 03:56 AM, Kevin O'Connor wrote:
>> On Fri, Jul 24, 2009 at 03:07:12PM +0200, Jes Sorensen wrote:
>>> With this patch applied, I can boot KVM using the latest upstream QEMU
>>> and Seabios.
>>
>> Thanks Jes.
>>
>> The bochs commit shows this wrapped in "#ifdef BX_QEMU".  Will this
>> patch break bochs, or were the ifdefs just bochs paranoia?
>
> Hi Kevin,
>
> I don't actually know for sure. I didn't add the #ifdefs since I got the
> impression that Seabios doesn't have them in many places. The original
> code was designed by Beth Kon, so maybe she can comment on whether we
> should add them to Seabios.
>
> Beth?
>
> Thanks,
> Jes
Hi Jes. These bios changes rely on matching userspace/kernel changes 
that translate irq0->inti2. So assuming seabios is used for systems 
other than kvm and qemu, this code would need to be conditionally removed.

But I'm not clear on what you're doing with this patch. You didn't 
include the irq0override flag that is needed by kvm because there are 
circumstances under which kvm turns off irq0override (i.e., old kernels 
that don't support irq routing). So this patch is fine for qemu, since 
it is permanently enabled there, but it is not a permanent solution for kvm.