On Mon, 13 Sep 2021 13:18:10 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
>> Masanori Yano has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains three additional >> commits since the last revision: >> >> - 8238274: (sctp) JDK-7118373 is not fixed for SctpChannel >> - Merge branch 'master' of https://github.com/masyano/jdk into 8238274 >> - 8238274: (sctp) JDK-7118373 is not fixed for SctpChannel > > test/jdk/com/sun/nio/sctp/SctpChannel/CloseDescriptors.java line 29: > >> 27: * @summary Potential leak file descriptor for SCTP >> 28: * @requires (os.family == "linux") >> 29: * @run main CloseDescriptors > > The test SctpMultiChannel/CloseDescriptors seems to work - at least it > doesn't fail all the time - but it's using `@run main/othervm`. I believe you > should make this test run in /othervm mode for more stability. I will test > again with your later change and /othervm mode. I pushed a fix to run the test in othervm mode. Please test again. ------------- PR: https://git.openjdk.java.net/jdk/pull/5274
