RFR (XS): 8021823 G1: Concurrent marking crashes with -XX:ObjectAlignmentInBytes>=32 in 64bit VMs (patch for hs24+hs25)

[Tony Printezis]

Thomas,

The fix in concurrentMark.cpp looks OK (how did it work before?!?!?!).

Regarding the test: If I understand the test correctly, the only reason 
why it works is because a conc marking cycle in G1 starts with a young 
GC (the assert seems to happen during an evac pause, right?). So, note 
that this test will probably behave different with CMS (if that matters).

Tony

On 8/28/13 9:32 AM, Thomas Schatzl wrote:
> Hi all,
>
> On Thu, 2013-08-01 at 07:49 +0200, Bengt Rutisson wrote:
>> Hi Thomas,
>>
>> The code change looks good.
>>
>> One question about the test. I know that the current failure was with
>> G1, but all of our GC should work with all the supported
>> ObjectAlignments, right? Why do we only want to test this with G1?
>> Maybe we could just remove " -XX:+UseG1GC" from the command line and
>> get testing for all GCs? In that case I guess you probably don't need
>> to use the ProcessTools either. You could have serveral @run commands
>> instead.
>    I still need another review on this issue, anyone?
>
> I only updated the test according to Bengt's suggestion, i.e. allow
> running all collectors through the tests and using several @run
> statements.
>
> bugs.sun.com:
> http://bugs.sun.com/view_bug.do?bug_id=8021823
>
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8021823/webrev.4/
>
> Testing:
> jprt, jtreg test
>
> Thanks,
> Thomas
>
>

-- 
Tony Printezis | Staff Software Engineer | Twitter

@TonyPrintezis
tprintezis at twitter.com