Bug 160025

Summary: battery plasmoid uses deprecated /proc/acpi files
Product: [Unmaintained] plasma4 Reporter: David <StormByte>
Component: widget-batteryAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED NOT A BUG    
Severity: wishlist    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description David 2008-03-28 17:14:45 UTC 
Version:           svn (using Devel)
Installed from:    Compiled sources
OS:                Linux

I want to post this bug not as a real bug, but as a warning.

I realized that battery plasmoid uses the deprecated /proc/acpi files to get its informations about battery. As I can see in config screen of kernel, this is considered deprecated (at least in 2.6.24).

So I suggest starting to be prepared for future /sys interface (/sys/bus/acpi/) if possible in case this dissapears from kernel's code not having this program useless.

What about it? If you already know about this issue, I'm sorry, I just wanted to let you know
Comment 1 Pino Toscano 2008-03-28 20:20:51 UTC 
> I realized that battery plasmoid uses the deprecated /proc/acpi files

Show us the code, please, as I see it using a plasma dataengine using Solid only (no opening of files in /proc). And Solid in turn uses HAL to get the information from the system.
Comment 2 Kevin Ottens 2008-03-28 20:29:46 UTC 
And HAL indeed still uses /proc/acpi files for those information, but parts read /sys already (hence the "I see my batterie twice" bug). But don't worry, they're slowly getting in order to fix that.

Closing as INVALID, it's out of our code.
Comment 3 David 2008-03-29 04:18:12 UTC 
OK, I can't show you the affecting code, but what I can say is that if I disable deprecated /proc/acpi functions in kernel, I'll get a battery plasmoid unusable (it says N/A in battery charge left, and so on)...