RFR(9): 8176098: Deprecate FlatProfiler

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR(9): 8176098: Deprecate FlatProfiler

Robbin Ehn
Hi all, please review:

The VM Option "-Xprof" no longer relevant and is a source of root scanning and contains bugs.

The FC Extension Request have been approved and CCC is pending approval.

Webrev:
http://cr.openjdk.java.net/~rehn/8176098/jdk/webrev/
http://cr.openjdk.java.net/~rehn/8176098/hotspot/webrev/

Bug:
https://bugs.openjdk.java.net/browse/JDK-8176098

No test is affected by this new warning. (there is an internal test but it's no in use)

Thanks!

/Robbin
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(9): 8176098: Deprecate FlatProfiler

Aleksey Shipilev-4
On 03/10/2017 06:10 PM, Robbin Ehn wrote:
> The VM Option "-Xprof" no longer relevant and is a source of root scanning and
> contains bugs.
>
> The FC Extension Request have been approved and CCC is pending approval.
>
> Webrev:
> http://cr.openjdk.java.net/~rehn/8176098/jdk/webrev/
> http://cr.openjdk.java.net/~rehn/8176098/hotspot/webrev/

YES. Looks good.

Thanks,
-Aleksey

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(9): 8176098: Deprecate FlatProfiler

Robbin Ehn
Thanks!

/Robbin

On 03/10/2017 06:31 PM, Aleksey Shipilev wrote:

> On 03/10/2017 06:10 PM, Robbin Ehn wrote:
>> The VM Option "-Xprof" no longer relevant and is a source of root scanning and
>> contains bugs.
>>
>> The FC Extension Request have been approved and CCC is pending approval.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~rehn/8176098/jdk/webrev/
>> http://cr.openjdk.java.net/~rehn/8176098/hotspot/webrev/
>
> YES. Looks good.
>
> Thanks,
> -Aleksey
>
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(9): 8176098: Deprecate FlatProfiler

coleen.phillimore
In reply to this post by Robbin Ehn
This change looks great!
thanks!
Coleen

On 3/10/17 12:10 PM, Robbin Ehn wrote:

> Hi all, please review:
>
> The VM Option "-Xprof" no longer relevant and is a source of root
> scanning and contains bugs.
>
> The FC Extension Request have been approved and CCC is pending approval.
>
> Webrev:
> http://cr.openjdk.java.net/~rehn/8176098/jdk/webrev/
> http://cr.openjdk.java.net/~rehn/8176098/hotspot/webrev/
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8176098
>
> No test is affected by this new warning. (there is an internal test
> but it's no in use)
>
> Thanks!
>
> /Robbin

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(9): 8176098: Deprecate FlatProfiler

Robbin Ehn
Thanks Coleen!

/Robbin

On 03/13/2017 10:54 PM, [hidden email] wrote:

> This change looks great!
> thanks!
> Coleen
>
> On 3/10/17 12:10 PM, Robbin Ehn wrote:
>> Hi all, please review:
>>
>> The VM Option "-Xprof" no longer relevant and is a source of root scanning and contains bugs.
>>
>> The FC Extension Request have been approved and CCC is pending approval.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~rehn/8176098/jdk/webrev/
>> http://cr.openjdk.java.net/~rehn/8176098/hotspot/webrev/
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8176098
>>
>> No test is affected by this new warning. (there is an internal test but it's no in use)
>>
>> Thanks!
>>
>> /Robbin
>
Loading...