From: Maxim Levitsky <maximlevitsky@gmail.com>
To: Matthew Garrett <mjg@redhat.com>,
Jussi Kivilinna <jussi.kivilinna@mbnet.fi>,
"Luis R. Rodriguez" <mcgrof@bombadil.infradead.org>
Cc: linux-pci@vger.kernel.org, jbarnes@virtuousgeek.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] pci: Don't enable aspm before drivers have had a chance to veto it
Date: Fri, 18 Jun 2010 15:06:51 +0300 [thread overview]
Message-ID: <1276862811.19918.10.camel@maxim-laptop> (raw)
In-Reply-To: <1276113907-22432-1-git-send-email-mjg@redhat.com>
On Wed, 2010-06-09 at 16:05 -0400, Matthew Garrett wrote:
> The aspm code will currently set the configured aspm policy before drivers
> have had an opportunity to indicate that their hardware doesn't support it.
> Unfortunately, putting some hardware in L0 or L1 can result in the hardware
> no longer responding to any requests, even after aspm is disabled. It makes
> more sense to leave aspm policy at the BIOS defaults at initial setup time,
> reconfiguring it after pci_enable_device() is called. This allows the
> driver to blacklist individual devices beforehand.
>
> Signed-off-by: Matthew Garrett <mjg@redhat.com>
> ---
Hi,
I recently discovered that my aspire one wireless troubles (card just
dies after a while) are caused by ASPM L0S state.
The device (AR5001) seems to have a hardware bug, and it also disables
L0S in windows driver.
Unfortenuly BIOS (news at 11) enables L0S.
Its easy to disable ASPM from driver. It just a matter of calling
pci_disable_link_state.
However, that depends on CONFIG_PCIEASPM.
How about making pci_disable_link_state always available or even better,
just make CONFIG_PCIEASPM unconditional?
Btw, this is my last linux problem that had slim chances to be solved.
The other one was s2ram hang on my primary notebook.
This just shows that in linux everyone helps each other.
I fixed almost impossible s2ram problem on my notebook.
Jussi Kivilinna fixed almost impossible to solve problem on my aspire
one.
Best regards,
Maxim Levitsky
next prev parent reply other threads:[~2010-06-18 12:06 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-09 20:05 Matthew Garrett
2010-06-10 2:05 ` Kenji Kaneshige
2010-06-15 15:11 ` Tomas Henzl
2010-06-15 15:18 ` Matthew Garrett
2010-06-15 15:29 ` Tomas Henzl
2010-06-15 15:32 ` Matthew Garrett
2010-06-18 12:06 ` Maxim Levitsky [this message]
2010-06-18 16:12 ` Jesse Barnes
2010-06-18 17:05 ` Maxim Levitsky
2010-06-18 17:08 ` Jesse Barnes
2010-06-18 17:15 ` Jesse Barnes
2010-06-18 17:44 ` Maxim Levitsky
2010-06-18 17:15 ` Jesse Barnes
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=1276862811.19918.10.camel@maxim-laptop \
--to=maximlevitsky@gmail.com \
--cc=jbarnes@virtuousgeek.org \
--cc=jussi.kivilinna@mbnet.fi \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=mcgrof@bombadil.infradead.org \
--cc=mjg@redhat.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
all inboxes | Powered by JetHome®