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