Re: Changes to PM layer break userspace
- From: Greg KH <gregkh@xxxxxxx>
- Date: Tue, 19 Dec 2006 21:34:17 -0800
On Tue, Dec 19, 2006 at 09:14:49PM -0800, David Brownell wrote:
On Tuesday 19 December 2006 8:26 pm, Matthew Garrett wrote:
On Tue, Dec 19, 2006 at 07:59:42PM -0800, David Brownell wrote:
It's perfectly reasonable to
refer to it as a flawed interface, or perhaps even a buggy one. But in
itself, it's clearly not a bug.
This class of bug is also called a "design bug" or sometimes "mistake".
Exactly, those "power" files actually pre-date the actual tree of
devices itself. They were just holders for what the original developer
thought was going to be needed, but was never properly implemented due
to some job changes (note, this was not myself...)
In contrast, the /sys/devices/.../power/state API has never had many
users beyond developers trying to test their drivers (without taking
the whole system into a low power state, which probably didn't work
in any case), and has *always* been problematic. And the change you
object to doesn't "break" anything fundamental, either. Everything
still works.
It's used on every Ubuntu and Suse system,
Odd how the relevant Suse developers didn't mention any issues with
those files going away, any of the times problems with them were
discussed on the PM list. Also, I have a Suse system that doesn't
use those files for anything ... maybe only newer release use it.
I would be very interested to see any newer SuSE programs using that
interface. Just point them out to me and I'll quickly fix them.
And yes, as a SuSE developer (and one of the people in charge of the
SuSE kernels), I have no problem with these files just going away.
Because, as David keeps repeating, they are broken and wrong.
thanks,
greg k-h
-
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/
- Follow-Ups:
- Re: Changes to PM layer break userspace
- From: Matthew Garrett
- Re: Changes to PM layer break userspace
- References:
- Changes to sysfs PM layer break userspace
- From: Matthew Garrett
- Re: Changes to PM layer break userspace
- From: David Brownell
- Re: Changes to PM layer break userspace
- From: Matthew Garrett
- Re: Changes to PM layer break userspace
- From: David Brownell
- Changes to sysfs PM layer break userspace
- Prev by Date: Re: [2.6 patch] powerpc: remove the broken Gemini support
- Next by Date: Re: BUG: wedged processes, test program supplied
- Previous by thread: Re: Changes to PM layer break userspace
- Next by thread: Re: Changes to PM layer break userspace
- Index(es):
Relevant Pages
|