Re: [PATCH] 3c59x and resume

Jeff Garzik (
Mon, 25 Mar 2002 19:57:19 -0500

christophe barbé wrote:

>Here is a small patch tested with 2.4.18 and 2.4.19-pre4.
>It was proposed by Andrew but not integrated in pre4.
>The problem is when using the vortex driver and suspend/resuming the
>machine. Without this patch the card id is each time greater. To resume
>correctly this driver need the option enable_wol=1 but as-is it will
>only be true for the first ID. You can enable it for the first 8 IDs
>with enable_wol=1,1,1,1,1,1,1,1 but you can't do it for all IDs.
>Said another way without this patch you can't suspend/resume more than
>eight times your machine.
>This is a fix for the most common use. The proper fix would be IMO to
>keep a bitmap of used IDs but I don't know if it worsts it.
>Also a fix would be to separate the suspend/resume functionality from
>the wol functionality (wake up on lan).
>--- linux/drivers/net/3c59x.c Sat Mar 23 10:24:56 2002
>+++ linux/drivers/net/3c59x.c Sat Mar 23 10:57:00 2002
>@@ -2891,6 +2891,9 @@
> vp = dev->priv;
>+ if (vp->card_idx == vortex_cards_found - 1)
>+ vortex_cards_found--;
> /* AKPM: FIXME: we should have
> * if (vp->cb_fn_base) iounmap(vp->cb_fn_base);
> * here

This patch causes module defaults to be reused -- potentially incorrectly.

This is a personal solution, that might live on temporary as an
outside-the-tree patch... but we cannot apply this to the stable kernel.

I agree the card idx is wrong on remove. Insert and remove a 3c59x
cardbus card several times, and you will lose your module options too.
However... take note that this problem cannot be solved "the easy way"
-- because one solution people may desire will potentially result in
module options getting re-used incorrectly. The above is one such solution.

If you want WOL options to "stick" or vary per-interface, we already
have an API for that -- ethtool. Check out drivers/net/natsemi.c for an
example implementation. _Tested_ patches to 3c59x that add WOL ethtool
support are welcome, pending Andrew's approval. Do not remove
enable_wol for now in a stable series, but we will deprecate its use
once ethtool support appears.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at