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.

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

Add notes for testing on windows #3658

Merged
merged 2 commits into from
May 21, 2024
Merged

Add notes for testing on windows #3658

merged 2 commits into from
May 21, 2024

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented May 19, 2024

No description provided.

@zanieb zanieb added documentation Improvements or additions to documentation windows Specific to the Windows platform labels May 19, 2024
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@zanieb zanieb enabled auto-merge (squash) May 20, 2024 23:18
CONTRIBUTING.md Outdated Show resolved Hide resolved
@konstin
Copy link
Member

konstin commented May 21, 2024

For anyone who comes looking at this in future: The cause is that nested futures in debug mode are massive, while they get compiled down to a reasonable size in release mode. You can debug this with some variation of (unix only). Install from https://github.com/loyd/top-type-sizes:

RUSTFLAGS=-Zprint-type-sizes cargo +nightly build -p uv -j 1 > type-sizes.txt && top-type-sizes -w -s -h 10 < type-sizes.txt > sizes.txt

You can debug the overflowing stack with lldb using https://forums.swift.org/t/getting-stack-size-using-lldb/63207:

settings set frame-format "frame #${frame.index}: sp=${frame.sp} fp=${frame.fp} pc=${frame.pc}{ ${module.file.basename}{`${function.name-with-args}${function.pc-offset}}}{ at ${line.file.basename}:${line.number}}{${function.is-optimized} [opt]}\n"
thread backtrace
session save backtrace.txt

Feed this to https://gist.github.com/konstin/448492d131bd346f2d387257c47fa312 to get the stack frame sizes.

@zanieb zanieb merged commit acde048 into main May 21, 2024
43 checks passed
@zanieb zanieb deleted the zb/contrib-win branch May 21, 2024 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation windows Specific to the Windows platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants