Re: [Suspend-devel] vt switch during image write

2007-08-08 Thread Tim Dijkstra
On Tue, 7 Aug 2007 01:50:12 +0300 Alon Bar-Lev [EMAIL PROTECTED] wrote: The question is: Why cannot I switch from one vt to another during image write? I guess some lock out somewhere at freeze something... I will be glad to be able to switch from silent to verbose modes during image

Re: [Suspend-devel] [PATCH] Set cpufreq to max during suspend/resume

2007-07-22 Thread Tim Dijkstra
On Sun, 22 Jul 2007 07:46:40 +0300 Alon Bar-Lev [EMAIL PROTECTED] wrote: On 7/21/07, Tim Dijkstra [EMAIL PROTECTED] wrote: No. The vbetool code is linked in. It doesn't run as regular processes. At the time we do those hacks, we are the only not-frozen part of userspace. This is what makes

[Suspend-devel] Fw: Bug#392168: Resume device on external USB drive causes the boot process to stop

2007-06-18 Thread Tim Dijkstra
Hi, What do people think about this? ---BeginMessage--- I have an old system (eMachine 433MH) with a new external USB drive (which is actually faster than the internal hard drive that came with the box). It takes a few seconds for the Linux to bring it online, due in some part I think to a 5

Re: [Suspend-devel] Patch, make s2ram --test return exit status.

2007-06-17 Thread Tim Dijkstra
On Fri, 8 Jun 2007 13:56:25 +0200 Stefan Seyfried [EMAIL PROTECTED] wrote: On Thu, Jun 07, 2007 at 10:51:38PM +0200, Tim Dijkstra wrote: Hi, I thought it was useful to be able to get the exit status from s2ram --test. Yes, sounds sane. Maybe we should return failure for those UNSURE

[Suspend-devel] Patch, make s2ram --test return exit status.

2007-06-07 Thread Tim Dijkstra
Hi, I thought it was useful to be able to get the exit status from s2ram --test. grts Tim Index: s2ram-ppc.c === RCS file: /cvsroot/suspend/suspend/s2ram-ppc.c,v retrieving revision 1.1 diff -u -r1.1 s2ram-ppc.c --- s2ram-ppc.c 13

Re: [Suspend-devel] [Patch] fix s2ram -n / be a bit more explicit wrt. UNSURE

2007-05-31 Thread Tim Dijkstra
On Thu, 31 May 2007 18:24:43 +0200 Stefan Seyfried [EMAIL PROTECTED] wrote: Hi, the first hunk fixes s2ram -n, without it does not print what hacks would actually be applied. The other hunks let s2ram complain louder about machines that are still UNSURE. The fedora guys made pretty bad

Re: [Suspend-devel] s2ram broken for Linux 2.6.22, libx86 broken on x86-64

2007-05-20 Thread Tim Dijkstra
On Sun, 20 May 2007 15:27:18 +0200 Stefan Richter [EMAIL PROTECTED] wrote: Rafael wrote: Actually, on x86-64 you have to compile it with $ make BACKEND=x86emu Seife, perhaps we should add this information to README? It is already clearly documented in the HOWTO. Yes please, or add

Re: [Suspend-devel] s2ram broken for Linux 2.6.22, libx86 broken on x86-64

2007-05-20 Thread Tim Dijkstra
On Sun, 20 May 2007 21:02:23 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Sunday, 20 May 2007 20:31, Tim Dijkstra wrote: On Sun, 20 May 2007 15:27:18 +0200 Stefan Richter [EMAIL PROTECTED] wrote: Rafael wrote: Actually, on x86-64 you have to compile it with $ make

Re: [Suspend-devel] s2ram works s2both doesn't

2007-05-17 Thread Tim Dijkstra
On Thu, 17 May 2007 11:59:37 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Thursday, 17 May 2007 00:14, Tim Dijkstra wrote: Hi, I've got an report of a laptop that works with s2ram --vbe_save --vbe_post But doesn't for s2both with the same options. What does work

[Suspend-devel] s2ram works s2both doesn't

2007-05-16 Thread Tim Dijkstra
Hi, I've got an report of a laptop that works with s2ram --vbe_save --vbe_post But doesn't for s2both with the same options. What does work however is s2both -a 1 Somehow the difference in code path in s2both is significant from that of s2ram (which basically does echo mem

Re: [Suspend-devel] x86_64

2007-05-15 Thread Tim Dijkstra
On Tue, 15 May 2007 12:55:04 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Tuesday, 15 May 2007 08:40, Arkadiusz Miskiewicz wrote: Makefile: 1.57 (tdykstra 13-May-07): ARCH := $(shell uname -m | sed -e s/i.86/x86/ -e s/ppc.*/ppc/) x86_64 not supported? Suspend builds

Re: [Suspend-devel] Bug in option handling

2007-05-14 Thread Tim Dijkstra
On Sun, 13 May 2007 22:52:37 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Saturday, 12 May 2007 22:18, Tim Dijkstra wrote: Hi, OK, it turns out that my last patch for adding comments to usage had a horrible bug. Although (struct option_descr) would yield a (struct option

[Suspend-devel] Bug in option handling

2007-05-12 Thread Tim Dijkstra
Hi, OK, it turns out that my last patch for adding comments to usage had a horrible bug. Although (struct option_descr) would yield a (struct option), as anticipated, the increment of the pointers of these structs ofcourse wouldn't... Anyway the patch below fixes this, although also in a bit

Re: [Suspend-devel] Splashy features

2007-05-11 Thread Tim Dijkstra
On Thu, 10 May 2007 22:17:29 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: - } else if (!splashy_open()) { + } else if (!(error = splashy_open(mode))) { I'd prefer error = error = splashy_open(mode); if (error) { and analogously below. huh? Why is that? grts

Re: [Suspend-devel] Comments in usage()

2007-04-23 Thread Tim Dijkstra
On Mon, 23 Apr 2007 11:20:51 +0200 Stefan Seyfried [EMAIL PROTECTED] wrote: On Sat, Apr 21, 2007 at 12:25:24AM +0200, Rafael J. Wysocki wrote: On Saturday, 21 April 2007 00:14, Tim Dijkstra wrote: Hi, S2ram could also use the usage() function from config.c to display its message

Re: [Suspend-devel] swapfiles

2007-04-22 Thread Tim Dijkstra
On Sun, 22 Apr 2007 21:18:46 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Sunday, 22 April 2007 11:40, Rafael J. Wysocki wrote: On Sunday, 22 April 2007 01:02, Tim Dijkstra wrote: On Sat, 21 Apr 2007 23:14:48 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Saturday, 21

Re: [Suspend-devel] swapfiles

2007-04-21 Thread Tim Dijkstra
On Sat, 21 Apr 2007 23:14:48 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Saturday, 21 April 2007 22:36, Tim Dijkstra wrote: On Sat, 21 Apr 2007 11:30:22 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Saturday, 21 April 2007 00:59, Tim Dijkstra wrote: Hi, I'm

[Suspend-devel] swapfiles

2007-04-20 Thread Tim Dijkstra
Hi, I'm trying out sdisk with a swap file. $ cat /proc/swaps FilenameTypeSizeUsedPriority /dev/hda2 partition 979956 189972 -1 /local/tmp/swapfile file1048568 0 -2

Re: [Suspend-devel] Bug#415441: s2disk and raid

2007-04-17 Thread Tim Dijkstra
On Thu, 12 Apr 2007 00:37:53 -0500 Luis Rodrigo Gallardo Cruz [EMAIL PROTECTED] wrote: On Thu, Apr 05, 2007 at 12:47:49AM +0400, Michael Tokarev wrote: Neil Brown wrote: On Tuesday April 3, [EMAIL PROTECTED] wrote: [] After the power cycle the kernel boots, devices are discovered,

Re: [Suspend-devel] [PATCH 2/2] s2ram: add PPC support

2007-04-16 Thread Tim Dijkstra
On Sun, 15 Apr 2007 02:36:12 +0200 Luca Tettamanti [EMAIL PROTECTED] wrote: @@ -27,7 +27,16 @@ int s2ram_do(void) { int ret = 0; - FILE *f = fopen(/sys/power/state, w); + FILE *f; + + /* If this works we're done. Else we just continue as if nothing +* happened,

Re: [Suspend-devel] [PATCH 2/2] s2ram: add PPC support

2007-04-14 Thread Tim Dijkstra
by default, can be overridden). PPC code is based on original patch from Tim Dijkstra. Signed-Off-By: Luca Tettamanti [EMAIL PROTECTED] --- Makefile| 22 +++- s2ram-ppc.c | 75 s2ram-ppc.h |2 + 3 files

Re: [Suspend-devel] [PATCH 2/2] s2ram: add PPC support

2007-04-11 Thread Tim Dijkstra
on original patch from Tim Dijkstra. I haven't looked to closely at it, I can't access my machine right now. But this patch doesn't seem correct. IIRC, s2ram_hacks is called before we really suspend the system, but the ppc version would. I'll take a closer look tomorrow night. Signed-Off-By: Luca

Re: [Suspend-devel] [patch] pass down argv[0] to error messages

2007-04-09 Thread Tim Dijkstra
On Tue, 3 Apr 2007 21:32:39 +0200 Stefan Seyfried [EMAIL PROTECTED] wrote: #define suspend_error(msg, args...) \ do { \ - fprintf(stderr, suspend: msg Reason: %m\n, ## args); \ + fprintf(stderr, %s: msg Reason: %m\n, my_name, ## args); \ } while (0); What is this do { }

[Suspend-devel] CONFIG_PPC

2007-04-05 Thread Tim Dijkstra
Hi, A while back I proposed a patch to support powerpc. The comments where that it was a bit ugly with all those #ifdef's. Now the question is how do I do this neatly? Of all the functions in s2ram there is only one that would be really duplicated. Then there is main() en a few functions that we

Re: [Suspend-devel] s2disk and raid

2007-04-04 Thread Tim Dijkstra
On Wed, 4 Apr 2007 15:20:56 +1000 Neil Brown [EMAIL PROTECTED] wrote: On Tuesday April 3, [EMAIL PROTECTED] wrote: Hi, I've got a bugreport [0] from a user trying to use raid and uswsusp. He's using initramfs-tools available in debian. I'll describe the problem and my analysis, maybe

Re: [Suspend-devel] s2both on powerpc

2007-04-03 Thread Tim Dijkstra
On Tue, 3 Apr 2007 00:39:04 +0200 Cédric Boutillier [EMAIL PROTECTED] wrote: Hi all ! Thanks to Alan Stern's help on usb-devel list, I have a working suspend-to-disk. My USB issues seem to have gone away thanks to his patch. That is good news. I need to unload (resp. reload) some modules

[Suspend-devel] s2disk and raid

2007-04-03 Thread Tim Dijkstra
Hi, I've got a bugreport [0] from a user trying to use raid and uswsusp. He's using initramfs-tools available in debian. I'll describe the problem and my analysis, maybe you can comment on what you think. A warning: I only have a casual understanding of raid, never looked at any code related to

[Suspend-devel] s2ram style options for s2both v2

2007-04-02 Thread Tim Dijkstra
Hi, I polished the patch a bit. I found one additional problem; also `-s' was already in use by both s2disk and s2ram. This let me to decide not to allow any short options for the `s2ram hacks' in s2both. For s2ram they are still supported. If however we would start using the usage function from

Re: [Suspend-devel] s2both on powerpc

2007-04-01 Thread Tim Dijkstra
On Sun, 01 Apr 2007 10:15:22 +0200 Johannes Berg [EMAIL PROTECTED] wrote: On Sat, 2007-03-31 at 22:17 +0200, Tim Dijkstra wrote: Hmm, ok I misunderstood then. I thought that in our previous tests you said it used echo mem ... maybe you were using your patches;) Yes, I do of course run

Re: [Suspend-devel] s2both on powerpc

2007-03-31 Thread Tim Dijkstra
Cédric Boutillier schreef: Hi ! summary Cedric gets a `colorful' screen and no suspend when using echo mem /s/p/s, but using the ioctl on /dev/pmu works OK /summary On 3/30/07, Rafael J. Wysocki [EMAIL PROTECTED] wrote: Which version of the suspend utilities do you use? Well, I do not

Re: [Suspend-devel] s2both on powerpc

2007-03-29 Thread Tim Dijkstra
On Thu, 29 Mar 2007 11:37:52 +0200 Cédric Boutillier [EMAIL PROTECTED] wrote: I have not seen any message from resume. I unpacked the initramfs using your script: everything seems in place: ./etc/uswsusp.conf And has the same contents as the one in your regular /-partition, right? Just

Re: [Suspend-devel] s2both on powerpc

2007-03-29 Thread Tim Dijkstra
On Thu, 29 Mar 2007 00:28:07 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Thursday, 29 March 2007 00:14, Tim Dijkstra wrote: On Tue, 27 Mar 2007 23:11:41 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: diff -u -r1.70 suspend.c --- suspend.c 16 Mar 2007 16:02:22 -

Re: [Suspend-devel] [PATCH] encourage the usage of --long_options

2007-03-29 Thread Tim Dijkstra
Op Thu, 29 Mar 2007 21:58:10 +0200 schreef Stefan Seyfried [EMAIL PROTECTED]: 2) Also pass the short option string to usage and check for each `val' if it is included in the short option string. At first I thought this was a bit overkill, but thinking about it, it is more correct...

Re: [Suspend-devel] s2both on powerpc

2007-03-29 Thread Tim Dijkstra
Op Thu, 29 Mar 2007 23:54:13 +0200 schreef Rafael J. Wysocki [EMAIL PROTECTED]: Okay, so the patch works. Tim, if you don't mind, I'd like to apply this one. Sure, go ahead. If I read it right the cast is now done when calling set_swap_file(). --- suspend.c |2 +- 1 file

Re: [Suspend-devel] s2both on powerpc

2007-03-28 Thread Tim Dijkstra
Hi Cédric, On Wed, 28 Mar 2007 23:03:12 +0200 Cédric Boutillier [EMAIL PROTECTED] wrote: When I reboot after having executed s2both, I see the following message: Invalidating stale suspend image or something similar (I cannot find it in the logs though). I don't know if this helps

Re: [Suspend-devel] s2both on powerpc

2007-03-28 Thread Tim Dijkstra
On Tue, 27 Mar 2007 23:11:41 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: diff -u -r1.70 suspend.c --- suspend.c 16 Mar 2007 16:02:22 - 1.70 +++ suspend.c 27 Mar 2007 20:36:52 - @@ -208,7 +208,7 @@ swap.offset = offset; error = ioctl(dev,

[Suspend-devel] Fw: Bug#416475: Samsung Q35 is not in the whitelist : workaround

2007-03-28 Thread Tim Dijkstra
Hi, Any further question to Jérémie before we can add it to the whitelist? grts Tim Jérémie Delaitre op Wed, 28 Mar 2007 10:57:30 +0200 Package: uswsusp Version: 0.6~cvs20070202-1 The Samsung Q35 is not in the whitelist of s2ram but

[Suspend-devel] powerpc support

2007-03-27 Thread Tim Dijkstra
Hi, This is a first cut at supporting powerpc for s2*. For the s2ram bit it basically removes the vbetool code. This has two reasons. First, there isn't a powerpc version of libx86 yet. Second, the machines that have support for s2ram in the kernel don't need those hacks. As you noticed, I've

[Suspend-devel] What did we resume from?

2007-03-27 Thread Tim Dijkstra
Hi. Sometimes it seems relevant for the thing calling s2both to know what route we took in the end, be it s2ram or s2disk. Would you think returning a value is a suitable way of communicating that to the user? Would some scripts maybe break, because they only expect 0 in case of success? grts

Re: [Suspend-devel] powerpc support

2007-03-27 Thread Tim Dijkstra
On Tue, 27 Mar 2007 11:55:03 +0200 Luca Tettamanti [EMAIL PROTECTED] wrote: On 3/27/07, Tim Dijkstra [EMAIL PROTECTED] wrote: This is a first cut at supporting powerpc for s2*. For the s2ram bit it basically removes the vbetool code. This has two reasons. First, there isn't a powerpc

Re: [Suspend-devel] What did we resume from?

2007-03-27 Thread Tim Dijkstra
On Tue, 27 Mar 2007 12:11:34 +0200 Rafael J. Wysocki [EMAIL PROTECTED] wrote: On Tuesday, 27 March 2007 11:51, Tim Dijkstra wrote: Hi. Sometimes it seems relevant for the thing calling s2both to know what route we took in the end, be it s2ram or s2disk. Would you think returning

Re: [Suspend-devel] s2both on powerpc

2007-03-27 Thread Tim Dijkstra
On Tue, 27 Mar 2007 13:23:44 +0200 Cédric Boutillier [EMAIL PROTECTED] wrote: Hi ! Apparently, resume_dev = 268462812 before calling set_swap_file(). I've no idea where these 0:0 come from, but I do have idea why it doesn't work. I don't know how to fix that right now, though.

Re: [Suspend-devel] s2both on powerpc

2007-03-27 Thread Tim Dijkstra
On Tue, 27 Mar 2007 21:18:17 +0200 Cédric Boutillier [EMAIL PROTECTED] wrote: Hi ! Tim said: Apparently it should be an `unsigned long long', which would make printf(blkdev: %llu \n, blkdev); the correct way to print it. héhé ! I got a non zero result !!! Thanks Tim ! (Should I be

Re: [Suspend-devel] [patch] Make the error printing more useful (was: s2both on powerpc)

2007-03-26 Thread Tim Dijkstra
On Mon, 26 Mar 2007 11:07:59 +0200 Stefan Seyfried [EMAIL PROTECTED] wrote: Hi, after reading this thread, i got the impression that we do not report errors verbosely enough. I mean - we normally never see those messages, but if we do, due to an error having happened, we always need to

Re: [Suspend-devel] rip out libx86 build options from the Makefile

2007-03-26 Thread Tim Dijkstra
On Thu, 22 Mar 2007 13:29:15 +0100 Stefan Seyfried [EMAIL PROTECTED] wrote: Hi, (sending a new mail, so that it does not get lost in the old thread) If nobody objects, i will rip out the libx86 build option from the Makefile. I will not do this today or tomorrow, but as time permits (or

[Suspend-devel] s2both on powerpc

2007-03-24 Thread Tim Dijkstra
Hi Cédric, First of all, thanks for testing! I'm cc-ing the suspend-devel list maybe they can chime in if they have some idea... On Fri, 23 Mar 2007 17:03:34 +0100 Cédric Boutillier [EMAIL PROTECTED] wrote: Hi ! I compiled successfully uswsusp on my iBook (mid 2005), and installed the

[Suspend-devel] s2both on powerpc -- another one

2007-03-24 Thread Tim Dijkstra
On Fri, 23 Mar 2007 14:40:02 +0100 Johannes Berg [EMAIL PROTECTED] wrote: Hi Tim, I gave it a try now... Here are a few things: - compression is totally messed up on my machine, it writes about 8% of the image and then gets an -ENOSPC error from the compression algorithm (I put

Re: [Suspend-devel] [patch] move to vbetool-1.0

2007-03-22 Thread Tim Dijkstra
On Tue, 20 Mar 2007 23:53:37 +0100 Tim Dijkstra [EMAIL PROTECTED] wrote: On Tue, 20 Mar 2007 22:43:37 +0100 Rafael J. Wysocki [EMAIL PROTECTED] wrote: Hm, well. I think that if we don't ship something, we should not have the option to compile it in our Makefile. Instead, we should

Re: [Suspend-devel] [patch] move to vbetool-1.0

2007-03-20 Thread Tim Dijkstra
On Tue, 20 Mar 2007 08:56:45 +0100 Stefan Seyfried [EMAIL PROTECTED] wrote: On Mon, Mar 19, 2007 at 10:21:16PM +0100, Rafael J. Wysocki wrote: I can't see how people can `depend' on it. Especially since this is only in CVS since a few days. Also libx86 isn't in CVS anymore so they will

[Suspend-devel] ioctl(snaptshot_fd, SNAPSHOT_S2RAM, 0);

2007-03-20 Thread Tim Dijkstra
Hi, I just noticed I have misunderstood how the machine is put in S3 for s2both. I thought it would use a function from s2ram.c to do that, but it seems it uses ioctl(snaptshot_fd, SNAPSHOT_S2RAM, 0); Just to be sure, this will probably do precisely the same thing as echo 'mem

Re: [Suspend-devel] ioctl(snaptshot_fd, SNAPSHOT_S2RAM, 0);

2007-03-20 Thread Tim Dijkstra
On Tue, 20 Mar 2007 16:38:43 +0100 Pavel Machek [EMAIL PROTECTED] wrote: On Tue 2007-03-20 11:37:36, Tim Dijkstra wrote: Hi, I just noticed I have misunderstood how the machine is put in S3 for s2both. I thought it would use a function from s2ram.c to do that, but it seems it uses

Re: [Suspend-devel] [patch] move to vbetool-1.0

2007-03-20 Thread Tim Dijkstra
On Tue, 20 Mar 2007 22:43:37 +0100 Rafael J. Wysocki [EMAIL PROTECTED] wrote: Hm, well. I think that if we don't ship something, we should not have the option to compile it in our Makefile. Instead, we should update the documentation to tell the users that they should obtain the thing

Re: [Suspend-devel] [patch] move to vbetool-1.0

2007-03-19 Thread Tim Dijkstra
On Mon, 19 Mar 2007 22:21:16 +0100 Rafael J. Wysocki [EMAIL PROTECTED] wrote: Hm, I think I misunderstood the issue. Can anyone please explain to me what the current situation is and why it may be a good idea to change it (or not)? libx86 is a dependency of vbetool. Since the new release of

Re: [Suspend-devel] [PATCH] Switch to VBETool 1.0 codebase

2007-03-05 Thread Tim Dijkstra
Hi, Our diff to VBETool 1.0 is virtually non-existent, the only 'significant' change is the do_get_mode / __get_mode change. Can't we persuade Matthew to take these changes? than we can just point at vbetool code and drop it from out repository. Then maybe I can persuade him to create a

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 --mode=755 resume

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 the laptop

[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 'instead of

[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

[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

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

2007-01-24 Thread Tim Dijkstra
+1,63 @@ +/* loglevel.c - routines to modify kernel console loglevel + * + * Released under GPL v2. + * (c) 2007 Tim Dijkstra + */ + +#include unistd.h +#include stdio.h +#include errno.h +#include sys/types.h +#include sys/stat.h +#include sys/mount.h + + +static FILE *printk_file; +static int

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

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

[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

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 by: sys_vendor = FUJITSU SIEMENS sys_product = D1547

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 incomplete as i would put it :-) I haven't tried

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. grep

[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,

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 complains loudly. suspend: pm_ops-prepare returned error -1

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

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 splashy

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]: Hi, The appended patch allows the users of suspend to abort

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]: On Thursday, 2 November 2006 14:49, Tim Dijkstra wrote: Op Thu, 2 Nov

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't

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 swap,

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 good idea. I

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

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's a waste of battery power. s2ram only takes about

[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

[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 +++

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 seems to work

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! This patch will lock the VT that is active just after splash

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

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

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 is no #ifdef SPLASH_CONFIG anymore. SPLASH_PARAM still

[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

[Suspend-devel] Need #include linux/fs.h to compile

2006-08-09 Thread Tim Dijkstra
Hi, With the libc6-dev in current debian sid, I need to include linux/fs.h 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

[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

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: Hi, Apparently we have used wrond console ioctls around freeze

[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

[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

[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