----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105111/#review14741 -----------------------------------------------------------
Ship it! i am not overly familiar with this exact bit of code, but it seems to have been working on my system just fine since the patch was posted. - Aaron J. Seigo On May 31, 2012, 8:48 a.m., Andriy Gapon wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105111/ > ----------------------------------------------------------- > > (Updated May 31, 2012, 8:48 a.m.) > > > Review request for Plasma. > > > Description > ------- > > This patch tries to a fix a race in the session saving and shutdown code of > ksmserver. > The race may happen on wmPhase1WaitingCount variable when WM-type clients are > fast enough to complete their save requests while the loop in > KSMServer::shutdown is still iterating. As a result wmPhase1WaitingCount may > become zero too early and that would kick off the next save/shutdown phase in > a client's callback. In fact, it might even go multiple times from zero to > non-zero and back during the WM-specific loop. > > > This addresses bug 206511. > http://bugs.kde.org/show_bug.cgi?id=206511 > > > Diffs > ----- > > ksmserver/shutdown.cpp a5a1ef9 > > Diff: http://git.reviewboard.kde.org/r/105111/diff/ > > > Testing > ------- > > > Thanks, > > Andriy Gapon > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel