> > > Another question - if we have reassembly_conf_set() would it make sense to > > > have also reassembly_conf_get? > > > So user can retrieve current ip_reassembly config values? > > > > > The set/supported values can be retrieved using rte_eth_dev_info :: > reass_capa > > Hmm, I thought rte_eth_dev_info :: reass_capa reports > max supported values, not currently set values. > Did I misunderstand something? > Reassembly configuration is expected to be a one-time setting and is not expected to change multiple times in the application. You are correct that rte_eth_dev_info :: reass_capa reports max supported values by the PMD. But if somebody uses the _set API, dev_info values will be overwritten. However, a get API can be added, if we have some use case. IMO, we can add it later if it will be required.
- [PATCH 0/8] ethdev: introduce IP reassembly offload Akhil Goyal
- [PATCH 1/8] ethdev: introduce IP reassembly offlo... Akhil Goyal
- RE: [PATCH 1/8] ethdev: introduce IP reassemb... Ananyev, Konstantin
- [PATCH 2/8] ethdev: add dev op for IP reassembly ... Akhil Goyal
- RE: [PATCH 2/8] ethdev: add dev op for IP rea... Ananyev, Konstantin
- RE: [PATCH 2/8] ethdev: add dev op for IP... Akhil Goyal
- RE: [PATCH 2/8] ethdev: add dev op fo... Ananyev, Konstantin
- RE: [PATCH 2/8] ethdev: add dev ... Akhil Goyal
- RE: [PATCH 2/8] ethdev: add ... Ananyev, Konstantin
- RE: [PATCH 2/8] ethdev: ... Akhil Goyal
- RE: [PATCH 2/8] ethdev: ... Akhil Goyal
- RE: [PATCH 2/8] ethdev: ... Ananyev, Konstantin
- [PATCH 3/8] ethdev: add mbuf dynfield for incompl... Akhil Goyal
- RE: [PATCH 3/8] ethdev: add mbuf dynfield for... Ananyev, Konstantin
- RE: [PATCH 3/8] ethdev: add mbuf dynfield... Akhil Goyal
- RE: [PATCH 3/8] ethdev: add mbuf dynf... Ananyev, Konstantin
- RE: [PATCH 3/8] ethdev: add mbuf... Akhil Goyal
- RE: [PATCH 3/8] ethdev: add ... Ananyev, Konstantin