From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1ItrCq-00075K-ES for qemu-devel@nongnu.org; Sun, 18 Nov 2007 15:57:28 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1ItrCp-000733-3t for qemu-devel@nongnu.org; Sun, 18 Nov 2007 15:57:27 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1ItrCp-00072v-0Q for qemu-devel@nongnu.org; Sun, 18 Nov 2007 15:57:27 -0500 Received: from pop-altamira.atl.sa.earthlink.net ([207.69.195.62]) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1ItrCo-0001tp-JC for qemu-devel@nongnu.org; Sun, 18 Nov 2007 15:57:26 -0500 Received: from user-142h2k8.cable.mindspring.com ([72.40.138.136] helo=earthlink.net) by pop-altamira.atl.sa.earthlink.net with esmtp (Exim 3.36 #1) id 1ItrCn-00031z-00 for qemu-devel@nongnu.org; Sun, 18 Nov 2007 15:57:25 -0500 Message-ID: <4740A735.8060508@earthlink.net> Date: Sun, 18 Nov 2007 15:57:25 -0500 From: Robert Reif MIME-Version: 1.0 Subject: Re: [Qemu-devel] [RFC][PATCH] fix sparc32 mxcc 64 bit read word order References: <473C4027.3030608@earthlink.net> <473CD151.6050000@earthlink.net> In-Reply-To: Content-Type: multipart/mixed; boundary="------------010508090307070308020704" Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org This is a multi-part message in MIME format. --------------010508090307070308020704 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Blue Swirl wrote: >On 11/16/07, Robert Reif wrote: > > >>>This patch fixes the word order for 64 bit reads of the mxcc registers. >>> >>> > >Otherwise everything seems OK, but it breaks NetBSD version 3 on SS10: >clock0 at obio0 slot 0 offset 0x200000: mk48t08 >timer0 at obio0 slot 0 offset 0x300000data fault: pc=0xf0111a0c >addr=0x0 sfsr=126 >panic: kernel fault >halted > >halt, power off > >Without the patch I get: >clock0 at obio0 slot 0 offset 0x200000: mk48t08 >timer0 at obio0 slot 0 offset 0x300000: delay constant 99 >zs0 at obio0 slot 0 offset 0x100000 level 12 softpri 6 >zstty0 at zs0 channel 0 (console i/o) >zstty1 at zs0 channel 1 >scsi-disk: Unsupported command length, command 79 > > > > > This is a classic case of two wrongs make a right. OpenBios need to be fixed to set mbus module id to start at 8, not 0 for mbus based machines. --------------010508090307070308020704 Content-Type: text/plain; name="obio.diff.txt" Content-Transfer-Encoding: base64 Content-Disposition: inline; filename="obio.diff.txt" SW5kZXg6IGRyaXZlcnMvb2Jpby5jCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIGRyaXZlcnMvb2Jpby5j CShyZXZpc2lvbiAxNzgpCisrKyBkcml2ZXJzL29iaW8uYwkod29ya2luZyBjb3B5KQpAQCAt ODkxLDcgKzg5MSwxNSBAQAogICAgICAgICBwdXNoX3N0cigiY2FjaGUtY29oZXJlbmNlPyIp OwogICAgICAgICBmd29yZCgicHJvcGVydHkiKTsKIAotICAgICAgICBQVVNIKGkpOworCXN3 aXRjaCAobWFjaGluZV9pZCkgeworICAgICAgICBjYXNlIDB4NzE6CisgICAgICAgIGNhc2Ug MHg3MjoKKyAgICAgICAgICAgIFBVU0goaSArIDgpOworICAgICAgICAgICAgYnJlYWs7Cisg ICAgICAgIGNhc2UgMHg4MDoKKyAgICAgICAgICAgIFBVU0goaSk7CisgICAgICAgICAgICBi cmVhazsKKyAgICAgICAgfQogICAgICAgICBmd29yZCgiZW5jb2RlLWludCIpOwogICAgICAg ICBwdXNoX3N0cigibWlkIik7CiAgICAgICAgIGZ3b3JkKCJwcm9wZXJ0eSIpOwo= --------------010508090307070308020704--