D8732: Fix race that starts kactivitymanagerd multiple times

This revision was automatically updated to reflect the committed changes. Closed by commit R6:b4bdd796079c: Fix race that starts kactivitymanagerd multiple times (authored by davidedmundson). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D8732?vs=22112=22819#toc REPOSITORY R6

D8732: Fix race that starts kactivitymanagerd multiple times

apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R6 KActivities BRANCH master REVISION DETAIL https://phabricator.kde.org/D8732 To: davidedmundson, apol Cc: apol, #frameworks

D8732: Fix race that starts kactivitymanagerd multiple times

apol added a comment. +1 I can see the same issue in my bootchart. INLINE COMMENTS > manager_p.cpp:82 > +if (!reply.isValid()) { > +//pre 5.12 the daemon did not support DBus > activation. Fall back to manually forking > +

D8732: Fix race that starts kactivitymanagerd multiple times

davidedmundson added a dependency: D8733: DBus activate KActivityManagerd. REPOSITORY R6 KActivities REVISION DETAIL https://phabricator.kde.org/D8732 To: davidedmundson Cc: #frameworks

D8732: Fix race that starts kactivitymanagerd multiple times

davidedmundson added a comment. bootchart screenshot of the problem (courtesy of apol) F5483212: Spectacle.GxYIDa.png REPOSITORY R6 KActivities REVISION DETAIL https://phabricator.kde.org/D8732 To: davidedmundson Cc: #frameworks

D8732: Fix race that starts kactivitymanagerd multiple times

davidedmundson created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY During plasma startup we have multiple processes using kactivities at once. The current code looks to see if the bus is active then