RFR(S): 8025942: Implement os::Bsd:available_memory
    Dmitry Samersoff 
    dmitry.samersoff at oracle.com
       
    Thu Oct 10 12:36:18 PDT 2013
    
    
  
Gerard,
Please move
uint64_t available = physical_memory() >> 2;
under #else
Besides that looks good for me
-Dmitry
On 2013-10-10 23:14, Gerard Ziemski wrote:
> Please review this fix that implements os::Bsd:available_memory()
> 
> Description:
> 
> This is a simple change - we use a similar implementation that Apple
> itself uses (see
> http://opensource.apple.com/source/system_cmds/system_cmds-498.2/vm_stat.tproj/vm_stat.c)
> to implement BSD (Apple platform only) implementation for finding out
> available (free) memory.
> 
> 
> Testing:
> 
> Passed JPRT, vm quicklooks
> 
> 
> References:
> 
> http://cr.openjdk.java.net/~dsamersoff/sponsorship/gziemski/JDK-8025942/webrev.01/
> 
> https://bugs.openjdk.java.net/browse/JDK-8025942
> 
> 
> cheers
> 
-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.
    
    
More information about the hotspot-dev
mailing list