Re: RFR: 8294821: Class load improvement for AES crypto engine [v2]

2022-10-11 Thread Xue-Lei Andrew Fan
On Tue, 11 Oct 2022 00:05:05 GMT, Valerie Peng wrote: > Mach5 result looks ok. There is one unexpected test failure but it seems > unrelated > (https://mach5.us.oracle.com:10060/api/v1/results/vpeng-jdkOh-20221010-1957-37280778-open_test_lib-test-linux-x64-122-1665432715-16/log) > . So, it sho

Re: RFR: 8294821: Class load improvement for AES crypto engine [v2]

2022-10-10 Thread Valerie Peng
On Mon, 10 Oct 2022 17:43:52 GMT, Xue-Lei Andrew Fan wrote: >> Hi, >> >> May I have the code clean up reviewed? >> >> There is a lot of computation in AESCrypt class load, which could be avoid >> by using the computation result directly. The computation takes 6.971875 >> milliseconds in a Ma

Re: RFR: 8294821: Class load improvement for AES crypto engine [v2]

2022-10-10 Thread Valerie Peng
On Mon, 10 Oct 2022 19:05:45 GMT, Xue-Lei Andrew Fan wrote: > Anyone can help run Mach5 testing in case I missed something? Sure, I can help with that. - PR: https://git.openjdk.org/jdk/pull/10568

Re: RFR: 8294821: Class load improvement for AES crypto engine [v2]

2022-10-10 Thread Xue-Lei Andrew Fan
On Mon, 10 Oct 2022 17:43:52 GMT, Xue-Lei Andrew Fan wrote: >> Hi, >> >> May I have the code clean up reviewed? >> >> There is a lot of computation in AESCrypt class load, which could be avoid >> by using the computation result directly. The computation takes 6.971875 >> milliseconds in a Ma

Re: RFR: 8294821: Class load improvement for AES crypto engine [v2]

2022-10-10 Thread Valerie Peng
On Mon, 10 Oct 2022 17:43:52 GMT, Xue-Lei Andrew Fan wrote: >> Hi, >> >> May I have the code clean up reviewed? >> >> There is a lot of computation in AESCrypt class load, which could be avoid >> by using the computation result directly. The computation takes 6.971875 >> milliseconds in a Ma

Re: RFR: 8294821: Class load improvement for AES crypto engine [v2]

2022-10-10 Thread Valerie Peng
On Mon, 10 Oct 2022 17:43:52 GMT, Xue-Lei Andrew Fan wrote: >> Hi, >> >> May I have the code clean up reviewed? >> >> There is a lot of computation in AESCrypt class load, which could be avoid >> by using the computation result directly. The computation takes 6.971875 >> milliseconds in a Ma

Re: RFR: 8294821: Class load improvement for AES crypto engine [v2]

2022-10-10 Thread Xue-Lei Andrew Fan
> Hi, > > May I have the code clean up reviewed? > > There is a lot of computation in AESCrypt class load, which could be avoid by > using the computation result directly. The computation takes 6.971875 > milliseconds in a MacOS M1 laptop. Although it is a one-time computation, but > removing