Page MenuHomeFreeBSD

Intel 10G SRIOV support
ClosedPublic

Authored by jfv on May 8 2015, 5:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 8, 2:11 AM
Unknown Object (File)
Fri, Dec 6, 8:59 AM
Unknown Object (File)
Sun, Nov 24, 12:20 PM
Unknown Object (File)
Sun, Nov 24, 11:06 AM
Unknown Object (File)
Nov 16 2024, 6:39 AM
Unknown Object (File)
Nov 14 2024, 12:16 PM
Unknown Object (File)
Oct 2 2024, 5:57 AM
Unknown Object (File)
Oct 1 2024, 5:47 PM
Subscribers
None

Details

Summary

This revision adds support for PF (host-side) SRIOV support to the ixgbe driver. Thanks belongs to Ryan
Stone of Sandvine who did all the early implementation, and to Marc Snyder of Dell who did some further
polish and debugging.

Test Plan

Due to internal schedules Intel has not yet been able to do more than touch test this feature,
so please understand that this is a Technology Preview, not fully tested functionaliy. We will
be continuing to test and enhance this over this year, and any early adapters are welcome to
submit changes and fixes to the code.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

jfv retitled this revision from to Intel 10G SRIOV support.
jfv updated this object.
jfv edited the test plan for this revision. (Show Details)
jfv added reviewers: erj, rstone, jhb, gnn, emaste, karels.

Remove x550 device that is not yet available. Also remove unneeded shared code prototype,
and some added whitespace.

head/sys/dev/ixgbe/if_ix.c
4990

Hm. There's a good chance I am ultimately the person who commented this line out, but I have no idea why. :(

5337

this can go. :)

5337

that was supposed to be on the CTR3 call about

5467

bad indentation here?

gnn requested changes to this revision.May 11 2015, 12:56 AM
gnn edited edge metadata.

I believe there is definitely a bug in the !! case (my first comment).

Also, can you split this commit into two, one for the IOV add and another for the non IOV changes?

head/sys/dev/ixgbe/if_ix.c
3760

Don't you have too many ! there?

head/sys/dev/ixgbe/ix_txrx.c
624–625

Is there a carriage return issue here or is that because of Phabricator?

head/sys/dev/ixgbe/ixgbe.h
450

Remove extra cr/lf

This revision now requires changes to proceed.May 11 2015, 12:56 AM
head/sys/dev/ixgbe/if_ix.c
3760

It's supposed to be like that because this value gets exposed to the user in the eee.enable sysctl; we want to init the value here to either 1 or 0 instead a function pointer or 0.

jfv marked 2 inline comments as done.May 11 2015, 6:25 PM
jfv added inline comments.
head/sys/dev/ixgbe/if_ix.c
3760

Eric did this, its as intended.

4990

Well, are you saying it should be uncommented, or removed?

5337

What do you mean can go exactly?

head/sys/dev/ixgbe/ix_txrx.c
624–625

It was a long line, i divided it to keep it under 80, there's no extra line inbetween
if that's what you mean?

jfv edited edge metadata.

Correct a couple minor points in the last revision.

gnn edited edge metadata.
This revision is now accepted and ready to land.May 12 2015, 1:20 PM