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=20113
REVISION
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
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 ;-)
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,
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
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,
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,
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, );
^
Yes,
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
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
10 matches
Mail list logo
