Skip to content

Enable coredumps on Windows#47611

Closed
staticfloat wants to merge 2 commits into
masterfrom
sf/windows_coredumps
Closed

Enable coredumps on Windows#47611
staticfloat wants to merge 2 commits into
masterfrom
sf/windows_coredumps

Conversation

@staticfloat

Copy link
Copy Markdown
Member

This commit provides two pieces; first, it enables building libuv with SIGQUIT support on Windows (useful for [0]), and second it enables coredumping when exceptions are hit, after our exception handler has finished printing out a backtrace. This is useful for using WER to create dump files when we segfault, etc...

This commit requires libuv PRs [1] and [2] to properly function.

[0] #45864
[1] JuliaLang/libuv#30
[2] JuliaLang/libuv#31

This commit provides two pieces; first, it enables building libuv with
`SIGQUIT` support on Windows (useful for [0]), and second it enables
coredumping when exceptions are hit, after our exception handler has
finished printing out a backtrace.  This is useful for using WER to
create dump files when we segfault, etc...

This commit requires libuv PRs [1] and [2] to properly function.

[0] #45864
[1] JuliaLang/libuv#30
[2] JuliaLang/libuv#31
Comment thread Make.inc Outdated
HAVE_SSP := 1
OSLIBS += -Wl,--export-all-symbols -Wl,--version-script=$(JULIAHOME)/src/julia.expmap \
$(NO_WHOLE_ARCHIVE) -lpsapi -lkernel32 -lws2_32 -liphlpapi -lwinmm -ldbghelp -luserenv -lsecur32 -latomic
$(NO_WHOLE_ARCHIVE) -lpsapi -lkernel32 -lws2_32 -liphlpapi -lwinmm -ldbghelp -luserenv -lsecur32 -latomic -ldbghelp -lshell32 -lole32 -luuid

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be in the libuv update PR instead (also, dbghelp is in the list twice now)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I intend to update libuv within this PR once the two libuv PRs are merged.

@ViralBShah

Copy link
Copy Markdown
Member

Do we want to get this one merged?

@vtjnash

vtjnash commented Nov 22, 2023

Copy link
Copy Markdown
Member

Waiting on #49937

vtjnash added a commit that referenced this pull request Feb 10, 2024
Notable additions:
 - Linux now uses io_uring for many fs calls
 - Windows now supports WTF-8
 - new cross-platform uv_clock_gettime API
 - Windows now emits a coredump when sending SIGQUIT, if configured
 - Windows now can execute file given by an exact path name that do not contain a `.` (unlike cmd.exe, which cannot)

Closes #47611
Fixes #33486
@ViralBShah
ViralBShah deleted the sf/windows_coredumps branch February 10, 2024 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

system:windows Affects only Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants