Re: [ 01/10] [Suspend2] kernel/power/modules.h

[Pavel Machek <pavel@ucw.cz>]

Hi!

> > > - Is there any way you could support doing a full image of memory with
> > > this approach? Would you take patches?
> >
> > Doing full image is certainly possible; it is not important if kernel
> > does the writing or userspace does it. Taking patches definitely
> > depends how they'd look like...
> 
> Would you actually do some work on generating them? 

Well, I'm afraid that full image of memory is not that high on
priority list. Rafael's solution seems to work pretty much okay.

> I'm concerned that I'm 
> going to end up putting more work into making patches, only to have you nack 
> them for (what seem to me like) arbitrary reasons. I feel like all you're 
> doing at the moment is pouring cold water on my work. Seeing you put
> some 

Well, sorry about that. I thought I gave you fair warning at the end
of last year, when I posted uswsusp patches to the lists...

> work into making mileage from the work I've put into Suspend2 would encourage 
> me a lot. Forgive me if I sound pretty negative - between a couple of months 
> of humid weather, lack of sleep and the event of the last few weeks, I'm not 
> in the best frame of mind at the moment. Perhaps it would be better
> for me 

Well, you did some pretty amazing work with suspend2.net community...

> just to say "I give full permission for you and Rafael to rip out of Suspend2 
> anything you find useful.", leave it at that and go find another hobby for a 
> while.

We could use some help with userland tools. Helping with userland
parts is probably the fastest way to get usable/nice/fast suspend to
the users.

If you think suspend2 can be easily ported to userspace... yes, that
would be useful/welcome.
									Pavel
-- 
Thanks, Sharp!
-
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/