|From:||Chris Leech <chris.leech-AT-gmail.com>|
|To:||Jeff Garzik <jgarzik-AT-pobox.com>, Li Shaohua <shaohua.li-AT-intel.com>, Andrew Morton <akpm-AT-osdl.org>, lkml <linux-kernel-AT-vger.kernel.org>, "David S. Miller" <davem-AT-redhat.com>|
|Subject:||Re: hotplug e1000 failed after 32 times|
|Date:||Sun, 19 Sep 2004 21:15:43 -0700|
On Sun, 19 Sep 2004 22:55:51 -0400, Jeff Garzik <email@example.com> wrote: > Li Shaohua wrote: > > I'm not familiar with the NIC driver, but this problem really is > > annoying. The gurus, please consider a solution. It's not an uncommon > > case. I believe it's common in a big system with hotplug support. I can > > understand why the driver doesn't support more than 32 a card, but one > > card with 32 times hotplug failed is a little ugly. > > There should be no problem at all with the driver supporting 32 NICs... > in fact if it cannot support at least 99 NICs, I would consider that a > bug. I'd like to hear more about what the failure condition actually is, see any system messages logged. The 32 NIC limitation is for configuration via the module parameters, but e1000 itself doesn't impose any other limitations on the number of NICs. I know of testing situations where 360 e1000 ports were in use on a large system. I've run hotplug tests, powering slots on and off while passing traffic in a fail-over teaming configuration, for weeks without problem. I'll run a hotplug test myself as soon as I get a chance, but I don't see any reason why it should be failing. As for the module parameter limitations, yes only 32 ports can be configured that way and the counter doesn't roll back with hot-removes. I think we can live with those limitations. Indexed parameters make no sense in a hotplug environment, use ethtool. The module parameters in e1000 work for a large number or users who can live with those limitations. That's why they're still in the driver. As long as they work, I think they should stay. But I'm not in favor of adding more code, or making the existing code more complex, to try and remove those limitations. That's why everything can be configured through ethtool. - Chris - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to firstname.lastname@example.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Copyright © 2004, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds