[Suspend-devel] openssl and licence issues

2006-06-26 Thread Tim Dijkstra
Hi, At the moment I'm making a debian package for suspend/uswsusp. Suspend has an an option for encrypting which make use of the libssl library. Unfortunately linking libssl into an GPL application makes the resulting binary undistributable (see [0] for more info.). Of course this is not a proble

[Suspend-devel] s2both and s2disk

2006-06-27 Thread Tim Dijkstra
Hi all, While packaging suspend I ran in a few design problems, which I solved (IMHO) in an elegant way. First the fact that if you want to both suspend-to-disk and suspend-to-both in a regular basis you need to config files which are identical except for a line 'suspend to both ='. This is not n

[Suspend-devel] whitelist entries

2006-06-27 Thread Tim Dijkstra
Hi, I added my machines to the whitelist.c. One is a desktop system, 's2ram -n' only identifies it with the bios version. Also the work around is likely to be only necessary because of the video card (nvidia riva tnt) I use, isn't it? So I don't know if it is wise to add it. Seems like a deficienc

Re: [Suspend-devel] s2both and s2disk

2006-06-27 Thread Tim Dijkstra
On Tue, 27 Jun 2006 21:08:32 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > While packaging suspend I ran in a few design problems, which I > > solved (IMHO) in an elegant way. > > > > First the fact that if you want to both suspend-to-disk and > > suspend-to-both in a regular basis y

Re: [Suspend-devel] s2both and s2disk

