From: "Rafa? Bilski" <rafalbilski@interia.pl>
To: linux-kernel@vger.kernel.org
Cc: Dave Jones <davej@codemonkey.org.uk>
Subject: Re: [ PATCH ] Longhaul - call suspend(PMSG_FREEZE) before and resume() after
Date: Sun, 28 May 2006 13:38:01 +0200 [thread overview]
Message-ID: <44798B99.9070608@interia.pl> (raw)
> This is an horrible hack that breaks so many defined semantics that it's
> not even funny.
> If you want something like that, then you need to freeze/resume _all_
> devices with the proper ordering defined by the bus linkage. It has a
> number of side effects though, can't be done that easily. Maybe cpufreq
> should have the necessary infrastructure for that ?
> That's the wrong approach. If you need to stop
> DMA's during the frequency change, you either need to fix all drivers to
> register cpufreq notifiers that do so (ick !) or if you want to reuse
> the PM callbacks, you need to respect their semantics, notably vs. call
> ordering, or very bad things will happen.
> If we want to go that way, we probably need to add a bit of
> infrastructure to cpufreq to cooperate with the PM code to trigger a
> "light" machine suspend/resume, though expect delays and artifacts, it's
> not something that code be done lightly.
> Ben.
I'm toys salesman. I don't think that I'm capable.
There is already necessary infrastructure (PM). I can do freeze for all
devices with just one function call. Problem is that only block devices
implement freeze. Most devices do suspend insteed of freeze. Some
devices (Speedtouch) don't implement suspend/resume. After USB power
down You have to unplug modem.
Block devices are at top level? If I remove PCI suspend/resume
(network card compatible) will this be OK? Other subsystems
are visible. Can block subsystem be visible too?
> But you should really add that preempt_disable and not try this on smp
> system...
> Pavel
Datasheet for my C3 Nehemiah says that this processor don't have local
APIC and is not SMP capable. I have assumed (based on original longhaul.c)
that all VIA C3 are not SMP capable.
Would You consider appling part of this patch if I add all my assumptions
to Kconfig?
depends on EXPERIMENTAL && (HZ_100 || HZ_250) && (PREEMPT_NONE || PREEMPT_VOLUNTARY)
Rafal
----------------------------------------------------------------------
Potrzebujesz gotowki? Halogotowka to nawet 50 000 bez wizyty w banku.
Rata od 35 zl, bez poreczycieli. Wypelnij formularz. Oddzwonimy.
>>> http://link.interia.pl/f1942
next reply other threads:[~2006-05-28 11:38 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-05-28 11:38 Rafa? Bilski [this message]
2006-05-28 14:56 ` [ PATCH ] Longhaul - call suspend(PMSG_FREEZE) before and resume() after Dave Jones
2006-05-28 16:23 ` Rafał Bilski
-- strict thread matches above, loose matches on Subject: below --
2006-05-28 14:42 [PATCH] " Rafał Bilski
2006-05-27 22:30 Rafał Bilski
2006-05-28 7:16 ` Pavel Machek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=44798B99.9070608@interia.pl \
--to=rafalbilski@interia.pl \
--cc=davej@codemonkey.org.uk \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox