RE: RFR: JDK-8163143: illegal bci error with interpreted frames in SA due to mirror being stored in interpreted frames

2016-08-11 Thread Jini Susan George
> >> -Original Message- > >> From: Dmitry Samersoff > >> Sent: Wednesday, August 10, 2016 1:42 PM > >> To: Jini Susan George; serviceability-dev@openjdk.java.net; Lindenmaier, > >> Goetz > >> Subject: Re: RFR: JDK-8163143: illegal

Re: RFR: JDK-8163143: illegal bci error with interpreted frames in SA due to mirror being stored in interpreted frames

2016-08-11 Thread Dmitry Samersoff
Jini Susan George; serviceability-dev@openjdk.java.net; Lindenmaier, >> Goetz >> Subject: Re: RFR: JDK-8163143: illegal bci error with interpreted frames in >> SA due to mirror being stored in interpreted frames >> >> Jini, >> >> Code changes looks good for me. &g

RE: RFR: JDK-8163143: illegal bci error with interpreted frames in SA due to mirror being stored in interpreted frames

2016-08-10 Thread Jini Susan George
/webrev.01/ Thanks, -Jini. > -Original Message- > From: Dmitry Samersoff > Sent: Wednesday, August 10, 2016 1:42 PM > To: Jini Susan George; serviceability-dev@openjdk.java.net; Lindenmaier, > Goetz > Subject: Re: RFR: JDK-8163143: illegal bci error with interpreted

Re: RFR: JDK-8163143: illegal bci error with interpreted frames in SA due to mirror being stored in interpreted frames

2016-08-10 Thread Dmitry Samersoff
Jini, Code changes looks good for me. CC'in Goetz for PPC changes. BasicLauncherTest.java: I'm not sure we need two extra parameters to launch() just to check for jstack Assertion Failure. Please either create a separate launchJStack method or just check exit code of jhsdb jstack. -Dmitry

RFR: JDK-8163143: illegal bci error with interpreted frames in SA due to mirror being stored in interpreted frames

2016-08-10 Thread Jini Susan George
Hi all, Please do review the changes for the SA defect: https://bugs.openjdk.java.net/browse/JDK-8163143 The webrev is at: http://cr.openjdk.java.net/~sballal/sponsorship/8163143/webrev.00/ Need testing with PPC64 for this. I have implemented the changes but have not tested