Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[23.0] Windows build needs to account for new extended socket options in JDK 17.0.10 #611

Closed
jerboaa opened this issue Nov 9, 2023 · 1 comment · Fixed by #615
Closed

[23.0] Windows build needs to account for new extended socket options in JDK 17.0.10 #611

jerboaa opened this issue Nov 9, 2023 · 1 comment · Fixed by #615
Labels
bug Something isn't working
Milestone
23.0.3.0-Final

Comments

@jerboaa
Copy link
Collaborator

jerboaa commented Nov 9, 2023

In CI we see this failure:

    Creating library C:\Users\RUNNER~1\AppData\Local\Temp\SVM-18141433008043533303\quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-runner.lib and object C:\Users\RUNNER~1\AppData\Local\Temp\SVM-18141433008043533303\quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-runner.exp
quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-runner.obj : error LNK2001: unresolved external symbol Java_jdk_net_WindowsSocketOptions_getTcpKeepAliveIntvl0
quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-runner.obj : error LNK2001: unresolved external symbol Java_jdk_net_WindowsSocketOptions_getTcpKeepAliveProbes0
quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-runner.obj : error LNK2001: unresolved external symbol Java_jdk_net_WindowsSocketOptions_getTcpKeepAliveTime0
quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-runner.obj : error LNK2001: unresolved external symbol Java_jdk_net_WindowsSocketOptions_setTcpKeepAliveIntvl0
quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-runner.obj : error LNK2001: unresolved external symbol Java_jdk_net_WindowsSocketOptions_setTcpKeepAliveProbes0
quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-runner.obj : error LNK2001: unresolved external symbol Java_jdk_net_WindowsSocketOptions_setTcpKeepAliveTime0
D:\a\mandrel\mandrel\quarkus\integration-tests\resteasy-jackson\target\quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-native-image-source-jar\quarkus-integration-test-resteasy-jackson-999-SNAPSHOT-runner.exe : fatal error LNK1120: 6 unresolved externals
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.image.NativeImageViaCC.handleLinkerFailure(NativeImageViaCC.java:204)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.image.NativeImageViaCC.runLinkerCommand(NativeImageViaCC.java:151)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.image.NativeImageViaCC.write(NativeImageViaCC.java:117)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGenerator.doRun(NativeImageGenerator.java:724)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGenerator.run(NativeImageGenerator.java:539)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGeneratorRunner.buildImage(NativeImageGeneratorRunner.java:408)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGeneratorRunner.build(NativeImageGeneratorRunner.java:612)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGeneratorRunner.start(NativeImageGeneratorRunner.java:134)
	at org.graalvm.nativeimage.builder/com.oracle.svm.hosted.NativeImageGeneratorRunner.main(NativeImageGeneratorRunner.java:94)

See: https://github.com/graalvm/mandrel/actions/runs/6810306032/job/18519936281?pr=609#step:13:2725

This is likely due to the new backport in JDK 17.0.10 adding new extended socket options for Windows. JDK bug:
https://bugs.openjdk.org/browse/JDK-8308593

Since the Linux code already had support for this, whatever is done for Linux in GraalVM will now likely also apply to Windows.
@jerboaa jerboaa added the bug Something isn't working label Nov 9, 2023
@jerboaa jerboaa added this to the 23.0.3.0-Final milestone Nov 9, 2023
@jerboaa jerboaa mentioned this issue Nov 10, 2023
Merged
@jerboaa jerboaa linked a pull request Nov 10, 2023 that will close this issue
Add support for Windows extended socket options #615
Merged
zakkak pushed a commit that referenced this issue Nov 10, 2023
@jerboaa @zakkak
Add support for Windows extended socket options
adfcdc7 
This feature has been backported to JDK 17.0.10 and the GraalVM code
needs to account for it.

Closes: #611
@zakkak
Copy link
Collaborator

zakkak commented Nov 10, 2023

Closed via #615

@zakkak zakkak closed this as completed Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
23.0.3.0-Final
Development

Successfully merging a pull request may close this issue.

Add support for Windows extended socket options jerboaa/graal
2 participants
@jerboaa @zakkak