On 5/3/2021 2:12 PM, Kevin Bowling wrote:
> Should we enable this by default?  My primary concern in the review is
> not making things like this require expert tuning.  If it is expected to
> be necessary, which Gallatin@ makes a case for, let’s toggle it on by
> default.
> 
> On Mon, May 3, 2021 at 10:56 AM Mark Johnston <ma...@freebsd.org
> <mailto:ma...@freebsd.org>> wrote:
> 
>     The branch main has been updated by markj:
> 
>     URL:
>     
> https://cgit.FreeBSD.org/src/commit/?id=64881da478071431a2d9e62613997a5772c56cdf
>     
> <https://cgit.FreeBSD.org/src/commit/?id=64881da478071431a2d9e62613997a5772c56cdf>
> 
>     commit 64881da478071431a2d9e62613997a5772c56cdf
>     Author:     Sai Rajesh Tallamraju <stall...@netapp.com
>     <mailto:stall...@netapp.com>>
>     AuthorDate: 2021-05-03 17:45:00 +0000
>     Commit:     Mark Johnston <ma...@freebsd.org>
>     CommitDate: 2021-05-03 17:47:14 +0000
> 

It looks like AIM was on by default in 11.x and earlier:
hw.ix.enable_aim: 1
hw.igb.enable_aim: 1

So I think enabling it does make sense in light of Drew's comments about
mbuf sorting only being useful in the face of something like AIM where
there will be a sufficient number of mbufs to sort.

-- 
Allan Jude
_______________________________________________
dev-commits-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-all
To unsubscribe, send any mail to "dev-commits-src-all-unsubscr...@freebsd.org"

Reply via email to