Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

intel10g: Enable TX prefetch for 14.8 Mpps #628

Merged
merged 2 commits into from
Oct 5, 2015

Conversation

lukego
Copy link
Member

@lukego lukego commented Sep 28, 2015

Enable the "TX descriptor prefetching" feature. This is required for the 82599 NIC to achieve line-rate transmission with 64-byte packets.

Resolves #622.

Enable the "TX descriptor prefetching" feature. This is required for
the 82599 NIC to achieve line-rate transmission with 64-byte packets.

Resolves snabbco#622.
@eugeneia
Copy link
Member

From the SnabbBot report:

OK: packetblaster-64 on c29f053 : 1.12294 of 9.468

Translation: 12% faster than master.

@kbara
Copy link
Contributor

kbara commented Sep 28, 2015

Nice one!

@lukego
Copy link
Member Author

lukego commented Sep 28, 2015

Progress at least...

The packetblaster benchmark should score 14.8 (Mpps) with 64-byte packets if all is well. That would be more like +55%. I suspect the issue is that packetblaster is sensitive to the other end of the connection and we need something there that is actually up and quickly receiving packets -- but not certain about that.

@lukego
Copy link
Member Author

lukego commented Sep 28, 2015

... or really should that be with 60 byte packets in the input file? (the other 4 bytes being the automatic/implicit ethernet CRC rather than part of the packetblaster config.)

lukego added a commit to lukego/snabb that referenced this pull request Oct 1, 2015
@eugeneia eugeneia merged commit c29f053 into snabbco:master Oct 5, 2015
dpino pushed a commit to dpino/snabb that referenced this pull request Dec 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants