From owner-freebsd-current  Wed Nov 29 16:15:19 2000
Delivered-To: freebsd-current@freebsd.org
Received: from mail.du.gtn.com (mail.du.gtn.com [194.77.9.57])
	by hub.freebsd.org (Postfix) with ESMTP id 9F22437B400
	for <freebsd-current@freebsd.org>; Wed, 29 Nov 2000 16:15:15 -0800 (PST)
Received: from mail.cicely.de (cicely.de [194.231.9.142])
	by mail.du.gtn.com (8.11.0.Beta3/8.11.0.Beta3) with ESMTP id eAU0F2p17934
	(using TLSv1/SSLv3 with cipher EDH-RSA-DES-CBC3-SHA (168 bits) verified OK);
	Thu, 30 Nov 2000 01:15:04 +0100 (MET)
Received: from cicely8.cicely.de (cicely8.cicely.de [10.1.2.10])
	by mail.cicely.de (8.11.0.Beta1/8.11.0.Beta1) with ESMTP id eAU0F8M63928;
	Thu, 30 Nov 2000 01:15:09 +0100 (CET)
Received: (from ticso@localhost)
	by cicely8.cicely.de (8.11.0/8.9.2) id eAU0F4i47497;
	Thu, 30 Nov 2000 01:15:04 +0100 (CET)
	(envelope-from ticso)
Date: Thu, 30 Nov 2000 01:15:03 +0100
From: Bernd Walter <ticso@cicely8.cicely.de>
To: Andrew Gallatin <gallatin@cs.duke.edu>
Cc: Wilko Bulte <wkb@freebie.demon.nl>,
	Mike Eldridge <diz@cafes.net>, freebsd-current@freebsd.org
Subject: Re: Patch for current on LCA based alphas
Message-ID: <20001130011503.A47460@cicely8.cicely.de>
References: <20001125193430.A40717@cicely8.cicely.de> <Pine.LNX.4.10.10011271439450.21970-100000@mail.cafes.net> <20001127224321.F1846@freebie.demon.nl> <20001128014038.A44759@cicely8.cicely.de> <14885.23282.73569.898582@grasshopper.cs.duke.edu>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
X-Mailer: Mutt 1.0.1i
In-Reply-To: <14885.23282.73569.898582@grasshopper.cs.duke.edu>; from gallatin@cs.duke.edu on Wed, Nov 29, 2000 at 02:47:54PM -0500
Sender: owner-freebsd-current@FreeBSD.ORG
Precedence: bulk
X-Loop: FreeBSD.ORG

On Wed, Nov 29, 2000 at 02:47:54PM -0500, Andrew Gallatin wrote:
> 
> Bernd Walter writes:
>  > 
>  >     trap entry = 0x2 (memory management fault)
>  >     a0         = 0xfffffbf1e0000018
>  >     a1         = 0x1
>  >     a2         = 0x0
>  >     pc         = 0xfffffc0000557a10
>  >     ra         = 0xfffffc000055791c
>  >     curproc    = 0xfffffc000062f118
>  >         pid = 0, comm = swapper
> 
> Bernd,
> 
> Note the faulting address.  It is not a k0seg address.  This means
> something went negative, most likely in the LCA_CFGOFF macro.  Here's
> why..  

What is k0seg?
How can I see that it's not one?

> After Doug's busspace changes, the *pcib_{read,write}_config()
> functions were introduced.  They take signed args and pass them to the
> CFGOFF macro.  Previously, the arguments to the LCA_CFGOFF macro were
> provided by the lca_cfg{read,write}{b,w,l} functions and were
> unsigned.

Just to be clear the values given to lca_read_config were:
b=0, s=20, f=0, reg=0, width=4
That means b in LCA_CFGOFF is false and the second formular will be applied.
The first part is 1<<n while n is calculated to be 31 in our case and
this makes -1 for int - your theory seems to be right.
But are you shure that changing our variables to unsigned will help?
Don't we need to make '1' unsigned?

> Can you try the following patch (after backing yours out).  A similar
> patch fixes the same problem on an AlphaServer 2100.  I'd like to fix
> this in all the pcib_{read,write}_config functions at once...

I will test it tommorow.

-- 
B.Walter              COSMO-Project         http://www.cosmo-project.de
ticso@cicely.de         Usergroup           info@cosmo-project.de



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message