Re: [swsusp] separate snapshot functionality to separate file

[Nigel Cunningham <ncunningham@cyclades.com>]

Hi.

On Wed, 2005-10-05 at 06:53, Pavel Machek wrote:
> > > That does not belong to snaphost. The rest is notthat clear, but I have it
> > > working in userspace.
> > 
> > Of course it is doable in the userland, but this does not mean it should be
> > done in the userland.  Personally I don't think so (please see
> > below).
> 
> Even if you don't agree with putting it to userland (and that's
> neccessary -- if we want some features from suspend2), split still
> makes sense.

Not necessary, but desirable in your eyes. I can see that you can make
it work if you're only talking about implementing eye candy, but if
you're serious about adding the substantial improvements from Suspend2
(support for multiple swap partitions, swap files, block sizes != 4096,
asynchronous I/O, readhead where I/O must be synchronous, support for
writing to a network or a generic file (again with block size != 4096)
etc, - let alone support for saving a full image of memory - this is
just going to get uglier and uglier. We can see this already because
you've already dropped swap support, obviously because it's too hard
from userspace.

The tidy up you're proposing is a nice step. But it seems to me to be 
the only good thing and really useful thing to come of this so far.

Pavel, at the PM summit, we agreed to work toward getting Suspend2
merged. I've been working since then on cleaning up the code, splitting
the patches up nicely and so on. In the meantime, you seem to have gone
off on a completely different tangent, going right against what we
agreed then. Can I get you to at least try to come back from that? I'd
be more than willing to help you with cherry picking some changes and
getting them in ahead of the rest of the code. Would you consider
working together to do that? In particular, I'm thinking that I could
send you the refrigerator patches as I have them at the moment, and a
patch to remove the reliance on PageReserved, at least for a start. Any
willingness on your part to give that a try?

Regards,

Nigel
-- 


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