D7757: Use real-time scheduling policy for kwin_wayland

2017-09-29 Thread Martin Flöser
This revision was automatically updated to reflect the committed changes. Closed by commit R108:7c8003f7f621: Use real-time scheduling policy for kwin_wayland (authored by graesslin). REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7757?vs=19375&id=20113 REVISION

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-19 Thread Christoph Feck
cfeck added a comment. I was about to suggest that it should queue lower than "real" realtime processes, but reading the description, this was actually taken care of using the lowest priority. +1 to this feature. I think Windows also favors GUI processes over background processes, so ha

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-19 Thread Martin Flöser
graesslin added a comment. In https://phabricator.kde.org/D7757#146952, @tcberner wrote: > Looks good to me. Sorry for the timeout :) The ping was more meant to other (Linux) developers on whether there is agreement that we should try making KWin a real time process ;-) REPOSITOR

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-19 Thread Tobias C . Berner
tcberner accepted this revision. tcberner added a comment. This revision is now accepted and ready to land. Looks good to me. Sorry for the timeout :) REPOSITORY R108 KWin BRANCH wayland-realtime REVISION DETAIL https://phabricator.kde.org/D7757 To: graesslin, #kwin, #plasma, tcberner

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-19 Thread Martin Flöser
graesslin added a comment. ping REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D7757 To: graesslin, #kwin, #plasma Cc: tcberner, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-10 Thread Martin Flöser
graesslin updated this revision to Diff 19375. graesslin added a comment. Restricted Application edited projects, added Plasma; removed KWin. Check whether we have SCHED_RESET_ON_FORK and ifdef accordingly REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7757?vs

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-10 Thread Tobias C . Berner
tcberner added a comment. I could not find anything similar. So that's probably the way to go. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D7757 To: graesslin, #kwin, #plasma Cc: tcberner, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, harde

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-10 Thread Martin Flöser
graesslin added a comment. In https://phabricator.kde.org/D7757#144415, @tcberner wrote: >^ > /home/Tobias/git/kwin/main_wayland.cpp:452:38: error: use of undeclared identifier 'SCHED_RESET_ON_FORK' > sched_setscheduler(0, SCHED_RR | SCHED_RESET_ON_FORK, &s

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-10 Thread Tobias C . Berner
tcberner added a comment. ^ /home/Tobias/git/kwin/main_wayland.cpp:452:38: error: use of undeclared identifier 'SCHED_RESET_ON_FORK' sched_setscheduler(0, SCHED_RR | SCHED_RESET_ON_FORK, &sp); ^ Yes, `SCHED_RESET_ON_F

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-10 Thread Martin Flöser
graesslin added a subscriber: tcberner. graesslin added a comment. @tcberner could you please try to compile this change on FreeBSD? I just want to be sure that it doesn't introduce a linux-ism. The most prominent dangerous part could be SCHED_RESET_ON_FORK where I'm not sure whether it is li

D7757: Use real-time scheduling policy for kwin_wayland

2017-09-10 Thread Martin Flöser
graesslin created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY The base idea behind this change is to keep the system responsive no matter what other processes do. All input and rendering needs to go