Messages in this thread | | | Date | Sat, 8 Jul 2006 02:04:41 +0200 | From | Pavel Machek <> | Subject | Re: [Suspend2-devel] Re: swsusp / suspend2 reliability |
| |
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 - 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/
| |