Re: [Pm-utils] code available to solve sound-after-suspend problems

2007-03-14 Thread Mark Stosberg
Olivier Blin wrote: > Mark Stosberg <[EMAIL PROTECTED]> writes: > >>> But maybe just because nobody even bothered to file a bug against the >>> kernel. >>> It might also very well be that it is just not documented well enough. I >>> figure that you would not google for "pm-utils custom hook" if y

[Pm-utils] Re: HAL Crasher

2007-03-14 Thread Richard Hughes
On Thu, 2007-03-15 at 00:06 +, Richard Hughes wrote: > HAL really shouldn't crash. I've got no time to debug this now. Release > blocker? (gdb) bt #0 0x007d4402 in __kernel_vsyscall () #1 0x00288f60 in raise () from /lib/libc.so.6 #2 0x0028a7b1 in abort () from /lib/libc.so.6 #3 0x02df8f55

[Pm-utils] HAL Crasher

2007-03-14 Thread Richard Hughes
HAL master, CK master, pm-utils from rawhide (basically a rawhide system): HAL crashes as soon as g-p-m is started up: /home/hughsie/Code/hal/hal/hald/../tools/hal-system-power-set-power-save exited 00:01:14.749 [I] hald_dbus.c:3228: No more methods in queue 00:01:14.749 [I] hald_dbus.c:3291: fai

Re: [Pm-utils] [Patch] add userspace suspend signature to pm-reset-swap

2007-03-14 Thread Stefan Seyfried
On Wed, Mar 14, 2007 at 07:06:29PM +0100, Olivier Blin wrote: > Stefan Seyfried <[EMAIL PROTECTED]> writes: > > > On Wed, Mar 14, 2007 at 10:53:17AM -0400, Peter Jones wrote: > >> On Tue, 2007-02-20 at 19:36 +0100, Stefan Seyfried wrote: > >> > >> > The printf for "no signature found" is not real

Re: [Pm-utils] [Patch] add userspace suspend signature to pm-reset-swap

2007-03-14 Thread Olivier Blin
Stefan Seyfried <[EMAIL PROTECTED]> writes: > On Wed, Mar 14, 2007 at 10:53:17AM -0400, Peter Jones wrote: >> On Tue, 2007-02-20 at 19:36 +0100, Stefan Seyfried wrote: >> >> > The printf for "no signature found" is not really necessary, but i found >> > it useful :-) >> >> Eh, I actually like th

Re: [Pm-utils] Re: pm-utils' 55battery: WTF?

2007-03-14 Thread Stefan Seyfried
On Wed, Mar 14, 2007 at 05:17:31PM +0100, Holger Macht wrote: > > For the record, I'd much rather that pm-suspend went away (e.g. moved > > into /usr/sbin) and command line users started used something like > > xdg-suspend (that don't exist yet, but look at Portland) to poke the > > desktop session

Re: [Pm-utils] fix cpufreq governor setting on dualcore machines

2007-03-14 Thread Stefan Seyfried
On Wed, Mar 14, 2007 at 11:36:41AM -0400, Peter Jones wrote: > On Tue, 2007-03-13 at 18:23 +0100, Stefan Seyfried wrote: > > > https://bugzilla.novell.com/show_bug.cgi?id=227854#c13 sums it up nicely. > > On dualcore cpus, where the governor setting affects more than one core, > > the cpufreq poli

Re: [Pm-utils] Re: s2both

2007-03-14 Thread Stefan Seyfried
On Wed, Mar 14, 2007 at 11:14:18AM -0400, Peter Jones wrote: > On Wed, 2007-03-14 at 08:16 +0100, Stefan Seyfried wrote: > > > - you cannot stopservice / restartservice services with a "-" in their name > > (patch attached) > > (I will reply to the other points in a separate mail) > > I really

Re: [Pm-utils] [Patch] add userspace suspend signature to pm-reset-swap

