Opened 4 years ago

Closed 4 years ago

#11411 closed bug (fixed)

Debugger doesn't exit after saving report.

Reported by: pulkomandy Owned by: anevilyak
Priority: normal Milestone: R1
Component: Applications/Debugger Version: R1/Development
Keywords: Cc: jessicah
Blocked By: Blocking:
Has a Patch: no Platform: All


This seems to be reproductible by running the WebKit testsuite. debug_server is set to automatically save reports rather than prompting the user for what to do.

The report is saved but Debugger doesn't exit after doing so. After a while (about 22000 tests run) my system is out of memory because of all the leftover Debugger instances, and the testsuite stops.

Attaching the DumpRenderTree report that was saved (in case it matters) and matching debug report from the "dead" Debugger with all threads debugged.

Attachments (3) (18.4 KB) - added by pulkomandy 4 years ago. (21.4 KB) - added by pulkomandy 4 years ago.
debugger_instances.png (72.7 KB) - added by Giova84 4 years ago.

Download all attachments as: .zip

Change History (10)

Changed 4 years ago by pulkomandy

comment:1 Changed 4 years ago by anevilyak

Not quite sure what to make of this one right now. Generating the report completed successfully, and we're waiting for TeamDebugger to terminate the target team. However, that one appears to be waiting to acquire the team lock in order to save settings, which, based on the semaphore counts, in the report, should succeed. Will have to see if I can replicate this one. Out of interest, though it theoretically shouldn't matter, if you have debug_server set to prompt, and pick save manually, do the instances go away, or do they hang in that instance as well?

comment:2 Changed 4 years ago by axeld

A benaphore, maybe?

comment:3 in reply to:  2 Changed 4 years ago by anevilyak

Replying to axeld:

A benaphore, maybe?

It is a BLocker, so yes. By the looks of things it got as far as calling acquire_sem_etc() though, where I would expect the count of the associated semaphore to be -1 at this point, which it isn't. Am I wrong? (The semaphore in question is: 109500 0 16206 team lock )

Last edited 4 years ago by anevilyak (previous) (diff)

comment:4 Changed 4 years ago by Giova84

just for the record: it happens also for me. When I save the debug report for any application which crashes, the debugger won't exit, and will create multiple instances for any app which crashes. So I have to manually kill these instances using ProcessController. Look at the attached screenshot "debugger_instances.png"

Changed 4 years ago by Giova84

Attachment: debugger_instances.png added

comment:5 Changed 4 years ago by anevilyak

Status: newin-progress

I'm able to reproduce it, looking into it...

comment:6 Changed 4 years ago by jessicah

Cc: jessicah added

comment:7 Changed 4 years ago by anevilyak

Resolution: fixed
Status: in-progressclosed

Should be fixed in hrev48222.

Note: See TracTickets for help on using tickets.