Hi Yipeng, On 09/04/2018 02:55 PM, Wang, Yipeng1 wrote:
Do we need both of the state and istate struct? struct rte_hash_iterator_state seems not doing much. How about we only have one "state" struct and just not expose the internals to the public API, similar to the rte_hash struct or rte_member_setsum struct. And in _init function use rte_malloc to allocate the state and add a _free function to free it.
The purpose of have struct state is to enable applications to allocate iterator states on their execution stack or embedding iterator states in larger structs to avoid an extra malloc()/free().
Do you foresee that the upcoming new underlying algorithm of hash tables will need to dynamically allocate iterator states?
[ ]'s Michel Machado