2007-03-14 Thread Stefan Seyfried
On Wed, Mar 14, 2007 at 10:53:17AM -0400, Peter Jones wrote: > On Tue, 2007-02-20 at 19:36 +0100, Stefan Seyfried wrote: > > > The printf for "no signature found" is not really necessary, but i found > > it useful :-) > > Eh, I actually like the "no output if no failure" better... Feel free to >

Re: [Pm-utils] [PATCH] use service unconditionally

2007-03-14 Thread Peter Jones
On Wed, 2007-03-14 at 18:26 +0200, S.Çağlar Onur wrote: > 14 Mar 2007 Çar tarihinde, Peter Jones şunları yazmıştı: > > Yeah, I think that's the right idea, but I think the patch below is > > slightly better (also untested!). > > > > BTW, isn't the $": blah blah blah" the right thing to do rather t

Re: [Pm-utils] [PATCH] use service unconditionally

2007-03-14 Thread S.Çağlar Onur
14 Mar 2007 Çar tarihinde, Peter Jones şunları yazmıştı: > Yeah, I think that's the right idea, but I think the patch below is > slightly better (also untested!). > > BTW, isn't the $": blah blah blah" the right thing to do rather than > setting LC_ALL? > > Obviously not applied yet. Attached one

Re: [Pm-utils] Re: pm-utils' 55battery: WTF?

2007-03-14 Thread Holger Macht
On Wed 14. Mar - 12:13:39, David Zeuthen wrote: > On Wed, 2007-03-14 at 12:05 -0400, Peter Jones wrote: > > On Wed, 2007-03-14 at 13:56 +0100, Olivier Blin wrote: > > > David Zeuthen <[EMAIL PROTECTED]> writes: > > > > It tells HAL to rescan the battery - in case someone changed the battery > > > >

Re: [Pm-utils] Re: pm-utils' 55battery: WTF?

2007-03-14 Thread Holger Macht
On Wed 14. Mar - 12:05:50, Peter Jones wrote: > On Wed, 2007-03-14 at 13:56 +0100, Olivier Blin wrote: > > David Zeuthen <[EMAIL PROTECTED]> writes: > > > It tells HAL to rescan the battery - in case someone changed the battery > > > we want the new serial number etc... if I recall Peter's reasonin

Re: [Pm-utils] Re: pm-utils' 55battery: WTF?

2007-03-14 Thread David Zeuthen
On Wed, 2007-03-14 at 12:05 -0400, Peter Jones wrote: > On Wed, 2007-03-14 at 13:56 +0100, Olivier Blin wrote: > > David Zeuthen <[EMAIL PROTECTED]> writes: > > > It tells HAL to rescan the battery - in case someone changed the battery > > > we want the new serial number etc... if I recall Peter's

Re: [Pm-utils] Re: pm-utils' 55battery: WTF?

2007-03-14 Thread Peter Jones
On Wed, 2007-03-14 at 13:56 +0100, Olivier Blin wrote: > David Zeuthen <[EMAIL PROTECTED]> writes: > > It tells HAL to rescan the battery - in case someone changed the battery > > we want the new serial number etc... if I recall Peter's reasoning > > correctly. I think this check is better done in

Re: [Pm-utils] [PATCH] use service unconditionally

2007-03-14 Thread Peter Jones
On Mon, 2007-03-12 at 08:25 +0100, Stefan Seyfried wrote: > > It appears that is inevitable that we introduce distro specific > > functions here, because you do not appear to have /etc/init.d and we > > don't have service. > > how about (untested!): Yeah, I think that's the right idea, but I thi

Re: [Pm-utils] [PATCH] use service unconditionally

2007-03-14 Thread Peter Jones
On Mon, 2007-03-12 at 08:25 +0100, Stefan Seyfried wrote: > On Sat, Mar 10, 2007 at 09:11:14PM +0100, Tim Dijkstra wrote: > > On Sat, 10 Mar 2007 16:49:04 +0200 > > "S.Çağlar Onur" <[EMAIL PROTECTED]> wrote: > > > * Ignore all threads/patches/comments so distros/people can > > > fork/alternate >

