linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* 3c509 resume from hibernation broken somewhere between 2.6.25 and 2.6.28
@ 2009-02-06 13:22 Ondrej Zary
  2009-02-07  6:04 ` David Miller
  0 siblings, 1 reply; 3+ messages in thread
From: Ondrej Zary @ 2009-02-06 13:22 UTC (permalink / raw)
  To: netdev, Linux Kernel

Hello,
last year, I posted a patch which fixed hibernation on 3c509 cards. That was 
back in 2.6.24. It worked fine in 2.6.25. But then I stopped using 
hibernation (as it did not work with my new IT8212 RAID controller).

Now I fixed it and noticed that 3c509 does not wake up properly anymore (in 
2.6.28) - neither in PnP nor in ISA modes. ifconfig down/up makes the card 
work again in PnP mode. However, in ISA mode, ifconfig up ends with "No such 
device" error.

Comparing the 3c509 driver between 2.6.25 and 2.6.28, there's only some 
statistics-related change. So the cause of the problem must be somewhere 
else.

This patch makes the resume work in PnP mode, but it's still not enough for 
ISA mode.

--- linux-2.6.28-orig/drivers/net/3c509.c	2009-01-02 17:19:07.000000000 +0100
+++ linux/drivers/net/3c509.c	2009-02-05 22:36:20.000000000 +0100
@@ -1477,6 +1477,7 @@
 	spin_lock_irqsave(&lp->lock, flags);
 
 	outw(PowerUp, ioaddr + EL3_CMD);
+	EL3WINDOW(0);
 	el3_up(dev);
 
 	if (netif_running(dev))

-- 
Ondrej Zary

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: 3c509 resume from hibernation broken somewhere between 2.6.25 and 2.6.28
  2009-02-06 13:22 3c509 resume from hibernation broken somewhere between 2.6.25 and 2.6.28 Ondrej Zary
@ 2009-02-07  6:04 ` David Miller
  2009-02-08 20:08   ` Ondrej Zary
  0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2009-02-07  6:04 UTC (permalink / raw)
  To: linux; +Cc: netdev, linux-kernel

From: Ondrej Zary <linux@rainbow-software.org>
Date: Fri, 6 Feb 2009 14:22:13 +0100

> Hello,
> last year, I posted a patch which fixed hibernation on 3c509 cards. That was 
> back in 2.6.24. It worked fine in 2.6.25. But then I stopped using 
> hibernation (as it did not work with my new IT8212 RAID controller).
> 
> Now I fixed it and noticed that 3c509 does not wake up properly anymore (in 
> 2.6.28) - neither in PnP nor in ISA modes. ifconfig down/up makes the card 
> work again in PnP mode. However, in ISA mode, ifconfig up ends with "No such 
> device" error.
> 
> Comparing the 3c509 driver between 2.6.25 and 2.6.28, there's only some 
> statistics-related change. So the cause of the problem must be somewhere 
> else.
> 
> This patch makes the resume work in PnP mode, but it's still not enough for 
> ISA mode.

Patch applied, thank you.

Let us know if you figure out how to get the ISA case working
too :-)

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: 3c509 resume from hibernation broken somewhere between 2.6.25 and 2.6.28
  2009-02-07  6:04 ` David Miller
@ 2009-02-08 20:08   ` Ondrej Zary
  0 siblings, 0 replies; 3+ messages in thread
From: Ondrej Zary @ 2009-02-08 20:08 UTC (permalink / raw)
  To: David Miller; +Cc: netdev, linux-kernel

On Saturday 07 February 2009 07:04:26 David Miller wrote:
> From: Ondrej Zary <linux@rainbow-software.org>
> Date: Fri, 6 Feb 2009 14:22:13 +0100
>
> > Hello,
> > last year, I posted a patch which fixed hibernation on 3c509 cards. That
> > was back in 2.6.24. It worked fine in 2.6.25. But then I stopped using
> > hibernation (as it did not work with my new IT8212 RAID controller).
> >
> > Now I fixed it and noticed that 3c509 does not wake up properly anymore
> > (in 2.6.28) - neither in PnP nor in ISA modes. ifconfig down/up makes the
> > card work again in PnP mode. However, in ISA mode, ifconfig up ends with
> > "No such device" error.
> >
> > Comparing the 3c509 driver between 2.6.25 and 2.6.28, there's only some
> > statistics-related change. So the cause of the problem must be somewhere
> > else.
> >
> > This patch makes the resume work in PnP mode, but it's still not enough
> > for ISA mode.
>
> Patch applied, thank you.
>
> Let us know if you figure out how to get the ISA case working
> too :-)

Found that when the driver is compiled as module, ISA resume works fine. But 
when the driver is compiled in the kernel, it does not work. I'm unable to 
find why.

When resuming, el3_isa_resume() is called first. It then calls 
el3_isa_id_sequence(). This function fails when the driver is compiled-in 
because id_read_eeprom(7) returns 0xFFFF - as if the card is not present 
there. I'm out of ideas.

-- 
Ondrej Zary

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-02-08 20:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-02-06 13:22 3c509 resume from hibernation broken somewhere between 2.6.25 and 2.6.28 Ondrej Zary
2009-02-07  6:04 ` David Miller
2009-02-08 20:08   ` Ondrej Zary

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).