Re: Linux 2.6.9-ac16

From: Francois Romieu
Date: Fri Dec 17 2004 - 13:59:27 EST

Arjan van de Ven <arjan@xxxxxxxxxxxxx> :
> On Thu, 2004-12-16 at 18:43 +0000, Alan Cox wrote:
> > o Acenic must use __devinitdata for hotplug (Alan Cox)
> > | based on an RH patch
> MODULE_PARM_DESC(tx_ratio, "AceNIC/3C985/GA620 ratio of NIC memory used
> for TX/RX descriptors (range 0-63)");
> -static char version[] __initdata =
> +static char version[] __devinitdata =
> "acenic.c: v0.92 08/05/2002 Jes Sorensen, linux-acenic@xxxxxxxxxx\n"
> "\n";;
> you broke this one... :-)
> the version var *cannot* be initdata of any kind, since the ethtool
> ioctl uses the variable. End Of Story(tm)

Episode 2

Actually ethtool_drvinfo.get_drvinfo() does not use _this_ version var
in the acenic driver. Btw it would be waaaaay too long for the 32 bytes
allowed in the misc fields of struct ethtool_drvinfo) (I messed the
r8169 driver this way).

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