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

ClangdTests.exe/TUSchedulerTests.ManyUpdates flaky on windows #50117

Closed
nico opened this issue Jun 19, 2021 · 6 comments
Closed

ClangdTests.exe/TUSchedulerTests.ManyUpdates flaky on windows #50117

nico opened this issue Jun 19, 2021 · 6 comments
Labels

Comments

@nico
Copy link
Contributor

nico commented Jun 19, 2021

Bugzilla Link 50773
Version unspecified
OS All
CC @sam-mccall

Extended Description

https://lab.llvm.org/buildbot/#/builders/123/builds/4487/steps/12/logs/stdio :

Reusing preamble version 0 for version 5 of C:\clangd-test\foo2.cpp
Assertion failed: Requests.empty() && "No new normal tasks can be scheduled concurrently with " "blockUntilIdle(): ASTWorker isn't threadsafe", file C:\b\slave\clang-x64-windows-msvc\llvm-project\clang-tools-extra\clangd\TUScheduler.cpp, line 1390
********************
Testing:  0.. 10.. 20.. 30.. 40.. 50.. 60.. 70.. 80.. 90.. 
********************
Failed Tests (1):
  Clangd Unit Tests :: ./ClangdTests.exe/TUSchedulerTests.ManyUpdates

Same on https://lab.llvm.org/buildbot/#/builders/123/builds/4485/steps/12/logs/stdio

Same on https://lab.llvm.org/buildbot/#/builders/123/builds/4481/steps/12/logs/stdio

Same on https://lab.llvm.org/buildbot/#/builders/123/builds/4564/steps/12/logs/stdio

From a random sample, it looks like the majority of failures on https://lab.llvm.org/buildbot/#/builders/123 is due to this.

@nico
Copy link
Contributor Author

nico commented Jun 19, 2021

(#48455 is about a hang in this test suite -- looks like this code has a concurrency bug somewhere)

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
@nico
Copy link
Contributor Author

nico commented Jun 26, 2023

@nico
Copy link
Contributor Author

nico commented Jun 26, 2023

@ilya-biryukov added this test over 5 years ago in https://reviews.llvm.org/D42174.

@HighCommander4
Copy link
Collaborator

See also clangd/clangd#1712

@kadircet
Copy link
Member

Hi all, sorry for missing those.

Hi @nico, I was checking the failures in the past week, and they don't seem to be from clangd. Is this still relevant for the build bot you're running?

@kadircet
Copy link
Member

kadircet commented Sep 5, 2023

closing as this doesn't seem to be showing up anymore on the relevant build bots at least, and despite the original bug being about a correctness issue, the recent failures are all timeouts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants