On Wed, 22 Dec 2021 04:16:49 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> I don't have access to AIX, but there are 500+ tests in test/jdk/jdk/jfr you 
>> might want to try out.
>
> @egahlin  beat me to it - yes the JFR testing is primarily under jdk not 
> hotspot, so we don't really need the new sanity tests.
> 
> Also are you aware that we have `src/hotspot/os/aix/libperfstat_aix.*` ?

@dholmes-ora, That is a fair point. I implemented the functionality before I 
knew about the project library, but now that I know about it, the cleaner 
option would be to refactor my solution. I am expecting to make some changes to 
the code this week, so I will incorporate this one as well. Thanks for the 
suggestion.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6885

Reply via email to