From owner-freebsd-mips Wed Feb 12 16:46:36 2003 Delivered-To: freebsd-mips@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 931) id 4E6FC37B405; Wed, 12 Feb 2003 16:46:33 -0800 (PST) Date: Wed, 12 Feb 2003 18:46:33 -0600 From: Juli Mallett To: freebsd-mips@FreeBSD.org Subject: [jmallett@FreeBSD.org: PERFORCE change 25068 for review] Message-ID: <20030212184633.A58569@FreeBSD.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i Organisation: The FreeBSD Project X-Alternate-Addresses: , , , , X-Towel: Yes X-LiveJournal: flata, jmallett X-Negacore: Yes Sender: owner-freebsd-mips@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG The moral of the story: don't trust yourself to have done the right thing in the first place. ----- Forwarded message from Juli Mallett ----- Delivered-To: jmallett@freebsd.org Delivered-To: perforce@freebsd.org Date: Wed, 12 Feb 2003 16:43:57 -0800 (PST) X-Authentication-Warning: repoman.freebsd.org: perforce set sender to jmallett@freebsd.org using -f From: Juli Mallett Subject: PERFORCE change 25068 for review To: Perforce Change Reviews http://perforce.freebsd.org/chv.cgi?CH=25068 Change 25068 by jmallett@jmallett_dalek on 2003/02/12 16:43:14 "Fool, fool, fool -- that's what I am." Here I thought I'd run into a gas bug, and I trusted the cc -S output was OK, and I thought everything was fine, and it was blowing up for a dumb reason. So I dig into the gas code and I get it to talk back in interesting ways, and I find out why this code was failing to assemble: I MADE A TYPO. Ugh. Forgot to parenthesise the register holding our pointer so I would end up with a memory address. Instead I wound up with a register. Damnit. Anyway, add the parens that are needed here. Now we're back to failing over DDB stuff in kern_mutex.c. Affected files ... .. //depot/projects/mips/sys/mips/include/atomic.h#8 edit Differences ... ==== //depot/projects/mips/sys/mips/include/atomic.h#8 (text+ko) ==== @@ -83,9 +83,9 @@ \ __asm __volatile ( \ "1:\n\t" \ - "ll %[temp], %[p]\n\t" \ + "ll %[temp], (%[p])\n\t" \ asmop "\n\t" \ - "sc %[temp], %[p]\n\t" \ + "sc %[temp], (%[p])\n\t" \ "beqz %[temp], 1b\n\t" \ : [val] "=r"(val) \ : [temp] "r"(temp), [p] "r"(p) \ @@ -117,10 +117,10 @@ __asm __volatile ( "1:\n\t" - "ll %[temp], %[p]\n\t" + "ll %[temp], (%[p])\n\t" "move %[res], %[temp]\n\t" "move %[temp], 0\n\t" - "sc %[temp], %[p]\n\t" + "sc %[temp], (%[p])\n\t" "beqz %[temp], 1b\n\t" : : [temp] "r"(temp), [p] "r"(p), [res] "r"(res) @@ -142,9 +142,9 @@ \ __asm __volatile ( \ "1:\n\t" \ - "lld %[temp], %[p]\n\t" \ + "lld %[temp], (%[p])\n\t" \ asmop "\n\t" \ - "scd %[temp], %[p]\n\t" \ + "scd %[temp], (%[p])\n\t" \ "beqz %[temp], 1b\n\t" \ : [val] "=r"(val) \ : [temp] "r"(temp), [p] "r"(p) \ @@ -176,10 +176,10 @@ __asm __volatile ( "1:\n\t" - "lld %[temp], %[p]\n\t" + "lld %[temp], (%[p])\n\t" "move %[res], %[temp]\n\t" "move %[temp], 0\n\t" - "scd %[temp], %[p]\n\t" + "scd %[temp], (%[p])\n\t" "beqz %[temp], 1b\n\t" : : [temp] "r"(temp), [p] "r"(p), [res] "r"(res) ----- End forwarded message ----- -- Juli Mallett AIM: BSDFlata -- IRC: juli on EFnet OpenDarwin, Mono, FreeBSD Developer ircd-hybrid Developer, EFnet addict FreeBSD on MIPS-Anything on FreeBSD To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-mips" in the body of the message