Re: [PING] RFR: 8231111: Cgroups v2: Rework Metrics in java.base so as to recognize unified hierarchy

2020-02-14 Thread Mandy Chung
Hi Severin, On 2/11/20 10:04 AM, Severin Gehwolf wrote: Updated webrev: Full: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-823/10/webrev/ incremental: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-823/10/incremental/webrev/ Thanks for updating this.  This patch looks okay in ge

RE: RFR(S) 8238585: Use handshake for JvmtiEventControllerPrivate::enter_interp_only_mode() and don't make compiled methods on stack not_entrant

2020-02-14 Thread Reingruber, Richard
Hi Patricio, > > I'm really glad you noticed the problematic nesting. This seems to be a general issue: currently a > > handshake cannot be nested in a vm operation. Maybe it should be asserted in the > > Handshake::execute() methods that they are not called by the vm thread evaluating a

Re: RFR(S) 8238585: Use handshake for JvmtiEventControllerPrivate::enter_interp_only_mode() and don't make compiled methods on stack not_entrant

2020-02-14 Thread Patricio Chilano
Hi Richard, On 2/14/20 9:58 AM, Reingruber, Richard wrote: Hi Patricio, thanks for having a look. > I’m only commenting on the handshake changes. > I see that operation VM_EnterInterpOnlyMode can be called inside > operation VM_SetFramePop which also allows nested operations. Here is

RE: RFR(S) 8238585: Use handshake for JvmtiEventControllerPrivate::enter_interp_only_mode() and don't make compiled methods on stack not_entrant

2020-02-14 Thread Reingruber, Richard
Hi Patricio, thanks for having a look. > I’m only commenting on the handshake changes. > I see that operation VM_EnterInterpOnlyMode can be called inside > operation VM_SetFramePop which also allows nested operations. Here is a > comment in VM_SetFramePop definition: > > // Nested