On Thursday 16 February 2012 17:42:49 Ilya Chernykh wrote: > On Thursday 16 February 2012 20:27:19 Serghei Amelian wrote: > > > > I think the patch is a mess, because qtcreator made automatically > > > > some cleanup in source file. Howewer, upower code is embraced between > > > > "#ifdef WITH_UPOWER" directives. > > > > > > Also I wonder whether the patch affects all KDE places where one can > > > shutdown the computer or only the shutdown dialog. I.e. whether it > > > would work for hibernating from a menu such as kickoff. > > > > Right now is implemented only on ksmserver. Calvin propose to make a dcop > > interface to it, and in this case will be accesibile everywhere. > > Currently kickoff detects well if kpowersave is running. I thought you > patched kpowersave to work through upower instead of hal. Would not it be a > better solution? > > Previously it used its own daemon, then it was ported to hal, why not to > port it to upower? Hybernate/suspend dialog circumventing kpowersave is not > good I think. Actually I start to develop a replacement for kpowersave, but is not so simple to migrate everything from hal to upower. For example, upower do not offer any method to query/set CPU governors. We need to implement our own server as a wrapper over cpufreq (maybe named ucpufreq). Now I consider to create something like ksmserver, which will handle all things related to power, and will expose a DCOP API to user applications. -- Serghei.