svn commit: r347418 - head/sys/net

[Andrew Gallatin]

On 2020-05-19 04:21, Kristof Provost wrote:

> The if_bnxt driver initialises |.isc_nrxd_max = {INT32_MAX, INT32_MAX, 
> INT32_MAX},|, so presumably that’s the cause.
> I don’t know what a sane value would be though. I’ve defaulted to 4096 
> (because that’s what some other iflib users seems to do) for now, and 
> that seems to work. It doesn’t panic and I can get traffic through it at 
> least:

You seem to be setting the max, not the default, and 4K max descriptors 
on a 100g device is going to basically cripple it.

How about setting to the next power of 2 below max int so as to keep 
with the authors intent?

If we don't already have a macro, something like  (INT32_MAX >> 1) + 1

Drew