[SOLVED] dmesg warning about ACPI_PROCFS_POWER for kernel-generic-4.4.4

brobr · 03-09-2016, 12:41 PM

I noticed this in my dmesg. Dunno whether it is crucial, but maybe something to change in future configs?

Code:

bash-4.3# dmesg | grep "procfs"
[    7.541400] ACPI: Deprecated procfs I/F for AC is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared
[    7.580077] ACPI: Deprecated procfs I/F for battery is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared

bash-4.3# tail -n 1000 /var/log/syslog | grep "procfs"
Mar  8 15:26:07 knotsUL kernel: [    7.395114] ACPI: Deprecated procfs I/F for AC is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared
Mar  8 15:26:07 knotsUL kernel: [    7.417477] ACPI: Deprecated procfs I/F for battery is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared
Mar  8 21:01:42 knotsUL kernel: [    7.874882] ACPI: Deprecated procfs I/F for AC is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared
Mar  8 21:01:42 knotsUL kernel: [    8.110683] ACPI: Deprecated procfs I/F for battery is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared
Mar  9 09:45:28 knotsUL kernel: [    7.541400] ACPI: Deprecated procfs I/F for AC is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared
Mar  9 09:45:28 knotsUL kernel: [    7.580077] ACPI: Deprecated procfs I/F for battery is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared


bash-4.3# cat /boot/config-generic-4.4.4 | grep "PROCFS_POWER"
CONFIG_ACPI_PROCFS_POWER=y

I am running slackware64-current on a laptop with an 8 core cpu:

Code:

vendor_id	: GenuineIntel
cpu family	: 6
model		: 58
model name	: Intel(R) Core(TM) i7-3632QM CPU @ 2.20GHz
stepping	: 9
microcode	: 0x15
cpu MHz		: 1200.031
cache size	: 6144 KB

smallpond · 03-10-2016, 01:10 PM

For my kernel:

Code:

 grep ACPI_PROCFS_POWER /boot/config-`uname -r`
# CONFIG_ACPI_PROCFS_POWER is not set

brobr · 03-10-2016, 01:26 PM

Quote:

Originally Posted by smallpond

For my kernel:

which is???

As mentioned in the thread title the =y setting is present in the config for the generic kernel shipped with slackware64-current....

volkerdi · 03-10-2016, 02:36 PM

Quote:

Originally Posted by brobr

I noticed this in my dmesg. Dunno whether it is crucial, but maybe something to change in future configs?

Code:

bash-4.3# dmesg | grep "procfs"
[    7.541400] ACPI: Deprecated procfs I/F for AC is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared
[    7.580077] ACPI: Deprecated procfs I/F for battery is loaded, please retry with CONFIG_ACPI_PROCFS_POWER cleared

Well, here's what the Kconfig has to say about this option:

Quote:

For backwards compatibility, this option allows
deprecated power /proc/acpi/ directories to exist, even when
they have been replaced by functions in /sys.
The deprecated directories (and their replacements) include:
/proc/acpi/battery/* (/sys/class/power_supply/*)
/proc/acpi/ac_adapter/* (sys/class/power_supply/*)
This option has no effect on /proc/acpi/ directories
and functions, which do not yet exist in /sys
This option, together with the proc directories, will be
deleted in the future.

Say N to delete power /proc/acpi/ directories that have moved to /sys/

Generally I tend to leave options like this enabled, figuring that backwards compatibility is probably good until such time as the kernel actually deletes the feature. Seems like the kernel is being a bit too noisy about it here, but otherwise it's a harmless (and possibly helpful) option.

brobr · 03-10-2016, 04:48 PM

If I understand this correctly, info like this is becoming deprecated?

Quote:

bash-4.3# cat /proc/acpi/battery/BAT0/info
present: yes
design capacity: 5600 mAh
last full capacity: 3384 mAh
battery technology: rechargeable
design voltage: 11100 mV
design capacity warning: 560 mAh
design capacity low: 0 mAh
cycle count: 0
capacity granularity 1: 64 mAh
capacity granularity 2: 64 mAh
model number: BAT
serial number: 0001
battery type: LION
OEM info: NOTEBOOK

But I understand that !-)

And this appears more geared up for machine-digestion....

Quote:

bash-4.3# ls /sys/class/power_supply/BAT0/
alarm current_now present uevent
capacity cycle_count serial_number voltage_min_design
capacity_level device/ status voltage_now
charge_full manufacturer subsystem/
charge_full_design model_name technology
charge_now power/ type
bash-4.3# cat /sys/class/power_supply/BAT0/current_now
0
bash-4.3# cat /sys/class/power_supply/BAT0/charge_full
3384000
bash-4.3# cat /sys/class/power_supply/BAT0/present
1
bash-4.3# cat /sys/class/power_supply/BAT0/status
Full
bash-4.3# cat /sys/class/power_supply/BAT0/serial_number
0001
bash-4.3# cat /sys/class/power_supply/BAT0/technology
Li-ion
bash-4.3# cat /sys/class/power_supply/BAT0/type
Battery
bash-4.3# cat /sys/class/power_supply/BAT0/manufacturer
NOTEBOOK
bash-4.3# cat /sys/class/power_supply/BAT0/capacity_level
Full

Don't we love progress!-)

Thanks for the =y setting and the whole of slackware!

bassmadrigal · 03-10-2016, 10:10 PM

I think the new change is intended to be exactly for machine-digestion. Most normal users don't need to browse through /proc/ or /sys/, but many programs rely on information in there. The new change makes it easier for other programs to gain needed information without needing to parse a single output with a multitude of information (this of all that sed/awk/sort/etc fun you may have had in the past with that).

That being said, until this option is completely removed from the kernel, it doesn't hurt anything to keep it enabled, so I'm also glad Pat has kept it in our kernels

GazL · 03-11-2016, 05:01 AM

The option is not enabled when you do a make defconfig, but Linus doesn't like to break userspace so I can't see him yanking it out from under people completely. I guess making a little noise to encourage folks to disable it is their only approach for getting people to move on: short of giving them a nasty surprise one day!