RFR: 8152491: Convert TracePageSizes to use UL

[Stefan Karlsson]

Hi all,

Here's a new patch, which addresses StefanJ's issues:

http://cr.openjdk.java.net/~stefank/8152491/webrev.02.delta
http://cr.openjdk.java.net/~stefank/8152491/webrev.02

Thanks,
StefanK

On 2016-04-08 10:57, Stefan Johansson wrote:
> Hi StefanK,
>
> On 2016-04-06 20:16, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please review to convert TracePageSizes to use UL.
>>
>> http://cr.openjdk.java.net/~stefank/8152491/webrev.01
>> https://bugs.openjdk.java.net/browse/JDK-8152491
>>
>> TracePageSizes used to be a develop flag, but I've decided to create 
>> a UL tag accessible from release builds. We've recently needed to 
>> debug some large pages code and wanted this information in release 
>> builds.
>>
> Looks good in general and I like that we make it available in product 
> mode. Just a few comments.
> os.cpp: 1534 void os::trace_page_sizes_for_requested_size(const char* str,
> 1535 const size_t requested_size, ...
> 1541 log_info(pagesizes)("%s:"
> 1542 " req_size=" SIZE_FORMAT "%s"
> 1543 " base=" PTR_FORMAT
> ...
> 1547 str,
> 1548 trace_page_size_params(size),
> 1549 p2i(base),
> On line 1548 we should use requested_size instead of size.---
> genCollectedHeap.cpp:
> 170
> 171   os::trace_page_sizes("Heap", 
> collector_policy()->min_heap_byte_size(),
> 172                        total_reserved,
> 173                        alignment,
> 174                        heap_rs->base(),
> 175                        heap_rs->size());
>
> Move min_heap_byte_size() to its own line.
> ---
>
> I also think that it would be better to use 'pagesize' instead of 
> 'pagesizes' as the tag, even though the old flag said pagesizes. But 
> there are a lot of other flags that also have an ending s, exceptions 
> for example, so I'm not sure how we should handle it. It would be good 
> to make the tags feel "unified".
>
> Thanks,
> Stefan
>
>> Changes in the patch:
>>
>> 1) Replaced TracePageSizes with the 'pagesizes' UL tag
>> 2) Cleaned up the memory region description strings
>> 3) Used B/K/M/G suffixes when the size is exactly divisible by those
>> 4) Removed trace_gen_sizes, since the output was too obscure to be 
>> that helpful
>> 5) Added tracing for Serial and CMS
>> 6) Fixed a jtreg test that verified the output from TracePageSizes
>>
>> Example output from 'java -Xlog:pagesizes -Xmx512m -XX:+UseLargePages 
>> -version':
>>
>> [0.018s][info][pagesizes] CodeHeap 'non-nmethods':  min=2496K max=8M 
>> base=0x00007fd735000000 page_size=4K size=8M
>> [0.018s][info][pagesizes] CodeHeap 'profiled nmethods': min=2496K 
>> max=116M base=0x00007fd735800000 page_size=4K size=116M
>> [0.018s][info][pagesizes] CodeHeap 'non-profiled nmethods': min=2496K 
>> max=116M base=0x00007fd73cc00000 page_size=4K size=116M
>> [0.056s][info][pagesizes] Heap:  min=8M max=512M 
>> base=0x00000000e0000000 page_size=2M size=512M
>> [0.056s][info][pagesizes] Block Offset Table: req_size=1M 
>> base=0x00007fd6d5fe1000 page_size=4K alignment=4K size=1M
>> [0.056s][info][pagesizes] Card Table: req_size=1M 
>> base=0x00007fd6d5de1000 page_size=4K alignment=4K size=1M
>> [0.056s][info][pagesizes] Card Counts Table: req_size=1M 
>> base=0x00007fd6d5ce1000 page_size=4K alignment=4K size=1M
>> [0.056s][info][pagesizes] Prev Bitmap: req_size=8M 
>> base=0x00007fd6d5400000 page_size=2M alignment=2M size=8M
>> [0.056s][info][pagesizes] Next Bitmap: req_size=8M 
>> base=0x00007fd6d4c00000 page_size=2M alignment=2M size=8M
>>
>> The patch uses the LogStreamCHeap from the following patch that is 
>> out for review:
>> http://mail.openjdk.java.net/pipermail/hotspot-dev/2016-April/022477.html
>>
>> Tested with JPRT and new internal VM test for the suffixes.
>>
>> Thanks,
>> StefanK
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20160411/da7e2b65/attachment.htm>