2006-06-27 Thread Tim Dijkstra
[ Not need to cc, I joined the list ] On Tue, 27 Jun 2006 21:22:40 +0200 Luca <[EMAIL PROTECTED]> wrote: > Il Tue, Jun 27, 2006 at 01:11:23PM +0200, Tim Dijkstra ha scritto: > > Second, suspend is not a good name for a binary. Because (as other > > people have brought up

Re: [Suspend-devel] s2both and s2disk

2006-06-28 Thread Tim Dijkstra
[ Not need to cc, I joined the list ] On Wed, 28 Jun 2006 00:36:10 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > I guess that having s2ram and s2disk makes sense. Not sure about > s2both, I hate hardlink tricks... but I guess we can do that. They're actually symlink, but I guess you hate those

Re: [Suspend-devel] whitelist entries

2006-06-28 Thread Tim Dijkstra
On Wed, 28 Jun 2006 00:29:13 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > I added my machines to the whitelist.c. One is a desktop system, > > 's2ram -n' only identifies it with the bios version. Also the work > > around is likely to be only necessary because of the video card > > (n

Re: [Suspend-devel] s2both and s2disk

2006-06-29 Thread Tim Dijkstra
On Wed, 28 Jun 2006 21:51:08 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Wednesday 28 June 2006 16:10, Pavel Machek wrote: > > Hi! > > > > > [ Not need to cc, I joined the list ] > > > > I'll cc you anyway; because otherwise _other_ Ccs will get lost (and > > some people -- like me

[Suspend-devel] Which archs are supported?

2006-06-30 Thread Tim Dijkstra
Hi all, As you all know by now I'm perfecting my debian package of uswsusp. Debian currently supprts on the order of 10 archs. In was wondering what archs are supported by uswsusp. I guess the s2ram bit is powerpc, i386 and amd. But for s2disk you more or less only need swap, right? Does anybody

Re: [Suspend-devel] Which archs are supported?

2006-06-30 Thread Tim Dijkstra
On Fri, 30 Jun 2006 11:22:20 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Hi, > > On Friday 30 June 2006 10:44, Tim Dijkstra wrote: > > As you all know by now I'm perfecting my debian package of uswsusp. > > Debian currently supprts on the ord

Re: [Suspend-devel] Which archs are supported?

2006-06-30 Thread Tim Dijkstra
On Fri, 30 Jun 2006 22:44:42 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > > > > As you all know by now I'm perfecting my debian package of > > > > > uswsusp. Debian currently supprts on the order of 10 archs. > > > > > In was wondering what archs are supported by uswsusp. I guess > >

Re: [Suspend-devel] [PATCH] initramfs generation with yaird

2006-07-03 Thread Tim Dijkstra
On Mon, 03 Jul 2006 11:46:53 -0400 Marten van Kerkwijk <[EMAIL PROTECTED]> wrote: > Hi, > > before anything, congratulations and thanks for making this nice > project! > > But this is mostly to confirm that the patch for yaird suggested by > Jason Lunz works well, though I had to do two fairly o

[Suspend-devel] Userspace splash?

2006-07-07 Thread Tim Dijkstra
Hi, I'm trying to integrate another splash system into suspend. It's a userspace splash program (splashy) so it should be easier for people to use (no kernel patches), but 'm not really sure if it's going to work at all. Being a userspace program, is it going to be frozen? I guess it will fro

Re: [Suspend-devel] [PATCH] Fix console ioctls in suspend.c:suspend_system()

2006-07-07 Thread Tim Dijkstra
On Fri, 7 Jul 2006 13:25:20 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Hi, > > Apparently we have used wrond console ioctls around freeze() in > suspend_system() and in a wrong order. The appended patch fixes > this. > > Comments welcome. Just some questions. You told me this was a

Re: [Suspend-devel] [PATCH] Fix console ioctls in suspend.c:suspend_system()

2006-07-07 Thread Tim Dijkstra
On Fri, 7 Jul 2006 14:44:19 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Friday 07 July 2006 14:30, Tim Dijkstra wrote: > > On Fri, 7 Jul 2006 13:25:20 +0200 > > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > > > >

Re: [Suspend-devel] [PATCH] Fix console ioctls in suspend.c:suspend_system()

2006-07-07 Thread Tim Dijkstra
On Fri, 7 Jul 2006 23:23:07 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Friday 07 July 2006 21:13, Rafael J. Wysocki wrote: > > On Friday 07 July 2006 15:02, Tim Dijkstra wrote: > > > On Fri, 7 Jul 2006 14:44:19 +0200 > > > &q

[Suspend-devel] Need #include to compile

2006-08-09 Thread Tim Dijkstra
Hi, With the libc6-dev in current debian sid, I need to include to compile. Else gcc will complain swsusp.h:141: error: "SYNC_FILE_RANGE_WRITE" undeclared This problem was hidden by the fact that that constant is defined in swsusp.h if SYS_sync_file_range was not, which wasn't defined

[Suspend-devel] s2disk: potential problems with HPA (host protected area)

2006-08-09 Thread Tim Dijkstra
Hi, I got a bug report in the debian BTS which is beyond my knowledge. Could one of you shed some light on the issue? Thanks, Tim Forwarded message: Machine: IBM Thinkpad R50p I recognized following issues concerning s2ram/s2disk/s2both: - calling s2ram works fine, resuming from RAM work

Re: [Suspend-devel] s2disk: potential problems with HPA (host protected area)

2006-08-11 Thread Tim Dijkstra
On Wed, 9 Aug 2006 17:13:47 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > It is not related to uswsusp, it is related to suspend to ram. The kernel > disables the HPA on boot. The harddrive is power cycled during suspend to > RAM. After resume, HPA is not disabled again. => boom. OK, so the

Re: [Suspend-devel] s2disk: potential problems with HPA (host protected area)

2006-08-11 Thread Tim Dijkstra
On Fri, 11 Aug 2006 12:04:14 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > On Fri, Aug 11, 2006 at 11:50:50AM +0200, Tim Dijkstra wrote: > > On Wed, 9 Aug 2006 17:13:47 +0200 > > Stefan Seyfried <[EMAIL PROTECTED]> wrote: > > > I will have a novell bu

[Suspend-devel] Splash support

2006-08-28 Thread Tim Dijkstra
Hi, As I announced a long time ago, I'm trying to get uswsusp to work with splashy. I've been working on getting splashy to expose a nice library interface which uswsusp can hook into. It is now more or less working, but before I produce a patch I have a few questions. The splash stucture as it i

[Suspend-devel] Fw: Does not properly resume from S3 with kernel 2.6.17

2006-08-28 Thread Tim Dijkstra
Hi, Christian can't get s2ram to work to work with a 2.6.17 kernel, although it is in the whitelist. It used to work with 2.6.16. Do you now of any regressions? Could anybody give him a hand in trying to get it to work again? In the mean time I asked him to try with minimal number modules loaded

Re: [Suspend-devel] Splash support

2006-08-28 Thread Tim Dijkstra
On Mon, 28 Aug 2006 11:28:19 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > As I announced a long time ago, I'm trying to get uswsusp to work with > > splashy. I've been working on getting splashy to expose a nice library > > interface which uswsusp can hook into. It is now more or les

[Suspend-devel] [PATCH] PARAM_NO of by one

2006-09-03 Thread Tim Dijkstra
Hi, If I can count, than PARAM_NO is of by one;) GEN_PARAM is defined as 8, which is correct because there is no #ifdef SPLASH_CONFIG anymore. SPLASH_PARAM still adds one to PARAM_NO though. This crashes s2disk/resume when there's an error in the config file. --- swsusp.h-- 2006-09-03 20:56:3

Re: [Suspend-devel] [PATCH] PARAM_NO of by one

2006-09-03 Thread Tim Dijkstra
On Sun, 3 Sep 2006 22:49:57 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > On Sun 2006-09-03 21:06:12, Tim Dijkstra wrote: > > Hi, > > > > If I can count, than PARAM_NO is of by one;) > > > > GEN_PARAM is defined as 8, which is correct because there

Re: [Suspend-devel] Makefile cleanup

