Hi Götz and JC, thanks for reviewing.
Best regards, Martin From: Jean Christophe Beyler <[email protected]> Sent: Montag, 20. Mai 2019 18:34 To: Lindenmaier, Goetz <[email protected]> Cc: Doerr, Martin <[email protected]>; [email protected]; Volker Simonis ([email protected]) <[email protected]> Subject: Re: RFR(S): 8224230: [PPC64, s390] Support AsyncGetCallTrace Hi Martin, Looks good to me too :) Jc On Mon, May 20, 2019 at 8:56 AM Lindenmaier, Goetz <[email protected]<mailto:[email protected]>> wrote: LGTM Goetz! > -----Original Message----- > From: Doerr, Martin > Sent: Montag, 20. Mai 2019 17:49 > To: Lindenmaier, Goetz > <[email protected]<mailto:[email protected]>>; serviceability- > [email protected]<mailto:[email protected]>; JC Beyler > <[email protected]<mailto:[email protected]>>; Volker Simonis > ([email protected]<mailto:[email protected]>) > <[email protected]<mailto:[email protected]>> > Subject: RE: RFR(S): 8224230: [PPC64, s390] Support AsyncGetCallTrace > > Hi Götz, > > thanks for reviewing it so quickly. > > Thanks for pointing me to the obsolete comment. I've also fixed in on PPC64 > and also removed a redundant assertion on PPC64: > http://cr.openjdk.java.net/~mdoerr/8224230_ppc_s390_AsyncCallTrace/webr > ev.01/ > > I haven't used async profiler, but I've ran JTREG tests for hotspot runtime on > linux PPC64 (Big and Little Endian) and s390 and all ones have passed. > > Best regards, > Martin > > > -----Original Message----- > From: Lindenmaier, Goetz > Sent: Montag, 20. Mai 2019 17:36 > To: Doerr, Martin <[email protected]<mailto:[email protected]>>; > serviceability- > [email protected]<mailto:[email protected]>; JC Beyler > <[email protected]<mailto:[email protected]>>; Volker Simonis > ([email protected]<mailto:[email protected]>) > <[email protected]<mailto:[email protected]>> > Subject: RE: RFR(S): 8224230: [PPC64, s390] Support AsyncGetCallTrace > > Hi Martin, > > the changes look good. > > Please fix this comment: > // Forte Analyzer AsyncGetCallTrace profiling support is not implemented on > Linux/S390x. > I guess it's no more true. No new webrev needed. > > Did you verify that the test works, or did you also try > async with the VM on ppc? > > Best regards, > Goetz. > > > -----Original Message----- > > From: Doerr, Martin > > Sent: Montag, 20. Mai 2019 17:32 > > To: > > [email protected]<mailto:[email protected]>; > > JC Beyler <[email protected]<mailto:[email protected]>>; > > Volker Simonis ([email protected]<mailto:[email protected]>) > > <[email protected]<mailto:[email protected]>>; > > Lindenmaier, Goetz > > <[email protected]<mailto:[email protected]>> > > Subject: RFR(S): 8224230: [PPC64, s390] Support AsyncGetCallTrace > > > > Hi, > > > > > > > > please review my change which allows usage of AsyncGetCallTrace on PPC64 > > and s390. > > > > > > > > Webrev: > > > > > http://cr.openjdk.java.net/~mdoerr/8224230_ppc_s390_AsyncCallTrace/webr > > ev.00/ > > > > > > > > Bug with more background information: > > > > https://bugs.openjdk.java.net/browse/JDK-8224230 > > > > > > > > Best regards, > > > > Martin > > > > -- Thanks, Jc
