D14302: Don't block forever in ensureKdeinitRunning

[Jaime Torres Amate]

jtamate added a comment.


  In D14302#297120 <https://phabricator.kde.org/D14302#297120>, @thiago wrote:
  
  > Can you print the contents of the timer object inside tryLock()?
  
  
  
  
    (gdb) info frame
    Stack level 2, frame at 0x7ffdda9967c0:
     rip = 0x7f54be8bc985 in QLockFile::tryLock (io/qlockfile.cpp:274); saved rip = 0x7f54c0a23afb
     called by frame at 0x7ffdda996860, caller of frame at 0x7ffdda996730
     source language c++.
     Arglist at 0x7ffdda996728, args: this=<optimized out>, timeout=<optimized out>
     Locals at 0x7ffdda996728, Previous frame's sp is 0x7ffdda9967c0
     Saved registers:
      rbx at 0x7ffdda996788, rbp at 0x7ffdda996790, r12 at 0x7ffdda996798, r13 at 0x7ffdda9967a0, r14 at 0x7ffdda9967a8, r15 at 0x7ffdda9967b0, rip at 0x7ffdda9967b8
    (gdb) info locals
    remainingTime = <optimized out>
    d = 0x557a916a98f0
    timer = {t1 = 9223372036854775807, t2 = 0, type = 1}
    sleepTime = 6400
    (gdb) up
    #3  0x00007f54c0a23afb in KDEInitInterface::ensureKdeinitRunning() () from /usr/lib64/libKF5DBusAddons.so.5
    (gdb) info locals
    No symbol table info available.
    (gdb) info frame
    Stack level 3, frame at 0x7ffdda996860:
     rip = 0x7f54c0a23afb in KDEInitInterface::ensureKdeinitRunning(); saved rip = 0x7f54c2d16ffb
     called by frame at 0x7ffdda9968a0, caller of frame at 0x7ffdda9967c0
     Arglist at 0x7ffdda9967b8, args: 
     Locals at 0x7ffdda9967b8, Previous frame's sp is 0x7ffdda996860
     Saved registers:
      rbx at 0x7ffdda996828, rbp at 0x7ffdda996830, r12 at 0x7ffdda996838, r13 at 0x7ffdda996840, r14 at 0x7ffdda996848, r15 at 0x7ffdda996850, rip at 0x7ffdda996858
  
  
  
  In D14302#297119 <https://phabricator.kde.org/D14302#297119>, @thiago wrote:
  
  > No, because your statement is incorrect. setPreciseRemainingTime **does** assign to t1:
  >
  >   t1 += secs + toSecsAndNSecs(nsecs).first;
  >   
  
  
  Yes, but this is assuming t1 = 0, I mean, it is not t1 = secs.... (not with +=).

REPOSITORY
  R271 KDBusAddons

REVISION DETAIL
  https://phabricator.kde.org/D14302

To: jtamate, dfaure, #frameworks, thiago
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180724/dd29fac0/attachment.html>