Re: uswsusp history lesson

From: Jan Rychter
Date: Sat Jul 08 2006 - 05:29:09 EST


>>>>> "Pavel" == Pavel Machek <pavel@xxxxxx> writes:
Pavel> Hi!
>> > > > So what Pavel wants can be translated as 'please use already
>> > > > merged code, it can already do what you want without further
>> > > > changing kernel'.
>> > >
>> > > Like we'd want to use unreviewed, extremely new and risky code
>> > > for something that happily destroy filesystems.
>> >
>> > You can either use suspend2 (14000 lines of unreviewed kernel
>> > code, old) or uswsusp (~500 lines of reviewed kernel code, ~2000
>> > lines of unreviewed userspace code, new).
>>
>> I was going to keep quiet, but I have to say this: If Suspend2 can
>> rightly be called unreviewed code, it's only because you've been too
>> busy flaming etc to give it serious review. Personally, though, I
>> don't think it's right

Pavel> I really looked at suspend2 hard, year or so ago, when I was
Pavel> pretty tired of the flamewars. At that point I decided it is way
Pavel> too big to be acceptable to mainline, and got that crazy idea
Pavel> about uswsusp, that surprisingly worked out at the end.

I hate these kinds of discussions, but since no one else did, I'm going
to say this very openly: I don't think you should be the one "deciding"
this. I don't know who and why named you the maintainer of all software
suspend solutions that might go into the mainline kernel, but facts are
that after at least two years of your maintainership (or is it more?):

-- we still don't have working software suspend in the kernel (listen
to the users),
-- we have two implementations in the kernel (swsusp and uswsusp),
none of which people are happy about,
-- the implementation that many people are very happy with is being
kept out because of your "decisions",

Again, those are facts. Forget about hand waving, concentrate on the
facts.

Finally, I personally would hold kernel maintainers to higher
standards. Not everyone has the right to say that something is "ugly"
without providing alternate solutions. Read some of Linus' E-mails: if
anyone has the right to wave hands without arguments it is him, but
still if he says something is ugly, he always provides a better
solution, sometimes also implementing it. And it works.

Re-stating the obvious: I'd like to see a maintainer of software suspend
that produces a working software suspend implementation.

I think most kernel developers underestimate how crucially important
suspend is for notebook users. It is the lifeblood of a notebook user's
life. Unless you develop the kernel and reboot your machine regularly,
you want a stable system that you can suspend and resume at any time,
reliably and predictably. This is way more important than new
schedulers or faster filesystems. It's about basic stability.

I know several people who got tired of the constant fight to keep the
machine running and switched to a Mac. I'm about to do the same as well.

As I don't expect any new arguments to appear in this thread
(unfortunately), and as I expect only more hand-waving from your side, I
won't be bothering you all anymore. EOT from my side.

--J.
-
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/