RFR: 8351491: Add info from release file to hserr file [v4]
David Holmes
dholmes at openjdk.org
Tue Apr 8 08:03:25 UTC 2025
On Tue, 8 Apr 2025 07:58:02 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>>
>> use stringStream in os.cpp, add a couple of changes suggested by David
>
> src/hotspot/share/runtime/os.cpp line 1538:
>
>> 1536: ss.print("%s/release", home);
>> 1537:
>> 1538: if (_image_release_file_content == nullptr) {
>
> Shouldn't we check before doing anything else i.e. make line 1538 first.
>
> Also for race concerns use load_acquire on `image_release_file_content`.
Though actually shouldn't this just be an assert as we only expect this to be called once.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24244#discussion_r2032626089
More information about the hotspot-dev
mailing list