Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 3 May 2021 17:09:42 -0400
From:      Mark Johnston <markj@freebsd.org>
To:        Allan Jude <allanjude@freebsd.org>
Cc:        Kevin Bowling <kevin.bowling@kev009.com>, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org, src-committers@freebsd.org
Subject:   Re: git: 64881da47807 - main - ixgbe: Restore AIM support
Message-ID:  <YJBmlpn33Np1Bs4n@nuc>
In-Reply-To: <295b05d0-58ba-6e04-6fbf-cb997c9ab112@freebsd.org>
References:  <202105031755.143Htj33046938@gitrepo.freebsd.org> <CAK7dMtCRTHW5f9qe-ZPaRZGaoUGNoQvLFKeMQcrDesevghEQ4g@mail.gmail.com> <295b05d0-58ba-6e04-6fbf-cb997c9ab112@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, May 03, 2021 at 03:30:55PM -0400, Allan Jude wrote:
> 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 <markj@freebsd.org
> > <mailto:markj@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 <stallamr@netapp.com
> >     <mailto:stallamr@netapp.com>>
> >     AuthorDate: 2021-05-03 17:45:00 +0000
> >     Commit:     Mark Johnston <markj@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.

See https://reviews.freebsd.org/D30094



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