diff mbox

[0/3] Fully support 47-bit physical addresses.

Message ID 20130930142243.GK29427@zareason
State RFC
Delegated to: David Miller
Headers show

Commit Message

Bob Picco Sept. 30, 2013, 2:22 p.m. UTC
Hi,
David Miller wrote:	[Sat Sep 28 2013, 03:34:12PM EDT]
> From: Gurudas Pai <gurudas.pai@oracle.com>
> Date: Fri, 27 Sep 2013 21:07:11 -0700
> 
> > This is on a t4-1, machine becomes unusable with hundreds of
> > "WARNING: CPU: 0 PID: 441 at mm/mmap.c:2729 exit_mmap+0x14c/0x160()"
> 
> Strange, it boots perfectly fine on my T4-2.  Can you reply to
> this email with the ".config" you used?
> 
> Also, did you make sure to have the PAGE_OFFSET patches applied
> first?  They are a prerequisite for this series.
> 
> Thanks.
It seems to be the forever confusing MMU hole. Debian wheezy has no issue with
the patch below.

thanx,

bob

 arch/sparc/include/asm/page_64.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

David Miller Sept. 30, 2013, 3:47 p.m. UTC | #1
From: Bob Picco <bpicco@meloft.net>
Date: Mon, 30 Sep 2013 10:22:43 -0400

> Hi,
> David Miller wrote:	[Sat Sep 28 2013, 03:34:12PM EDT]
>> From: Gurudas Pai <gurudas.pai@oracle.com>
>> Date: Fri, 27 Sep 2013 21:07:11 -0700
>> 
>> > This is on a t4-1, machine becomes unusable with hundreds of
>> > "WARNING: CPU: 0 PID: 441 at mm/mmap.c:2729 exit_mmap+0x14c/0x160()"
>> 
>> Strange, it boots perfectly fine on my T4-2.  Can you reply to
>> this email with the ".config" you used?
>> 
>> Also, did you make sure to have the PAGE_OFFSET patches applied
>> first?  They are a prerequisite for this series.
>> 
>> Thanks.
> It seems to be the forever confusing MMU hole. Debian wheezy has no issue with
> the patch below.

Yes, that is the patch I was just about to test as well:

> @@ -101,7 +101,7 @@ typedef pte_t *pgtable_t;
>   * the actual exclusion region we enforce, wherein we use a 4GB red
>   * zone on each side of the VA hole.
>   */
> -#define SPARC64_VA_HOLE_TOP	_AC(0xfffff80000000000,UL)
> +#define SPARC64_VA_HOLE_TOP	_AC(0xfffffc0000000000,UL)
>  #define SPARC64_VA_HOLE_BOTTOM	_AC(0x0000080000000000,UL)

You have to adjust VA_HOLE_BOTTOM as well, shifting it down by one bit,
to be complete.

So my calculations were off by one, we need 44-bits to support the
current SPARC64_VA_HOLE_{TOP,BOTTOM} settings.

Anyways, I'm going to respin my series with minor adjustments plus
the VA_HOLE changes and post them here.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/sparc/include/asm/page_64.h b/arch/sparc/include/asm/page_64.h
index 2a6c45a..65d0ded 100644
--- a/arch/sparc/include/asm/page_64.h
+++ b/arch/sparc/include/asm/page_64.h
@@ -101,7 +101,7 @@  typedef pte_t *pgtable_t;
  * the actual exclusion region we enforce, wherein we use a 4GB red
  * zone on each side of the VA hole.
  */
-#define SPARC64_VA_HOLE_TOP	_AC(0xfffff80000000000,UL)
+#define SPARC64_VA_HOLE_TOP	_AC(0xfffffc0000000000,UL)
 #define SPARC64_VA_HOLE_BOTTOM	_AC(0x0000080000000000,UL)
 
 #define VA_EXCLUDE_START (SPARC64_VA_HOLE_BOTTOM - (1UL << 32UL))