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

Vladimir Kozlov kvn at openjdk.org
Fri Mar 1 17:26: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.

Good.

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

Marked as reviewed by kvn (Reviewer).

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


More information about the hotspot-compiler-dev mailing list