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 19:46:31 +0300 Message-ID: <48E257E7.8000600@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> <48E22A32.7060007@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; 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]:49178 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753542AbYI3Qqc (ORCPT ); Tue, 30 Sep 2008 12:46:32 -0400 In-Reply-To: <48E22A32.7060007@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Avi Kivity wrote: > 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. > It works, so I pushed it out. Alex, can you rebase your bios patches on top of current HEAD? -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.