Re: PATCH: VLAN support for 3c59x/3c90x

From: Jeff Garzik (jgarzik_at_pobox.com)
Date: 07/31/04

  • Next message: Zwane Mwaikambo: "Re: 2.6.8-rc2-mm1" 
    Date:	Sat, 31 Jul 2004 12:26:05 -0400
To: Willy Tarreau <willy@w.ods.org>

    Willy Tarreau wrote:
    > On Sat, Jul 31, 2004 at 12:05:03PM -0400, Jeff Garzik wrote:
    >
    >>>Ok, sorry, I've just checked, they are 6. But I incidentely used the
    >>>feature
    >>>on 2 of them (dl2k and starfire). But more drivers still have the
    >>>'static int mtu=1500' preceeded by a comment stating "allow the user to
    >>>change
    >>>the mtu". Why is it not a #define then, if nobody can change it anymore ?
    >>
    >>People can change it all the time with ifconfig.
    >
    >
    > no, they change dev->mtu, not the 'static' mtu. I think it has been

    Correct. The user _should not_ be allowed to change the driver limit,
    because it is precisely that a driver (and hardware) limit.

    > inherited from the old days without MODULE_PARM.

    Using MODULE_PARM to select max MTU limit was always the wrong thing to
    do. The user generally has no idea of the hardware limits involve.

    Patches to remove these incorrect module parameters are welcome.

            Jeff

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  • Next message: Zwane Mwaikambo: "Re: 2.6.8-rc2-mm1"