This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Re: Patch: Add new _Jv_SetGCFreeSpaceDivisor method to libgcj.
- From: Andrew Haley <aph-gcc at littlepinkcloud dot COM>
- To: David Daney <ddaney at avtrex dot com>
- Cc: Java Patch List <java-patches at gcc dot gnu dot org>
- Date: Mon, 22 Oct 2007 10:26:36 +0100
- Subject: Re: Patch: Add new _Jv_SetGCFreeSpaceDivisor method to libgcj.
- References: <471AEA80.4040301@avtrex.com>
David Daney writes:
> As I mentioned previously, GC_free_space_divisor is no longer exported
> from libgcj.so.
>
> Access to this GC tuning parameter is needed on systems where the
> maximum heap size is set. If GC_free_space_divisor is not set to a
> value larger than its default value of three, the heap can easily get
> fragmented in such a manner that OutOfMemoryErrors are thrown even
> though there is plenty of free memory.
>
> Since the loss of the ability to adjust GC_free_space_divisor is a
> regression, this should be committed even though we are in stage three.
>
> Tested on x86_64-pc-linux-gnu in libjava with no regressions.
>
> OK to commit?
>
> 2007-10-20 David Daney <ddaney@avtrex.com>
>
> * include/jvm.h (_Jv_SetGCFreeSpaceDivisor): Declare new function.
> * boehm.cc (_Jv_SetGCFreeSpaceDivisor): Define it.
Sure.
Incidentally, my ceasing to export GC_* wasn't a particularly well
thought-out decision: I just looked at the symbols we were exporting
and decided on a subset I thought was useful. Maybe there are some
GC_ symbols that should be exported. Thoughts?
Andrew.