Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 20 Aug 2008 16:41:51 -0700
From:      Nathan Whitehorn <nwhitehorn@freebsd.org>
To:        John Baldwin <jhb@freebsd.org>
Cc:        freebsd-arch@freebsd.org
Subject:   Re: UMA MD Small Allocator Runtime Switching
Message-ID:  <48ACABBF.3070705@freebsd.org>
In-Reply-To: <200808131214.43326.jhb@freebsd.org>
References:  <48981C19.8060009@freebsd.org> <200808051024.27043.jhb@freebsd.org> <48A2E62A.9060604@freebsd.org> <200808131214.43326.jhb@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
John Baldwin wrote:
> [snip]
>> I thought about it, but we can only use 4K pages on the G5 so this would 
>> put a large amount of pressure on the page table. IBM removed the block 
>> translation mechanism from the G5 and the CPU's superpage support is not 
>> available in the 32-bit compatibility mode under which we currently run.
>>     
>
> Hmm, I didn't know you weren't running in full 64-bit mode.  Is that a 
> property of the G5 CPU that it only supports the 32-bit compat mode with 
> 64-bit extensions?
>   
(see other email about the properties of the G5)

I think the following one-line patch provides a reasonable solution to 
this problem. Is there any reason this is a bad idea?
-Nathan

Index: uma_core.c
===================================================================
--- uma_core.c  (revision 181929)
+++ uma_core.c  (working copy)
@@ -1667,7 +1667,7 @@
 
        bucket_init();
 
-#ifdef UMA_MD_SMALL_ALLOC
+#if defined(UMA_MD_SMALL_ALLOC) && !defined(UMA_MD_SMALL_ALLOC_NEEDS_VM)
        booted = 1;
 #endif
 




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?48ACABBF.3070705>