Bug 213211 - REGRESSION (r263045): TestWTF.Signals.SignalsWorkOnExit crashing
Summary: REGRESSION (r263045): TestWTF.Signals.SignalsWorkOnExit crashing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Keith Miller
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-06-15 13:35 PDT by Ryan Haddad
Modified: 2020-06-15 15:24 PDT (History)
8 users (show)

See Also:


Attachments
crash log (54.41 KB, text/plain)
2020-06-15 13:35 PDT, Ryan Haddad
no flags Details
Patch (1.46 KB, patch)
2020-06-15 13:59 PDT, Keith Miller
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2020-06-15 13:35:05 PDT
TestWTF.Signals.SignalsWorkOnExit is crashing on macOS bots after https://trac.webkit.org/changeset/263045/webkit

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   TestWTF                             0x000000010ca03253 WTFCrashWithInfo(int, char const*, char const*, int) + 19 (Assertions.h:671)
1   TestWTF                             0x000000010ce365ee WTF::SignalHandlers::add(WTF::Signal, WTF::Function<WTF::SignalAction (WTF::Signal, WTF::SigInfo&, __darwin_x86_thread_state64&)>&&) + 206
2   TestWTF                             0x000000010ce36a9a WTF::addSignalHandler(WTF::Signal, WTF::Function<WTF::SignalAction (WTF::Signal, WTF::SigInfo&, __darwin_x86_thread_state64&)>&&) + 186
3   TestWTF                             0x000000010ccadf27 Signals_SignalsWorkOnExit_Test::TestBody() + 53 (Signals.cpp:43)
4   TestWTF                             0x000000010cea55d9 void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) + 72
5   TestWTF                             0x000000010cea5542 testing::Test::Run() + 184
6   TestWTF                             0x000000010cea619f testing::TestInfo::Run() + 199
7   TestWTF                             0x000000010cea6817 testing::TestCase::Run() + 249
8   TestWTF                             0x000000010ceaef83 testing::internal::UnitTestImpl::RunAllTests() + 667
9   TestWTF                             0x000000010ceaebd1 bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) + 72
10  TestWTF                             0x000000010ceaeb5c testing::UnitTest::Run() + 108
11  TestWTF                             0x000000010cd5fc82 TestWebKitAPI::TestsController::run(int, char**) + 120 (TestsController.cpp:91)
12  TestWTF                             0x000000010ce9bcc5 main + 400 (mainMac.mm:60)
13  libdyld.dylib                       0x00007fff6a282cc9 start + 1

https://results.webkit.org/?suite=api-tests&test=TestWTF.Signals.SignalsWorkOnExit
Comment 1 Radar WebKit Bug Importer 2020-06-15 13:35:41 PDT
<rdar://problem/64376951>
Comment 2 Ryan Haddad 2020-06-15 13:35:46 PDT
Created attachment 401932 [details]
crash log
Comment 3 Keith Miller 2020-06-15 13:59:12 PDT
Created attachment 401934 [details]
Patch
Comment 4 Mark Lam 2020-06-15 14:06:24 PDT
Comment on attachment 401934 [details]
Patch

r+ to unbreak the API test using Signal::Usr.  However, but looks like the only place where we add a signal handler for Signal::Usr is the API tests.  Can we just make the test use a different signal and get rid of Signal::Usr altogether.  Let's do that in a separate bug though.
Comment 5 Keith Miller 2020-06-15 15:05:31 PDT
(In reply to Mark Lam from comment #4)
> Comment on attachment 401934 [details]
> Patch
> 
> r+ to unbreak the API test using Signal::Usr.  However, but looks like the
> only place where we add a signal handler for Signal::Usr is the API tests. 
> Can we just make the test use a different signal and get rid of Signal::Usr
> altogether.  Let's do that in a separate bug though.

Yeah, it does look unused now. I think we had it because some ports used to use it as their thread suspension method.
Comment 6 EWS 2020-06-15 15:24:35 PDT
Committed r263065: <https://trac.webkit.org/changeset/263065>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 401934 [details].