RFR: 8362087: Test containers/docker/ShareTmpDir.java intermittent fails [v2]
SendaoYan
syan at openjdk.org
Sun Aug 3 12:43:43 UTC 2025
On Sat, 2 Aug 2025 15:51:07 GMT, Francesco Andreuzzi <duke at openjdk.org> wrote:
>> SendaoYan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Add synchronized lock for addClassOptions
>
> test/hotspot/jtreg/containers/docker/ShareTmpDir.java line 86:
>
>> 84: Thread t1 = new Thread() {
>> 85: public void run() {
>> 86: opts.addClassOptions("1");
>
> Is `addClassOptions` thread safe? I had a quick look and it didn't seem so
Thanks for the reviews. I have add synchronized for both `opts.addClassOptions`
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26605#discussion_r2249959172
More information about the hotspot-runtime-dev
mailing list