Skip to content

use SO_REUSEADDR on gdb stub socket#6101

Merged
OBattler merged 1 commit into
86Box:masterfrom
sherpya:gdbstub-reuseaddr
Aug 29, 2025
Merged

use SO_REUSEADDR on gdb stub socket#6101
OBattler merged 1 commit into
86Box:masterfrom
sherpya:gdbstub-reuseaddr

Conversation

@sherpya

@sherpya sherpya commented Aug 29, 2025

Copy link
Copy Markdown
Contributor

Summary

I'm trying to debug a real bios with gdb stub, I restart often and sometime 86Box doesn't wait for debugger.
Looks like the problem is missing SO_REUSEADDR on the socket.
Could you please check on other platform (it's only tested on Linux)

@Cacodemon345

Copy link
Copy Markdown
Contributor

You should ifdef out the part for Windows platforms; this should be fine otherwise.

@sherpya

sherpya commented Aug 29, 2025

Copy link
Copy Markdown
Contributor Author

You should ifdef out the part for Windows platforms; this should be fine otherwise.

windows has SO_REUSEADDR I think the difference is in the function signature (it needs const char * instead)

@sherpya

sherpya commented Aug 29, 2025

Copy link
Copy Markdown
Contributor Author

I've updated the code path for windows (also added SO_EXCLUSIVEADDRUSE), but I'm unable to trigger the action for msys2 in my fork

@OBattler

Copy link
Copy Markdown
Member

I approved the actions on your pull request now, so the one for Msys2 is going to be performed here now. Waiting for them all to finish now.

@OBattler
OBattler merged commit 5c6c8e7 into 86Box:master Aug 29, 2025
47 checks passed
@OBattler

Copy link
Copy Markdown
Member

It passed all checks, therefore, merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants