<div dir="ltr">Thank you for reply, Thomas! Please see inline. It looks there are at least two ideas that can be good suggestions here.<div><div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 29, 2015 at 5:29 AM, Thomas Schatzl <span dir="ltr"><<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Tao,<br>
<span class=""><br>
On Fri, 2015-06-26 at 17:46 -0700, Tao Mao wrote:<br>
> Hi there,<br>
><br>
> Recently in using G1 on our realtime-focused production servers, I<br>
> noticed some full GCs were triggered by humongous allocation failure<br>
> due to fragmentation of free regions while it's still having enough<br>
> space in number.<br>
><br>
> I'm not suggesting that we move H-objects around. Just wonder if we<br>
> have the following or similar in place or plan to have. e.g.,<br>
<br>
</span>In such cases, moving around (small) humongous objects might actually be<br>
a good idea (compared to a full gc).<br>
<span class="">><br>
> (1) For evacuation, we grab free regions closer to the start of heap<br>
> and/or try to grab free regions with fewer neighboring free regions.<br>
<br>
</span>1a) has been implemented in some 8u release, either u20 or u40. If I<br>
remember correctly, new old gen regions (that includes humongous) are<br>
allocated from the end of the heap downwards, new young regions from the<br>
start of the heap.<br></blockquote><div><br></div><div>I saw that piece of code. Nice to have.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
The second option has not been explored (try to grab free regions<br>
according to other policies), but seems straightforward (but some work)<br>
to do. Check out G1CollectedHeap::new_region().<br></blockquote><div><br></div><div>Might not be that significant if the above is there. The above mechanism realizes that in the end, IMO.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Controlling the destination regions only solves half of the problem<br>
though, but a useful mechanism also requires control of the source<br>
regions: in a given evacuation there are often a lot of almost equally<br>
expensive regions to collect, so any change according to e.g. "try to<br>
create as much as possible contiguous free areas" might help a lot<br>
without a lot of performance impact.<br></blockquote><div><br></div><div>Good idea! You may want to record this idea somewhere.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span class=""><br>
> (2) Humongous object allocation tries to find the most suitable<br>
> location, i.e., least waste of its neighboring free regions after<br>
> H-allocation.<br>
<br>
</span>Check out G1CollectedHeap::humongous_obj_allocate(). The entry point<br>
that might be most interesting would be to change<br>
HeapRegionManager::find_contiguous_empty_or_available().<br></blockquote><div><br></div><div>I checked the latest code. It looks like the second point (finding the most suitable contiguous regions for humongous allocation) would be straightforward to implement in HeapRegionManager::find_contiguous().</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span class=""><br>
> Is there something along this line to fight region-level<br>
> fragmentation, or related fixed bugs? If so, any pointers?<br>
<br>
</span>Above mentioned is what I know in that area.<br>
<br>
Thanks,<br>
  Thomas<br>
<br>
<br>
</blockquote></div><br></div></div></div></div>