Request for reviews (S): 6826736: CMS: core dump with -XX:+UseCompressedOops

[Tom Rodriguez]

Looks good.

tom

On Jul 22, 2009, at 12:35 PM, Vladimir Kozlov wrote:

>
> http://cr.openjdk.java.net/~kvn/6826736/webrev.01
>
> Fixed 6826736: CMS: core dump with -XX:+UseCompressedOops
>
> Problem:
> Compiled code may produce decoded oop = narrow_oop_base
> when a narrow oop implicit null check is used.
> The narrow_oop_base could be NULL or be the address
> of the page below heap depending on the used mode of
> compressed oops. Such decoded oop could live across
> a safepoint and as result referenced in oopMap and debug info.
> The implicit NULL exception code checks for such oop value
> but deoptimization code and GC does not. And this cause
> the problem.
>
> Solution:
> Fix deoptimization code and GC to check for such oops values.
> During deoptimization convert them to NULL.
> And ignore them for GC (like NULL values are ignored).
>
> Added the regression test which shows the problem.
>
> Reviewed by:
>
> Fix verified (y/n): y, regression test
>
> Other testing:
> JPRT, CTW with/without UseCompressedOops and +/- ScavengeALot
>