review (XS) for 6970566: runThese fails with SIGSEGV
Vladimir Kozlov
vladimir.kozlov at oracle.com
Wed Jul 21 19:04:42 PDT 2010
Tom,
You did not remove CodeCache::commit() from AdapterBlob::create().
Vladimir
Tom Rodriguez wrote:
> http://cr.openjdk.java.net/~never/6970566
>
> 6970566: runThese fails with SIGSEGV
> Reviewed-by:
>
> In the fix for 6965184 I added some code to track the number of
> adapters in the code cache by calling CodeCache::commit after the
> AdapterBlob was created. The code was missing a null check in case
> the allocation failed and that would lead to crashes if the code cache
> filled up. Instead of adding a null check I moved it into the
> AdapterBlob constructor to mirror other calls to commit. I also
> deleted a dead constructor.
More information about the hotspot-compiler-dev
mailing list