Messages in this thread |  | | Subject | Re: suspend2 merge [was Re: [RFC] Fix Device Power Management States] | From | Nigel Cunningham <> | Date | Wed, 11 Aug 2004 15:05:54 +1000 |
| |
Hi.
On Wed, 2004-08-11 at 09:36, Pavel Machek wrote: > At one point I was pretty unhappy with swsusp state (that was just > before I wrote trivial highmem support), and was willing to merge > suspend2 pretty much at all costs. (Okay, I was never willing to let > LZO-suspend2 sneak into 2.6). I'm now way happier with merged > pmdisk+swsusp works.
The LZF licensing has been corrected so it's not a problem.
> I feared big problems with highmem support, but surprisingly, trivial > support thats in current code does not cause problems for > people. People seem to like pmdisk+swsusp, too...
That will be because you're eating most of the memory anyway; there's not problem finding enough memory to copy the Highmem too. I guess that once we start seeing people trying to suspend 2GB to disk or you try to eat less memory, Highmem will become more of a pain.
> Now, people like suspend2 even more, and for good reasons: it is > extremely fast, it provides nice feedback and its refrigerator is > superior. > > I also realized that suspend2 is fundamentally more complex than > swsusp: it introduces additional time period where page cache must not > be touched. I did not realize this sooner.
Sorry. I said it in so many ways! It's not really an issue though; processes are stopped and suspend's own I/O doesn't touch page cache.
> I do longer think that "merge at all costs" is good idea now: in > kernel code is small, simple and nice by now, and I'd like to keep it > that way.
I want it to be simple and clear too. I'm getting there. As part of the cleanup, I'd just made suspend modular, which has further untangled some of the interdependancies between sections of code. I still have more work to do, but I'm increasingly feeling like removing the debugging code.
> Now, there are some parts of swsusp that are not quite okay. One of > them is refrigerator -- it fails (in non-critical way but still) in > some cases where it should not fail. suspend2 seems to have this > solved, and I'd like to merge its refrigerator.
I'll submit a patch. I need to look at how you use the refrigerator first. I refrigerate processes prior to resuming as well, and might need to adjust things if you don't do that. I also need to check it will work okay with S3.
> Other parts... I'm not so sure. Incremental patches would certainly > help a lot here.
Well, if you insist, I'll do what I can. I looked at doing this before, but was like trying to describe how to transform a mini into a station wagon while using it every day! It will, of course, make the merge _much_ slower too.
Nigel -- Nigel Cunningham Christian Reformed Church of Tuggeranong PO Box 1004, Tuggeranong, ACT 2901
Many today claim to be tolerant. But true tolerance can cope with others being intolerant.
- 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/
|  |