[Update][PATCH 0/5] ACPI / PM: Export power information to user space

From: Rafael J. Wysocki
Date: Wed Jan 23 2013 - 12:56:16 EST


On Tuesday, January 22, 2013 03:15:52 AM Rafael J. Wysocki wrote:
> On Monday, January 21, 2013 02:03:03 PM Rafael J. Wysocki wrote:
> > On Monday, January 21, 2013 01:48:55 AM Rafael J. Wysocki wrote:
> > > On Sunday, January 20, 2013 03:51:40 PM Rafael J. Wysocki wrote:
> > > > Hi Greg,
> > > >
> > > > The following patch series is about exporting ACPI power resources to user
> > > > space that will be necessary for PM diagnostics on new platforms (powertop is
> > > > the target). Still, existing systems can also benefit from that, like my
> > > > oldish HP nx6325 on which the patches have been tested.
> > > >
> > > > Patch [1/3] shouldn't be really controverisal, as it only makes it possible
> > > > to read power resources' reference counters from user space in a quite
> > > > straightforward way.
> > > >
> > > > Patch [3/3], however, exposes lists of power resources associated with
> > > > different power states of devices using ACPI PM as sysfs subdirectories
> > > > containing symbolic links and that requires some sysfs changes made by
> > > > patch [2/3].
> > > >
> > > > Admittedly, my sysfs skills are limited so please let me know if those
> > > > changes make sense and are acceptable from your perspective. If not, then
> > > > please kindly advise what I can do instead of them.
> > > >
> > > > The patches apply on top of the current linux-next branch of the linux-pm.git
> > > > tree.
> > >
> > > Refreshed with Documentation/ABI files as requested.
> >
> > Well, that version wasn't really a good one, due to mistakes in the docs
> > mostly, so a new one follows (with better docs hopefully).
> >
> > Besides, I thought I'd include a patch that exposes ACPI power state
> > information, because it kind of belongs to the lot. It goes as the new [1/4].
> >
> > [2/4] is the previous [1/3] and so on.
>
> Here it goes again after the changes we discussed earlier today.
>
> I think I've addressed all of the comments, but if I overlooked something,
> please let me know.

So, the following is an update containing a patch ([1/5]) that fixes the
"creation of sysfs files after the device have been registered" problem.
Hopefully. :-)

Patches [2-3/5] are the previous [1-2/4] and they are almost identical as
before, so I added the ACKs you gave for them. Patch [4/5] is the sysfs one
with the internal function placed in the internal header. Patch [5/5] is
a rework of the one exposing the lists of power resources as directories
containing symbolic links.

Please let me know if you are fine with this.

Thanks,
Rafael


--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/