All of lore.kernel.org
 help / color / mirror / Atom feed
From: Len Brown <lenb@kernel.org>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Shaohua Li <shaohua.li@intel.com>,
	linux acpi <linux-acpi@vger.kernel.org>
Subject: Re: suspend order - again
Date: Thu, 27 Mar 2008 11:43:28 -0400	[thread overview]
Message-ID: <200803271143.28972.lenb@kernel.org> (raw)
In-Reply-To: <200803141254.55597.rjw@sisk.pl>

On Friday 14 March 2008, Rafael J. Wysocki wrote:
> On Friday, 14 of March 2008, Shaohua Li wrote:
> > I just found a system (Asus A6B00VC) suffers a regression of suspend
> > order adjust. In the system, if _PTS is called before pci device
> > suspend, pci config read of slot 01:01.0 will always return 0xffffffff
> > (only this slot, not other devices). Adding acpi_new_pts_ordering fixes
> > this. I checked the log, _PTS itself doesn't generate any pci config
> > access, it appears _PTS call into SMBIOS and changes something. Note,
> > this is ACPI 1.0 table. Should we just blacklist the system or re-think
> > the suspend order?
> 
> I don't want to change the ordering of code.  It's been changed for many times
> and it always turned out that some systems didn't work.
> 
> If we can implement the blacklisting in a reasonable fashion, I'd prefer to do
> just that.

It isn't obvious to me why this regression is exempt from the
normal response we have to regressions found during -rc.
Particullarly sinced it was root caused to show that
we did the right thing before and we do the wrong thing now.

-Len

  reply	other threads:[~2008-03-27 15:44 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1205482960.23855.4.camel@sli10-desk.sh.intel.com>
2008-03-14 11:54 ` suspend order - again Rafael J. Wysocki
2008-03-27 15:43   ` Len Brown [this message]
2008-03-27 16:42     ` Rafael J. Wysocki
2008-03-27 17:06       ` Len Brown
2008-03-27 17:36         ` Carlos Corbacho

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=200803271143.28972.lenb@kernel.org \
    --to=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=rjw@sisk.pl \
    --cc=shaohua.li@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.