RFR: 8203290: [PPC64, s390] Check functionality of JDK-8199712 (Flight Recorder) [v10]
Tyler Steele
duke at openjdk.java.net
Mon Jan 24 16:55:09 UTC 2022
On Fri, 21 Jan 2022 09:28:27 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:
>> Tyler Steele has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Revert changes to jfrThreadSampler.cpp
>> - Update copyright years. Remove 'TODO' where no longer needed.
>
> Thanks for enhancing libperfstat_aix and making it consistent.
> I just noticed that JavaThread::pd_get_top_frame_for_profiling does not yet contain all the required checks. I think they should be taken from thread_linux_ppc.cpp (note that there were recent fixes).
Thanks for your comments @TheRealMDoerr. I agree that 131 files changed is too many to review; most of them are not files I actually changed, so I will do some digging to understand what happened there. My typical git workflow does not seem to be working well for this project.
Regarding the changes requested to `JavaThread::pd_get_top_frame_for_profiling`, is there an issue/pr for tracking the changes you are requesting?
-------------
PR: https://git.openjdk.java.net/jdk/pull/6885
More information about the build-dev
mailing list