Re: [Pm-utils] fix cpufreq governor setting on dualcore machines

2007-03-14 Thread Holger Macht
On Wed 14. Mar - 11:36:41, Peter Jones wrote: > On Tue, 2007-03-13 at 18:23 +0100, Stefan Seyfried wrote: > > > https://bugzilla.novell.com/show_bug.cgi?id=227854#c13 sums it up nicely. > > On dualcore cpus, where the governor setting affects more than one core, > > the cpufreq policy will be set

Re: [Pm-utils] Re: pm-utils' 55battery: WTF?

2007-03-14 Thread David Zeuthen
On Wed, 2007-03-14 at 11:00 -0400, Peter Jones wrote: > On Tue, 2007-03-13 at 14:45 -0400, David Zeuthen wrote: > > > It tells HAL to rescan the battery - in case someone changed the battery > > we want the new serial number etc... if I recall Peter's reasoning > > correctly. I think this check is

Re: [Pm-utils] [patch] fix pm-reset-swap

2007-03-14 Thread Peter Jones
On Wed, 2007-03-14 at 17:24 +0200, S.Çağlar Onur wrote: > 14 Mar 2007 Çar tarihinde, Stefan Seyfried şunları yazmıştı: > > Yes, this is probably the best approach, i'll try to get something like > > that into our util-linux package, too. > > > > However, pm-reset-swap as it is in CVS is really dan

Re: [Pm-utils] fix cpufreq governor setting on dualcore machines

2007-03-14 Thread Peter Jones
On Tue, 2007-03-13 at 18:23 +0100, Stefan Seyfried wrote: > https://bugzilla.novell.com/show_bug.cgi?id=227854#c13 sums it up nicely. > On dualcore cpus, where the governor setting affects more than one core, > the cpufreq policy will be set to performance after resume. The idea is certainly corr

Re: [Pm-utils] [patch] fix pm-reset-swap

2007-03-14 Thread S.Çağlar Onur
14 Mar 2007 Çar tarihinde, Stefan Seyfried şunları yazmıştı: > Yes, this is probably the best approach, i'll try to get something like > that into our util-linux package, too. > > However, pm-reset-swap as it is in CVS is really dangerous, so it should > either be removed or fixed. Although we ar

Re: [Pm-utils] Re: s2both

2007-03-14 Thread Peter Jones
On Wed, 2007-03-14 at 08:16 +0100, Stefan Seyfried wrote: > - you cannot stopservice / restartservice services with a "-" in their name > (patch attached) (I will reply to the other points in a separate mail) I really hate using "foo" in real (not "here's some arbitrary example") code. So, ap

Re: [Pm-utils] Re: pm-utils' 55battery: WTF?

2007-03-14 Thread Peter Jones
On Tue, 2007-03-13 at 14:45 -0400, David Zeuthen wrote: > It tells HAL to rescan the battery - in case someone changed the battery > we want the new serial number etc... if I recall Peter's reasoning > correctly. I think this check is better done in HAL than in pm-utils; it > doesn't make sense to

Re: [Pm-utils] Re: s2both

2007-03-14 Thread Rafael J. Wysocki
On Wednesday, 14 March 2007 12:45, Tim Dijkstra wrote: > On Tue, 13 Mar 2007 23:28:11 +0100 > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > > For uswsusp, you can try to open /dev/snapshot and do some tests. > > For example, you can check if the SNAPSHOT_SET_IMAGE_SIZE ioctl works > > (should

Re: [Pm-utils] [Patch] add userspace suspend signature to pm-reset-swap

2007-03-14 Thread Peter Jones
On Tue, 2007-02-20 at 19:36 +0100, Stefan Seyfried wrote: > The printf for "no signature found" is not really necessary, but i found > it useful :-) Eh, I actually like the "no output if no failure" better... Feel free to convince me otherwise, though ;) Until then, applied as: --- src/pm-reset

