From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: Re: [PATCH 0/5] bios: >4G updates Date: Tue, 30 Sep 2008 16:31:30 +0300 Message-ID: <48E22A32.7060007@redhat.com> References: <1222188118.8138.101.camel@2710p.home> <48DA1F6A.40105@redhat.com> <1222273063.16470.157.camel@lappy> <1222358913.8138.245.camel@2710p.home> <48DFA094.6040208@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm-devel , gleb@redhat.com To: Alex Williamson Return-path: Received: from mx2.redhat.com ([66.187.237.31]:39105 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752099AbYI3Nb2 (ORCPT ); Tue, 30 Sep 2008 09:31:28 -0400 In-Reply-To: <48DFA094.6040208@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Avi Kivity wrote: > > I tried to merge it, but it the new bios won't boot. I traced this to > ata_reset: it toggles the soft reset bit in the command register and > expects to see the busy bit set, but the ide device model returns zero > status if the selected device has no drive. > > No idea if the device model or the bios is wrong. > > If anyone (hi Gleb) wants to take a look, it's in the branch > bios-merge on git.kernel.org. > > btw, commenting this out leads to boot failure as well, but that may > be related. > Okay, I think I have it -- the bios code was using .org to set certain entry points to well known addresses, and one code section overflowed due to enabling the cache, which is a kvm addition. Moving that bit out-of-line allows the bios to boot. -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.