2006-09-11 Thread Tim Dijkstra
On Mon, 11 Sep 2006 11:33:52 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > I also tried to make the use and ordering of "-c" and "-o" to $(CC) a bit > more consistent throughout the Makefile. > > Any objections? Why don't you use something like vbetool/vbetool.o: vbetool/vbetool.c

[Suspend-devel] 1/5 splashy support

2006-09-12 Thread Tim Dijkstra
Hi, I'm sending some patches adding support for splashy (http://splashy.alioth.debian.org). It is a bootsplash system implemented in user space using the directfb library. The experimental branch in svn (svn://svn.debian.org/svn/splashy/branches/0.3) now produces a library to which uswsusp can lin

[Suspend-devel] 3/5 splashy support

2006-09-12 Thread Tim Dijkstra
Some changes related to output and user interaction. - Change some printf(...)s into fprintf(stderr,...)s - Remove one "Press enter to continue", if it's triggered it would always trigger a second instance further down the code path. - Make print_checksum print to a buffer instead of stdout Ind

[Suspend-devel] 2/5 splashy support

2006-09-12 Thread Tim Dijkstra
The progress would halt at 20% for a while. This will make it run nice and continuously. Index: suspend.c === --- suspend.c (revision 158) +++ suspend.c (revision 159) @@ -425,8 +425,7 @@ break;

[Suspend-devel] 4/5 splashy support

2006-09-12 Thread Tim Dijkstra
Splashy can display a popup if it needs user interaction. Make user queries go through the splash struct to be able to use that. - Remove to_silent/to_verbose these are private to bootsplash - Add read_password/getchar to the struct - Ask the password not from within encrypt_init, but separately.

[Suspend-devel] 5/5 splashy support

2006-09-12 Thread Tim Dijkstra
+ * + * Copyright (C) 2006 Tim Dijkstra <[EMAIL PROTECTED]> + * + * This file is released under the GPLv2. + * + */ + +#ifdef CONFIG_SPLASHY +#include + +#include + +#include "encrypt.h" +#include "splashy_funcs.h" + +int splashy_open() //char *mode) +{ +char * mod

Re: [Suspend-devel] 1/5 splashy support

2006-09-14 Thread Tim Dijkstra
On Wed, 13 Sep 2006 22:52:26 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Hi, > > On Tuesday, 12 September 2006 11:02, Tim Dijkstra wrote: > > > > Also it removes the ioctls around freeze(), IIRC this is an attempt > > to lock against VT-sw

Re: [Suspend-devel] 1/5 splashy support

2006-09-14 Thread Tim Dijkstra
On Thu, 14 Sep 2006 10:34:38 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > > > Also it removes the ioctls around freeze(), IIRC this is an attempt > > > > to lock against VT-switching. The way it is implemented now will > > > > make s2disk switch away from the splash VT. IMHO it is

Re: [Suspend-devel] 1/5 splashy support

2006-09-14 Thread Tim Dijkstra
On Thu, 14 Sep 2006 12:55:06 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Hi, > > On Thursday, 14 September 2006 11:04, Pavel Machek wrote: > > Hi! > > > > > > I thought Rafael is going to apply them... he has CVS write access, > > too. Or just send me a patch (one, I'm lazy) and a ch

Re: [Suspend-devel] 1/5 splashy support

2006-09-14 Thread Tim Dijkstra
On Thu, 14 Sep 2006 14:29:34 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > On Thu 2006-09-14 13:22:29, Tim Dijkstra wrote: > > On Thu, 14 Sep 2006 12:55:06 +0200 > > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > > > > Hi, > > &

Re: [Suspend-devel] 1/5 splashy support

2006-09-14 Thread Tim Dijkstra
On Thu, 14 Sep 2006 17:10:18 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > -(a) [optional] If you want to use the compression and/or encryption > > -capabilities of the suspend tools and you have installed the necessary > > -packages referred to in subsection 2), you need to set > > -CO

Re: [Suspend-devel] 1/5 splashy support

2006-09-14 Thread Tim Dijkstra
On Thu, 14 Sep 2006 17:26:45 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Thursday, 14 September 2006 17:22, Tim Dijkstra wrote: > > On Thu, 14 Sep 2006 17:10:18 +0200 > > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > >

[Suspend-devel] Lock VT against switching

2006-09-14 Thread Tim Dijkstra
Hi, This patch will lock the VT that is active just after splash is initialized and release it just before the splash system is stopped. I haven't tested it extensively with bootsplash.org, but it seems to work with that (in vmware) too. Comments? Index: suspend.c ==

Re: [Suspend-devel] Lock VT against switching

2006-09-15 Thread Tim Dijkstra
On Fri, 15 Sep 2006 11:06:36 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > This patch will lock the VT that is active just after splash is > > initialized and release it just before the splash system is stopped. > > > > I haven't tested it extensively with bootsplash.org, but it seem

Re: [Suspend-devel] Lock VT against switching

