Re: [Suspend2-devel] Re: swsusp / suspend2 reliability

[Pavel Machek <pavel@ucw.cz>]

On Sat 2006-07-08 09:33:00, Nigel Cunningham wrote:
> Hi.
> 
> On Saturday 08 July 2006 09:25, Pavel Machek wrote:
> > > > 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 to call 
> it unreviewed. I've had and applied feedback from lots of people over time 
> (hch, Rafael, Pekka(sp?), Nick, Con and Hugh to name just a few). If they 
> weren't reviewing the code, what were they doing?

Well, you are right that Rafael did some quite serious reviewing of
latest incarnation... OTOH you got some pretty big comments ("may not
use /proc") that were not included, yet (and will need big changes),
so it is not really reviewed, either. (Certainly swsusp in-kernel code
got more reviewing over the years).
								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html