RFR: 8327108: compiler.lib.ir_framework.shared.TestFrameworkSocket should listen on loopback address only

Christian Hagedorn chagedorn at openjdk.org
Fri Mar 1 13:08:52 UTC 2024


On Fri, 1 Mar 2024 12:50:58 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Can I please get a review of this test-only change which proposes to address https://bugs.openjdk.org/browse/JDK-8327108?
> 
> As noted in the JBS issue, before this proposed change, the internal test framework code in `compiler.lib.ir_framework.shared.TestFrameworkSocket` was binding a `java.net.ServerSocket` to "any address". This can lead to interference  from other hosts on the network, when the tests are run. The change here proposes to bind this `ServerSocket` to loopback address and reduce the chances of such interference.
> 
> Originally, the interference issues were noticed in CI when `tier3` was run. With the change proposed in this PR, I've run `tier1`, `tier2` and `tier3` in our CI environment and they all passed.

That looks reasonable, thanks for fixing this!

-------------

Marked as reviewed by chagedorn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18078#pullrequestreview-1911090301


More information about the hotspot-compiler-dev mailing list