2006-09-15 Thread Tim Dijkstra
On Fri, 15 Sep 2006 22:15:09 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Friday, 15 September 2006 21:39, Tim Dijkstra wrote: > > On Fri, 15 Sep 2006 11:06:36 +0200 > > Pavel Machek <[EMAIL PROTECTED]> wrote: > > > > > Hi! >

[Suspend-devel] Lock VT against switching, new patch

2006-09-16 Thread Tim Dijkstra
Hi, Here's a new version of the patch: Index: suspend.c === RCS file: /cvsroot/suspend/suspend/suspend.c,v retrieving revision 1.52 diff -u -r1.52 suspend.c --- suspend.c 14 Sep 2006 14:18:58 - 1.52 +++ suspend.c 16 Sep

[Suspend-devel] Small fixes, clean ups

2006-09-16 Thread Tim Dijkstra
Hi, Three small cleanups: - Remove s2both binary at 'make clean' - Remove unused #define - Add \n to message, looks better imho. Index: Makefile === diff -u -r1.36 Makefile --- Makefile14 Sep 2006 14:58:21 - 1.36 +++ Ma

Re: [Suspend-devel] Lock VT against switching, new patch

2006-09-16 Thread Tim Dijkstra
On Sat, 16 Sep 2006 10:00:19 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Hi, > > On Saturday, 16 September 2006 09:37, Tim Dijkstra wrote: > > Hi, > > > > Here's a new version of the patch: > > Looks good to me. > > Gree

[Suspend-devel] Make splash.getchar show prompt

2006-09-16 Thread Tim Dijkstra
Hi, Because splashy by default doesn't show the printf statements, the user wouldn't know that he/she would have to push a button on splash.getchar. This adds a prompt to splash.getchar Index: resume.c === diff -u -r1.29 resume.c ---

Re: [Suspend-devel] Make splash.getchar show prompt

2006-09-18 Thread Tim Dijkstra
On Sat, 16 Sep 2006 22:41:08 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Can we rename this thing to splash.dialog()? I don't think .getchar() > should print. Sound sensible ;) > The rest looks good, but I think we should ask Holger if he agrees with it. _This_ patch doesn't changes

Re: [Suspend-devel] [PATCH] smooth image writeout progress display

2006-09-18 Thread Tim Dijkstra
On Sun, 17 Sep 2006 20:05:29 -0400 Jason Lunz <[EMAIL PROTECTED]> wrote: > > Increase the granularity of calls to start_writeout() when > early_writeout is used. This results in a smooth progress display > during writeout, rather than a noticeable pause each 20% of the way > through. Yes, I noti

Re: [Suspend-devel] Make splash.getchar show prompt

2006-09-18 Thread Tim Dijkstra
On Mon, 18 Sep 2006 11:35:56 +0200 Tim Dijkstra <[EMAIL PROTECTED]> wrote: > Index: bootsplash.h > === > diff -u -r1.2 bootsplash.h > --- bootsplash.h 14 Sep 2006 14:18:58 - 1.2 > +++ bootsplash.h

Re: [Suspend-devel] [PATCH] smooth image writeout progress display

2006-09-18 Thread Tim Dijkstra
On Mon, 18 Sep 2006 12:01:05 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > > I think performance shouldn't be hurt, since we're still batching > > > pages 1% at a time, down from 20%. A normal image has 10's of > > > thousands of pages (mine 512M laptop is ~55000 pages, for example) s

Re: [Suspend-devel] [PATCH] smooth image writeout progress display

2006-09-18 Thread Tim Dijkstra
On Mon, 18 Sep 2006 13:27:19 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Hi, > > On Monday, 18 September 2006 13:17, Pavel Machek wrote: > > > > "It is an option, so it is not important to get it right" is > > > > ugly. It would be nice to actually benchmark this, > > > > > > Agreed.

Re: [Suspend-devel] Make splash.getchar show prompt

2006-09-18 Thread Tim Dijkstra
On Mon, 18 Sep 2006 11:50:45 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > > > > Can we rename this thing to splash.dialog()? I don't think .getchar() > > > should print. > > > > Sound sensible ;) > > > > > The rest looks good, bu

Re: [Suspend-devel] Suspend read/write "benchmarks"

2006-09-19 Thread Tim Dijkstra
On Tue, 19 Sep 2006 21:33:14 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Hi, > > On Tuesday, 19 September 2006 20:31, Pavel Machek wrote: > > > > > I'd say this is in agreement with the LZF documentation I > > > > > read. It says the _decompression_ should be (almost) as fast > > > >

Re: [Suspend-devel] Suspend read/write "benchmarks"

2006-09-20 Thread Tim Dijkstra
On Wed, 20 Sep 2006 01:01:26 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Wednesday, 20 September 2006 00:07, Pavel Machek wrote: > > Hi! > > > > > > > > > Yes, certainly. > > > > > > > > > > > > I'd go for 1% steps. If someone finds it slows his machine down, > > > > > > he's the o

[Suspend-devel] Don't make /dev/snapshot patch

