Bug 160025 - battery plasmoid uses deprecated /proc/acpi files
Summary: battery plasmoid uses deprecated /proc/acpi files
Status: RESOLVED NOT A BUG
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-battery (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-28 17:14 UTC by David
Modified: 2008-03-29 04:18 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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)...