Re: [aarch64-port-dev ] RFR: 8209413: AArch64: NPE in clhsdb jstack command

2019-02-17 Thread Nick Gasson (Arm Technology China)
Hi Andrew, Please take a look at this updated patch: http://cr.openjdk.java.net/~ngasson/8209413/webrev.03/ It contains your suggested change below, as well as an extra check in the AARCH64Frame two-argument constructor that the PC from SP[-1] is valid. Also changed SharedRuntime::generate_nat

Re: [RFR]8215622: Add dump to file support for jmap histo

2019-02-17 Thread 臧琳
Dear All, Thanks a lot for your reviewing. Here I refined the patch to fix a minor bug that cause BasicJMapTest fail with jmap -histo:all. May I ask your help again to review it? http://cr.openjdk.java.net/~xiaofeya/8215622/webrev.09/ Thanks! Cheers, Lin