2006-09-20 Thread Tim Dijkstra
Hi, Many people have udev, which will make /dev/snapshot dynamically if support is available in the kernel. And if it didn't, making it from the Makefile wouldn't be a solution because it would disappear on the next reboot. This patch adds another CONFIG option to the Makefile to not make /dev/sna

Re: [Suspend-devel] Don't make /dev/snapshot patch

2006-09-20 Thread Tim Dijkstra
On Wed, 20 Sep 2006 12:44:46 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > Many people have udev, which will make /dev/snapshot dynamically if > > support is available in the kernel. And if it didn't, making it from the > > Makefile wouldn't be a solution because it would disappear on

Re: [Suspend-devel] Don't make /dev/snapshot patch

2006-09-20 Thread Tim Dijkstra
On Wed, 20 Sep 2006 14:38:53 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > On Wed 2006-09-20 14:20:19, Stefan Seyfried wrote: > > On Wed, Sep 20, 2006 at 02:12:30PM +0200, Tim Dijkstra wrote: > > > > > But I think those two lines do not hurt and saves packagers (m

Re: [Suspend-devel] freeze everything but s2ram?

2006-10-06 Thread Tim Dijkstra
On Fri, 6 Oct 2006 15:54:08 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > Hi, > > i just had an idea :-) > > Is it possible, by using the uswsusp infrastructure, to freeze everything > but the s2ram process during suspend to RAM? I mean the following: > > start s2ram > - chvt away from X >

Re: [Suspend-devel] freeze everything but s2ram?

2006-10-06 Thread Tim Dijkstra
On Fri, 6 Oct 2006 21:58:40 +0200 Tim Dijkstra <[EMAIL PROTECTED]> wrote: > On Fri, 6 Oct 2006 15:54:08 +0200 > Stefan Seyfried <[EMAIL PROTECTED]> wrote: > > > Hi, > > > > i just had an idea :-) > > > > Is it possible, by using the uswsu

Re: [Suspend-devel] freeze everything but s2ram?

2006-10-07 Thread Tim Dijkstra
On Sat, 7 Oct 2006 14:51:31 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > On Fri, Oct 06, 2006 at 10:30:25PM +0200, Tim Dijkstra wrote: > > > > Just curious; why do you ever want to do s2ram instead of s2disk? > > > > I meant s2both, duh... > > It

Re: [Suspend-devel] refactor console code / get rid of vt.[ch]

2006-10-08 Thread Tim Dijkstra
On Sat, 7 Oct 2006 20:39:22 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > good. I'll wait for some mor ACKs before committing, so whoever knows > something about consoles or had something to do with splash code, please > take a look at that patch. Thanks. Looks good to me. It's mostly moving

[Suspend-devel] [PATCH] Minor change in wording

2006-10-11 Thread Tim Dijkstra
Hi, I got a bug report telling me that one of the prompts in resume is confusing and misses a newline. This fixes that. Index: resume.c === RCS file: /cvsroot/suspend/suspend/resume.c,v retrieving revision 1.32 diff -u -r1.32 resume.

Re: [Suspend-devel] Release, maybe?

2006-10-13 Thread Tim Dijkstra
On Fri, 13 Oct 2006 00:43:55 +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > > Unfortunately I now have only a little time for programming due to my work > > on the university, but I hope to be less busy around 11 November, so I'd > > like > > to make a release at that time. That's a go

Re: [Suspend-devel] Suspend with cryptsetup encrypted swap?

2006-10-16 Thread Tim Dijkstra
On Mon, 16 Oct 2006 13:26:16 +0200 Matej Cepl <[EMAIL PROTECTED]> wrote: > Hi, > > is something like this possible? I have here notebook with Fedora Core > 5 with swap encrypted using cryptsetup create. Is it possible to > suspend/resume such notebook with ususpend? > Dunno much about encrypted

Re: [Suspend-devel] Suspend with cryptsetup encrypted swap?

2006-10-30 Thread Tim Dijkstra
On Sun, 29 Oct 2006 23:15:15 + (UTC) "Matej Cepl" <[EMAIL PROTECTED]> wrote: > Jason Lunz scripst: > > Yes, it works just fine. I've been running a laptop with dm-crypt on > > swap and root for months now - only /boot is unencrypted. > > That's cool -- more I am thinking about that more I don

Re: [Suspend-devel] [PATCH] Avoid modifying file access time after creating the image

2006-10-30 Thread Tim Dijkstra
On Mon, 30 Oct 2006 00:20:50 +0100 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Unfortunately in its current form s2disk causes the access time of the resume > device special file to be updated after the suspend image has been created, > which is potentially dangerous. Why is that dangerous?

Re: [Suspend-devel] [PATCH] Allow users to abort image saving

