From owner-p4-projects Wed Feb 12 16:44: 1 2003 Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 5CAD037B405; Wed, 12 Feb 2003 16:43:58 -0800 (PST) Delivered-To: perforce@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id E781637B401 for ; Wed, 12 Feb 2003 16:43:57 -0800 (PST) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 7927943FAF for ; Wed, 12 Feb 2003 16:43:57 -0800 (PST) (envelope-from jmallett@freebsd.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.12.6/8.12.6) with ESMTP id h1D0hvbv048619 for ; Wed, 12 Feb 2003 16:43:57 -0800 (PST) (envelope-from jmallett@freebsd.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.12.6/8.12.6/Submit) id h1D0hvow048616 for perforce@freebsd.org; Wed, 12 Feb 2003 16:43:57 -0800 (PST) Date: Wed, 12 Feb 2003 16:43:57 -0800 (PST) Message-Id: <200302130043.h1D0hvow048616@repoman.freebsd.org> 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 Sender: owner-p4-projects@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG 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) To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe p4-projects" in the body of the message