*> p.s. Lately someone involved with DPDK said KNI would be deprecated in future DPDK releases; I haven't read or listen to this before, is this true? What would be the natural replacement then?*
KNI is a non-trivial part of the product I'm in the process of building. I'd appreciate someone "in the know" addressing this one please. Are there specific roadmap plans relative to KNI that we need to be aware of? Regards, Jay On Tue, Sep 23, 2014 at 4:27 AM, Marc Sune <marc.sune at bisdn.de> wrote: > Hi all, > > So we are having some problems with KNI. In short, we have a DPDK > application that creates KNI interfaces and destroys them during its > lifecycle and connecting them to DOCKER containers. Interfaces may > eventually be even named the same (see below). > > We were wondering why even calling rte_kni_release() the hugepages memory > was rapidly being exhausted, and we also realised even after destruction, > you cannot use the same name for the interface. > > After close inspection of the rte_kni lib we think the core issue and is > mostly a design issue. rte_kni_alloc ends up calling kni_memzone_reserve() > that calls at the end rte_memzone_reserve() which cannot be unreserved by > rte_kni_relese() (by design of memzones). The exhaustion is rapid due to > the number of FIFOs created (6). > > If this would be right, we would propose and try to provide a patch as > follows: > > * Create a new rte_kni_init(unsigned int max_knis); > > This would preallocate all the FIFO rings(TX, RX, ALLOC, FREE, Request > and Response)*max_knis by calling kni_memzone_reserve(), and store them in > a kni_fifo_pool. This should only be called once by DPDK applications at > bootstrapping time. > > * rte_kni_allocate would just use one of the kni_fifo_pool (one => meaning > a a set of 6 FIFOs making a single slot) > * rte_kni_release would return to the pool. > > This should solve both issues. We would base the patch on 1.7.2. > > Thoughts? > marc > > p.s. Lately someone involved with DPDK said KNI would be deprecated in > future DPDK releases; I haven't read or listen to this before, is this > true? What would be the natural replacement then? >