Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 21 Jun 2011 09:34:06 -0700
From:      Jack Vogel <jfvogel@gmail.com>
To:        Guy Helmer <guy.helmer@palisadesystems.com>
Cc:        Jack F Vogel <jfv@freebsd.org>, svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, John Baldwin <jhb@freebsd.org>
Subject:   Re: svn commit: r223350 - head/sys/dev/e1000
Message-ID:  <BANLkTi=8uBXJECU7_15wdAZBHzhvAwOTFg@mail.gmail.com>
In-Reply-To: <5B6CCF15-05B9-418A-8B84-6A2FF441A39C@palisadesystems.com>
References:  <201106202259.p5KMxT1h069297@svn.freebsd.org> <201106210907.18414.jhb@freebsd.org> <BANLkTim-_hqXBC=3oC2dwi7YxC7aSJ_BPg@mail.gmail.com> <5B6CCF15-05B9-418A-8B84-6A2FF441A39C@palisadesystems.com>

next in thread | previous in thread | raw e-mail | index | archive | help
There IS a global default for flow control, its just hardcoded.. :)

Flow control was something my validation group specifically ragged on me
about... and was the reason I changed to a per-adapter setting.

Let me think about it.

Jack


On Tue, Jun 21, 2011 at 9:17 AM, Guy Helmer
<guy.helmer@palisadesystems.com>wrote:

> On Jun 21, 2011, at 11:07 AM, Jack Vogel wrote:
>
> LOL, the old adage is true, you just can't please all the people... :)
>
> The way the code now is it assigns a default, but you could still have
> a loader entry that would change it for all adapters if you wanted to,
> but ok, if you prefer the older for this. What other globals do you
> think should be retained?
>
>
> I'd like to be able to set the global default for flow control.  I'd also
> appreciate a global default for disabling TCP checksum offload, but that's
> above and beyond what we had before :-)
>
> Jack
>
>
> On Tue, Jun 21, 2011 at 6:07 AM, John Baldwin <jhb@freebsd.org> wrote:
>
>> On Monday, June 20, 2011 6:59:29 pm Jack F Vogel wrote:
>> > Author: jfv
>> > Date: Mon Jun 20 22:59:29 2011
>> > New Revision: 223350
>> > URL: http://svn.freebsd.org/changeset/base/223350
>> >
>> > Log:
>> >   Eliminate some global tuneables in favor of adapter-specific,
>> >   particular flow control and dma coalesce. Also improve the
>> >   sysctl operation on those too.
>> >
>> >   Add IPv6 detection in the ioctl code, this was done for
>> >   ixgbe first, carrying that over.
>> >
>> >   Add resource ability to disable particular adapter.
>> >
>> >   Add HW TSO capability so vlans can make use of TSO
>>
>> The tunables are useful for setting defaults for all interfaces. :(
>>
>> I use hw.igb.rx_processing_limit=-1 in loader.conf at work so that we can
>> ensure that all igb interfaces in a given system have that setting.   This
>> is
>> more scalable than having to set the right number of entries in
>> /etc/sysctl.conf.local on different machines, etc, without spamming the
>> console during boot with warnings about tweaking non-existing sysctls,
>> etc.
>>
>> Please consider keeping the tunables where the tunables are used to set
>> default settings for all adapters from the loader but per-device sysctls
>> are
>> used post-boot to provide runtime, per-device settings.
>>
>> --
>> John Baldwin
>>
>
>
>
>
>
> ------------------------------
> *This message has been scanned by ComplianceSafe, powered by Palisade's
> PacketSure.*
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?BANLkTi=8uBXJECU7_15wdAZBHzhvAwOTFg>