review request (S) 6866585 debug code in ciObjectFactory too slow
Tom Rodriguez
Thomas.Rodriguez at Sun.COM
Thu Aug 6 11:36:50 PDT 2009
>>> Would it be possible to put the check into the GC epilogue?
>>> Probably not, since ciEnv guys are not statically allocated and
>>> not easy to enumerate (one per active compile-thread task). I
>>> guess I do like the idea of checking a GC tick counter from the
>>> CI, to throttle the paranoid check. Best of all would be to have
>>> some explicit linkage from the GC epilogue code to the CI, which
>>> somehow documents and enforces the order invariance. You can see
>>> why this is awkward: It is an invariant shared by the CI and the
>>> GC.
>
> Yes, i agree, if it's one per active compile task, may be it's best
> to use
> yr strategy of eliding the check unless the full gc count changes.
John, couldn't this strategy allow us to reunify the handling of oops
into a single table instead of needing the NonPermOop stuff? If we
simply re-sort the table when needed that code becomes simple again.
It also makes us impervious to reordering concerns.
tom
>
> -- ramki
>
>> -- John
>
More information about the hotspot-compiler-dev
mailing list