public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Jones <davej@redhat.com>
To: "Pallipadi, Venkatesh" <venkatesh.pallipadi@intel.com>
Cc: linux-pm@lists.osdl.org, Pavel Machek <pavel@ucw.cz>
Subject: Re: suspending to disk on FC6 not working
Date: Sun, 1 Oct 2006 01:22:03 -0400	[thread overview]
Message-ID: <20061001052203.GF28868@redhat.com> (raw)
In-Reply-To: <EB12A50964762B4D8111D55B764A8454ABBCA8@scsmsx413.amr.corp.intel.com>

On Sat, Sep 30, 2006 at 10:07:16PM -0700, Pallipadi, Venkatesh wrote:

 > >I had not noticed it was not loaded. This maybe a bigger problem. With
 > >the latest FC6 kernel this module gets loaded and thus suspend fails. I
 > >rebuilt the same kernel but plopped this patch in. The module does not
 > >want to load:
 > >
 > ># /sbin/modprobe acpi-cpufreq
 > >FATAL: Error inserting acpi_cpufreq
 > >(/lib/modules/2.6.18-1.2708.cpufreq/kernel/arch/i386/kernel/cpu
 > >/cpufreq/acpi-cpufreq.ko): No such device
 > >
 > >This is why suspend works. But now this module won't load.
 > >
 > 
 > The module was not working on your laptop even when suspend was failing.
 > That is what the absence of /sys.../cpufreq says. If this module was
 > working for you earlier and stopped working recently or even otherwise
 > both this and speedstep-centrino does not work on your platform, that is
 > a separate issue. Was this module working and cpufreq supported on your
 > system with any earlier kernels?

One problem acpi-cpufreq faces when built modular compared to the
other cpufreq drivers is that there's no way of knowing before modprobe'ing
whether or not it's going to do anything or not, so userspace never knows
if it's really 'safe' to load this module.

In Fedora, we modprobe the acpi-cpufreq module if all other scaling
drivers have failed their init routines. Ie, it's treated as a fallback
"Well, nothing else worked, lets try this, and if this don't work out, bail..."
When we added this userspace 'fallback', we started getting a bunch of
reports like the above, from users who had CPUs that couldn't do any scaling
at all. The STICKY tag was added to the driver so that it would stop
making noise for those users.

Looking at the problem differently, we *could* add back the sticky tag,
and add some explicit checks for !data in the suspend/resume paths
to abort cleanly. It's a bit of a hack though.

hmm?

	Dave

  reply	other threads:[~2006-10-01  5:22 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-01  5:07 suspending to disk on FC6 not working Pallipadi, Venkatesh
2006-10-01  5:22 ` Dave Jones [this message]
2006-10-01 12:28   ` Rafael J. Wysocki
2006-10-01 18:36     ` Dave Jones
2006-10-01 18:38       ` Pavel Machek
2006-10-01 18:56         ` Dave Jones
2006-10-01  5:22 ` Louis Garcia
  -- strict thread matches above, loose matches on Subject: below --
2006-09-29 22:55 Pallipadi, Venkatesh
2006-10-01  2:49 ` Louis Garcia
2006-10-01  3:21   ` Dave Jones
2006-10-01  5:00     ` Louis Garcia
2006-10-01  5:05       ` Dave Jones
2006-10-01  5:30         ` Pavel Machek
2006-09-29 22:23 Pallipadi, Venkatesh
2006-09-29 21:35 Pallipadi, Venkatesh
2006-09-29 22:14 ` Pavel Machek
2006-09-29 22:25 ` Dave Jones
2006-09-24 22:55 Louis Garcia
2006-09-25  7:16 ` Rafael J. Wysocki
2006-09-25 17:10   ` Louis Garcia
2006-09-25 18:43     ` Rafael J. Wysocki
2006-09-25 19:34       ` Louis Garcia
2006-09-25 20:15         ` Rafael J. Wysocki
2006-09-25 23:04           ` Louis Garcia
2006-09-26 10:00             ` Rafael J. Wysocki
2006-09-26 21:47               ` Louis Garcia
2006-09-26 22:01                 ` Rafael J. Wysocki
2006-09-26 22:15                   ` Pavel Machek
2006-09-26 22:26                     ` Rafael J. Wysocki
2006-09-26 22:39                       ` Louis Garcia
2006-09-26 22:45                         ` Pavel Machek
2006-09-26 22:54                           ` Rafael J. Wysocki
2006-09-26 22:58                             ` Pavel Machek
2006-09-26 23:09                               ` Louis Garcia
2006-09-26 23:12                                 ` Pavel Machek
2006-09-27  2:36                     ` Dave Jones
2006-09-27  2:51                       ` Dave Jones
2006-09-27  6:55                         ` Louis Garcia
2006-09-29 20:54                           ` Dave Jones

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=20061001052203.GF28868@redhat.com \
    --to=davej@redhat.com \
    --cc=linux-pm@lists.osdl.org \
    --cc=pavel@ucw.cz \
    --cc=venkatesh.pallipadi@intel.com \
    /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