Re: [patch] e1000 TSO parameter

David Mosberger (davidm@napali.hpl.hp.com)
Mon, 14 Jul 2003 21:57:05 -0700


>>>>> On Mon, 14 Jul 2003 21:45:10 -0700, "David S. Miller" <davem@redhat.com> said:

DaveM> On Mon, 14 Jul 2003 21:42:40 -0700 "Feldman, Scott"
DaveM> <scott.feldman@intel.com> wrote:

>> > Note that I had to move the e1000_check_options() call to a >
>> slighly earlier place. You may want to double-check that > it's
>> really OK.

>> I'm not too keen on adding another module parameter. Maybe a
>> CONFIG_E1000_TSO option?

DaveM> Extend ethtool please :-)

ethtool would be ideal, agreed.

I absolutely think that this should be a runtime option, not a
compile-time option.

--david
-
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/