Skip to content

try to debug crash on CI#38

Closed
Guest0x0 wants to merge 49 commits intomainfrom
debug
Closed

try to debug crash on CI#38
Guest0x0 wants to merge 49 commits intomainfrom
debug

Conversation

@Guest0x0
Copy link
Collaborator

No description provided.

@coveralls
Copy link

coveralls commented Aug 15, 2025

Pull Request Test Coverage Report for Build 166

Details

  • 3 of 9 (33.33%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.5%) to 80.769%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/internal/event_loop/event_loop.mbt 3 9 33.33%
Totals Coverage Status
Change from base Build 162: -0.5%
Covered Lines: 693
Relevant Lines: 858

💛 - Coveralls

@Guest0x0 Guest0x0 marked this pull request as draft August 15, 2025 05:34
@Guest0x0 Guest0x0 force-pushed the debug branch 3 times, most recently from 882e6db to 0c68bd8 Compare August 15, 2025 07:50
@Guest0x0
Copy link
Collaborator Author

close via #40. The actual cause is a bug in MacOS's signal implementation, and there is no way to work around the bug. Now the runtime uses condition variable for thread wake up on MacOS. Interestingly, condition variable is also buggy on MacOS. Fortunately the condition variable bug can be worked around.

On Linux, the runtime still uses signal, for smaller overhead, and the condition variable implementation of glibc seems to be buggy too.

@Guest0x0 Guest0x0 closed this Aug 19, 2025
@Guest0x0 Guest0x0 deleted the debug branch September 1, 2025 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants