* [PATCH 2.6.{32,31}] iwlwifi: power up all devices for EEPROM read
@ 2010-01-21 12:22 Stanislaw Gruszka
2010-01-21 18:02 ` reinette chatre
2010-01-21 20:51 ` [stable] [PATCH 2.6.{32, 31}] " Greg KH
0 siblings, 2 replies; 6+ messages in thread
From: Stanislaw Gruszka @ 2010-01-21 12:22 UTC (permalink / raw)
To: stable, linux-wireless
Cc: Reinette Chatre, John W. Linville, Stanislaw Gruszka
commit f8701fe3aec24fcfb0dfa19aab47904611f96daf upstream
On 2.6.31 and 2.6.32 iwl3945 crash during kdump, I discovered
above upstream commit fix that.
Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
---
I tested patch on 2.6.32 with iwl3945 - it fix kdump, and on 2.6.31
with iwl5300 - no regression notice.
drivers/net/wireless/iwlwifi/iwl-eeprom.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/iwlwifi/iwl-eeprom.c b/drivers/net/wireless/iwlwifi/iwl-eeprom.c
index 18dc3a4..870c0db 100644
--- a/drivers/net/wireless/iwlwifi/iwl-eeprom.c
+++ b/drivers/net/wireless/iwlwifi/iwl-eeprom.c
@@ -486,6 +486,7 @@ int iwl_eeprom_init(struct iwl_priv *priv)
}
e = (__le16 *)priv->eeprom;
+ priv->cfg->ops->lib->apm_ops.init(priv);
ret = priv->cfg->ops->lib->eeprom_ops.verify_signature(priv);
if (ret < 0) {
IWL_ERR(priv, "EEPROM not found, EEPROM_GP=0x%08x\n", gp);
@@ -556,6 +557,8 @@ done:
err:
if (ret)
iwl_eeprom_free(priv);
+ /* Reset chip to save power until we load uCode during "up". */
+ priv->cfg->ops->lib->apm_ops.stop(priv);
alloc_err:
return ret;
}
--
1.6.2.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 2.6.{32,31}] iwlwifi: power up all devices for EEPROM read
2010-01-21 12:22 [PATCH 2.6.{32,31}] iwlwifi: power up all devices for EEPROM read Stanislaw Gruszka
@ 2010-01-21 18:02 ` reinette chatre
2010-01-22 7:43 ` Stanislaw Gruszka
2010-01-21 20:51 ` [stable] [PATCH 2.6.{32, 31}] " Greg KH
1 sibling, 1 reply; 6+ messages in thread
From: reinette chatre @ 2010-01-21 18:02 UTC (permalink / raw)
To: Stanislaw Gruszka
Cc: stable@kernel.org, linux-wireless@vger.kernel.org,
John W. Linville
Hi Stanislaw,
On Thu, 2010-01-21 at 04:22 -0800, Stanislaw Gruszka wrote:
> commit f8701fe3aec24fcfb0dfa19aab47904611f96daf upstream
This is strange. As the commit you refer to indicate, it fixes problems
introduced by "iwlwifi: remove power-wasting calls to apm_ops.init()"
and "iwlagn: power up device before initializing EEPROM" - these patches
have only been introduced in 2.6.33, so targeting this patch for earlier
kernels does not seem right.
Note that what the abovementioned patches did was move the device "power
up" routines from probe to be closer to where they are needed
(iwl_eeprom_init) in an effort to optimize power usage.
In the code you are changing, the above patches are not present, so the
original behavior of device power up during probe is still present.
The 2.6.32 and 2.6.31 code you are changing here adds a device "power
up" call into iwl_eeprom_init. This is not necessary since as you can
see, in iwl3945_pci_probe this exact same call is made right before
iwl_eeprom_init is called.
>
> On 2.6.31 and 2.6.32 iwl3945 crash during kdump, I discovered
> above upstream commit fix that.
This does not make sense. Was this testing done in the stable kernel
from linux-2.6-stable or perhaps a kernel that had those other power
saving patches backported?
>
> Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
> ---
> I tested patch on 2.6.32 with iwl3945 - it fix kdump, and on 2.6.31
> with iwl5300 - no regression notice.
>
> drivers/net/wireless/iwlwifi/iwl-eeprom.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/iwlwifi/iwl-eeprom.c b/drivers/net/wireless/iwlwifi/iwl-eeprom.c
> index 18dc3a4..870c0db 100644
> --- a/drivers/net/wireless/iwlwifi/iwl-eeprom.c
> +++ b/drivers/net/wireless/iwlwifi/iwl-eeprom.c
> @@ -486,6 +486,7 @@ int iwl_eeprom_init(struct iwl_priv *priv)
> }
> e = (__le16 *)priv->eeprom;
>
> + priv->cfg->ops->lib->apm_ops.init(priv);
> ret = priv->cfg->ops->lib->eeprom_ops.verify_signature(priv);
> if (ret < 0) {
> IWL_ERR(priv, "EEPROM not found, EEPROM_GP=0x%08x\n", gp);
> @@ -556,6 +557,8 @@ done:
> err:
> if (ret)
> iwl_eeprom_free(priv);
> + /* Reset chip to save power until we load uCode during "up". */
> + priv->cfg->ops->lib->apm_ops.stop(priv);
> alloc_err:
> return ret;
> }
Reinette
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 2.6.{32,31}] iwlwifi: power up all devices for EEPROM read
2010-01-21 18:02 ` reinette chatre
@ 2010-01-22 7:43 ` Stanislaw Gruszka
0 siblings, 0 replies; 6+ messages in thread
From: Stanislaw Gruszka @ 2010-01-22 7:43 UTC (permalink / raw)
To: reinette chatre
Cc: stable@kernel.org, linux-wireless@vger.kernel.org,
John W. Linville
Hi
On Thu, Jan 21, 2010 at 10:02:24AM -0800, reinette chatre wrote:
> The 2.6.32 and 2.6.31 code you are changing here adds a device "power
> up" call into iwl_eeprom_init. This is not necessary since as you can
> see, in iwl3945_pci_probe this exact same call is made right before
> iwl_eeprom_init is called.
Sorry, missed that. I should wrote in the changelog that this is backport
of more commits and remove power up stuff from pci proble as well.
> >
> > On 2.6.31 and 2.6.32 iwl3945 crash during kdump, I discovered
> > above upstream commit fix that.
>
> This does not make sense. Was this testing done in the stable kernel
> from linux-2.6-stable or perhaps a kernel that had those other power
> saving patches backported?
I tested backported patch on 2.6.32.4. It fix kdump, this issue is
100% reproducible on my laptop with iwl3945.
The functional change introduced in this patch is we disable device
after EEPROM read. I think that make difference with kdump. Not sure
if this right fix or just workaround.
Debugging kdump kernel is somehow limited, I have no netconsole or serial
console option. I can only see last part of call trace where are iwlwifi
functions. I'm going dig into kernel to make calltrace shorter and see
where exact the oops is.
Cheers
Stanislaw
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [stable] [PATCH 2.6.{32, 31}] iwlwifi: power up all devices for EEPROM read
2010-01-21 12:22 [PATCH 2.6.{32,31}] iwlwifi: power up all devices for EEPROM read Stanislaw Gruszka
2010-01-21 18:02 ` reinette chatre
@ 2010-01-21 20:51 ` Greg KH
2010-01-21 20:59 ` reinette chatre
1 sibling, 1 reply; 6+ messages in thread
From: Greg KH @ 2010-01-21 20:51 UTC (permalink / raw)
To: Stanislaw Gruszka
Cc: stable, linux-wireless, Reinette Chatre, John W. Linville
On Thu, Jan 21, 2010 at 01:22:25PM +0100, Stanislaw Gruszka wrote:
> commit f8701fe3aec24fcfb0dfa19aab47904611f96daf upstream
>
> On 2.6.31 and 2.6.32 iwl3945 crash during kdump, I discovered
> above upstream commit fix that.
>
> Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
> ---
> I tested patch on 2.6.32 with iwl3945 - it fix kdump, and on 2.6.31
> with iwl5300 - no regression notice.
Thanks for the backport, I've queued this up now.
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [stable] [PATCH 2.6.{32, 31}] iwlwifi: power up all devices for EEPROM read
2010-01-21 20:51 ` [stable] [PATCH 2.6.{32, 31}] " Greg KH
@ 2010-01-21 20:59 ` reinette chatre
2010-01-21 21:05 ` Greg KH
0 siblings, 1 reply; 6+ messages in thread
From: reinette chatre @ 2010-01-21 20:59 UTC (permalink / raw)
To: Greg KH
Cc: Stanislaw Gruszka, stable@kernel.org,
linux-wireless@vger.kernel.org, John W. Linville
Hi Greg,
On Thu, 2010-01-21 at 12:51 -0800, Greg KH wrote:
> On Thu, Jan 21, 2010 at 01:22:25PM +0100, Stanislaw Gruszka wrote:
> > commit f8701fe3aec24fcfb0dfa19aab47904611f96daf upstream
> >
> > On 2.6.31 and 2.6.32 iwl3945 crash during kdump, I discovered
> > above upstream commit fix that.
> >
> > Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
> > ---
> > I tested patch on 2.6.32 with iwl3945 - it fix kdump, and on 2.6.31
> > with iwl5300 - no regression notice.
>
> Thanks for the backport, I've queued this up now.
This backport is on top of patches that are not present in 2.6.31 or
2.6.32. Please see the message I sent in response to the original post.
Could we please finish that discussion first before looking into merging
this patch?
Thank you
Reinette
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [stable] [PATCH 2.6.{32, 31}] iwlwifi: power up all devices for EEPROM read
2010-01-21 20:59 ` reinette chatre
@ 2010-01-21 21:05 ` Greg KH
0 siblings, 0 replies; 6+ messages in thread
From: Greg KH @ 2010-01-21 21:05 UTC (permalink / raw)
To: reinette chatre
Cc: Stanislaw Gruszka, stable@kernel.org,
linux-wireless@vger.kernel.org, John W. Linville
On Thu, Jan 21, 2010 at 12:59:55PM -0800, reinette chatre wrote:
> Hi Greg,
>
> On Thu, 2010-01-21 at 12:51 -0800, Greg KH wrote:
> > On Thu, Jan 21, 2010 at 01:22:25PM +0100, Stanislaw Gruszka wrote:
> > > commit f8701fe3aec24fcfb0dfa19aab47904611f96daf upstream
> > >
> > > On 2.6.31 and 2.6.32 iwl3945 crash during kdump, I discovered
> > > above upstream commit fix that.
> > >
> > > Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
> > > ---
> > > I tested patch on 2.6.32 with iwl3945 - it fix kdump, and on 2.6.31
> > > with iwl5300 - no regression notice.
> >
> > Thanks for the backport, I've queued this up now.
>
> This backport is on top of patches that are not present in 2.6.31 or
> 2.6.32. Please see the message I sent in response to the original post.
> Could we please finish that discussion first before looking into merging
> this patch?
Oops, ok, I'll go drop it, thanks for the quick response.
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2010-01-22 7:45 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-21 12:22 [PATCH 2.6.{32,31}] iwlwifi: power up all devices for EEPROM read Stanislaw Gruszka
2010-01-21 18:02 ` reinette chatre
2010-01-22 7:43 ` Stanislaw Gruszka
2010-01-21 20:51 ` [stable] [PATCH 2.6.{32, 31}] " Greg KH
2010-01-21 20:59 ` reinette chatre
2010-01-21 21:05 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).