From owner-freebsd-mips@FreeBSD.ORG  Wed Jan 12 18:46:14 2011
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 56A8D106567A
	for <freebsd-mips@FreeBSD.org>; Wed, 12 Jan 2011 18:46:14 +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 036298FC16
	for <freebsd-mips@FreeBSD.org>; Wed, 12 Jan 2011 18:46:13 +0000 (UTC)
Received: from [127.0.0.1] (localhost [127.0.0.1])
	by harmony.bsdimp.com (8.14.3/8.14.1) with ESMTP id p0CIdgSw066129
	for <freebsd-mips@freebsd.org>; Wed, 12 Jan 2011 11:39:42 -0700 (MST)
	(envelope-from imp@bsdimp.com)
Message-ID: <4D2DF56E.6080207@bsdimp.com>
Date: Wed, 12 Jan 2011 11:39:42 -0700
From: Warner Losh <imp@bsdimp.com>
User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US;
	rv:1.9.2.12) Gecko/20101029 Thunderbird/3.1.6
MIME-Version: 1.0
To: freebsd-mips@FreeBSD.org
References: <AANLkTi=Ab+sUez+CH0_2=LJh5bQeRXF-Fv2C6t9-DPVy@mail.gmail.com>
	<AANLkTiksezVkcnwvxv6Fnr9cHAnACZWCH6_jc0buy08h@mail.gmail.com>
In-Reply-To: <AANLkTiksezVkcnwvxv6Fnr9cHAnACZWCH6_jc0buy08h@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
X-Content-Filtered-By: Mailman/MimeDel 2.1.5
Cc: 
Subject: Re: [PATCH] Use vm_paddr_t for physical address on MIPS
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: Wed, 12 Jan 2011 18:46:14 -0000

On 01/12/2011 04:43, Jayachandran C. wrote:
> On Wed, Jan 12, 2011 at 5:08 PM, Jayachandran C.
> <c.jayachandran@gmail.com>  wrote:
>> There are a few places in MIPS where vm_offset_t is used for physical
>> addresses. The attached patch fixes them, and also contains changes to
>> use uintmax_t when printing PTE entries and physical addresses.
>>
>> I plan to check this in if there are no objections, before checking in
>> the 64 bit PTE changes.  These changes will be useful if we decide to
>> have 64 bit physical address on n32(32bit) compilation.
> Re-sending with attachment.

These changes look good to me.  Looks like arm makes the same mistake 
about vm_offset_t vs vm_paddr_t for phys_avail too (well in places, it 
appears to be muddled between both depending on the subport).  ia64, 
i386, amd64 and sparc64 get this right.  powerpc uses vm_offset_t, and 
likely should be changed as well.

Warner
> JC.
>
>
> _______________________________________________
> freebsd-mips@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-mips
> To unsubscribe, send any mail to "freebsd-mips-unsubscribe@freebsd.org"