Re: RFR: 8154041: JVMTI to Unified Logging

2016-04-19 Thread Robbin Ehn
Hi Serguei, Thanks! /Robbin On 04/19/2016 11:47 AM, serguei.spit...@oracle.com wrote: Hi Robbin, The fix looks good. Also, I'm Ok with the removal of comments. Thanks, Serguei On 4/19/16 02:43, Robbin Ehn wrote: Hi Marcus, On 04/19/2016 11:07 AM, Marcus Larsson wrote: Hi, On

Re: RFR: 8154041: JVMTI to Unified Logging

2016-04-19 Thread serguei.spit...@oracle.com
Hi Robbin, The fix looks good. Also, I'm Ok with the removal of comments. Thanks, Serguei On 4/19/16 02:43, Robbin Ehn wrote: Hi Marcus, On 04/19/2016 11:07 AM, Marcus Larsson wrote: Hi, On 04/14/2016 01:36 PM, Robbin Ehn wrote: Hi all, Please review: This moves jvmti trace output to

Re: RFR: 8154041: JVMTI to Unified Logging

2016-04-19 Thread Robbin Ehn
Hi Marcus, On 04/19/2016 11:07 AM, Marcus Larsson wrote: Hi, On 04/14/2016 01:36 PM, Robbin Ehn wrote: Hi all, Please review: This moves jvmti trace output to the tag jvmti on trace level. The jvmti trace argument e.g. "-XX:TraceJVMTI=ec+ieost,all+ieost" have a lot of functionality, e.g.

Re: RFR: 8154041: JVMTI to Unified Logging

2016-04-19 Thread Marcus Larsson
Hi, On 04/14/2016 01:36 PM, Robbin Ehn wrote: Hi all, Please review: This moves jvmti trace output to the tag jvmti on trace level. The jvmti trace argument e.g. "-XX:TraceJVMTI=ec+ieost,all+ieost" have a lot of functionality, e.g. can filter on func, in/out, etc, so it if left as is.

RFR: 8154041: JVMTI to Unified Logging

2016-04-14 Thread Robbin Ehn
Hi all, Please review: This moves jvmti trace output to the tag jvmti on trace level. The jvmti trace argument e.g. "-XX:TraceJVMTI=ec+ieost,all+ieost" have a lot of functionality, e.g. can filter on func, in/out, etc, so it if left as is. TraceJVMTI enables jvmti trace to stdout if no