Review Request 126362: Allow timeout in reset_oom_protection while waiting for SIGUSR1

[Michael Pyne]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126362/#review89577
-----------------------------------------------------------

Ship it!


I hacked my kinit.git to not send the SIGUSR1 at all and the patch still allowed kdeinit to exec processes (albeit noticeably slower), so seems to me it should work. No other ill effects noted either.

- Michael Pyne


On Dec. 15, 2015, 10:19 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126362/
> -----------------------------------------------------------
> 
> (Updated Dec. 15, 2015, 10:19 a.m.)
> 
> 
> Review request for KDE Frameworks and Michael Pyne.
> 
> 
> Repository: kinit
> 
> 
> Description
> -------
> 
> If for whatever reason the child process does not get SIGUSR1, it
> should not block an infinite time, but timeout and continue.
> 
> 
> Diffs
> -----
> 
>   src/kdeinit/kinit.cpp a18008a11bf00a35aa0cab450180926217cd58f5 
> 
> Diff: https://git.reviewboard.kde.org/r/126362/diff/
> 
> 
> Testing
> -------
> 
> Seems to not work correctly yet as I don't hit the timeout condition with the broken kwin_wayland.
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151216/b40104c7/attachment.html>