By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 1603

Summary: klaptopdaemon is broken
Product: TDE Reporter: Darrell <darrella>
Component: tdeutilsAssignee: Timothy Pearson <kb9vqf>
Status: NEW ---    
Severity: major CC: albator78, bugwatch, darrella, michele.calgaro
Priority: P5    
Version: R14.0.x [Trinity]   
Hardware: Other   
OS: Linux   
See Also: http://bugs.pearsoncomputing.net/show_bug.cgi?id=1848
Compiler Version: TDE Version String:
Application Version: Application Name:
Bug Depends on:    
Bug Blocks: 2967    

Description Darrell 2013-07-31 19:48:40 CDT
Thinkpad T400 laptop

* I'm unable to suspend to ram with klaptop, even as root. I can suspend to ram as root with tdepowersave.

* As root, in the ACPI config tab, selecting the "Setup Helper Application" button results in a dialog to supply a root passowrd (I'm already root):

"You need to supply a root password to allow the privileges of the klaptop_acpi_helper to change."

Selecting the dialog Continue button does nothing.

* In the ACPI Config tab the NOTE text is outdated, with references to the 2.4 and 2.5 kernels.

* /opt/trinity/bin/klaptop_acpi_helper is set uid.
Comment 1 Darrell 2013-07-31 21:25:15 CDT
* As root and non-root, selecting Suspend from the klaptop popup menu results in the Desktop Session Locked dialog appearing (or the screen saver and then the dialog). No attempt to suspend to ram.

I have upower installed and for both root and non-root upower -d shows can-suspend: yes. When upower -d shows can-suspend: no for a non-root user, the Suspend option does not appear in the popup menu.
Comment 2 Darrell 2013-07-31 21:50:05 CDT
* The klaptop deamon sources support a slew of TQToolTips but the only tooltip that always appears everywhere is "This module allows you to configure ACPI for your system."
Comment 3 Francois Andriot 2013-08-01 06:00:59 CDT
On my laptop, klaptop does not work at all
On all tabs except the first one (Battery), I have a generic message telling that ACPI support exists but may be incomplete on my computer...
Comment 4 Darrell 2013-08-01 14:59:55 CDT
As klaptop deamon is a part of the foundational package set (tdeutils), should this report be added to the R14 etherpad?
Comment 5 Francois Andriot 2013-08-02 02:51:53 CDT
It looks like klaptop functions are based on the /proc/acpi directory.
I believe this is obsolete, and that newer linux kernel only show information under /sys .

What features does klaptop have, that tdepowersave doesn't ? Is the former totally obsolete ?
Maybe it should be modified to use the TDE Hardware library.
Comment 6 Darrell 2013-08-02 10:20:37 CDT
> It looks like klaptop functions are based on the /proc/acpi directory.
> I believe this is obsolete, and that newer linux kernel only show
> information under /sys .

/proc/acpi exists in my Slackware 14.0 systems.

> What features does klaptop have, that tdepowersave doesn't ? Is the former
> totally obsolete ?

As this is my first laptop, I'm no expert to answer. :-) Seems to me klaptop is simpler and more straightforward, yet providing some similar features, especially the basic features. I like the layout of klaptop better --- easier to comprehend and nowhere near as complicated.

> Maybe it should be modified to use the TDE Hardware library.

Perhaps. I don't know whether that is the cause of the breakage. My system does not have HAL, only upower/udisks/etc.
Comment 7 Darrell 2013-08-09 12:58:20 CDT
Bumping to Major because I added the report to the R14 etherpad road map.
Comment 8 Darrell 2013-09-01 11:05:25 CDT
Proverbial questions: would we better in the long run purging klaptopdaemon and merging tdepowersave into tdeutils as a replacement? Or should klaptop be pulled from tdeutils and offered as a separate app?

Note: klaptopdaemon is hard-coded to /usr/sbin/pmi and /usr/sbin/hibernate. Looks like those executables were part of debian-specific packages. I have searched high and low for those files in Slackware and find no history anywhere, not even in third party repositories. That hard-coding would explain why klaptop is broken on many systems.
Comment 9 Michele Calgaro 2018-07-31 20:49:17 CDT
Slavek and I have agreed on removing klaptop daemon.
It should be consider to make tdepowersave the default by moving it into tdeutils. This may require some work to allow tdepowersave to build with HAL support for *BSD distro.
Comment 10 Michele Calgaro 2018-08-01 05:30:09 CDT
Also need to make sure that all references to klaptopdaemon in TDE code are removed, if any.