Review Request 121286: Adding support for lldb in DrKonqi (step 1)

René J.V. Bertin rjvbertin at gmail.com
Sun Nov 30 19:50:52 GMT 2014



> On Nov. 29, 2014, 11:20 p.m., Pino Toscano wrote:
> > drkonqi/drkonqibackends.cpp, line 175
> > <https://git.reviewboard.kde.org/r/121286/diff/3/?file=331101#file331101line175>
> >
> >     This macros won't be defined outside Mac, so please check for its existency as well.

On the compilers I have experience with this evaluates to false f the token isn't defined...


> On Nov. 29, 2014, 11:20 p.m., Pino Toscano wrote:
> > drkonqi/parser/backtraceparser.cpp, line 204
> > <https://git.reviewboard.kde.org/r/121286/diff/3/?file=331103#file331103line204>
> >
> >     Most probably not needed?

No, not needed. But as the complexity of this code grows, so does the computational cost if we do all checks, even those that are clearly irrelevant or even in inappropriate (non MS Windows software might well have reasons to define a function called WinMain).
Hence  my suggestion above to group the checks per platform/OS. That could be done in a single if statement, but it could of course also be

```
#ifdef Q_OS_MAC
    if (/*...*/) {
        return true;
    }
#elif defined(Q_OS_WIN)
    if (/*...*/) {
        return true;
    }
#elif defined(Q_OS_WHATEVER)
// ...
#else
    // standard Unix/Linux
    if (/*...*/) {
        return true;
    }
#endif
    return false;
````


> On Nov. 29, 2014, 11:20 p.m., Pino Toscano wrote:
> > drkonqi/parser/backtraceparser.cpp, line 206
> > <https://git.reviewboard.kde.org/r/121286/diff/3/?file=331103#file331103line206>
> >
> >     This does not need a regexp.

doh ...


- René J.V.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121286/#review71100
-----------------------------------------------------------


On Nov. 29, 2014, 11:03 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121286/
> -----------------------------------------------------------
> 
> (Updated Nov. 29, 2014, 11:03 p.m.)
> 
> 
> Review request for KDE Software on Mac OS X and KDE Runtime.
> 
> 
> Repository: kde-runtime
> 
> 
> Description
> -------
> 
> DrKonqi currently lacks support for lldb, which means KDE users on recent OS X versions cannot generate and submit post-mortem backtraces.
> 
> The patches in this RR introduce simple logic (based on *compile-time* OS version detection) to select either gdb or lldb, as well as appropriate lldbrc files.
> 
> This is the first step to be taken: determine when lldb should be launched, and how (to obtain a backtrace).
> 
> 
> Diffs
> -----
> 
>   drkonqi/backtracegenerator.cpp 1107e11 
>   drkonqi/data/debuggers/external/lldbrc PRE-CREATION 
>   drkonqi/data/debuggers/internal/lldbrc PRE-CREATION 
>   drkonqi/drkonqibackends.cpp 064d07d 
>   drkonqi/parser/CMakeLists.txt d08d0d7 
>   drkonqi/parser/backtraceparser.cpp 7f62c97 
>   drkonqi/parser/backtraceparserlldb.h PRE-CREATION 
>   drkonqi/parser/backtraceparserlldb.cpp PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/121286/diff/
> 
> 
> Testing
> -------
> 
> On OS X 10.9.4 with kdelibs git/4.14 .
> Launching lldb works, as does the BatchCommand to obtain a backtrace; parsing of that information will be tackled later.
> The backtrace isn't particularly useful though, because it doesn't (always/never/...?) display the location of the crash and steps leading up to it, *presumably* because of an issue in the interaction between KDE's crash reporter and lldb. This will need work...
> 
> ```
> Application: Kate (kate), signal: Segmentation fault: 11
> (lldb) process attach --pid 88853
> Process 88853 stopped
> Executable module set to "/opt/local/bin/kate".
> Architecture set to: x86_64-apple-macosx.
> (lldb) command source -s 0 '/private/var/folders/j1/1439ppj08xj8h6006s6drbq00000gs/T/kde-bertin/drkonqiB88857.tmp'
> Executing commands in '/private/var/folders/j1/1439ppj08xj8h6006s6drbq00000gs/T/kde-bertin/drkonqiB88857.tmp'.
> (lldb) set set term-width 200
> (lldb) set set interpreter.prompt-on-quit false
> (lldb) thread info
> thread #1: tid = 0x1bda48, 0x00007fff8cb85e20 libsystem_kernel.dylib`__wait4 + 8, queue = 'com.apple.main-thread', stop reason = signal SIGSTOP
> 
> (lldb) bt all
> * thread #1: tid = 0x1bda48, 0x00007fff8cb85e20 libsystem_kernel.dylib`__wait4 + 8, queue = 'com.apple.main-thread', stop reason = signal SIGSTOP
>   * frame #0: 0x00007fff8cb85e20 libsystem_kernel.dylib`__wait4 + 8
>     frame #1: 0x000000010272bc8e libkdeui.5.dylib`KCrash::startProcess(int, char const**, bool) [inlined] startProcessInternal(argc=<unavailable>, directly=<unavailable>) + 265 at kcrash.cpp:556
>     frame #2: 0x000000010272bb85 libkdeui.5.dylib`KCrash::startProcess(argc=<unavailable>, argv=<unavailable>, waitAndExit=<unavailable>) + 21 at kcrash.cpp:538
>     frame #3: 0x000000010272adb9 libkdeui.5.dylib`KCrash::defaultCrashHandler(sig=<unavailable>) + 1209 at kcrash.cpp:441
>     frame #4: 0x00007fff8fe965aa libsystem_platform.dylib`_sigtramp + 26
>     frame #5: 0x00007fff8a55c098 libobjc.A.dylib`objc_msgSend + 24
> 
>   thread #2: tid = 0x1bda4b, 0x00007fff8cb86662 libsystem_kernel.dylib`kevent64 + 10, queue = 'com.apple.libdispatch-manager'
>     frame #0: 0x00007fff8cb86662 libsystem_kernel.dylib`kevent64 + 10
>     frame #1: 0x00007fff905a1421 libdispatch.dylib`_dispatch_mgr_invoke + 239
>     frame #2: 0x00007fff905a1136 libdispatch.dylib`_dispatch_mgr_thread + 52
> 
>   thread #3: tid = 0x1bda4c, 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #0: 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #1: 0x00007fff8f5d6f08 libsystem_pthread.dylib`_pthread_wqthread + 330
>     frame #2: 0x00007fff8f5d9fb9 libsystem_pthread.dylib`start_wqthread + 13
> 
>   thread #4: tid = 0x1bda52, 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #0: 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #1: 0x00007fff8f5d6f08 libsystem_pthread.dylib`_pthread_wqthread + 330
>     frame #2: 0x00007fff8f5d9fb9 libsystem_pthread.dylib`start_wqthread + 13
> 
>   thread #5: tid = 0x1bda75, 0x00007fff8cb859aa libsystem_kernel.dylib`__select + 10, name = 'com.apple.CFSocket.private'
>     frame #0: 0x00007fff8cb859aa libsystem_kernel.dylib`__select + 10
>     frame #1: 0x00007fff8b12fa03 CoreFoundation`__CFSocketManager + 867
>     frame #2: 0x00007fff8f5d5899 libsystem_pthread.dylib`_pthread_body + 138
>     frame #3: 0x00007fff8f5d572a libsystem_pthread.dylib`_pthread_start + 137
>     frame #4: 0x00007fff8f5d9fc9 libsystem_pthread.dylib`thread_start + 13
> 
>   thread #6: tid = 0x1bda83, 0x00007fff8cb81a1a libsystem_kernel.dylib`mach_msg_trap + 10
>     frame #0: 0x00007fff8cb81a1a libsystem_kernel.dylib`mach_msg_trap + 10
>     frame #1: 0x00007fff8cb80d18 libsystem_kernel.dylib`mach_msg + 64
>     frame #2: 0x00007fff8b0e3f15 CoreFoundation`__CFRunLoopServiceMachPort + 181
>     frame #3: 0x00007fff8b0e3539 CoreFoundation`__CFRunLoopRun + 1161
>     frame #4: 0x00007fff8b0e2e75 CoreFoundation`CFRunLoopRunSpecific + 309
>     frame #5: 0x00007fff8e30d05e AppKit`_NSEventThread + 144
>     frame #6: 0x00007fff8f5d5899 libsystem_pthread.dylib`_pthread_body + 138
>     frame #7: 0x00007fff8f5d572a libsystem_pthread.dylib`_pthread_start + 137
>     frame #8: 0x00007fff8f5d9fc9 libsystem_pthread.dylib`thread_start + 13
> 
>   thread #7: tid = 0x1bdcba, 0x00007fff8cb85a3a libsystem_kernel.dylib`__semwait_signal + 10, name = 'com.apple.appkit-heartbeat'
>     frame #0: 0x00007fff8cb85a3a libsystem_kernel.dylib`__semwait_signal + 10
>     frame #1: 0x00007fff88fbcdc0 libsystem_c.dylib`nanosleep + 200
>     frame #2: 0x00007fff88fbccb2 libsystem_c.dylib`usleep + 54
>     frame #3: 0x00007fff8e3d117d AppKit`-[NSUIHeartBeat _heartBeatThread:] + 2132
>     frame #4: 0x00007fff9368776b Foundation`__NSThread__main__ + 1318
>     frame #5: 0x00007fff8f5d5899 libsystem_pthread.dylib`_pthread_body + 138
>     frame #6: 0x00007fff8f5d572a libsystem_pthread.dylib`_pthread_start + 137
>     frame #7: 0x00007fff8f5d9fc9 libsystem_pthread.dylib`thread_start + 13
> (lldb) detach
> Process 88853 detached
> (lldb) quit
> ```
> 
> Same crash but backtrace generated after starting the app in the debugger (instead of attaching to it):
> 
> ```
> * thread #1: tid = 0x1be8ca, 0x0000000101283430 QtGui`qt_mac_create_imagemask(pixmap=<unavailable>, sr=0x00007fff5fbfcbf0) + 752 at qpixmap_mac.cpp:1090, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=EXC_I386_GPFLT)
>     frame #0: 0x0000000101283430 QtGui`qt_mac_create_imagemask(pixmap=<unavailable>, sr=0x00007fff5fbfcbf0) + 752 at qpixmap_mac.cpp:1090
>    1087     for(int y = sy, offset=0; y < sh; ++y) {
>    1088         srow = sptr + (y * (sbpr / 4));
>    1089         for(int x = sx; x < sw; ++x)
> -> 1090             *(dptr+(offset++)) = (*(srow+x) & mask) ? 255 : 0;
>    1091     }
>    1092     QCFType<CGDataProviderRef> provider = CGDataProviderCreateWithData(0, dptr, nbytes, qt_mac_cgimage_data_free);
>    1093     px->cg_mask = CGImageMaskCreate(sw, sh, 8, 8, nbytes / sh, provider, 0, 0);
> [snip]
> (lldb) bt all
> * thread #1: tid = 0x1be8ca, 0x0000000101283430 QtGui`qt_mac_create_imagemask(pixmap=<unavailable>, sr=0x00007fff5fbfcbf0) + 752 at qpixmap_mac.cpp:1090, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=EXC_I386_GPFLT)
>   * frame #0: 0x0000000101283430 QtGui`qt_mac_create_imagemask(pixmap=<unavailable>, sr=0x00007fff5fbfcbf0) + 752 at qpixmap_mac.cpp:1090
>     frame #1: 0x000000010139dafb QtGui`qt_mac_draw_pattern(info=0x0000000105089e10, c=0x000000011232fd30) + 907 at qpaintengine_mac.cpp:494
>     frame #2: 0x00007fff93e1ca6f CoreGraphics`CGPatternDelegateDrawPattern + 66
>     frame #3: 0x00000001126d475a libPDFRIP.A.dylib`PDFPatternEmitDefinition + 1299
>     frame #4: 0x00000001126d49c6 libPDFRIP.A.dylib`emitPatternDefinition + 14
>     frame #5: 0x00007fff8b0d1382 CoreFoundation`__CFSetApplyFunction_block_invoke + 18
>     frame #6: 0x00007fff8b0b58fc CoreFoundation`CFBasicHashApply + 124
>     frame #7: 0x00007fff8b0d133d CoreFoundation`CFSetApplyFunction + 173
>     frame #8: 0x00000001126d4993 libPDFRIP.A.dylib`PDFPatternSetEmitDefinitions + 71
>     frame #9: 0x00000001126caa0c libPDFRIP.A.dylib`emit_page_resources(PDFDocument*) + 59
>     frame #10: 0x00000001126ca9b0 libPDFRIP.A.dylib`PDFDocumentEndPage + 73
>     frame #11: 0x00000001126c9570 libPDFRIP.A.dylib`pdf_EndPage + 17
>     frame #12: 0x00007fff8f7e7ea3 PrintCore`pdfSpoolingEndPage(void*, void*) + 345
>     frame #13: 0x00007fff8f7df835 PrintCore`PJCEndPage(OpaquePMPrintSession*) + 40
>     frame #14: 0x00007fff8f7bd3b6 PrintCore`PMSessionEndPageNoDialog + 80
>     frame #15: 0x00000001011812c7 QtGui`QMacPrintEngine::end() + 103
>     frame #16: 0x00000001012dad0e QtGui`QPainter::end(this=<unavailable>) + 142 at qpainter.cpp:1963
>     frame #17: 0x000000010d841159 libkatepartinterfaces.4.dylib`KatePrinter::print(doc=<unavailable>) + 17097 at kateprinter.cpp:669
>     frame #18: 0x000000010d69533a libkatepartinterfaces.4.dylib`KateDocument::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) [inlined] KateDocument::print() + 1994 at katedocument.cpp:1855
>     frame #19: 0x000000010d695332 libkatepartinterfaces.4.dylib`KateDocument::qt_static_metacall(_o=0x000000010508be40, _c=<unavailable>, _id=<unavailable>, _a=0x00007fff5fbfdd40) + 1986 at katedocument.moc:267
>     frame #20: 0x000000010276b3fd QtCore`QMetaObject::activate(sender=0x00000001036d1b20, m=<unavailable>, local_signal_index=<unavailable>, argv=<unavailable>) + 1693 at qobject.cpp:3567
>     frame #21: 0x00000001011bd419 QtGui`QAction::activate(QAction::ActionEvent) [inlined] QAction::triggered(this=0x00000001036d1b20, _t1=false) + 233 at moc_qaction.cpp:277
>     frame #22: 0x00000001011bd3f1 QtGui`QAction::activate(this=0x00000001036d1b20, event=<unavailable>) + 193 at qaction.cpp:1257
>     frame #23: 0x0000000101171c91 QtGui`-[QCocoaMenuLoader qtDispatcherToQAction:] + 65
>     frame #24: 0x00007fff8e38b260 AppKit`-[NSApplication sendAction:to:from:] + 327
>     frame #25: 0x00007fff8e3a61c8 AppKit`-[NSMenuItem _corePerformAction] + 394
>     frame #26: 0x00007fff8e3a5f04 AppKit`-[NSCarbonMenuImpl performActionWithHighlightingForItemAtIndex:] + 117
>     frame #27: 0x00007fff8e3a507c AppKit`-[NSMenu performKeyEquivalent:] + 289
>     frame #28: 0x00007fff8e3a18e1 AppKit`-[NSApplication _handleKeyEquivalent:] + 822
>     frame #29: 0x00007fff8e310acc AppKit`-[NSApplication sendEvent:] + 3293
>     frame #30: 0x00000001011720fe QtGui`-[QNSApplication sendEvent:] + 78
>     frame #31: 0x00007fff8e1609f9 AppKit`-[NSApplication run] + 646
>     frame #32: 0x000000010117aba0 QtGui`QEventDispatcherMac::processEvents(QFlags<QEventLoop::ProcessEventsFlag>) + 528
>     frame #33: 0x00000001027508ad QtCore`QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) [inlined] QFlags(this=0x00007fff00000024) + 9 at qglobal.h:2319
>     frame #34: 0x00000001027508a4 QtCore`QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) [inlined] QFlags(this=0x00007fff00000024) at qglobal.h:2319
>     frame #35: 0x00000001027508a4 QtCore`QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) [inlined] QFlags<QEventLoop::ProcessEventsFlag>::operator|(this=<unavailable>, f=EventLoopExec) const + 59 at qeventloop.cpp:149
>     frame #36: 0x0000000102750869 QtCore`QEventLoop::exec(this=0x00007fff5fbfe500, flags=(i = 0)) + 409 at qeventloop.cpp:204
>     frame #37: 0x0000000102753ac7 QtCore`QCoreApplication::exec() + 199 at qcoreapplication.cpp:1225
>     frame #38: 0x0000000100011693 libkdeinit4_kate.dylib`kdemain(argc=<unavailable>, argv=<unavailable>) + 13507 at katemain.cpp:381
>     frame #39: 0x00007fff8d6075fd libdyld.dylib`start + 1
>     frame #40: 0x00007fff8d6075fd libdyld.dylib`start + 1
> 
>   thread #2: tid = 0x1be8ee, 0x00007fff8cb86662 libsystem_kernel.dylib`kevent64 + 10, queue = 'com.apple.libdispatch-manager'
>     frame #0: 0x00007fff8cb86662 libsystem_kernel.dylib`kevent64 + 10
>     frame #1: 0x00007fff905a1421 libdispatch.dylib`_dispatch_mgr_invoke + 239
>     frame #2: 0x00007fff905a1136 libdispatch.dylib`_dispatch_mgr_thread + 52
> 
>   thread #3: tid = 0x1be8ef, 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #0: 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #1: 0x00007fff8f5d6f08 libsystem_pthread.dylib`_pthread_wqthread + 330
>     frame #2: 0x00007fff8f5d9fb9 libsystem_pthread.dylib`start_wqthread + 13
> 
>   thread #4: tid = 0x1be8f0, 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #0: 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #1: 0x00007fff8f5d6f08 libsystem_pthread.dylib`_pthread_wqthread + 330
>     frame #2: 0x00007fff8f5d9fb9 libsystem_pthread.dylib`start_wqthread + 13
> 
>   thread #5: tid = 0x1be8fe, 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #0: 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #1: 0x00007fff8f5d6f08 libsystem_pthread.dylib`_pthread_wqthread + 330
>     frame #2: 0x00007fff8f5d9fb9 libsystem_pthread.dylib`start_wqthread + 13
> 
>   thread #6: tid = 0x1be909, 0x00007fff8cb859aa libsystem_kernel.dylib`__select + 10, name = 'com.apple.CFSocket.private'
>     frame #0: 0x00007fff8cb859aa libsystem_kernel.dylib`__select + 10
>     frame #1: 0x00007fff8b12fa03 CoreFoundation`__CFSocketManager + 867
>     frame #2: 0x00007fff8f5d5899 libsystem_pthread.dylib`_pthread_body + 138
>     frame #3: 0x00007fff8f5d572a libsystem_pthread.dylib`_pthread_start + 137
>     frame #4: 0x00007fff8f5d9fc9 libsystem_pthread.dylib`thread_start + 13
> 
>   thread #7: tid = 0x1be915, 0x00007fff8cb81a1a libsystem_kernel.dylib`mach_msg_trap + 10
>     frame #0: 0x00007fff8cb81a1a libsystem_kernel.dylib`mach_msg_trap + 10
>     frame #1: 0x00007fff8cb80d18 libsystem_kernel.dylib`mach_msg + 64
>     frame #2: 0x00007fff8b0e3f15 CoreFoundation`__CFRunLoopServiceMachPort + 181
>     frame #3: 0x00007fff8b0e3539 CoreFoundation`__CFRunLoopRun + 1161
>     frame #4: 0x00007fff8b0e2e75 CoreFoundation`CFRunLoopRunSpecific + 309
>     frame #5: 0x00007fff8e30d05e AppKit`_NSEventThread + 144
>     frame #6: 0x00007fff8f5d5899 libsystem_pthread.dylib`_pthread_body + 138
>     frame #7: 0x00007fff8f5d572a libsystem_pthread.dylib`_pthread_start + 137
>     frame #8: 0x00007fff8f5d9fc9 libsystem_pthread.dylib`thread_start + 13
> 
>   thread #8: tid = 0x1be963, 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #0: 0x00007fff8cb85e6a libsystem_kernel.dylib`__workq_kernreturn + 10
>     frame #1: 0x00007fff8f5d6f08 libsystem_pthread.dylib`_pthread_wqthread + 330
>     frame #2: 0x00007fff8f5d9fb9 libsystem_pthread.dylib`start_wqthread + 13
> 
>   thread #9: tid = 0x1bea53, 0x00007fff8cb85a3a libsystem_kernel.dylib`__semwait_signal + 10, name = 'com.apple.appkit-heartbeat'
>     frame #0: 0x00007fff8cb85a3a libsystem_kernel.dylib`__semwait_signal + 10
>     frame #1: 0x00007fff88fbcdc0 libsystem_c.dylib`nanosleep + 200
>     frame #2: 0x00007fff88fbccb2 libsystem_c.dylib`usleep + 54
>     frame #3: 0x00007fff8e3d117d AppKit`-[NSUIHeartBeat _heartBeatThread:] + 2132
>     frame #4: 0x00007fff9368776b Foundation`__NSThread__main__ + 1318
>     frame #5: 0x00007fff8f5d5899 libsystem_pthread.dylib`_pthread_body + 138
>     frame #6: 0x00007fff8f5d572a libsystem_pthread.dylib`_pthread_start + 137
>     frame #7: 0x00007fff8f5d9fc9 libsystem_pthread.dylib`thread_start + 13
> ```
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20141130/62491641/attachment.htm>


More information about the kde-core-devel mailing list