Re: [Suspend-devel] [RFC/RFT][PATCH 0/7] s2disk: Various code cleanups

2007-08-16 Thread Pavel Machek
Hi! > I have seven patches indented to make the code in suspend.c a bit more > readable > (less levels of indentation, simpler loops etc.). > > I haven't tested them yet, so if anyone has the time and motivation, please do > so. ;-) > > Comments welcome. They look ok from the fast glance.

Re: [Suspend-devel] [RFC/RFT][PATCH 7/7] s2disk: Rename compress to do_compress

2007-08-16 Thread Pavel Machek
Hi! > @@ -1396,7 +1396,7 @@ int main(int argc, char *argv[]) > > mem_size = 3 * page_size + buffer_size; > #ifdef CONFIG_COMPRESS > - if (compress) > + if (do_compress) > mem_size += LZO1X_1_MEM_COMPRESS; > #endif > #ifdef CONFIG_ENCRYPT Can we kill the #ifdef? do

Re: [Suspend-devel] [RFC/RFT][PATCH 5/7] s2disk: save_image() cleanup

2007-08-16 Thread Pavel Machek
Hi! > Reduce the number of indentation levels in save_image(). > > --- > suspend.c | 54 -- > 1 file changed, 28 insertions(+), 26 deletions(-) > > Index: suspend/suspend.c > ==

Re: [Suspend-devel] [rft] Kill junk from s2ram resume paths

2007-08-16 Thread Pavel Machek
Hi! > > > > > > # Running in *copy* of this code, somewhere in low 1MB. > > > > > > > > > > > > - movb$0xa1, %al ; outb %al, $0x80 > > > > > > > > > > Well, what was this for? > > > > > > > > Debugging leds on port 80. I still have that card somewhere > > > > :-). Interesting part

Re: [Suspend-devel] [PATCH] Cleanup terminal if abort was selected

2007-08-16 Thread Pavel Machek
Hi! > > > > Current implementation does not cleanup console settings if abort was > > > > selected by user. > > > > This should fix it. > > > > > > I prefer the appended patch. > > > > One of the key rules of procedural programming: > > Don't write the same code at different places... > > The sam

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

2007-08-16 Thread Pavel Machek
Hi! > > This patch adds --splash argument to s2both, s2disk, resume. > It is required especially for resume when you want to disable splash in > initramfs > as result of kernel parameter, even if it usually turned on on suspend.conf > you > wish to turn it off. Patch looks pretty much ok. > --

Re: [Suspend-devel] [PATCH] Do not manually calculate config elements

2007-08-16 Thread Pavel Machek
Hi! > This allows you to specify an array of config_par, ending it with NULL > name, so that you don't have to calculate the number of entries manually. > > -int parse(char *my_name, char *file_name, int parc, struct config_par *parv) > +int parse(char *my_name, char *file_name, struct config_pa

Re: [Suspend-devel] [RFC] Modify switching into lzo compression

2007-08-16 Thread Pavel Machek
> On 8/15/07, Rafael J. Wysocki <[EMAIL PROTECTED]> wrote: > > > You can malloc as much as you wish at initialization directly into > > > the pointers... I just don't understand what may go wrong if you do > > > this... > > > > Nothing, really. I prefer to have two global pointers and one contigu

Re: [Suspend-devel] [PATCH] Cleanup terminal if abort was selected

2007-08-16 Thread Pavel Machek
Hi! > Current implementation does not cleanup console settings if abort was > selected by user. > This should fix it. > > Part of pending series at: > http://alon.barlev.googlepages.com/suspend-patches.tar.bz2 > > Best Regards, > Alon Bar-Lev. > > --- > > --- suspend.org/suspend.c 2007-07-

Re: [Suspend-devel] encryption docs

2007-08-14 Thread Pavel Machek
On Tue 2007-08-14 23:07:56, Alon Bar-Lev wrote: > On 8/14/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > > Hi! > > > > I'd like to commit > > > > Suspend encryption > > ~~ > > > > Encryption in suspend.sf.net uses

Re: [Suspend-devel] encryption docs

2007-08-14 Thread Pavel Machek
On Tue 2007-08-14 23:55:27, Alon Bar-Lev wrote: > On 8/14/07, Luca <[EMAIL PROTECTED]> wrote: > > IOW we don't use RSA to encrypt the whole image ;) > > Sure you don't. > I referred the symmetric decryption... > This can be done using gpg or any other external decryption software. > > Something l

Re: [Suspend-devel] encryption docs

2007-08-14 Thread Pavel Machek
Hi! > > I think that adding the PK logic into suspend is an overkill... > > Best to support only symmetric operations, and load symmetric key from > > a file/handle as losetup does. > > The image is always encrypted with symmetric algo. If RSA is used > (optional) then the key for the symmetric e

[Suspend-devel] encryption docs

