OpenFastPath / ofp

OpenFastPath project
BSD 3-Clause "New" or "Revised" License
349 stars 126 forks source link

Update to odp v1.35.0.0 level #278

Closed JereLeppanen closed 2 years ago

JereLeppanen commented 2 years ago

v2:

v3:

bogdanPricope commented 2 years ago

Cool. What about:

JereLeppanen commented 2 years ago

@bogdanPricope Possible improvements, but probably not in this PR. ODP and OFP init and term functions have always been separate, and calling the ODP init functions from OFP init functions would probably necessitate passing the parameters through, which would be a bit awkward.

bogdanPricope commented 2 years ago

@JereLeppanen I agree that those improvements are not for this PR. I did those and many others in 200+ commits on top OFP (including DHCP and now DNS support). For ODP/OFP init functions my approach was hybrid: caller either provides an ODP instance (configured with ODP init functions, etc.) or a NFP_ODP_INSTANCE_INVALID value (btw ODP_INSTANCE_INVALID is not defined) and ofp will create and manage the ODP instance with some default params.

Btw, what are your plans with OFP? As far I can tell, for whatever you are using it, I can give you a better version with a richer feature set and it will cost less then develop it yourself.

JannePeltonen commented 2 years ago

Now that ODP does not necessarily support scheduled queues as IPsec destination queues, ofp_ipsec_init_global() should probably set max_num_sa to zero if inbound_op_mode or outbound_op_mode is async and the queue_type_sched ipsec capability is not set in ODP.