Page MenuHomeFreeBSD

vmxnet3: add native netmap support
ClosedPublic

Authored by g.lettieri_iet.unipi.it on Thu, Feb 7, 1:32 PM.

Details

Summary

Sunnyvalley, Inc., is sponsoring native netmap support for the vmxnet3 adapter. This comes for free in 12, since the driver has been ported to iflib. This patch adds native support for 11 in the legacy way. To make it minimally intrusive, the native support is only enabled if vmxnet3.netmap_native is set at boot.

Tested on stable-11 running inside vmplayer.

Diff Detail

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

v.maffione_gmail.com added a comment.EditedThu, Feb 7, 2:06 PM

It would probably help to have more lines of contexts for the patch.
If you used git, you should probably use something like:

git diff -U30 [...]

If you used svn

svn diff -x '-U30'

This is the same patch as before, but with more context added.

sys/dev/vmware/vmxnet3/vmx_netmap.h
48 ↗(On Diff #53654)

nm_prdis

249 ↗(On Diff #53654)

nm_prlim()

335 ↗(On Diff #53654)

this can be removed, because netmap_attach_ext already logs "netmap queues/slots" using if_printf.

replaced old debug macros and removed a useless log message.

v.maffione_gmail.com added inline comments.
sys/dev/vmware/vmxnet3/if_vmx.c
2137 ↗(On Diff #53659)

Can you please break this long line? Second level indents are four spaces, see style(9)

This revision is now accepted and ready to land.Mon, Feb 18, 2:11 PM
This revision now requires review to proceed.Mon, Feb 18, 4:14 PM
This revision is now accepted and ready to land.Mon, Feb 18, 4:14 PM
bryanv accepted this revision.Mon, Feb 18, 6:52 PM
Closed by commit rS344272: vmx(4): add native netmap support (authored by vmaffione, committed by ). · Explain WhyTue, Feb 19, 10:08 AM
This revision was automatically updated to reflect the committed changes.