RE: 8215534: [testbug] some jfr test don't check @requires vm.hasJFR

2018-12-13 Thread Lindenmaier, Goetz
Subject: Re: 8215534: [testbug] some jfr test don't check @requires > vm.hasJFR > > Looks good. > > Erik > > > Hi, > > > > These tests lack @requires vm.hasJFR, thus they are failing on AIX. > > http://cr.openjdk.java.net/~goetz/wr18/8215334-JFR_requir

Re: 8215534: [testbug] some jfr test don't check @requires vm.hasJFR

2018-12-13 Thread Erik Gahlin
Looks good. Erik Hi, These tests lack @requires vm.hasJFR, thus they are failing on AIX. http://cr.openjdk.java.net/~goetz/wr18/8215334-JFR_requires/01/ Please review. I will push this to jdk12 as it is a testbug if I miss the RDP deadline. Best regards, Goetz.

RE: 8215534: [testbug] some jfr test don't check @requires vm.hasJFR

2018-12-13 Thread Lindenmaier, Goetz
gt; Subject: Re: 8215534: [testbug] some jfr test don't check @requires > vm.hasJFR > > Looks good. > > PS. I just checked that there are other tests with the same requires clause. > > -Sundar > > On 13/12/18, 1:20 PM, Lindenmaier, Goetz wrote: > > Hi, >

Re: 8215534: [testbug] some jfr test don't check @requires vm.hasJFR

2018-12-13 Thread Sundararajan Athijegannathan
Looks good. PS. I just checked that there are other tests with the same requires clause. -Sundar On 13/12/18, 1:20 PM, Lindenmaier, Goetz wrote: Hi, These tests lack @requires vm.hasJFR, thus they are failing on AIX. http://cr.openjdk.java.net/~goetz/wr18/8215334-JFR_requires/01/ Please revi

8215534: [testbug] some jfr test don't check @requires vm.hasJFR

2018-12-12 Thread Lindenmaier, Goetz
Hi, These tests lack @requires vm.hasJFR, thus they are failing on AIX. http://cr.openjdk.java.net/~goetz/wr18/8215334-JFR_requires/01/ Please review. I will push this to jdk12 as it is a testbug if I miss the RDP deadline. Best regards, Goetz.