Re: [patch] e1000 TSO parameter

[David Mosberger <davidm@xxxxxxxxxxxxxxxxx>]

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

  DaveM> On Mon, 14 Jul 2003 21:42:40 -0700 "Feldman, Scott"
  DaveM> <scott.feldman@xxxxxxxxx> 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