RFR (M): 8003420: NPG: make new GC root for pd_set

[Ioi Lam]

Thomas,

Looks good.Thank you!

- Ioi



On 10/04/2013 10:27 AM, Thomas Schatzl wrote:
> Hi Ioi,
>
> On Thu, 2013-10-03 at 07:30 -0700, Ioi Lam wrote:
>> Hi Thomas,
>>
>> One more suggestion:
>>
>> after calling dictionary()->unlink(is_alive), maybe we should have a debug build
>> verification loop that checks that all ProtectionDomainCacheEntry's
>> that are reachable from DictionaryEntry's live in the table? (i.e., we
>> don't have any dangling pointers).
> New version with this changes at
>
> http://cr.openjdk.java.net/~tschatzl/8003420/webrev.3/
>
> Look at SystemDictionary.cpp. Everything else is the same.
>
> Testing
> jprt, local testing with dacapo/scala benchmarks in debug mode, with
> VerifyBefore/AfterGC.
>
> Hth,
> Thomas
>
>