2006-11-02 Thread Tim Dijkstra
Op Thu, 2 Nov 2006 08:07:36 +0100 schreef "Rafael J. Wysocki" <[EMAIL PROTECTED]>: > Hi, > > The appended patch allows the users of suspend to abort the image > saving by pressing Ctrl+c. > > Comments welcome. This is not going to work for splashy. We will need wrap the call to 'read' in the sp

Re: [Suspend-devel] [PATCH] Allow users to abort image saving

2006-11-02 Thread Tim Dijkstra
Op Thu, 2 Nov 2006 16:10:04 +0100 schreef "Rafael J. Wysocki" <[EMAIL PROTECTED]>: > On Thursday, 2 November 2006 14:49, Tim Dijkstra wrote: > > Op Thu, 2 Nov 2006 08:07:36 +0100 > > schreef "Rafael J. Wysocki" <[EMAIL PROTECTED]>: > > >

Re: [Suspend-devel] [PATCH] Allow users to abort image saving

2006-11-02 Thread Tim Dijkstra
On Thu, 2 Nov 2006 17:20:39 +0100 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Thursday, 2 November 2006 16:55, Tim Dijkstra wrote: > > Op Thu, 2 Nov 2006 16:10:04 +0100 > > schreef "Rafael J. Wysocki" <[EMAIL PROTECTED]>: > > > &g

Re: [Suspend-devel] results report for Dell Inspiron 1300

2006-11-16 Thread Tim Dijkstra
On Thu, 16 Nov 2006 12:26:45 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > If the framebuffer is really a problem, it might of course be that Tim just > never tried it with a framebuffer driver. I don't remember. I will check this when I get my hands on this laptop again. grts Tim signatu

[Suspend-devel] Remove useless errror messages

2006-11-29 Thread Tim Dijkstra
Hi guys, If the kernel doesn't have support for platform mode, uswsusp complains loudly. suspend: pm_ops->prepare returned error -1 suspend: Snapshotting system ... suspend: Saving image data pages ... suspend: pm_ops->enter returned error -1, calling power_off These are harmless messages, becau

Re: [Suspend-devel] Remove useless errror messages

2006-11-29 Thread Tim Dijkstra
On Wed, 29 Nov 2006 12:09:02 +0100 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Hi, > > On Wednesday, 29 November 2006 11:59, Tim Dijkstra wrote: > > Hi guys, > > > > If the kernel doesn't have support for platform mode, uswsusp complai

Re: [Suspend-devel] Remove useless errror messages

2006-11-30 Thread Tim Dijkstra
Op Thu, 30 Nov 2006 08:11:09 +0100 schreef Stefan Seyfried <[EMAIL PROTECTED]>: > Hi, > > On Wed, Nov 29, 2006 at 10:16:12PM +0100, Tim Dijkstra wrote: > > On Wed, 29 Nov 2006 12:09:02 +0100 > > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > &

Re: [Suspend-devel] Remove useless errror messages

2006-11-30 Thread Tim Dijkstra
Op Thu, 30 Nov 2006 13:53:46 +0100 schreef Stefan Seyfried <[EMAIL PROTECTED]>: > > BTW, what is the first kernel with the correct code? It's not in > > 2.16.18, is it? Ah, I see they just released 2.6.19, is it in there? > > I think it is (i pushed it through Andrew), but i have not checked it.

[Suspend-devel] RSA_DATA_SIZE to small

2006-12-11 Thread Tim Dijkstra
Hi, suspend-keygen fails silently generating keys larger than approximately 3500 bits. The problem is that RSA_DATA_SIZE is to small for the test if (offset + size >= RSA_DATA_SIZE) to pass. Although RSA_DATA_SIZE is big enough to accommodate the biggest keys in practice. The theoretical size th

[Suspend-devel] Fw: My machine is not in the whitelist of the s2ram/s2both

2006-12-11 Thread Tim Dijkstra
Hi, I have one for the whitelist. OK, to commit? grts Tim Begin forwarded message: Date: Sat, 09 Dec 2006 22:22:44 +0100 From: Andreas Mainik <[EMAIL PROTECTED]> To: Debian Bug Tracking System <[EMAIL PROTECTED]> Subject: Bug#402365: uswsusp: My machine is not in the whitelist of the s2ram/s2

Re: [Suspend-devel] Fw: My machine is not in the whitelist of the s2ram/s2both

2006-12-11 Thread Tim Dijkstra
On Mon, 11 Dec 2006 14:43:46 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > On Mon, Dec 11, 2006 at 01:49:44PM +0100, Tim Dijkstra wrote: > > Please insert my machine in the whitelist. > > > > #s2ram -i > > > > This machine can be identified

Re: [Suspend-devel] Fw: My machine is not in the whitelist of the s2ram/s2both

2006-12-11 Thread Tim Dijkstra
On Mon, 11 Dec 2006 17:26:01 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > On Mon, Dec 11, 2006 at 05:01:49PM +0100, Tim Dijkstra wrote: > > > You are right of course. This of course means that the matching as done know > > is broken. > > It is "incompl

