lkml.org 
[lkml]   [2007]   [Jan]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Fix /sys/device/.../power/state regression
On Fri, Jan 26, 2007 at 04:42:56PM -0800, David Brownell wrote:
> On Friday 26 January 2007 3:15 pm, Matthew Garrett wrote:
> > It's certainly the case that fixing those drivers would result in a
> > better long-term situation - however, nobody currently seems to have any
> > interest in doing so...
>
> And the way these things work, unfortunately, is that merging your patch
> would ensure nobody ever gets such interest. Removing that "state" file
> (and its bogus infrastructure) has already taken a few years too long.

I'd argue that the onus is on those who wish to remove the interface to
ensure that equivalent functionality exists first. It's not as if it's
especially /hard/, just tedious and requires access to a large quantity
of hardware.

> > This patch restores useful functionality
> > without breaking the extra sanity checks that you've added. I appreciate
> > that it's not an interface that you want to support in the long term
> > (well, even the short term...),
>
> You imply that it _was_ once supported, which is not true. Like any
> other bug (in this case "design bug"), it was there and could be abused.
> And like some other bugs, fixing it can trigger complaints from (ab)users.
>
> It's been several years now that this interface has been well recognized as
> trouble. Years in which all _other_ users went and did the Right Thing:
> they stopped using it, or never started.

Now this is what throws me somewhat. Last May, you argued strongly in
favour of keeping the interface:

"Which IMO makes removing this a Bad Thing. It needs to have some
kind of replacement in place before the "magic numbers" go away."

(http://lists.osdl.org/pipermail/linux-pm/2006-May/002376.html)

"How could we schedule the removal before we have even had a couple
releases to fine-tune its replacement, and verify that the main issues
with the current thing are fully resolved?"

(http://lists.osdl.org/pipermail/linux-pm/2006-May/002382.html)

"> Maybe date will need to be shifted...

How about "one year after its replacement is ready"?"

(http://lists.osdl.org/pipermail/linux-pm/2006-May/002384.html)

I don't think there /has/ been any strong indication from the PM
community that this interface was going to go away in the near future -
the last time somebody tried to remove it (rather than incidentally
breaking it), there were complaints and the matter seemed to drop.

Now, the current situation is that the interface is scheduled for
removal in July (not something I especially agree with, and something
you seemed to disagree with less than a year ago). However, the changes
in 2.6.19 effectively crippled it. The current situation is that the
only widely-used bus where this interface still works is USB, so to a
large extent the interface has already been removed. If the interface is
that broken, then it should just be removed now - if it's going to be
carried around until July, it should be fixed so that it works as well
as it did in 2.6.18 and below. The patch does that.

--
Matthew Garrett | mjg59@srcf.ucam.org
-
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/

\
 
 \ /
  Last update: 2007-01-27 02:23    [W:0.625 / U:0.148 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site