sfxge(4): do not use RxQ index as label
ClosedPublic
Actions

Authored by arybchik on Apr 27 2016, 9:09 AM.

Details

Reviewers

philip
np
gnn
bz

Commits

rS298836: MFC r298735
rS298735: sfxge(4): do not use RxQ index as label

Summary

Labels are limitted by 32 on EF10. It is not sufficient on powerful hosts.
Since only one RxQ is running over each EvQ, zero label may be used.

Sponsored by: Solarflare Communications, Inc.
MFC after: 2 days
PR: 208267

Test Plan

Build tested, driver loaded/unloaded, interface up/down.
No problems with RxQ flush.
Traffic works fine.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

arybchik updated this revision to Diff 15648.Apr 27 2016, 9:09 AM

arybchik retitled this revision from to sfxge(4): do not use RxQ index as label.

arybchik updated this object.

arybchik edited the test plan for this revision. (Show Details)

arybchik added reviewers: gnn, philip, np, bz.

Herald added a subscriber: imp. · View Herald TranscriptApr 27 2016, 9:09 AM

gnn accepted this revision.Apr 27 2016, 10:48 PM

gnn edited edge metadata.

This revision is now accepted and ready to land.Apr 27 2016, 10:48 PM

Closed by commit rS298735: sfxge(4): do not use RxQ index as label (authored by arybchik). · Explain WhyApr 28 2016, 6:21 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

sfxge/

sfxge_ev.c

57 lines

sfxge_rx.c

2 lines

Diff 15685

View Options

head/sys/dev/sfxge/sfxge_ev.c