...
- Still looking for Nov 15 for dev complete.
- Release 5.0.2 is delayed. Ed Ting requested the commit id's for 5.0.2
- Discussion of Netronome's patch.
- Henry reviewed a diagram that tries to summarizes the proposal by Netronome. See the google document at https://docs.google.com/presentation/d/1o4FC2uPphAgR2mu8YNWEGWe6Mz4yDYyjgvLqkb5c4eM/edit#slide=id.p8 , and the current snapshot of that document on the Wiki at https://wiki.tungsten.io/download/attachments/1409118/Offload%20Design%20Options%20-%20Oct%2030%202018%20ver.pptx?api=v2
- There were still questions about the tradeoffs between supporting vendor-specific designs in the short term vs. retaining motivation for working vendor-independent solutions in sister industry groups (e.g., on the Linux TC Module and RTE flow).
- Joseph raised questions about testing.
- Sukhdev asked for a list of the specific patches and the blueprint. Can Netronome provide these please? (see response below)
- Sukhdev took the AI to have the key experts review the code before the next meeting and provide feedback.
- Paul raised the point that upstreaming the patches at this time (i.e., in time for Rel 5.1) raises some concerns, and asked whether there is a feasible option for this to initially be implemented by applying a patch to the upstreamed version. And hence providing additional time for review.
- Paul asked for vendors other than Netronome to review the code and identify areas where there might be alternate preferred solutions.
- Netronome code that needs review: https://review.opencontrail.org/#/c/47035/. Patch set 1 is the original work by Mellanox, patchsets 2 and 3 are the delta added by Netronome. We would like to get other NIC vendors to review this. This patch is also relevant: https://review.opencontrail.org/#/c/42850/. Blueprint: https://blueprints.launchpad.net/opencontrail/+spec/pluggable-offloads