2007-08-14 Thread Pavel Machek
Hi! I'd like to commit Suspend encryption ~~ Encryption in suspend.sf.net uses RSA internally; reason is that we want to only prompt for passphrase on resume. So, during suspend, image is effectively encrypted with public key, and during resume, user has to first decrypt private

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

2007-08-14 Thread Pavel Machek
--- suspend.org/COPYING 1970-01-01 02:00:00.0 +0200 > +++ suspend-0.6_beta1/COPYING 2007-08-03 00:36:13.00000 +0300 > @@ -0,0 +1,68 @@ > + > +suspend License > + > + Copyright (C) 2005-2007 Rafael J. Wysocki <[EMAIL PROTECTED]> > + Copyright (C) 2006-2007

Re: [Suspend-devel] DVD problem after resume from S3

2007-08-14 Thread Pavel Machek
Hi! > I have an ASUS-M2NPV motherboard and I have a SAMSUNG HD300LJ SATA hard-disk > and an LG GDR8164B IDE DVD-ROM. > > I am running kernel 2.6.21.5 and I noticed that after resume from S3, my > system > crashes as soon as I try to access the DVD. > Basically the crash is silent, that is there

Re: [Suspend-devel] getting s2ram to work on a Samsung X60

2007-08-13 Thread Pavel Machek
Hi! > I'm trying to get s2ram to work on a Samsung X60 running Ubuntu Gutsy, and > I think I'll need some help... > > firstly: > sudo s2ram -i > [sudo] password for atreju: > This machine can be identified by: > sys_vendor = "SAMSUNG ELECTRONICS CO., LTD." > sys_product = "SX60S" >

Re: [Suspend-devel] s2disk/s2ram not working with recent kernels on Acer Extensa 4100 WLMI

