Re: [Suspend2-devel] Re: uswsusp history lesson

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting "Rafael J. Wysocki" <[email protected]>:

The problem is he _can't_ do it on his own if he wants the code merged,
because for this purpose some people have to review it, and that's not
only me or Pavel, but also architecture maintainers, memory management
maintainers, and probably some other people too.  Moreover, Nigel needs
to address the issues raised by the reviewers.

Of course, of course. Nobody is going to merge anything until relevant maintainers approve. That's not what I proposed.

My point is something else. A few months back Pavel mentioned that he's thinking of developers more than users when it comes to Suspend2. In other words, he was concerned with maintenance of the thing. I'm also guessing nobody likes signing their name on something they have fundamental design beef with. All valid points, of course.

In order to avoid all this, my proposal introduces Nigel as the maintainer of Suspend2 code (i.e. *only* the non-shared bits with swsusp/uswsusp).

Given that Nigel:

- doesn't want to rip out/change neither swsusp nor uswsusp
- wants to share code as much as possible
- wants to fix things to be technically acceptable
- has shown to able to maintain Suspend2 codebase for users
- no swsusp/uswsup coder would have to worry about Suspend2 code beyond already shared bits they would worry about anyway

I think it would be appropriate to let him do so (once the initial technical issues are fixed).

The "your design sucks" argument between Pavel and Nigel is not likely to be resolved by more talk (this thread is quite appropriately called "history lesson" :-). These two have been at it for months now, with no resolution in sight. Yours truly also contributed by useless flaming from time to time ;-) No need for that any more.

However, Pavel is the one in the position of power here (being the maintainer), so I think he should, in the interest of users, decide to give Suspend2 a fair chance (after all those technical issues are addressed, of course), by letting Suspend2 be in the same position as swsusp or uswsusp - in other words, in the main tree (actually -mm, to start with, just as Nigel asked). And with my proposal Pavel and other swsusp/uswsusp coders, yourself included, would not have to spend any effort past reviewing the initial set of patches.

In the end, it's a win-win.

--
Bojan
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux