Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755442AbaDTNJn (ORCPT ); Sun, 20 Apr 2014 09:09:43 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:51491 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755321AbaDTNJb (ORCPT ); Sun, 20 Apr 2014 09:09:31 -0400 Date: Sun, 20 Apr 2014 15:09:30 +0200 From: Pavel Machek To: Ilia Mirkin Cc: Stefan Lippers-Hollmann , Lan Tianyu , "Rafael J. Wysocki" , "linux-acpi@vger.kernel.org" , kernel list Subject: Re: 3.14-rc: /proc/acpi/battery gone? Message-ID: <20140420130930.GD12135@amd.pavel.ucw.cz> References: <20140314211456.GA23213@amd.pavel.ucw.cz> <201403150217.50526.s.L-H@gmx.de> <532471EC.1070107@intel.com> <201403160357.53858.s.L-H@gmx.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi! > >> I just glance wmbattery code. I find the code in the acpi.c is already > >> using the new sysfs battery interfaces, right? > > > > By default, wmbattery appears to default to using upower as abstraction > > level, instead of querying sysfs itself directly. > > > > http://git.kitenet.net/?p=wmbattery.git;a=blob;f=autoconf/makeinfo.in;hb=HEAD > > > > which sets USE_UPOWER=1 by default. > > > > If USE_UPOWER=0 is set explicitly for the build, it reverts back to > > direct sysfs parsing - and yes, it does appear to adhere to the current > > sysfs API properly. > > > > The last remains, and the ability to parse procfs (which hasn't been > > default for quite some time already, in favour of using hal as > > abstraction layer) has finally been removed in > > > > http://git.kitenet.net/?p=wmbattery.git;a=commitdiff;h=833eb63a5ce4f2fb712a201b1db4f2db1700fddb > > > > The switch from procfs parsing to hal (by default at least) in turn > > happened with > > > > http://git.kitenet.net/?p=wmbattery.git;a=commitdiff;h=63c3d1a0b11e8ade1a5612bb5baa3d92e153bbbe > > > > in 2008 (before Debian squeeze/ oldstable). I have not investigated if > > hal then read from procfs or sysfs, but wmbattery at least didn't read > > from procfs itself, unless explicitly told to do so (USE_HAL=0) during > > the build since mid 2008. > > > > The current version of wmbattery however will never try to access > > /proc/acpi, the current version no longer knows of its existence. > > > > [Again, I'm not familiar with wmbattery myself and have never run it] > > Stefan, > > Thanks for looking into this. The newest wmbattery version indeed > supports upower. However, I haven't figured out how to get it to work. > That's obviously not the kernel's fault, but an unfortunate reality. > It seems to really want dbus to be running, but when I start dbus > (which nothing else on my system needs, apparently), it just hangs. My > knowledge of these things is, (un)fortunately non-existent, so I just > gave up on the upower approach. Running something as heavy as dbus > just for a silly dock app seems... silly as well. Any news on this one? It seems that delaying fsck on battery power also relies on /proc/acpi/battery... https://bbs.archlinux.org/viewtopic.php?id=12168 Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/