Adding serviceability-dev@... Since the proposed solution to the bug is to not post an event, I think the Serviceability Team (which owns JVM/TI) needs to be involved directly in the review.
Dan On 11/14/18 9:28 AM, Thomas Stüfe wrote:
Dear all, may I please have reviews on this small patch. Note that this is borderline serviceability. I try to avoid crossposting, but if you think this should be looked at by serviceability feel free to forward it there. Issue: https://bugs.openjdk.java.net/browse/JDK-8213834 CR: http://cr.openjdk.java.net/~stuefe/webrevs/8213834-jvmti-reourceexhausted-shall-not-be-posted-from-compiler-thread/webrev.00/webrev/ Short description: we may post ResourceExhausted from Compiler threads. Handlers of this event may call back into the JVM, which may cause problems since we run in the compiler thread. The proposed solution is not to post the event in that case. See previous discussion here: http://mail.openjdk.java.net/pipermail/serviceability-dev/2018-November/025898.html Thanks, Thomas