ixgbe(4): Update HEAD (p2) to 3.2.12-k

[cramerj_intel.com]

Summary

iflib-enabled ixgbe driver update, including:

• Initial iflib conversion provided by mmacy & sbruno (D5213)
• Support for X550EM devices.
• Support for Bypass adapters.
• Flow Director code moved to separate files
• SR-IOV code moved to separate files

Test Plan

Compile-tested with minimal touch-testing for the PF driver (none for VF). Respectfully request waiting for Jeff's team to perform a validation pass before committing.

[cramerj_intel.com]

• Moved code around to match if_ix.c and/or the OOT driver
• Wrapped KTR code with ifdef for those using GENERIC-NODEBUG

[cramerj_intel.com]

• Fixed reporting of 2.5G speeds

[cramerj_intel.com]

• Updated interrupt setup/servicing/detection to match OOT. This fixes the issue of the interface not coming back up after unplugging/re-plugging in the module/cable.

[smh]

Wow that's a lot of changes.

Done an initial pass, mostly style nits but would be nice to get them fixed.

sys/dev/ixgbe/if_bypass.c

48

Might be nice to extract the while loop to a little helper so you can do:

atomic_cmpset_int_ensure(&adapter->bypass.low, 0, 1, 3000);

128

Bracing issue, I would expect this to be:

if ((error = hw->mac.ops.bypass_rw(hw, cmd, &version)) != 0)

132

Bracing issue

173

excessive bracing

186

Bracing

194

Why the delay? If ixgbe_bypass_mutex_clear needs time to settle then would it not be better for it to do that instead of the caller?

234

bracing

253

As above

272

Should this be earlier?

276

bracing

295

As above

318

bracing

337

as above

360

bracing

379

as above

401

bracing

421

as above

456

bracing

514

bracing

564

bracing

618

IXGBE_ERR_INVALID_ARGUMENT is used above should they be the same?

621

0 as the value is not bool

sys/dev/ixgbe/if_fdir.c

59

0

sys/dev/ixgbe/if_ixv.c

290

== NULL

304

Long line

343

== NULL

350

formatting

392

I thought the kernel free checked this already? Same above

394

Move above free.

430

does this return a bool or an int if int != 0 check.

557

Given end only does return (error); just do that here and then return (0); at the end as that makes it clearer whats happening

568

brace return

600

return (EINVAL); and then return (0); at the end eliminating the else and error var makes the flow clearer

855

Indentation

891

Change to an if and return (0) then followed by the default case to make flow clearer.

1188

no need for return on void func

1214

brace

1314

what is the 3, need a constant?

1321

!= 0

1401

!= 0

1402

Long line.

1408

!= 0 or == 1?

1409

Long line

1638

As this always return 0 should it be a void?

1857

return not needed

sys/dev/ixgbe/if_sriov.c

281

break not needed

451

== 0

595

== 0

750

== 0

846

always returns 0, should it be a void

sys/dev/ixgbe/ix_txrx.c

103

!= 0

289

!= 0

348

Commented out code

357

!= 0

405

!= 0

sys/dev/ixgbe/ixgbe_api.c

1706

!= NULL

sys/dev/ixgbe/ixgbe_common.c

424

bracing

433

!= 0

446

bracing

1194

bracing more below.

sys/dev/ixgbe/ixgbe_x550.c

2809

Return early to avoid all the indentation?

[cramerj_intel.com]

• Fixed VF driver.
• Another attempt at fixing 2.5G reporting.

[cramerj_intel.com]

• Building the drivers fails because of the malloc-type change I performed in a previous commit. Reverting that change.

[cramerj_intel.com]

• Fixed Report Status accounting not being reset when interface down/up'd.

[cramerj_intel.com]

• Applied same ring accounting bug fix to ixv as was committed to the PF driver in previous commit.
• Modified ixv to indicate that we don't support full frame sizes when VLANs are enabled if the hardware is 82599 and jumbo frames is disabled.

[cramerj_intel.com]

• Fixed some panics when using VF driver. The VF driver still doesn't work, however.

[cramerj_intel.com]

• Removed code that is already performed in the iflib layer. This should help Tx function in the VF driver.

[cramerj_intel.com]

• Fixed updating OS stats issue in VF driver.

[cramerj_intel.com]

Another review will be created shortly to fit on top of D11232.