<table><tr><td style="">bruns added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22339">View Revision</a></tr></table><br /><div><div><p>I would have preferred this to only land when <a href="https://phabricator.kde.org/D22333" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D22333</a> had been addressed fully, I assumed this required no further notice.</p>

<p>So from a pure technical view, this indeed seems to be correct, but from an architectural view this seems less than ideal (to put it mildly).</p>

<p>One obvious issue is the multiple instantiation and initialization, i.e. when the UDisks2 backend is used from different threads each one has to introspect udisks itself, which seems wasteful for me.</p>

<p>I will extend this thought in <a href="https://phabricator.kde.org/D22333" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D22333</a>, can we move any further discussion there?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R245 Solid</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22339">https://phabricator.kde.org/D22339</a></div></div><br /><div><strong>To: </strong>apol, Frameworks, davidedmundson, bruns<br /><strong>Cc: </strong>lbeltrame, bruns, kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham<br /></div>