Re: [Pm-utils] Re: Resume via quirks, not using the DBUS method, Was: Release Candidates ?

2007-03-14 Thread Peter Jones
On Fri, 2007-03-09 at 16:47 +0100, Stefan Seyfried wrote: > On Tue, Mar 06, 2007 at 12:51:48PM -0500, Peter Jones wrote: > > On Tue, 2007-03-06 at 18:23 +0100, Michael Biebl wrote: > > > 2007/3/6, Peter Jones <[EMAIL PROTECTED]>: > > > > > > > > I find myself wondering if /etc/pm/config shouldn't r

Re: [Pm-utils] code available to solve sound-after-suspend problems

2007-03-14 Thread Olivier Blin
Mark Stosberg <[EMAIL PROTECTED]> writes: >> But maybe just because nobody even bothered to file a bug against the kernel. >> It might also very well be that it is just not documented well enough. I >> figure that you would not google for "pm-utils custom hook" if you had a >> problem with sound a

Re: [Pm-utils] Re: pm-utils' 55battery: WTF?

2007-03-14 Thread Olivier Blin
David Zeuthen <[EMAIL PROTECTED]> writes: > On Tue, 2007-03-13 at 19:35 +0100, Stefan Seyfried wrote: >> dbus-send --print-reply --system --reply-timeout=2000 \ >> --dest=org.freedesktop.Hal $x \ >> org.freedesktop.Hal.Device.Rescan string:$x

Re: [Pm-utils] Re: pm-utils' 55battery: WTF?

2007-03-14 Thread Stefan Seyfried
On Wed, Mar 14, 2007 at 01:56:02PM +0100, Olivier Blin wrote: > David Zeuthen <[EMAIL PROTECTED]> writes: > > It tells HAL to rescan the battery - in case someone changed the battery > > we want the new serial number etc... if I recall Peter's reasoning > > correctly. I think this check is better d

Re: [Pm-utils] [patch] fix pm-reset-swap

2007-03-14 Thread Stefan Seyfried
On Wed, Mar 14, 2007 at 12:52:45PM +0100, Tim Dijkstra wrote: > On Tue, 13 Mar 2007 18:26:15 +0100 > Stefan Seyfried <[EMAIL PROTECTED]> wrote: > > > Hi, > > > > pm-reset-swap is dangerously broken right now. This is the "rc = 1" hunk. > > > > Additinally, it does not fix userspace suspend image

Re: [Pm-utils] [patch] fix pm-reset-swap

2007-03-14 Thread Tim Dijkstra
On Tue, 13 Mar 2007 18:26:15 +0100 Stefan Seyfried <[EMAIL PROTECTED]> wrote: > Hi, > > pm-reset-swap is dangerously broken right now. This is the "rc = 1" hunk. > > Additinally, it does not fix userspace suspend images. > The third hunk is more a cosmetic issue, i'm not passionate about it. FW

Re: [Pm-utils] Re: s2both

2007-03-14 Thread Tim Dijkstra
On Tue, 13 Mar 2007 23:28:11 +0100 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > For uswsusp, you can try to open /dev/snapshot and do some tests. > For example, you can check if the SNAPSHOT_SET_IMAGE_SIZE ioctl works > (should work if uswsusp is supported). The SNAPSHOT_AVAIL_SWAP should > w

Re: [Pm-utils] Re: s2both

2007-03-14 Thread Richard Hughes
On Wed, 2007-03-14 at 08:16 +0100, Stefan Seyfried wrote: > Patches for all of the above have been posted, most of them multiple > times, on the pm-utils list. Peter, before the release can you review some of these? Thanks. Richard. ___ Pm-utils maili