llvm-project
[lldb] Script determinism of TestSwiftActorUnprioritisedJobs
#10435
Open

[lldb] Script determinism of TestSwiftActorUnprioritisedJobs #10435

kastiglione
kastiglione44 days ago
No description provided.
kastiglione kastiglione requested a review 44 days ago
kastiglione
kastiglione44 days ago

another attempt at #10417

kastiglione
kastiglione43 days ago

@swift-ci test macOS

artemcm
artemcm40 days ago

@swift-ci test Linux platform

artemcm
artemcm40 days ago

@swift-ci test Windows platform

kastiglione [lldb] Make TestSwiftActorUnprioritisedJobs deterministic
0335696f
kastiglione kastiglione force pushed from 12f13135 to 0335696f 33 days ago
kastiglione
kastiglione33 days ago
kastiglione
kastiglione33 days ago

Resuming work on this PR

kastiglione Run and show `bt all`
88ecc146
kastiglione
kastiglione33 days ago

@swift-ci test linux

kastiglione Attempt forward progress of other threads
1bb5bc77
kastiglione
kastiglione32 days ago

@swift-ci test linux

kastiglione Add missing thread.Suspend()
e8537a7e
kastiglione
kastiglione32 days ago

@swift-ci test linux

kastiglione Unconditionally run other threads
f7f225f7
kastiglione
kastiglione32 days ago
kastiglione
kastiglione32 days ago

@swift-ci test linux

kastiglione
kastiglione32 days ago

@swift-ci test macOS

kastiglione
kastiglione32 days ago
kastiglione
kastiglione32 days ago

@swift-ci test linux

kastiglione
kastiglione32 days ago

@swift-ci test macOS

kastiglione kastiglione requested a review from adrian-prantl adrian-prantl 32 days ago
kastiglione
kastiglione32 days ago

So far, tests are passing consistently.

kastiglione
kastiglione32 days ago

@swift-ci test linux

kastiglione
kastiglione31 days ago
kastiglione
kastiglione31 days ago

@swift-ci test macOS

kastiglione
kastiglione31 days ago

@swift-ci test linux

kastiglione
kastiglione31 days ago

@swift-ci test linux

kastiglione Increase sleep wait
56e53072
kastiglione
kastiglione31 days ago

@swift-ci test linux

kastiglione
kastiglione31 days ago

@swift-ci test macOS

kastiglione
kastiglione31 days ago (edited 27 days ago)

New set of logs after increasing sleep wait time from 2s to 5s.

macOS CI Logs:

Linux CI Logs:

kastiglione
kastiglione31 days ago

@swift-ci test linux

kastiglione
kastiglione30 days ago

@swift-ci test macOS

kastiglione
kastiglione30 days ago

@swift-ci test linux

kastiglione
kastiglione30 days ago

@swift-ci test macOS

kastiglione
adrian-prantl
adrian-prantl commented on 2025-04-19
lldb/test/API/lang/swift/async/actors/unprioritised_jobs/TestSwiftActorUnprioritisedJobs.py
33 self.dbg.SetAsync(True)
34 process.Continue()
35 # Wait - allowing other threads to run.
36
time.sleep(5)
adrian-prantl30 days ago👍 1

We usually do something like this (copied from another test)

DEFAULT_TIMEOUT = 20 * (10 if ("ASAN_OPTIONS" in os.environ) else 1)

to guard against even slower as an bots.

adrian-prantl24 days ago

Even better would be to wait for something the target actually does, like reading from stdin or creating a file.

kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione
kastiglione Add actor call to hopefully initialize swift concurrency threads
f9b00d3b
kastiglione
kastiglione New algorithm to ensure actor is sufficiently setup
4e5a1a39
kastiglione
kastiglione
kastiglione Add IsValid asserts
3d9f2180
kastiglione
kastiglione New approach did not work, restore previous
52f61f18
kastiglione
kastiglione
kastiglione
kastiglione

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
Milestone