All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stanislaw Gruszka <sgruszka@redhat.com>
To: ath9k-devel@lists.ath9k.org
Subject: [ath9k-devel] [RFC/RFT v2 00/12] ath9k: ASPM fixes
Date: Fri, 29 Jul 2011 15:20:04 +0200	[thread overview]
Message-ID: <20110729132004.GB2280@redhat.com> (raw)
In-Reply-To: <20110727091150.4e517f30@jbarnes-desktop>

On Wed, Jul 27, 2011 at 09:11:50AM -0700, Jesse Barnes wrote:
> On Wed, 27 Jul 2011 14:14:47 +0200
> Stanislaw Gruszka <sgruszka@redhat.com> wrote:
> 
> > This patch series try to fix ath9k ASPM. Some of patches are cleanup
> > only or do merging common code (with e1000e driver).
> > 
> > With CONFIG_PCIEASPM it's possible to change ASPM settings on runtime
> > via /sys/module/pcie_aspm/parameters/policy . However most drivers
> > I looked at assume this setting is constant. I add a callback to pci
> > driver to inform about the change, and allow driver to do needed hw
> > related changes. Currently only ath9k implement the callback, but
> > I think it will be useful for other drivers (i.e. iwlwifi, rtlwifi)
> > that do some own ASPM handling.
> > 
> > These patches needs wide testing as they may work on some systems
> > and not work on others, that can depend on PCIe bridges.
> 
> These look fine, I remember seeing a place where e1000e could use this
> just recently and it looks like you took care of it.

Yep, accually some pci changes I did was based on e1000e code.

> We may be able to sneak this in to 3.1 to make things easier on the
> dependent drivers, but it'll have to wait until I send my pull request
> for the first batch of PCI changes (probably today).

This could go 3.2, except first patch (ath9k specific) which directly
address system lockups. I'm going to a bit rework and post 1st patch
today. Rest patches from the series I'll repost next week.

Thanks
Stanislaw

WARNING: multiple messages have this Message-ID (diff)
From: Stanislaw Gruszka <sgruszka@redhat.com>
To: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: linux-wireless <linux-wireless@vger.kernel.org>,
	ath9k-devel@venema.h4ckr.net, camilo@mesias.co.uk,
	Jonathan Nieder <jrnieder@gmail.com>,
	Tony Houghton <h@realh.co.uk>,
	Rajkumar Manoharan <rmanoharan@atheros.com>,
	Adrian Chadd <adrian@freebsd.org>,
	linux-pci@vger.kernel.org,
	Jeff Kirsher <jeffrey.t.kirsher@intel.com>,
	e1000-devel@lists.sourceforge.net
Subject: Re: [RFC/RFT v2 00/12] ath9k: ASPM fixes
Date: Fri, 29 Jul 2011 15:20:04 +0200	[thread overview]
Message-ID: <20110729132004.GB2280@redhat.com> (raw)
In-Reply-To: <20110727091150.4e517f30@jbarnes-desktop>

On Wed, Jul 27, 2011 at 09:11:50AM -0700, Jesse Barnes wrote:
> On Wed, 27 Jul 2011 14:14:47 +0200
> Stanislaw Gruszka <sgruszka@redhat.com> wrote:
> 
> > This patch series try to fix ath9k ASPM. Some of patches are cleanup
> > only or do merging common code (with e1000e driver).
> > 
> > With CONFIG_PCIEASPM it's possible to change ASPM settings on runtime
> > via /sys/module/pcie_aspm/parameters/policy . However most drivers
> > I looked at assume this setting is constant. I add a callback to pci
> > driver to inform about the change, and allow driver to do needed hw
> > related changes. Currently only ath9k implement the callback, but
> > I think it will be useful for other drivers (i.e. iwlwifi, rtlwifi)
> > that do some own ASPM handling.
> > 
> > These patches needs wide testing as they may work on some systems
> > and not work on others, that can depend on PCIe bridges.
> 
> These look fine, I remember seeing a place where e1000e could use this
> just recently and it looks like you took care of it.

Yep, accually some pci changes I did was based on e1000e code.

> We may be able to sneak this in to 3.1 to make things easier on the
> dependent drivers, but it'll have to wait until I send my pull request
> for the first batch of PCI changes (probably today).

This could go 3.2, except first patch (ath9k specific) which directly
address system lockups. I'm going to a bit rework and post 1st patch
today. Rest patches from the series I'll repost next week.

Thanks
Stanislaw

  reply	other threads:[~2011-07-29 13:20 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-27 12:14 [ath9k-devel] [RFC/RFT v2 00/12] ath9k: ASPM fixes Stanislaw Gruszka
2011-07-27 12:14 ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 01/12] ath9k: skip ->config_pci_powersave() if PCIe port has ASPM disabled Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 02/12] ath9k: remove ->config_pci_powersave() redundant argument Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 03/12] ath9k: merge common ->config_pci_powersave() checks Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 04/12] ath9k: use common PCIe ASPM definces instead of custom ones Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 05/12] ath9k: do btcoex ASPM disabling at initialization time Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 06/12] ath9k: init PCIe PM and SERDES registers if ASPM is enabled Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 07/12] pci: aspm: add function for disabling ASPM Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 08/12] ath9k: use common " Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 09/12] e1000e: " Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 10/12] pci: aspm: add settings changed callback Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 11/12] ath9k: be prepare for dynamic ASPM settings change Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 12:14 ` [ath9k-devel] [RFC/RFT v2 12/12] ath9k: fix initialization ordering issues Stanislaw Gruszka
2011-07-27 12:14   ` Stanislaw Gruszka
2011-07-27 16:11 ` [ath9k-devel] [RFC/RFT v2 00/12] ath9k: ASPM fixes Jesse Barnes
2011-07-27 16:11   ` Jesse Barnes
2011-07-29 13:20   ` Stanislaw Gruszka [this message]
2011-07-29 13:20     ` Stanislaw Gruszka

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=20110729132004.GB2280@redhat.com \
    --to=sgruszka@redhat.com \
    --cc=ath9k-devel@lists.ath9k.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 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.