Allow IPoIB to work with multiple FIBs
ClosedPublic
Actions

Authored by rpokala on Oct 10 2020, 9:50 PM.

Details

Reviewers

• hselasky

Commits

rS367047: MFC r366686:
rS367046: MFC r366686:
rS366686: Allow IP over IB to work with multiple FIBs.

Summary

Call M_SETFIB() to make sure the IPoIB packet is directed to the correct
interface-specific FIB.

Test Plan

Tested at Panasas on FreeBSD 12.1; general-purpose routing using the default
FIB, and a separate FIB for routing between IPoIB on ib0 and IPoEth on mce0.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 34091
Build 31264: arc lint + arc unit

Event Timeline

rpokala created this revision.Oct 10 2020, 9:50 PM

Herald added a subscriber: imp. · View Herald TranscriptOct 10 2020, 9:50 PM

rpokala requested review of this revision.Oct 10 2020, 9:50 PM

Harbormaster completed remote builds in B34091: Diff 78091.Oct 10 2020, 9:50 PM

rpokala retitled this revision from Allow IPoIB to work with mutiple FIBs to Allow IPoIB to work with multiple FIBs.Oct 10 2020, 9:56 PM

This piece of code is about to change with D26254 . I've added the patch there instead.

In D26733#595994, @hselasky wrote:

This piece of code is about to change with D26254 . I've added the patch there instead.

Fair enough. Any objection to me submitting this, in the current location, to stable/12 (tested) and stable/11 (not tested, but ipoib_demux() is substantially the same, so it seems right)?

If you MFC it too it is fine. I'll rebase my patch.

This revision is now accepted and ready to land.Oct 13 2020, 7:24 AM

rpokala closed this revision.Oct 13 2020, 8:46 PM

rpokala added a commit: rS366686: Allow IP over IB to work with multiple FIBs..

rpokala added a commit: rS367046: MFC r366686:.Oct 25 2020, 11:26 PM

rpokala added a commit: rS367047: MFC r366686:.