From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Ll6DG-0006sl-Fi for qemu-devel@nongnu.org; Sat, 21 Mar 2009 14:46:30 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Ll6DF-0006sU-0D for qemu-devel@nongnu.org; Sat, 21 Mar 2009 14:46:30 -0400 Received: from [199.232.76.173] (port=45112 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Ll6DE-0006sP-Rv for qemu-devel@nongnu.org; Sat, 21 Mar 2009 14:46:28 -0400 Received: from static-71-162-243-5.phlapa.fios.verizon.net ([71.162.243.5]:40518 helo=grelber.thyrsus.com) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Ll6DE-0004T0-Fn for qemu-devel@nongnu.org; Sat, 21 Mar 2009 14:46:28 -0400 From: Rob Landley Subject: Re: [Qemu-devel] svn 6658 broke powerpc. Date: Sat, 21 Mar 2009 13:44:54 -0500 References: <200903171857.42875.rob@landley.net> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200903211344.55267.rob@landley.net> Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Blue Swirl Cc: The OpenBIOS Mailinglist , qemu-devel@nongnu.org On Saturday 21 March 2009 04:32:30 Blue Swirl wrote: > I accidentally (by using a wrong variable in a more complex patch) > found out a workaround for the bug, now -nographic boot works. But > still the other nodes have the same problem as before and the > workaround removes the "reg" node which is present on real tree. > > Now the escc node doesn't have the address appended: > 1bfac /pci@80000000/mac-io@4/nvram@0 (nvram) > 1c1c0 /pci@80000000/mac-io@4/escc (escc) > 1c2ac /pci@80000000/mac-io@4/escc/ch-a@13020 (serial) > 1c52c /pci@80000000/mac-io@4/escc/ch-b@13000 (serial) > > Maybe nodes under /pci/mac-io should use custom > encode-unit/decode-unit methods instead of PCI ones? My workaround at this end is to just revert openbios-ppc to the earlier version. I still think that bisecting to find the specific commit that broke it would be informative, but your repository's move to /trunk at commit 470 essentially wiped all repository history before that. Your development page doesn't say how to download the old versions, just the ones under "trunk": http://www.openfirmware.info/OpenBIOS Nor does your repository viewer view anything before 470 from the little pull- down menu: http://tracker.coreboot.org/trac/openbios/browser/trunk/openbios-devel?rev=470 http://tracker.coreboot.org/trac/openbios/browser/trunk/openbios-devel?rev=469 I guessed how to get the repository viewer to show me the earlier versions: http://tracker.coreboot.org/trac/openbios/browser/openbios-devel?rev=469 But this doesn't work: $ svn co -r 469 svn://openbios.org/openbios/openbios-devel svn: File not found: revision 480, path '/openbios-devel' Then again, I can always bisect by downloading tarballs from your repository viewer... Eh, why not. For easy cut and pasting, my build reproduction sequence is: chmod +x config/scripts/switch-arch config/scripts/switch-arch cross-ppc PATH=~/firmware/firmware/build/cross-compiler-powerpc/bin:$PATH \ make TARGET=powerpc- sudo cp obj-ppc/openbios-qemu.elf /usr/local/share/qemu/openbios-ppc 469 exhibits the bug, so it's before that. I believe 450 is the "known good" version, so let's try 460... Bug. 455... Bug. 452... Bug. 451... Works. So the bug was introduced (or at least triggered) by revision 452: http://tracker.coreboot.org/trac/openbios/changeset/452/openbios-devel Anything in there look broken to you? Rob P.S. Please at least put up a link to this on your development page: http://tracker.coreboot.org/trac/openbios/browser/openbios-devel?rev=469 Right now earlier versions of your repository are completely inaccessible to newbies like me...