Re: [patch] e1000 TSO parameter

From: David Mosberger (davidm@napali.hpl.hp.com)
Date: Mon Jul 14 2003 - 23:57:05 EST


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



This archive was generated by hypermail 2b29 : Tue Jul 15 2003 - 22:00:55 EST