[Suspend-devel] [PATCH] Move abort logic to spash struct

2007-01-09 Thread Tim Dijkstra
Hi, To be able to abort suspending while using splashy, we have to move the abort logic to the splashy struct so it can be overridden when libsplashy is used for splashy support. OK to commit? grts Tim Index: splash.c === RCS file:

Re: [Suspend-devel] [PATCH] Move abort logic to spash struct

2007-01-10 Thread Tim Dijkstra
On Wed, 10 Jan 2007 08:38:13 +0100 Pavel Machek <[EMAIL PROTECTED]> wrote: > > +ret = !tcgetattr(0, oldtrm); > > > +if (ret) { > > +*newtrm = *oldtrm; > > +newtrm->c_cc[VMIN] = 0; > > +newtrm->c_cc[VTIME] = 1; > > +

Re: [Suspend-devel] [PATCH] Move abort logic to spash struct

2007-01-10 Thread Tim Dijkstra
Hi, Here is an updated patch: Index: splash.c === RCS file: /cvsroot/suspend/suspend/splash.c,v retrieving revision 1.4 diff -u -r1.4 splash.c --- splash.c18 Sep 2006 12:00:09 - 1.4 +++ splash.c10 Jan 2007 11:43:45 -

[Suspend-devel] resume in /usr/sbin

2007-01-10 Thread Tim Dijkstra
Hi, I just noticed that the new version of the Makefile installs resume in /usr/sbin (or whatever you configured $(DESTDIR)$(SUSPEND_DIR) to be). Is there any reason to do that? It's not that you would be calling it by hand on a normal running system... Of course you need the binary on initrd or

Re: [Suspend-devel] resume in /usr/sbin

2007-01-10 Thread Tim Dijkstra
On Wed, 10 Jan 2007 13:10:49 +0100 Tim Dijkstra <[EMAIL PROTECTED]> wrote: > Hi, > > I just noticed that the new version of the Makefile installs resume > in /usr/sbin (or whatever you configured $(DESTDIR)$(SUSPEND_DIR) to be). > Is there any reason to do that? It&#x

Re: [Suspend-devel] [PATCH] writeout_time not saved with CONFIG_ENCRYPT=yes

2007-01-22 Thread Tim Dijkstra
On Mon, 22 Jan 2007 18:48:30 +0100 Michal Schmidt <[EMAIL PROTECTED]> wrote: > Consider adding this one too. It makes RSA_data itself much smaller. Not > all RSA components need to take 512 bytes: Yes I noticed this too, but it didn't seem relevant at the time. Patch looks good. grts Tim -

[Suspend-devel] Reordering in resume 0/2

2007-01-22 Thread Tim Dijkstra
Hi, I got some bug reports and unwanted behavior by resume. The problem boiled down to the fact that resume treats 'no image in partition' as a regular, while it is of course expected behavior on a normal boot. Some symptoms that will be fixed by these patches - resume sets console loglevel to so

[Suspend-devel] Reordering in resume 1/2

2007-01-22 Thread Tim Dijkstra
This adds some functions to get_kernel_console_loglevel, inspired by those in suspend.c. We could also move parts to a shared file, if desired. Also sets kernel_loglevel back to the original if there were no errors. --- ../suspend-cvs-1/resume.c 2007-01-15 16:07:41.0 +0100 +++ resume.c

[Suspend-devel] Reordering in resume 2/2

2007-01-22 Thread Tim Dijkstra
Split the opening of resume_dev and retrieval of the header from read_image into a new function. That way we can do something else if everything is OK, but there just isn't an image. The new function return ENOMEDIUM in case of no image, better suggestions welcome. diff -u ../suspend-cvs-1/resume

Re: [Suspend-devel] Reordering in resume 0/2

2007-01-23 Thread Tim Dijkstra
On Tue, 23 Jan 2007 10:08:44 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > On Mon, Jan 22, 2007 at 11:43:21PM +0100, Tim Dijkstra wrote: > > Hi, > > > First patch is somewhat unrelated. While fixing this I was trying to > > get the logic of read_image(). The la

Re: [Suspend-devel] Reordering in resume 2/2

2007-01-23 Thread Tim Dijkstra
On Tue, 23 Jan 2007 13:00:08 +0100 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Tuesday, 23 January 2007 10:51, Pavel Machek wrote: > > Hi! > > > > > Split the opening of resume_dev and retrieval of the header from > > > read_image into a new function. That way we can do something else if

Re: [Suspend-devel] Reordering in resume 1/2

2007-01-24 Thread Tim Dijkstra
loglevel.c 24 Jan 2007 12:52:19 - @@ -0,0 +1,63 @@ +/* loglevel.c - routines to modify kernel console loglevel + * + * Released under GPL v2. + * (c) 2007 Tim Dijkstra + */ + +#include +#include +#include +#include +#include +#include + + +static FILE *printk_file; +static int proc_mount

[Suspend-devel] getconsolefd

2007-01-24 Thread Tim Dijkstra
Hi, I was skimming trough the code some more when I found some more duplicated code. getconsolefd in vt.c and console_fd suspend.c. The differences are that console_fd opens like: fd = open(fname, O_RDONLY); if (fd < 0 && errno == EACCES) fd = open(fname, O_WRONLY); and g

[Suspend-devel] Cleaning up Makefile

2007-01-25 Thread Tim Dijkstra
Hi, While adding the loglevel.[ch], I've got a bit annoyed with the makefile, it has duplication of stuff all over it. I'm planning to clean it up a bit. Some question before I do that. Doe any body mind if I rename the target install-resume => install-resume-on-initrd and instal

Re: [Suspend-devel] Cleaning up Makefile

2007-01-25 Thread Tim Dijkstra
On Thu, 25 Jan 2007 19:04:37 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > > Why is it whitelist.c and whitelist.h? /\ forgot a NOT here ---/ > > I only have whitelist.c? I mean it is more a header file, we don't compile it, it is included. I would th

[Suspend-devel] New Makefile

2007-01-31 Thread Tim Dijkstra
Hi, Here is my shot at a new Makefile. What do you think? grts Tim === #CONFIG_COMPRESS=yes #CONFIG_ENCRYPT=yes #CONFIG_SPLASHY=yes #CONFIG_UDEV=yes #CONFIG_RESUME_DYN=yes SUSPEND_DIR=/usr/local/sbin RESUME_DIR=/usr/local/lib/suspend CONFIG_DIR=/etc RESUME_DEVICE= BO

[Suspend-devel] What kernel has support for ...?

2007-02-02 Thread Tim Dijkstra
Hi, Could someone please tell me what (stable) kernels have support for: a) Suspend/Resume with swap files b) Platform mode instead And is there a easy way to test for swap file support, some /sys entry perhaps? Thanks, Tim -

Re: [Suspend-devel] What kernel has support for ...?

2007-02-02 Thread Tim Dijkstra
On Fri, 2 Feb 2007 13:28:16 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > > > b) Platform mode instead > > > > I'm not sure what you mean here ... > > i guess it is SNAPSHOT_PMOPS... > > Ah, yes. The 'instead' is a remnant of a larger sentence that I cut out saying something like 'inst

[Suspend-devel] Using a swap file

2007-02-03 Thread Tim Dijkstra
Hi, I'm testing to see if I can suspend/resume using a swap file. I have created a swap file on one of my (lvm) partitions and have the the following lines in my config file (which I got with swap-offset): resume device = /dev/mapper/vg_cs-local resume offset = 12979490 s2disk refuses to suspend

Re: [Suspend-devel] Using a swap file

2007-02-03 Thread Tim Dijkstra
On Sat, 3 Feb 2007 23:52:18 +0100 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > Hi, > > On Saturday, 3 February 2007 23:10, Tim Dijkstra wrote: > > Hi, > > > > I'm testing to see if I can suspend/resume using a swap file. I have > > crea

[Suspend-devel] suspend on ppc /w pmu

2007-02-07 Thread Tim Dijkstra
Hi Guys, At the moment we support only ix86_{32,64} machines. I personally do not have a ppc machine, but apparently bringing it in a s2ram state is pretty easy. It is something along the lines of: fd = open("/dev/pmu", O_RDWR); ioctl(fd, PMU_IOC_SLEEP, arg); If we integrate this into the s2ram

Re: [Suspend-devel] Bug#410320: s2ram: whitelist entry

2007-02-12 Thread Tim Dijkstra
Hi Guys, I just got this report On Fri, 9 Feb 2007 18:19:59 +0100 Bill Allombert <[EMAIL PROTECTED]> wrote: > Package: uswsusp > Version: 0.3~cvs20060928-6 > Severity: wishlist > > Hello Tim, > > With etch-amd64 on my new laptop, s2ram work fine with that option: > > #s2ram -f -a 1 > > Here

Re: [Suspend-devel] Failure to resume from suspend to disk

2007-02-14 Thread Tim Dijkstra
On Wed, 14 Feb 2007 18:08:27 +0100 Christian Axelsson <[EMAIL PROTECTED]> wrote: > Stefan Seyfried wrote: > > On Wed, Feb 14, 2007 at 05:28:00PM +0100, Christian Axelsson wrote: > >> Hello! > >> > >> Im trying to resume from disk om my dell 420 but after suspending (using > >> s2disk) and passing

Re: [Suspend-devel] [patch] small trivial "make install" fixes

2007-02-14 Thread Tim Dijkstra
On Wed, 14 Feb 2007 12:33:24 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > Hi, > > i need these for a chrooted build (where $DESTDIR is not always present > before make install): > > > --- Makefile > +++ Makefile > @@ -137,10 +137,10 @@ > fi > > install-resume: > - install --m

  1   2   >