<table><tr><td style="">romangg 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/D26026">View Revision</a></tr></table><br /><div><div><p>That being said thinking about it some more having a file watcher can lead to races for example in the following case:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">User changes auto rotation and resolution at the same time while device is held in a position unequal to current rotation</li>
<li class="remarkup-list-item">KWin receives updated configuration (I)</li>
<li class="remarkup-list-item">KScreen daemon receives auto rotation change through file watcher before resolution change through KCM -> KWin -> daemon</li>
<li class="remarkup-list-item">KScreen sends updated configuration (II) with different rotation to KWin</li>
<li class="remarkup-list-item">KWin sends new configuration (i) and afterwards receives configuration (II)</li>
<li class="remarkup-list-item">KWin applies (II) and sends (II)</li>
<li class="remarkup-list-item">KScreen receives first new configuration (I) and then (II)</li>
</ul>

<p>In this case the resolution would not be set.</p>

<p>I'm unsure how to solve this best. I really don't want to send all auxiliary KScreen data through the KWin connection. Simple solution would be a small delay on control changes waiting for potential KWin changes. A likely better solution might be to introduce this libkscreen dbus out of process service found on RandR backend to the KWayland backend and then send auxiliary data through there but not to the compositor.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R104 KScreen</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26026">https://phabricator.kde.org/D26026</a></div></div><br /><div><strong>To: </strong>romangg, Plasma<br /><strong>Cc: </strong>davidedmundson, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>