2007-08-09 Thread Pavel Machek
Hi! > > I can nowsuspend and resume using the "nolapic" kernel option! (I > > remember that I needed that option way in the past as well > > as patching the DSDT to get the battery state; I didn't need both for a > > long time now...) > > > > Using the nolapic option there are some warnings at b

Re: [Suspend-devel] Asus A2D

2007-08-06 Thread Pavel Machek
On Thu 2007-08-02 19:28:27, Jean-Baptiste BUTET wrote: > GREAT NEWS ! > > A2D works out of the box (directly with s2ram -f) for both > suspend to ram & suspend to disk > > Only one condition (J. Puydt's bug report gives the hint) : > disable dri on Xorg. > (comment "load dri" line in /etc

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

2007-08-06 Thread Pavel Machek
On Mon 2007-08-06 12:17:17, Alon Bar-Lev wrote: > On 8/6/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > > And the license is GPLv2... for all subparts, too. They are part of > > GPLed project after all. > > No... > /* Radeontool v1.4 > * by Frederick Dean <[E

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

2007-08-06 Thread Pavel Machek
Hi! > Use sh and not bash. > Support BOOT_DIR for install-resume.sh > > --- > > diff -urNp suspend.org/scripts/create-resume-initrd.sh > suspend-0.6_beta1/scripts/create-resume-initrd.sh > --- suspend.org/scripts/create-resume-initrd.sh 2006-07-23 > 14:32:53.0 +0300 > +++ suspend

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

2007-08-06 Thread Pavel Machek
01-01 02:00:00.0 +0200 > > +++ suspend-0.6_beta1/COPYING 2007-08-03 00:36:13.0 +0300 > > @@ -0,0 +1,68 @@ > > + > > +suspend License > > + > > + Copyright (C) 2005-2007 Rafael J. Wysocki <[EMAIL PROTECTED]> > > + Copyright (C) 20

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

2007-08-05 Thread Pavel Machek
Hi! > > > I don't care too much, although i think this is much too much > > > actually changed code for just "automake conversion". I mean - > > > this is configuration stuff, why do we need to reorder whitelist.[hc]? > > > > > > I guess i know the answer - and now you know why everybody hates aut

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

2007-08-05 Thread Pavel Machek
Hi! > > 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. > > > > 2. Split whitelist.c into whitelist.h/whitelist.c as including .c is none > > standard/unsupported. > > These changes have to be look

Re: [Suspend-devel] Autoconf/automake support

2007-07-31 Thread Pavel Machek
On Tue 2007-07-31 21:45:14, Luca wrote: > On 7/31/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > > On Tue 2007-07-31 21:35:59, Alon Bar-Lev wrote: > > > On 7/31/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > > > > Hmm, IIRC we _do_ have some webmaster, al

Re: [Suspend-devel] Autoconf/automake support

2007-07-31 Thread Pavel Machek
On Tue 2007-07-31 21:35:59, Alon Bar-Lev wrote: > On 7/31/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > > Hmm, IIRC we _do_ have some webmaster, already. Ok, lets keep pages in > > suse wiki, then. > > Excuse me for not understanding such short messages... > Are yo

Re: [Suspend-devel] Autoconf/automake support

2007-07-31 Thread Pavel Machek
Hi! > > Right... so do you have sourceforge login? I guess we could use you as > > a webmaster for suspend.sf.net project ;-). > > My sf login is alonbl. > But I am far from being a web master... Or have the time to do this. > But if you like I can put the suggested html on your site so that you

Re: [Suspend-devel] 32-vs-64 bit affects quirks (was Re: Working but not in list)

2007-07-31 Thread Pavel Machek
Hi! > > > > Hmm, yes, I guess I want to now. So you have thinkpad that is broken > > > > with -a1 -m, in 64-bit mode only? And no other differences? (Like, > > > > same kernel framebuffer driver, both time from console?) > > > > > > I have a thinkpad on my desk (T61 with intel graphics) that is b

Re: [Suspend-devel] [rft] Kill junk from s2ram resume paths

2007-07-31 Thread Pavel Machek
Hi! > > > diff --git a/arch/i386/kernel/acpi/wakeup.S > > > b/arch/i386/kernel/acpi/wakeup.S > > > index 1415da1..9cebef7 100644 > > > --- a/arch/i386/kernel/acpi/wakeup.S > > > +++ b/arch/i386/kernel/acpi/wakeup.S > > > @@ -28,21 +28,6 @@ #define BEEP \ > > > movb$15, %al; \ > > >

Re: [Suspend-devel] [rft] Kill junk from s2ram resume paths

2007-07-31 Thread Pavel Machek
Hi! > > > > # Running in *copy* of this code, somewhere in low 1MB. > > > > > > > > - movb$0xa1, %al ; outb %al, $0x80 > > > > > > Well, what was this for? > > > > Debugging leds on port 80. I still have that card somewhere > > :-). Interesting parties can reinsert it. > > Ah

Re: [Suspend-devel] [rft] Kill junk from s2ram resume paths

2007-07-31 Thread Pavel Machek
;s machine :-). (VGA accesses > > could theoretically hurt if vga is not present / if it is in some > > strange state). > > > > Signed-off-by: Pavel Machek <[EMAIL PROTECTED]> > > > > diff --git a/arch/i386/kernel/acpi/wakeup.S b/arch/i386/kernel/acpi/wa

[Suspend-devel] [rft] Kill junk from s2ram resume paths

2007-07-31 Thread Pavel Machek
tically hurt if vga is not present / if it is in some strange state). Signed-off-by: Pavel Machek <[EMAIL PROTECTED]> diff --git a/arch/i386/kernel/acpi/wakeup.S b/arch/i386/kernel/acpi/wakeup.S index 1415da1..9719bd6 100644 --- a/arch/i386/kernel/acpi/wakeup.S +++ b/arch/i386/kernel/acpi/wa

Re: [Suspend-devel] 32-vs-64 bit affects quirks (was Re: Working but not in list)

2007-07-31 Thread Pavel Machek
Hi! > > Hmm, yes, I guess I want to now. So you have thinkpad that is broken > > with -a1 -m, in 64-bit mode only? And no other differences? (Like, > > same kernel framebuffer driver, both time from console?) > > I have a thinkpad on my desk (T61 with intel graphics) that is broken with > "-a 3"

Re: [Suspend-devel] Working but not in list

2007-07-31 Thread Pavel Machek
On Mon 2007-07-30 07:49:07, Stefan Seyfried wrote: > On Fri, Jul 27, 2007 at 07:38:52PM +0200, Pavel Machek wrote: > > > If the needed quirks differ between otherwise identical i386 and > > x86-64 installation, I'd like to know. That means there's serious > >

Re: [Suspend-devel] Autoconf/automake support

2007-07-31 Thread Pavel Machek
On Mon 2007-07-30 19:53:35, Alon Bar-Lev wrote: > On 7/30/07, Stefan Seyfried <[EMAIL PROTECTED]> wrote: > >> Why do you forward people to suse URLs on whitelist? > >> I would have expected all be forwarded to suspend.sf.net... > >> http://en.opensuse.org/S2ram belongs to the project site... not >

[Suspend-devel] 32-vs-64 bit affects quirks (was Re: Working but not in list)

2007-07-27 Thread Pavel Machek
> On Mon, Jul 23, 2007 at 07:44:10PM +0200, Hendrik-Jan Heins wrote: > > > > I just tested the following: > > > > Sidux 2007-2 (tartaros), amd64 on live cd. > > I loaded it to ram and installed uswsusp. > > next I did: "s2ram -f -m -a3", three times. > > Every time the machine suspended correctly

Re: [Suspend-devel] Working but not in list

2007-07-27 Thread Pavel Machek
> (i re-added suspend-devel to cc: to keep the conversation on-list, please > keep it that way) > > On Thu, Jul 19, 2007 at 10:40:47PM +0200, Hendrik-Jan Heins wrote: > > -BEGIN PGP SIGNED MESSAGE- > > Hash: SHA1 > > > > ah yes, I'm sorry, forgot to answer that one. > > I'm running i386

Re: [Suspend-devel] Autoconf/automake support

2007-07-27 Thread Pavel Machek
Hi! > > But i am not the one to decide this, as i am only maintaining a small piece > > of the code (mainly the whitelist). > > Why do you forward people to suse URLs on whitelist? Because it was easier that way :-). > I would have expected all be forwarded to suspend.sf.net... > http://en.open

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

2007-07-23 Thread Pavel Machek
On Mon 2007-07-23 23:16:00, Alon Bar-Lev wrote: > On 7/23/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > >> It works correctly now. > >> Previously it did not work because of SIGALARM and related issues. > >> I worked with cpufreqd developer to solve all these.

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

2007-07-23 Thread Pavel Machek
On Mon 2007-07-23 20:17:40, Alon Bar-Lev wrote: > On 7/23/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > >Well, so user explicitely configured his machine to run on 80% max, > >for some reason. Now you want suspend to explicitely override his > >setting. That strikes me

Re: [Suspend-devel] [PATCH] Add lzo compression support

2007-07-23 Thread Pavel Machek
Hi! > >> If you ask me... I think lzo will be better for most users... But I > > > >If you can argue lzo is significantly better, we can switch, but we do > >not want both. > > Well... So let's switch. :) So, what are advantages of lzo? Is it significantly faster? Better compression ratio? Stef

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

2007-07-23 Thread Pavel Machek
On Mon 2007-07-23 20:21:17, Alon Bar-Lev wrote: > On 7/23/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > >> These s2ram_* funcs in the end call code from vbetool/vbetool.c which > >> is indeed linked in. But partly I remembered it wrongly. The hacks > >> _before_

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

2007-07-23 Thread Pavel Machek
On Mon 2007-07-23 20:19:22, Alon Bar-Lev wrote: > On 7/23/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > >Okay, so this is done from userland, not from kernel. > > > >Does cpu frequency scaling work properly after we snapshot the system? > >If yes, we should not ne

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

2007-07-23 Thread Pavel Machek
On Mon 2007-07-23 20:14:15, Alon Bar-Lev wrote: > On 7/23/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > >> So these hacks can be removed from suspend code... And moved into the > >> hibernate-script or pm-utils. > > > >No, they can't (easily?), because

Re: [Suspend-devel] [PATCH] Add lzo compression support

2007-07-23 Thread Pavel Machek
On Mon 2007-07-23 20:12:21, Alon Bar-Lev wrote: > On 7/23/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > >I guess it is easiest way for gentoo, but... I'd prefer not to switch > >compression algorithms too often, unless there's good reason to. > > Support is a

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

2007-07-23 Thread Pavel Machek
On Sun 2007-07-22 22:09:50, Alon Bar-Lev wrote: > On 7/22/07, Tim Dijkstra <[EMAIL PROTECTED]> wrote: > > 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 does

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

2007-07-23 Thread Pavel Machek
Hi! > > > 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 it different from running it from a bunch of scripts. > > > > Are you sure? > > Can you please refer

Re: [Suspend-devel] [PATCH] Enable interactive reboot selection

2007-07-23 Thread Pavel Machek
On Fri 2007-07-20 18:54:50, Alon Bar-Lev wrote: > > Hello, > > This patch enables the user to press "r" during snapshot writing, > in order to reboot and not shutdown. > > One more step toward suspend2 functionality... :) Looks ok to me. -- (english) http://www.livejournal.com/~pavelmachek (ce

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

2007-07-23 Thread Pavel Machek
On Sat 2007-07-21 16:18:57, Alon Bar-Lev wrote: > On 7/21/07, Pavel Machek <[EMAIL PROTECTED]> wrote: > > I'm not sure it is good idea. We should either let cpu frequency > > scaling do its job during suspend (it should just scale to the max > > when it sees cpu

Re: [Suspend-devel] suspend - libz dependency

2007-07-23 Thread Pavel Machek
Hi! > > AFAICS, recent versions of pciutils depend on it. > > Oh... > You don't have to compile it with zlib... > > """The configure script will automatically enable support for a compressed > pci.ids if you have zlib installed. You can override its guess by using > "make ZLIB=no" or "make ZLIB

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

2007-07-23 Thread Pavel Machek
Hi! > > some battery life. But imho it is better to test it first, maybe the > > ondemand > > govenor is even better. > > You may have a configuration in which ondemand can be in range of > minimum to X, where X In this configuration letting the ondemand handle this will not result > in optimum

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

2007-07-23 Thread Pavel Machek
On Sat 2007-07-21 11:50:01, Pavel Machek wrote: > Hi! > > > > Hello Rafael, > > > > Thanks for all of the patches so far. :-) > > > > > Yet another patch in the suspend2 feature-set series. > > > It sets cpufreq to maximum during the cy

Re: [Suspend-devel] [PATCH] Add lzo compression support

2007-07-23 Thread Pavel Machek
Hi! > I am the maintainer of gentoo hibernate-script, tuxonice (suspend2). > I wish to add uswsup/suspend into portage. > > We do not currently maintain liblzf, as it seems upstream is not cooperative. > I've created a patch so that the user may select to use liblzf or lzo-2 > during compilation.

Re: [Suspend-devel] IBM ACPI suspend led is turned off during suspend

2007-07-23 Thread Pavel Machek
Hi! > IBM suspend led is turned on before suspend, but for some strange > reason after kernel snapshot (before the actual page write) it is > turned off, while it should stay blinking. > > You turn on the blink using: > # echo 7 blink > /proc/acpi/ibm/led > You turn off the blink using: > # echo

Re: [Suspend-devel] Slightly change S3_BIOS|S3_MODE handling wrt. beeping patch

2007-07-23 Thread Pavel Machek
On Tue 2007-07-17 22:39:57, Stefan Seyfried wrote: > Hi, > I am proposing two changes to s2ram-x86.c: > - first, if no "-a" is given, reset acpi_video_flags to 0, which will > override also the boot options. The reason i want to do this is that > people (me included) are sometimes confused by f

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

2007-07-21 Thread Pavel Machek
Hi! > > Hello Rafael, > > Thanks for all of the patches so far. :-) > > > Yet another patch in the suspend2 feature-set series. > > It sets cpufreq to maximum during the cycle. > > It is important to finish the process as quickly as possible (human > > engineer issue :) ) > > When running on b

Re: [Suspend-devel] s2ram works with init=/bin/bash but not otherwise

2007-07-13 Thread Pavel Machek
On Thu 2007-07-12 22:47:56, Jorge Visca wrote: > Pavel Machek wrote: > >> Ok, after some testing i found it was the sonypi module. Now the lid > >> close is not detected, but anyway i had no use for that when i couldnt > >> suspend... > >> > > >

Re: [Suspend-devel] s2ram works with init=/bin/bash but not otherwise

2007-07-12 Thread Pavel Machek
a replacement for this module being developped. > So, > > s2ram -f -a 1 > s2ram -f -a 3 > and > s2ram -f -p -m > > > works on a > > sys_vendor = "Sony Corporation" > sys_product = "VGN-C140G" > sys_version = "C3LMX4QW&qu

Re: [Suspend-devel] s2ram works with init=/bin/bash but not otherwise

2007-07-11 Thread Pavel Machek
Hi! > >> I can get s2ram to work from init=/bin/bash, but when I boot the full > >> OS and log in (on a text virtual terminal), the system will not > >> suspend: > >> > >> s2ram -f -a 1 > >> > > Is there any difference in modules between working and broken case? > > Any interesting messag

Re: [Suspend-devel] s2ram works with init=/bin/bash but not otherwise

2007-07-10 Thread Pavel Machek
Hi! > I can get s2ram to work from init=/bin/bash, but when I boot the full > OS and log in (on a text virtual terminal), the system will not > suspend: > > s2ram -f -a 1 Is there any difference in modules between working and broken case? Any interesting messages in dmesg?

Re: [Suspend-devel] [RFC] get rid of CONFIG_DISABLE_CONSOLE_SUSPEND

2007-07-04 Thread Pavel Machek
Hi! > > > > diff --git a/drivers/char/lp.c b/drivers/char/lp.c > > > > index 62051f8..8267ff8 100644 > > > > --- a/drivers/char/lp.c > > > > +++ b/drivers/char/lp.c > > > > @@ -144,7 +144,7 @@ static unsigned int lp_count = 0; > > > > static struct class *lp_class; > > > > > > > > #ifdef CONFI

Re: [Suspend-devel] [RFC] get rid of CONFIG_DISABLE_CONSOLE_SUSPEND

2007-07-04 Thread Pavel Machek
Pavel > > > > Kill CONFIG_DISABLE_CONSOLE_SUSPEND; it should not be configurable at > > all, instead, we should automatically keep console alive when > > possible. > > > > Signed-off-by: Pavel Machek <[EMAIL PROTECTED]> > > > > diff --git a/drivers/char/lp.c b/d

[Suspend-devel] [RFC] get rid of CONFIG_DISABLE_CONSOLE_SUSPEND

2007-06-28 Thread Pavel Machek
be configurable at all, instead, we should automatically keep console alive when possible. Signed-off-by: Pavel Machek <[EMAIL PROTECTED]> diff --git a/drivers/char/lp.c b/drivers/char/lp.c index 62051f8..8267ff8 100644 --- a/drivers/char/lp.c +++ b/drivers/char/lp.c @@ -144,7 +144,7 @@

Re: [Suspend-devel] s2ram (unknown machine)

2007-06-26 Thread Pavel Machek
On Tue 2007-06-26 14:04:00, Matthias Zehner wrote: > This is the output of my machine: > > This machine can be identified by: > sys_vendor = "ASUSTeK Computer INC." > sys_product = "A7N8X2.0" > sys_version = "REV 1.xx" > bios_version = "ASUS A7N8X2.0 Deluxe ACPI BIOS Rev 1008"

Re: [Suspend-devel] [PATCH, 3rd try] make disable_console_suspend runtime configurable

2007-06-21 Thread Pavel Machek
Hi! > > > Sorry for not screaming when CONFIG_DISABLE_CONSOLE_SUSPEND went in, > > > but please lets solve this correctly > > > > Ouch and sorry for not screaming at "try 1" time. But it still does > > not make the patch right, and I believe that even patch authors agree > > that "no-config-n

Re: [Suspend-devel] [PATCH, 3rd try] make disable_console_suspend runtime configurable

2007-06-17 Thread Pavel Machek
Hi! > > > I hate having to recompile the kernel, just to be able to debug suspend. > > > Remove CONFIG_DISABLE_CONSOLE_SUSPEND, replace it by a tunable in > > > /sys/power/disable_console_suspend. > > > > > > > > > Signed-off-by: Stefan Seyfried <[EMAIL PROTECTED]> > > > Signed-off-by: Frank Sei

Re: [Suspend-devel] [PATCH, 3rd try] make disable_console_suspend runtime configurable

2007-06-17 Thread Pavel Machek
On Fri 2007-06-15 00:20:28, Rafael J. Wysocki wrote: > On Thursday, 14 June 2007 15:59, Frank Seidel wrote: > > From: Stefan Seyfried <[EMAIL PROTECTED]> > > > > I hate having to recompile the kernel, just to be able to debug suspend. > > Remove CONFIG_DISABLE_CONSOLE_SUSPEND, replace it by a tuna

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

2007-06-15 Thread Pavel Machek
Hi! > > 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 > > That smells fishy, since i only see two

Re: [Suspend-devel] [PATCH, 2nd try] make disable_console_suspend runtime configurable

2007-06-13 Thread Pavel Machek
Hi! > I hate having to recompile the kernel, just to be able to debug suspend. > Remove CONFIG_DISABLE_CONSOLE_SUSPEND, replace it by a tunable in > /sys/power/disable_console_suspend. > Signed-off-by: Stefan Seyfried <[EMAIL PROTECTED]> > Signed-off-by: Frank Seidel <[EMAIL PROTECTED]> I wonder

Re: [Suspend-devel] Beeping patch (was: Toshiba Satellite A70 don't resume from s2ram)

2007-06-10 Thread Pavel Machek
Hi! > > My laptop is this: > > > > # s2ram -i > > This machine can be identified by: > > sys_vendor = "TOSHIBA" > > sys_product = "Satellite A70" > > sys_version = "* " > > bios_version = "V1.50" > > > > kernel: 2.6.22-rc3-git2-47 > > > > openSUSE 10.2 > > > > It get hanged

Re: [Suspend-devel] S2disk on Thinkpad R60 and suse 10.2

2007-05-14 Thread Pavel Machek
Hi! > > free command output is: > > total used free sharedbuffers cached > > Mem: 2075176 4415521633624 0 3512 115356 > > -/+ buffers/cache: 3226841752492 > > Swap: 2104472 474522057020 > > > > looks like

Re: [Suspend-devel] Thank You

2007-05-13 Thread Pavel Machek
hi! > > I have a Dell XPS M1210 box with Core Duo 2 Ghz, 2G RAM and an nVIDIA > > GeForce Go 7400 GPU (and I'm using the proprietary nvidia modules) running > > on a Debian box. > > > >>From the very beginning I'd never had success in getting this box to > >>support > > sw suspend. But amazingly

Re: [Suspend-devel] Toshiba Setellite U205-S5067 Suspend to RAM

2007-05-11 Thread Pavel Machek
Hi! > I have also tried the LFDk CD and my machine also suspended right away. > > I have been trying to hunt this bug down, adding many printk to my > kernel to see where the time is being spend. > > Up to now I can see that the system I get the following: > > enter_state in kernel/power/main.c

Re: [Suspend-devel] Toshiba Satellite A70 gets hanged trying to resume from s2ram

2007-05-10 Thread Pavel Machek
Hi! > > > I still can't suspend/resume to ram. > > > > > > The machine enter in suspend mode (leds blinking orange) but when I push > > > the power button to resume the leds gets blue (some sounds from CD) but > > > the system gets hanged, I have to force a shutdown. > > > > > > How can I debug th

Re: [Suspend-devel] s2ram Compaq D510 USDT

2007-05-09 Thread Pavel Machek
On Thu 2007-05-10 01:35:33, Martin Heimes wrote: > Hi Pavel, > > >>>My machine is not in the whitelist (...) > >>all S3 options in BIOS set to enabled > >> > >>revo:~# s2ram -i > >>This machine can be identified by: > >> sys_vendor = "Compaq" > >> sys_product = "Evo D510 USDT" > >>

Re: [Suspend-devel] S2ram working on unknown machine (fwd)

2007-05-09 Thread Pavel Machek
Well, as long as video works, it probably should be whitelisted. - Forwarded message from Per Waag? <[EMAIL PROTECTED]> - X-Original-To: [EMAIL PROTECTED] From: Per Waag? <[EMAIL PROTECTED]> Subject: Re: [Suspend-devel] S2ram working on unknown machine To: Pavel Machek <[

Re: [Suspend-devel] Toshiba Setellite U205-S5067 Suspend to RAM

2007-05-09 Thread Pavel Machek
Hi! > > No no... syslog timestamps are unreliable at this point. downing cpus > > take 704.5-704.4 seconds. You need to actually watch console to see > > when it hangs, and I suspect it is not here. > > > > OK. I'll try to print the messages to the console and watch it. I hope > the screen doesn

Re: [Suspend-devel] S2ram working on unknown machine

2007-05-09 Thread Pavel Machek
On Tue 2007-05-08 13:09:05, Per Waag? wrote: > s2ram -f simply works on my computer. > > s2ram -i gives this output: > This machine can be identified by: > sys_vendor = "Packard Bell NEC" > sys_product = "000" > sys_version = "P820008416" > bios_version = "R

Re: [Suspend-devel] s2ram Compaq D510 USDT

2007-05-09 Thread Pavel Machek
Hi! > > My machine is not in the whitelist (...) > > all S3 options in BIOS set to enabled > > > revo:~# s2ram -i > This machine can be identified by: > sys_vendor = "Compaq" > sys_product = "Evo D510 USDT" > sys_version = " " > bios_version = "686O1 v1.05" > See http://

Re: [Suspend-devel] Toshiba Setellite U205-S5067 Suspend to RAM

2007-05-09 Thread Pavel Machek
Hi! > I have the same laptop as him, a U205-S5067. I also get basically the > same behavior: is I try to suspend, the screen power off very fast but > the fan start moving like there is some processing going on. > > But, if I wait long enough (between 5 and 10 minutes) the system > suspends to r

Re: [Suspend-devel] Whitelisting no-pm-quirks

2007-05-08 Thread Pavel Machek
Hi! > > Still, what we definitely want is _one single_ architecture independent > > whitelist. And there is where hal-info comes in which is easy to update > > for distributions etc. IMHO. I'm seeing this difficulty with the two > > different lists for quite some time know and thought about possib

Re: [Suspend-devel] Compaq Presario V5120EU Works

2007-05-08 Thread Pavel Machek
Hi! >Works fine for me with just "sudo s2ram -f" >Output of s2ram -i is below: >[EMAIL PROTECTED]:/etc/acpi/resume.d$ sudo s2ram -i >This machine can be identified by: >sys_vendor = "Hewlett-Packard" >sys_product = "Presario V5000 (EW836EA#ABU) " >sy

Re: [Suspend-devel] Toshiba Satellite A70 gets hanged trying to resume from s2ram

2007-05-08 Thread Pavel Machek
Hi! > I still can't suspend/resume to ram. > > The machine enter in suspend mode (leds blinking orange) but when I push the > power button to resume the leds gets blue (some sounds from CD) but the > system gets hanged, I have to force a shutdown. > > How can I debug the problem? Beeping patc

Re: [Suspend-devel] [RFC][PATCH] swsusp: Reduce code duplication between user.c and disk.c

2007-05-07 Thread Pavel Machek
On Mon 2007-05-07 13:43:11, Johannes Berg wrote: > Hi, > > > However, this modifies the user-observable behavior in such a way that > > if the user sets 'shutdown mode = platform' in the s2disk's configuration > > file, > > the kernel will use the hibernation mode resulting from the sysfs interfa

Re: [Suspend-devel] Success in s2ram

2007-05-02 Thread Pavel Machek
On Fri 2007-04-27 18:06:06, Noam Raphael wrote: > Hello, > > I tried s2ram, and wasn't on the whitelist, but it seems to work fine. > Hurray! The first time suspend to ram worked for me on linux! > > Here's the output of s2ram -i: > This machine can be identified by: > sys_vendor = "IBM" >

Re: [Suspend-devel] Success in s2ram

2007-05-02 Thread Pavel Machek
Hi! > Sorry, it should probably not go in the white list yet. I once > suspended to ram and the display was problematic - it had problems > with repaint and moving, so things looked really strange. It went away > when I suspended to disk again. In all other times, it worked fine. It is probably "

Re: [Suspend-devel] Modification of the SNAPSHOT_PMOPS ioctl

2007-05-01 Thread Pavel Machek
Hi! > I'd like to modify the kernel's behavior related to the SNAPSHOT_PMOPS > ioctl. Namely, currently PMOPS_ENTER works regardless of the value > of "suspend mode" indicated by "cat /sys/power/disk". This doesn't seem to > be quite right to me, so I'd like to make s2disk set "/sys/power/disk"

Re: [Suspend-devel] s2ram on Thinkpad R60

2007-04-24 Thread Pavel Machek
Hi! > Marcus Better wrote: > > Stefan Seyfried wrote: > >> Does it also work from the text console, or do you rather need "s2ram -f > >> -a3" there? > > > With "s2ram -f -a3" from a text console it restored the screen correctly > > after resume. > > Update: This no longer works as of kernel 2.6.

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

2007-04-24 Thread Pavel Machek
Hi! > It's quite simple: a weak symbol may be discared by the linker if it > found another copy. > > This is the updated patch. You can either use the original 2/2 or this > one: Looks ok to me. Pavel -- (english) http://www.livejournal.c

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

2007-04-24 Thread Pavel Machek
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

Re: [Suspend-devel] Suspend failed due to task refuzing to freeze

2007-04-24 Thread Pavel Machek
s in logs. Hmm, weird. But it is good that it works for you. Pavel > On Thursday 12 April 2007 12:18:01 Pavel Machek wrote: > > Hi! > > > > > > > > > Task hald-addon-stor is in disk-sleep status. > > &

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

2007-04-14 Thread Pavel Machek
Hi! > >> Add s2ram support for PPC architecture. s2ram.{c,h} contain the > >> implementation of the required functions, used by the main file. The > >> Makefile selects the correct platform files using $(ARCH) variable > >> (autodetected by default, can be overridden). > >> > >> PPC code is based

Re: [Suspend-devel] Suspend failed due to task refuzing to freeze

2007-04-12 Thread Pavel Machek
Hi! > > > > > Task hald-addon-stor is in disk-sleep status. > > > > > > > > > > After preparing suspend and during stopping tasks, job jets aborted > > > > > (after 20 second) with "<1 task refusing to freeze>" > > > > > After, a list of few proccesses is dispayed as "Strange, > > > > > proccess_n

Re: [Suspend-devel] Suspend failed due to task refuzing to freeze

2007-04-11 Thread Pavel Machek
Hi! > > > Task hald-addon-stor is in disk-sleep status. > > > > > > After preparing suspend and during stopping tasks, job jets aborted > > > (after 20 second) with "<1 task refusing to freeze>" > > > After, a list of few proccesses is dispayed as "Strange, proccess_name > > > not stopped" > > > >

Re: [Suspend-devel] s2ram, Acer 3012, dead end?

2007-04-11 Thread Pavel Machek
Hi! > > > The process is always the same: Suspending, resuming, the HD and > > > CapsLock LED flash short, then the HD led flashes another 1 or 2 times, > > > then everything is dead. No CapsLock reaction, no SysRq reaction. 4 sec > > > power button necessary. > > > > Is it panic? ..no, that woul

Re: [Suspend-devel] Suspending only possible with ATI's fglrx driver

2007-04-11 Thread Pavel Machek
Hi1 > > > as opposed to my initial report in > > > 2006 using a 2.6.17.4 kernel the display won't wake up with the > > > xorg/radeon driver. (When using the 2.6.17.4 kernel last year with the > > > then current xorg > > > > Well, you could find out when xorg/radeon regression happened and ask > >

Re: [Suspend-devel] Toshiba Setellite U205-S5067 Suspend to RAM

2007-04-11 Thread Pavel Machek
Hi! > Since most of the troubleshooting I've read about seems to be about > fixing problems with resuming and I can never get the system to suspend, > I'm not sure where to go from here. Can anyone suggest any next steps? > Is there any more information I should post? printk and udelay are your

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

2007-04-11 Thread Pavel Machek
Hi! > Add s2ram support for PPC architecture. s2ram.{c,h} contain the > implementation of the required functions, used by the main file. The > Makefile selects the correct platform files using $(ARCH) variable > (autodetected by default, can be overridden). > > PPC code is based on original patch

Re: [Suspend-devel] [PATCH 1/2] s2ram: split generic and x86 code

2007-04-11 Thread Pavel Machek
Hi! > Split generic and x86 s2ram code: the new files (s2ram.{c,h}) contains > platform specific code moved out of the main file. > > Signed-Off-By: Luca Tettamanti <[EMAIL PROTECTED]> > --- a/s2ram-x86.c 1970-01-01 01:00:00.0 +0100 > +++ b/s2ram-x86.c 2007-04-10 21:38:51.000

Re: [Suspend-devel] s2ram: my laptop don't wake up, system hanged.

2007-04-10 Thread Pavel Machek
Hi! > My system gets hanged after try to resume from s2ram. My system is not in > whitelist, so I have tried everything in http://en.opensuse.org/S2ram but > with with the same bad results. Try plain s2ram -f from init=/bin/bash boot.

Re: [Suspend-devel] PM-Debugging doesn't seem to work

2007-04-10 Thread Pavel Machek
Hi! > My Lenovo X60T sometimes reboots or freezes on a s2ram resume. > Trying to get more clues i enabled PM-debugging and tracing in my > kernel but i only get this: > > Magic number: 0:798:378 > hash matches drivers/base/power/resume.c:46 > Time: tsc clocksource has been installed. > Time: hpet

  1   2   3   4   5   >