From owner-freebsd-mips@FreeBSD.ORG  Fri Jul  9 21:42:33 2010
Return-Path: <owner-freebsd-mips@FreeBSD.ORG>
Delivered-To: freebsd-mips@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 42239106564A;
	Fri,  9 Jul 2010 21:42:33 +0000 (UTC) (envelope-from imp@bsdimp.com)
Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85])
	by mx1.freebsd.org (Postfix) with ESMTP id E1F448FC19;
	Fri,  9 Jul 2010 21:42:32 +0000 (UTC)
Received: from localhost (localhost [127.0.0.1])
	by harmony.bsdimp.com (8.14.3/8.14.1) with ESMTP id o69LXoiH091718;
	Fri, 9 Jul 2010 15:33:50 -0600 (MDT) (envelope-from imp@bsdimp.com)
Date: Fri, 09 Jul 2010 15:34:09 -0600 (MDT)
Message-Id: <20100709.153409.59640143160064195.imp@bsdimp.com>
To: jmallett@FreeBSD.org
From: "M. Warner Losh" <imp@bsdimp.com>
In-Reply-To: <AANLkTimcCqP2SinyKJUqpT9l0j07AqGCLxMCuCGXc2wA@mail.gmail.com>
References: <AANLkTik8fBlxMyZ3AZETF_7DvlSpPsx2hozIWinRy0U0@mail.gmail.com>
	<20100709.144631.1141490504242621103.imp@bsdimp.com>
	<AANLkTimcCqP2SinyKJUqpT9l0j07AqGCLxMCuCGXc2wA@mail.gmail.com>
X-Mailer: Mew version 6.3 on Emacs 22.3 / Mule 5.0 (SAKAKI)
Mime-Version: 1.0
Content-Type: Text/Plain; charset=iso-8859-1
Content-Transfer-Encoding: quoted-printable
Cc: freebsd-mips@FreeBSD.org
Subject: Re: Merging 64 bit changes to -HEAD - part 4
X-BeenThere: freebsd-mips@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Porting FreeBSD to MIPS <freebsd-mips.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-mips>,
	<mailto:freebsd-mips-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-mips>
List-Post: <mailto:freebsd-mips@freebsd.org>
List-Help: <mailto:freebsd-mips-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-mips>,
	<mailto:freebsd-mips-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 09 Jul 2010 21:42:33 -0000

In message: <AANLkTimcCqP2SinyKJUqpT9l0j07AqGCLxMCuCGXc2wA@mail.gmail.c=
om>
            Juli Mallett <jmallett@FreeBSD.org> writes:
: On Fri, Jul 9, 2010 at 13:46, M. Warner Losh <imp@bsdimp.com> wrote:
: > pmap_map should likely have a comment to the effect:
: > =A0* For memory that's directly mappable, we return the direct map
: > =A0* address. =A0For other addresses, we create a map.
: =

: Doesn't it already?
: =

:  *	The value passed in '*virt' is a suggested virtual address for
:  *	the mapping. Architectures which can support a direct-mapped
:  *	physical to virtual region can return the appropriate address
:  *	within that region, leaving '*virt' unchanged. Other
:  *	architectures should map the pages starting at '*virt' and
:  *	update '*virt' with the first usable address after the mapped
:  *	region.

That's rather generic, and on some mips we do a mixed-mode here...

: > : mips-ddb-64.patch (from Juli's branch)
: > : Minor fixups for ddb support.
: >
: > Can you explain the casting for all the calls to kbdpeek*?
: =

: Yes.  sp is (or should be) a register_t.  You can't go from a
: register_t to a pointer on n32.  Even if we don't intend to support
: n32 kernels from head, we shouldn't make n32 harder by using the wron=
g
: types without casts :)

True, but can't we redefine these functions such that we don't need
these great big ugly casts?  They are written in assembler, after all,
but are specific to mips so the API can change here...

Warner