Re: [Suspend-devel] [PATCH] Enable splash setting via command-line

2007-08-15 Thread Tim Dijkstra
On Wed, 15 Aug 2007 19:30:27 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Saturday, 4 August 2007 23:17, Alon Bar-Lev wrote: > > > > Hello, > > > > This patch adds --splash argument to s2both, s2disk, resume. > > It is required especially for resume when you want to disable splash i

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 ima

Re: [Suspend-devel] [PATCH 6/6] - autoconf/automake glue - s2ram

2007-08-04 Thread Tim Dijkstra
On Fri, 3 Aug 2007 13:45:20 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > 1. Move main() out of s2ram.c into s2ram-main.c, this will enable s2ram.o > > to be included > > in both s2ram and s2both without recompilation. That way we get even more s2ram-* files. In s2ram.c there would no

Re: [Suspend-devel] [PATCH 6/6] - autoconf/automake glue - s2ram

2007-08-04 Thread Tim Dijkstra
On Fri, 3 Aug 2007 13:45:20 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > 2. Split whitelist.c into whitelist.h/whitelist.c as including .c is none > > standard/unsupported. > > These changes have to be looked at by Pavel and Stefan. At some point I suggested this too. Pavel was so

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 o

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

2007-07-21 Thread Tim Dijkstra
On Sat, 21 Jul 2007 23:08:30 +0300 "Alon Bar-Lev" <[EMAIL PROTECTED]> wrote: > On 7/21/07, Alon Bar-Lev <[EMAIL PROTECTED]> wrote: > > On 7/21/07, Holger Macht <[EMAIL PROTECTED]> wrote: > > > Right. So if we start putting things like cpufreq in the suspend package, > > > we would have an argument

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

2007-07-21 Thread Tim Dijkstra
On Sat, 21 Jul 2007 18:28:24 +0300 "Alon Bar-Lev" <[EMAIL PROTECTED]> wrote: > On 7/21/07, Holger Macht <[EMAIL PROTECTED]> wrote: > > This is already done in pm-utils. Upstream uses 'userspace' as governor, > > which I personally think is wrong. openSUSE uses the 'performance' > > governor and th

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

2007-07-20 Thread Tim Dijkstra
On Fri, 20 Jul 2007 22:20:51 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > Yet another patch in the suspend2 feature-set series. > > It sets cpufreq to maximum during the cycle. I haven't looked at the patch, but this doesn't really sound essential for uswsusp. Maybe it should go into

[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? --- Begin Message --- 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 s

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

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

[Suspend-devel] swap-offset is using a deprecated SCSI ioctl

2007-05-20 Thread Tim Dijkstra
Hi Luca, I just got this report from a swap-offset user: I've just stumbled upon this message in my syslog after today's upgrade of uswsusp: program swap-offset is using a deprecated SCSI ioctl, please convert it to SG_IO grts Tim signature.asc Description: PGP signature ---

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

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 ple

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 >

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

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

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

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

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 i

[Suspend-devel] Splashy features

2007-05-10 Thread Tim Dijkstra
Hi, libsplashy changed the ABI a bit, needed to change something. It also would want to know if we're suspend of resuming, it can then show a penguin going to bed or waking up. Index: resume.c === RCS file: /cvsroot/suspend/suspend/

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

2007-04-24 Thread Tim Dijkstra
On Tue, 24 Apr 2007 00:56:02 +0200 Luca Tettamanti <[EMAIL PROTECTED]> wrote: > Il Fri, Apr 20, 2007 at 06:59:11PM +0200, Stefan Seyfried ha scritto: > > On Wed, Apr 18, 2007 at 09:22:20PM +0200, Tim Dijkstra wrote: > > > Luca Tettamanti <[EMAIL PROTECTED]> wrote:

Re: [Suspend-devel] swapfiles

2007-04-24 Thread Tim Dijkstra
On Tue, 24 Apr 2007 21:24:28 +0200 Tim Dijkstra <[EMAIL PROTECTED]> wrote: > On Mon, 23 Apr 2007 23:34:58 +0200 > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > > On Monday, 23 April 2007 21:30, Tim Dijkstra wrote: > > > On Sun, 22 Apr 2007 23

Re: [Suspend-devel] swapfiles

2007-04-24 Thread Tim Dijkstra
On Mon, 23 Apr 2007 23:34:58 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Monday, 23 April 2007 21:30, Tim Dijkstra wrote: > > On Sun, 22 Apr 2007 23:22:30 +0200 > > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > >

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

Re: [Suspend-devel] swapfiles

2007-04-23 Thread Tim Dijkstra
On Sun, 22 Apr 2007 23:22:30 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Sunday, 22 April 2007 22:06, Tim Dijkstra wrote: > > On Sun, 22 Apr 2007 21:18:46 +0200 > > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > > >

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

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

Re: [Suspend-devel] swapfiles

2007-04-21 Thread Tim Dijkstra
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 trying out sdisk with a swap file. > > > > $ cat /proc/swaps > > Filename

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

[Suspend-devel] Comments in usage()

2007-04-20 Thread Tim Dijkstra
Hi, S2ram could also use the usage() function from config.c to display its message. The only problem is that to replicate its current output we need to print comments too. To make this possible I made a little patch. The patch only works for s2disk currently, I didn't feel like adding the comments

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

2007-04-18 Thread Tim Dijkstra
On Mon, 16 Apr 2007 23:22:32 +0200 Luca Tettamanti <[EMAIL PROTECTED]> wrote: > Il Mon, Apr 16, 2007 at 10:35:25PM +0200, Tim Dijkstra ha scritto: > > On Sun, 15 Apr 2007 02:36:12 +0200 > > Luca Tettamanti <[EMAIL PROTECTED]> wrote: > > > > > >

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 di

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

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

2007-04-14 Thread Tim Dijkstra
Hi, This patch fixes all my concerns I raised in the previous e-mail. It applies on top of lucas patch. So if you apply 1/2, I'll do this one on top. If everybody agrees that is... grts diff -urN a/Makefile b/Makefile --- a/Makefile 2007-04-14 22:55:32.0 +0200 +++ b/Makefile 2007-04-1

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

2007-04-14 Thread Tim Dijkstra
(ARCH) variable > (autodetected 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 ++

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

2007-04-11 Thread Tim Dijkstra
; > PPC code is based 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

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); > > > > Wh

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

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

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

2007-04-03 Thread Tim Dijkstra
On Tue, 3 Apr 2007 11:25:23 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > Hi, > > This is a RFC, i am not sure if printing the program name in all error > messages is really useful, or if just "suspend:" is better there, but > at least in usage() it should be the correct one. I think I woul

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

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

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 pat

Re: [Suspend-devel] s2both on powerpc

2007-03-31 Thread Tim Dijkstra
Johannes Berg schreef: > On Sat, 2007-03-31 at 21:08 +0200, Tim Dijkstra wrote: > >> And to ask your earlier question, s2both uses something slightly >> different >> than s2ram. s2ram tries first some ioctl on /dev/pmu, if that fails it >> tries >> echo mem >

Re: [Suspend-devel] s2both on powerpc

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

Re: [Suspend-devel] s2both on powerpc

2007-03-30 Thread Tim Dijkstra
On Fri, 30 Mar 2007 12:41:11 +0200 Johannes Berg <[EMAIL PROTECTED]> wrote: > On Fri, 2007-03-30 at 11:27 +0200, Tim Dijkstra wrote: > > On Sat, 24 Mar 2007 21:16:52 +0100 > > Tim Dijkstra <[EMAIL PROTECTED]> wrote: > > > > We're trying to debug s2dis

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

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

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

2007-03-29 Thread Tim Dijkstra
On Thu, 29 Mar 2007 14:30:04 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > On Thu, Mar 29, 2007 at 11:04:22AM +0200, Tim Dijkstra wrote: > > On Thu, 29 Mar 2007 10:32:59 +0200 > > Stefan Seyfried <[EMAIL PROTECTED]> wrote: > > > > > This is ugl

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

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

2007-03-29 Thread Tim Dijkstra
On Thu, 29 Mar 2007 10:32:59 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > This is ugly and long, but shows the long options (we could leave out > the short options, or go for something like: > Usage: suspend [-h|--help] > [-f|--config ] > [-s|--image_size ] >

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

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

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 und

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 ! (Sh

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

[Suspend-devel] s2ram style options for s2both

2007-03-27 Thread Tim Dijkstra
Hi, I think we agree that it would be nice for s2both to have the same commandline options as s2ram. Well, here is a patch;) A few comments: The logic I choose for s2both is to continue with suspend-to-disk in case of an error in the s2ram part. Even if the problem is with cli options. This is ho

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

2007-03-27 Thread Tim Dijkstra
Hi, I was just adding some command line options to s2both when I noticed that in fact it didn't went to s3 anymore. I noticed this before, but thought that my experimentation with pm-utils was to blams. Debugging a bit if found out that ioctl(dev, SNAPSHOT_S2RAM, 0); sets errno to EBUSY

Re: [Suspend-devel] [Pm-utils] powerpc support

2007-03-27 Thread Tim Dijkstra
Hi Stefan, Thanks for the comments, I'll integrate them and try to minimize the number of ifdefs. On Tue, 27 Mar 2007 13:23:03 +0200 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > > +#endif > > + case '?': > > Will probably not work, since it is not in options[] above? getopts return

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

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

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

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

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 permit

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 t

Re: [Suspend-devel] s2both on powerpc

2007-03-24 Thread Tim Dijkstra
On Sun, 25 Mar 2007 00:17:22 +0100 "Cédric Boutillier" <[EMAIL PROTECTED]> wrote: > > Which kernel are you using? Could you try to catch the error code that > > s2both returns? Or just add two printf's that print out the error code? > > I have modified the three lines you mentionned as follows: >

[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

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

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 th

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 th

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 fr

[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 > /sys/pow

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

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 rel

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

2007-03-19 Thread Tim Dijkstra
On Mon, 19 Mar 2007 11:27:50 +0100 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > On Monday, 19 March 2007 08:28, Stefan Seyfried wrote: > > On Sun, Mar 18, 2007 at 04:25:21PM +0100, Tim Dijkstra wrote: > > > On Wed, 14 Mar 2007 16:05:40 +0100 > > >

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

2007-03-18 Thread Tim Dijkstra
On Wed, 14 Mar 2007 16:05:40 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: Sorry I didn't respond to this earlier... > I removed all the removed x86emu files from the inlined patch, they are > included in the attached bzip'ed diff X86emu is no longer present in CVS right? Why do we then not

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

2007-03-05 Thread Tim Dijkstra
On Mon, 5 Mar 2007 12:44:22 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > ;-) So let's do a poll: what distributions already include libx86 as a > package? /me raises hand Debian has, that is probably related to the fact that Matthew is a debian developer... grts Tim -

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

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

2007-03-05 Thread Tim Dijkstra
On Mon, 5 Mar 2007 12:03:18 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > On Mon, Mar 05, 2007 at 11:35:10AM +0100, Arkadiusz Miskiewicz wrote: > > On Monday 05 of March 2007, Stefan Seyfried wrote: > > > > > Note: this patch will not compile right now, you'll need to unpack libx86 > > > int

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

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

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

[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

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

[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

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

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

[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

[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

  1   2   >