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

gopls: crashes in analysis with missing stacktrace #2847

Closed
findleyr opened this issue Jun 23, 2023 · 4 comments
Closed

gopls: crashes in analysis with missing stacktrace #2847

findleyr opened this issue Jun 23, 2023 · 4 comments

Comments

@findleyr
Copy link
Contributor

A recent change to gopls introduced a log.Fatal in the analysis driver when encountering an 'impossible state' (golang/go#60963). We've since discovered and fixed one such impossible state (golang/go#60952), but there may be others.

We believe this is the cause of the numerous blank crash reports in this issue tracker.

We're going to cut a new gopls release today containing a fix for the known failure mode, and downgrading this call to log.Fatal. Pinning this issue for visibility in the meantime. Sorry for the breakage.

If you encounter such a crash, we would greatly apprecate it if you could look in the gopls output panel and capture the fatal log message. This would help us determine if we have indeed fixed the root cause.
Screen Shot 2021-01-27 at 2 53 49 PM<!--

CC @hyangah @adonovan

@findleyr
Copy link
Contributor Author

We believe these crashes have been fixed by gopls@v0.12.4.

go install golang.org/x/tools/gopls@latest

However, I'll leave this issue open until we have confidence that the crash reports have resolved.

@sreerajsmenon01
Copy link

Crash_Gopls.txt
Please find the attachment

@findleyr
Copy link
Contributor Author

Thanks @sreerajsmenon01. That looks like an OOM:

runtime: out of memory: cannot allocate 4194304-byte block (1551564800 in use)
fatal error: out of memory

Notably the runtime is reporting 1.5G in-use. While that is high, it is not entirely unreasonable during an analysis pass. Are you working in a significantly memory-constrained environment?

@findleyr
Copy link
Contributor Author

Ok, closing this issue as we had no further crashes over the weekend.

@findleyr findleyr unpinned this issue Jun 26, 2023
@golang golang locked and limited conversation to collaborators Jun 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants