From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 57CC2685D for ; Mon, 20 Oct 2014 02:30:59 +0200 (CEST) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga103.fm.intel.com with ESMTP; 19 Oct 2014 17:28:55 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,862,1389772800"; d="scan'208";a="402677276" Received: from pgsmsx101.gar.corp.intel.com ([10.221.44.78]) by FMSMGA003.fm.intel.com with ESMTP; 19 Oct 2014 17:31:34 -0700 Received: from pgsmsx106.gar.corp.intel.com (10.221.44.98) by PGSMSX101.gar.corp.intel.com (10.221.44.78) with Microsoft SMTP Server (TLS) id 14.3.195.1; Mon, 20 Oct 2014 08:38:22 +0800 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by pgsmsx106.gar.corp.intel.com (10.221.44.98) with Microsoft SMTP Server (TLS) id 14.3.195.1; Mon, 20 Oct 2014 08:38:22 +0800 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.174]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.207]) with mapi id 14.03.0195.001; Mon, 20 Oct 2014 08:38:15 +0800 From: "Zhang, Helin" To: Marc Sune Thread-Topic: [dpdk-dev] [PATCH v3] KNI: use a memzone pool for KNI alloc/release Thread-Index: AQHP5sLCmPf2yW31Q0SId6PeoCB4E5wzeFQwgAAMrACABKkfgA== Date: Mon, 20 Oct 2014 00:38:14 +0000 Message-ID: References: <1413190190-10193-1-git-send-email-marc.sune@bisdn.de> <5440E094.3080008@bisdn.de> In-Reply-To: <5440E094.3080008@bisdn.de> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH v3] KNI: use a memzone pool for KNI alloc/release X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 20 Oct 2014 00:31:01 -0000 Hi Marc It's OK, as my comments is helpful confirmed by you. Thank you for the pati= ence! Regards, Helin > -----Original Message----- > From: Marc Sune [mailto:marc.sune@bisdn.de] > Sent: Friday, October 17, 2014 5:26 PM > To: Zhang, Helin > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3] KNI: use a memzone pool for KNI > alloc/release >=20 > Hi Helin, >=20 > You are right. The example application should properly call > rte_kni_init() with the right amount of #KNI ifaces. I overlooked it. I w= ill fix it > and circulate patch v4. >=20 > Apologize for the inconvenience. > marc >=20 > On 17/10/14 02:47, Zhang, Helin wrote: > > Hi Marc > > > > After I read part of KNI app code again, I still have comments for your= code > changes in KNI example applications. I guess you might need another versi= on of > patch. Sorry and Thanks! > > > >> -----Original Message----- > >> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Marc Sune > >> Sent: Monday, October 13, 2014 4:50 PM > >> To: dev@dpdk.org > >> Subject: [dpdk-dev] [PATCH v3] KNI: use a memzone pool for KNI > >> alloc/release > >> > >> This patch implements the KNI memzone pool in order to prevent > >> memzone exhaustion when allocating/deallocating KNI interfaces. > >> > >> It adds a new API call, rte_kni_init(max_kni_ifaces) that shall be > >> called before any call to rte_kni_alloc() if KNI is used. > >> > >> v2: Moved KNI fd opening to rte_kni_init(). Revised style. > >> v3: Adapted examples/kni to rte_kni_init(). > >> > >> Signed-off-by: Marc Sune > >> --- > >> examples/kni/main.c | 3 + > >> lib/librte_kni/rte_kni.c | 315 > >> +++++++++++++++++++++++++++++++++++++--------- > >> lib/librte_kni/rte_kni.h | 18 +++ > >> 3 files changed, 277 insertions(+), 59 deletions(-) > >> > >> diff --git a/examples/kni/main.c b/examples/kni/main.c index > >> cb17b43..f998b02 > >> 100644 > >> --- a/examples/kni/main.c > >> +++ b/examples/kni/main.c > >> @@ -872,6 +872,9 @@ main(int argc, char** argv) > >> rte_exit(EXIT_FAILURE, "Configured invalid " > >> "port ID %u\n", i); > >> > >> + /* Initialize KNI subsystem */ > >> + rte_kni_init(nb_sys_ports); > > Use 'nb_sys_ports' as the max number of KNI interfaces is not good enou= gh. > KNI example application supports multiple KNI interfaces per port, that w= ill use > more kernel threads for receiving packets from user space, then performan= ce > can be higher. > > I think the max number of KNI interfaces here should be "nb_sys_ports * > params[port_id]->nb_kni", note that 'params[port_id]->nb_kni' is calculat= ed in > kni_alloc(), you might need to put that calculation before calling rte_kn= i_init(). > > > >> + > >> /* Initialise each port */ > >> for (port =3D 0; port < nb_sys_ports; port++) { > >> /* Skip ports that are not enabled */ diff --git > >> a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c index > >> 76feef4..e339ef0 100644 > >> --- a/lib/librte_kni/rte_kni.c > >> +++ b/lib/librte_kni/rte_kni.c > >> @@ -40,6 +40,7 @@ > >> #include > >> #include > >> > >> +#include > >> #include > >> #include > >> #include > >> @@ -58,7 +59,7 @@ > >> > >> #define KNI_REQUEST_MBUF_NUM_MAX 32 > >> > >> -#define KNI_MZ_CHECK(mz) do { if (mz) goto fail; } while (0) > >> +#define KNI_MEM_CHECK(cond) do { if (cond) goto kni_fail; } while > >> +(0) > >> > >> /** > >> * KNI context > >> @@ -66,6 +67,7 @@ > >> struct rte_kni { > >> char name[RTE_KNI_NAMESIZE]; /**< KNI interface name > */ > >> uint16_t group_id; /**< Group ID of KNI devices > */ > >> + uint32_t slot_id; /**< KNI pool slot ID */ > >> struct rte_mempool *pktmbuf_pool; /**< pkt mbuf mempool */ > >> unsigned mbuf_size; /**< mbuf size */ > >> > >> @@ -88,10 +90,48 @@ enum kni_ops_status { > >> KNI_REQ_REGISTERED, > >> }; > >> > >> +/** > >> + * KNI memzone pool slot > >> + */ > >> +struct rte_kni_memzone_slot{ > >> + uint32_t id; > >> + uint8_t in_use : 1; /**< slot in use */ > >> + > >> + /* Memzones */ > >> + const struct rte_memzone *m_ctx; /**< KNI ctx */ > >> + const struct rte_memzone *m_tx_q; /**< TX queue */ > >> + const struct rte_memzone *m_rx_q; /**< RX queue */ > >> + const struct rte_memzone *m_alloc_q; /**< Allocated mbufs queue */ > >> + const struct rte_memzone *m_free_q; /**< To be freed mbufs queue > >> */ > >> + const struct rte_memzone *m_req_q; /**< Request queue */ > >> + const struct rte_memzone *m_resp_q; /**< Response queue */ > >> + const struct rte_memzone *m_sync_addr; > >> + > >> + /* Free linked list */ > >> + struct rte_kni_memzone_slot *next; /**< Next slot link.list */ > >> +}; > >> + > >> +/** > >> + * KNI memzone pool > >> + */ > >> +struct rte_kni_memzone_pool{ > >> + uint8_t initialized : 1; /**< Global KNI pool init flag *= / > >> + > >> + uint32_t max_ifaces; /**< Max. num of KNI ifaces */ > >> + struct rte_kni_memzone_slot *slots; /**< Pool slots */ > >> + rte_spinlock_t mutex; /**< alloc/relase mutex */ > >> + > >> + /* Free memzone slots linked-list */ > >> + struct rte_kni_memzone_slot *free; /**< First empty slot */ > >> + struct rte_kni_memzone_slot *free_tail; /**< Last empty slot */ > >> +}; > >> + > >> + > >> static void kni_free_mbufs(struct rte_kni *kni); static void > >> kni_allocate_mbufs(struct rte_kni *kni); > >> > >> static volatile int kni_fd =3D -1; > >> +static struct rte_kni_memzone_pool kni_memzone_pool =3D {0}; > >> > >> static const struct rte_memzone * > >> kni_memzone_reserve(const char *name, size_t len, int socket_id, @@ > >> -105,6 +145,161 @@ kni_memzone_reserve(const char *name, size_t len, > >> int socket_id, > >> return mz; > >> } > >> > >> +/* Pool mgmt */ > >> +static struct rte_kni_memzone_slot* > >> +kni_memzone_pool_alloc(void) > >> +{ > >> + struct rte_kni_memzone_slot* slot; > >> + > >> + rte_spinlock_lock(&kni_memzone_pool.mutex); > >> + > >> + if(!kni_memzone_pool.free) { > >> + rte_spinlock_unlock(&kni_memzone_pool.mutex); > >> + return NULL; > >> + } > >> + > >> + slot =3D kni_memzone_pool.free; > >> + kni_memzone_pool.free =3D slot->next; > >> + > >> + if(!kni_memzone_pool.free) > >> + kni_memzone_pool.free_tail =3D NULL; > >> + > >> + rte_spinlock_unlock(&kni_memzone_pool.mutex); > >> + > >> + return slot; > >> +} > >> + > >> +static void > >> +kni_memzone_pool_release(struct rte_kni_memzone_slot* slot) { > >> + rte_spinlock_lock(&kni_memzone_pool.mutex); > >> + > >> + if(kni_memzone_pool.free) > >> + kni_memzone_pool.free_tail->next =3D slot; > >> + else > >> + kni_memzone_pool.free =3D slot; > >> + > >> + kni_memzone_pool.free_tail =3D slot; > >> + slot->next =3D NULL; > >> + > >> + rte_spinlock_unlock(&kni_memzone_pool.mutex); > >> +} > >> + > >> + > >> +/* Shall be called before any allocation happens */ void > >> +rte_kni_init(unsigned int max_kni_ifaces) { > >> + uint32_t i; > >> + struct rte_kni_memzone_slot* it; > >> + const struct rte_memzone *mz; > >> +#define OBJNAMSIZ 32 > >> + char obj_name[OBJNAMSIZ]; > >> + char mz_name[RTE_MEMZONE_NAMESIZE]; > >> + > >> + if(max_kni_ifaces =3D=3D 0) { > >> + RTE_LOG(ERR, KNI, "Invalid number of max_kni_ifaces %d\n", > >> + max_kni_ifaces); > >> + rte_panic("Unable to initialize KNI\n"); > >> + } > >> + > >> + /* Check FD and open */ > >> + if (kni_fd < 0) { > >> + kni_fd =3D open("/dev/" KNI_DEVICE, O_RDWR); > >> + if (kni_fd < 0) { > >> + rte_panic("Can not open /dev/%s\n", KNI_DEVICE); > >> + } > >> + } > >> + > >> + /* Allocate slot objects */ > >> + kni_memzone_pool.slots =3D (struct > >> rte_kni_memzone_slot*)rte_malloc(NULL, > >> + sizeof(struct rte_kni_memzone_slot) * > >> + max_kni_ifaces, > >> + 0); > >> + KNI_MEM_CHECK(kni_memzone_pool.slots =3D=3D NULL); > >> + > >> + /* Initialize general pool variables */ > >> + kni_memzone_pool.initialized =3D 1; > >> + kni_memzone_pool.max_ifaces =3D max_kni_ifaces; > >> + kni_memzone_pool.free =3D &kni_memzone_pool.slots[0]; > >> + > >> + /* Pre-allocate all memzones of all the slots; panic on error */ > >> + for(i=3D0; i >> + > >> + /* Recover current slot */ > >> + it =3D &kni_memzone_pool.slots[i]; > >> + it->id =3D i; > >> + > >> + /* Allocate KNI context */ > >> + snprintf(mz_name, RTE_MEMZONE_NAMESIZE, "KNI_INFO_%d", i); > >> + mz =3D kni_memzone_reserve(mz_name, sizeof(struct rte_kni), > >> + SOCKET_ID_ANY, 0); > >> + KNI_MEM_CHECK(mz =3D=3D NULL); > >> + it->m_ctx =3D mz; > >> + > >> + /* TX RING */ > >> + snprintf(obj_name, OBJNAMSIZ, "kni_tx_%d", i); > >> + mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, > >> + SOCKET_ID_ANY, 0); > >> + KNI_MEM_CHECK(mz =3D=3D NULL); > >> + it->m_tx_q =3D mz; > >> + > >> + /* RX RING */ > >> + snprintf(obj_name, OBJNAMSIZ, "kni_rx_%d", i); > >> + mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, > >> + SOCKET_ID_ANY, 0); > >> + KNI_MEM_CHECK(mz =3D=3D NULL); > >> + it->m_rx_q =3D mz; > >> + > >> + /* ALLOC RING */ > >> + snprintf(obj_name, OBJNAMSIZ, "kni_alloc_%d", i); > >> + mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, > >> + SOCKET_ID_ANY, 0); > >> + KNI_MEM_CHECK(mz =3D=3D NULL); > >> + it->m_alloc_q =3D mz; > >> + > >> + /* FREE RING */ > >> + snprintf(obj_name, OBJNAMSIZ, "kni_free_%d", i); > >> + mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, > >> + SOCKET_ID_ANY, 0); > >> + KNI_MEM_CHECK(mz =3D=3D NULL); > >> + it->m_free_q =3D mz; > >> + > >> + /* Request RING */ > >> + snprintf(obj_name, OBJNAMSIZ, "kni_req_%d", i); > >> + mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, > >> + SOCKET_ID_ANY, 0); > >> + KNI_MEM_CHECK(mz =3D=3D NULL); > >> + it->m_req_q =3D mz; > >> + > >> + /* Response RING */ > >> + snprintf(obj_name, OBJNAMSIZ, "kni_resp_%d", i); > >> + mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, > >> + SOCKET_ID_ANY, 0); > >> + KNI_MEM_CHECK(mz =3D=3D NULL); > >> + it->m_resp_q =3D mz; > >> + > >> + /* Req/Resp sync mem area */ > >> + snprintf(obj_name, OBJNAMSIZ, "kni_sync_%d", i); > >> + mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, > >> + SOCKET_ID_ANY, 0); > >> + KNI_MEM_CHECK(mz =3D=3D NULL); > >> + it->m_sync_addr =3D mz; > >> + > >> + if(i+1 =3D=3D max_kni_ifaces) { > >> + it->next =3D NULL; > >> + kni_memzone_pool.free_tail =3D it; > >> + }else > >> + it->next =3D &kni_memzone_pool.slots[i+1]; > >> + } > >> + > >> + return; > >> + > >> +kni_fail: > >> + rte_panic("Unable to allocate memory for max_kni_ifaces:%d." > >> + "increase the amount of hugepages memory\n", max_kni_ifaces); } > >> + > >> /* It is deprecated and just for backward compatibility */ struct > >> rte_kni * rte_kni_create(uint8_t port_id, @@ -140,34 +335,37 @@ > >> rte_kni_alloc(struct rte_mempool *pktmbuf_pool, > >> struct rte_kni_device_info dev_info; > >> struct rte_kni *ctx; > >> char intf_name[RTE_KNI_NAMESIZE]; > >> -#define OBJNAMSIZ 32 > >> - char obj_name[OBJNAMSIZ]; > >> char mz_name[RTE_MEMZONE_NAMESIZE]; > >> const struct rte_memzone *mz; > >> + struct rte_kni_memzone_slot* slot=3DNULL; > >> > >> if (!pktmbuf_pool || !conf || !conf->name[0]) > >> return NULL; > >> > >> - /* Check FD and open once */ > >> - if (kni_fd < 0) { > >> - kni_fd =3D open("/dev/" KNI_DEVICE, O_RDWR); > >> - if (kni_fd < 0) { > >> - RTE_LOG(ERR, KNI, "Can not open /dev/%s\n", > >> - KNI_DEVICE); > >> - return NULL; > >> - } > >> + /* Check if KNI subsystem has been initialized */ > >> + if (kni_memzone_pool.initialized !=3D 1) { > >> + RTE_LOG(ERR, KNI, "KNI subsystem has not been initialized. " > >> + "Invoke rte_kni_init() first\n"); > >> + return NULL; > >> } > >> > >> + /* Get an available slot from the pool */ > >> + slot =3D kni_memzone_pool_alloc(); > >> + if(!slot) { > >> + RTE_LOG(ERR, KNI, "Cannot allocate more KNI interfaces; " > >> + "increase the number of max_kni_ifaces(current %d) or " > >> + "release unusued ones.\n", > >> + kni_memzone_pool.max_ifaces); > >> + return NULL; > >> + } > >> + > >> + /* Recover ctx */ > >> + ctx =3D slot->m_ctx->addr; > >> snprintf(intf_name, RTE_KNI_NAMESIZE, "%s", conf->name); > >> - snprintf(mz_name, RTE_MEMZONE_NAMESIZE, "KNI_INFO_%s", > >> intf_name); > >> - mz =3D kni_memzone_reserve(mz_name, sizeof(struct rte_kni), > >> - SOCKET_ID_ANY, 0); > >> - KNI_MZ_CHECK(mz =3D=3D NULL); > >> - ctx =3D mz->addr; > >> > >> if (ctx->in_use) { > >> RTE_LOG(ERR, KNI, "KNI %s is in use\n", ctx->name); > >> - goto fail; > >> + return NULL; > >> } > >> memset(ctx, 0, sizeof(struct rte_kni)); > >> if (ops) > >> @@ -190,83 +388,72 @@ rte_kni_alloc(struct rte_mempool > *pktmbuf_pool, > >> RTE_LOG(INFO, KNI, "pci: %02x:%02x:%02x \t %02x:%02x\n", > >> dev_info.bus, dev_info.devid, dev_info.function, > >> dev_info.vendor_id, dev_info.device_id); > >> - > >> /* TX RING */ > >> - snprintf(obj_name, OBJNAMSIZ, "kni_tx_%s", intf_name); > >> - mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, SOCKET_ID_ANY, > >> 0); > >> - KNI_MZ_CHECK(mz =3D=3D NULL); > >> + mz =3D slot->m_tx_q; > >> ctx->tx_q =3D mz->addr; > >> kni_fifo_init(ctx->tx_q, KNI_FIFO_COUNT_MAX); > >> dev_info.tx_phys =3D mz->phys_addr; > >> > >> /* RX RING */ > >> - snprintf(obj_name, OBJNAMSIZ, "kni_rx_%s", intf_name); > >> - mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, SOCKET_ID_ANY, > >> 0); > >> - KNI_MZ_CHECK(mz =3D=3D NULL); > >> + mz =3D slot->m_rx_q; > >> ctx->rx_q =3D mz->addr; > >> kni_fifo_init(ctx->rx_q, KNI_FIFO_COUNT_MAX); > >> dev_info.rx_phys =3D mz->phys_addr; > >> > >> /* ALLOC RING */ > >> - snprintf(obj_name, OBJNAMSIZ, "kni_alloc_%s", intf_name); > >> - mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, SOCKET_ID_ANY, > >> 0); > >> - KNI_MZ_CHECK(mz =3D=3D NULL); > >> + mz =3D slot->m_alloc_q; > >> ctx->alloc_q =3D mz->addr; > >> kni_fifo_init(ctx->alloc_q, KNI_FIFO_COUNT_MAX); > >> dev_info.alloc_phys =3D mz->phys_addr; > >> > >> /* FREE RING */ > >> - snprintf(obj_name, OBJNAMSIZ, "kni_free_%s", intf_name); > >> - mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, SOCKET_ID_ANY, > >> 0); > >> - KNI_MZ_CHECK(mz =3D=3D NULL); > >> + mz =3D slot->m_free_q; > >> ctx->free_q =3D mz->addr; > >> kni_fifo_init(ctx->free_q, KNI_FIFO_COUNT_MAX); > >> dev_info.free_phys =3D mz->phys_addr; > >> > >> /* Request RING */ > >> - snprintf(obj_name, OBJNAMSIZ, "kni_req_%s", intf_name); > >> - mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, SOCKET_ID_ANY, > >> 0); > >> - KNI_MZ_CHECK(mz =3D=3D NULL); > >> + mz =3D slot->m_req_q; > >> ctx->req_q =3D mz->addr; > >> kni_fifo_init(ctx->req_q, KNI_FIFO_COUNT_MAX); > >> dev_info.req_phys =3D mz->phys_addr; > >> > >> /* Response RING */ > >> - snprintf(obj_name, OBJNAMSIZ, "kni_resp_%s", intf_name); > >> - mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, SOCKET_ID_ANY, > >> 0); > >> - KNI_MZ_CHECK(mz =3D=3D NULL); > >> + mz =3D slot->m_resp_q; > >> ctx->resp_q =3D mz->addr; > >> kni_fifo_init(ctx->resp_q, KNI_FIFO_COUNT_MAX); > >> dev_info.resp_phys =3D mz->phys_addr; > >> > >> /* Req/Resp sync mem area */ > >> - snprintf(obj_name, OBJNAMSIZ, "kni_sync_%s", intf_name); > >> - mz =3D kni_memzone_reserve(obj_name, KNI_FIFO_SIZE, SOCKET_ID_ANY, > >> 0); > >> - KNI_MZ_CHECK(mz =3D=3D NULL); > >> + mz =3D slot->m_sync_addr; > >> ctx->sync_addr =3D mz->addr; > >> dev_info.sync_va =3D mz->addr; > >> dev_info.sync_phys =3D mz->phys_addr; > >> > >> + > >> /* MBUF mempool */ > >> snprintf(mz_name, sizeof(mz_name), RTE_MEMPOOL_OBJ_NAME, > >> pktmbuf_pool->name); > >> mz =3D rte_memzone_lookup(mz_name); > >> - KNI_MZ_CHECK(mz =3D=3D NULL); > >> + KNI_MEM_CHECK(mz =3D=3D NULL); > >> dev_info.mbuf_va =3D mz->addr; > >> dev_info.mbuf_phys =3D mz->phys_addr; > >> ctx->pktmbuf_pool =3D pktmbuf_pool; > >> ctx->group_id =3D conf->group_id; > >> + ctx->slot_id =3D slot->id; > >> ctx->mbuf_size =3D conf->mbuf_size; > >> > >> ret =3D ioctl(kni_fd, RTE_KNI_IOCTL_CREATE, &dev_info); > >> - KNI_MZ_CHECK(ret < 0); > >> + KNI_MEM_CHECK(ret < 0); > >> > >> ctx->in_use =3D 1; > >> > >> return ctx; > >> > >> -fail: > >> - > >> +kni_fail: > >> + if(slot) > >> + kni_memzone_pool_release(&kni_memzone_pool.slots[slot->id]); > >> + > >> return NULL; > >> } > >> > >> @@ -287,6 +474,7 @@ int > >> rte_kni_release(struct rte_kni *kni) > >> { > >> struct rte_kni_device_info dev_info; > >> + uint32_t slot_id; > >> > >> if (!kni || !kni->in_use) > >> return -1; > >> @@ -302,8 +490,19 @@ rte_kni_release(struct rte_kni *kni) > >> kni_free_fifo(kni->rx_q); > >> kni_free_fifo(kni->alloc_q); > >> kni_free_fifo(kni->free_q); > >> + > >> + slot_id =3D kni->slot_id; > >> + > >> + /* Memset the KNI struct */ > >> memset(kni, 0, sizeof(struct rte_kni)); > >> > >> + /* Release memzone */ > >> + if(slot_id > kni_memzone_pool.max_ifaces) { > >> + rte_panic("KNI pool: corrupted slot ID: %d, max: %d\n", > >> + slot_id, kni_memzone_pool.max_ifaces); > >> + } > >> + kni_memzone_pool_release(&kni_memzone_pool.slots[slot_id]); > >> + > >> return 0; > >> } > >> > >> @@ -437,23 +636,21 @@ rte_kni_get_port_id(struct rte_kni *kni) > >> struct rte_kni * rte_kni_get(const char *name) { > >> - struct rte_kni *kni; > >> - const struct rte_memzone *mz; > >> - char mz_name[RTE_MEMZONE_NAMESIZE]; > >> - > >> - if (!name || !name[0]) > >> - return NULL; > >> - > >> - snprintf(mz_name, RTE_MEMZONE_NAMESIZE, "KNI_INFO_%s", name); > >> - mz =3D rte_memzone_lookup(mz_name); > >> - if (!mz) > >> - return NULL; > >> - > >> - kni =3D mz->addr; > >> - if (!kni->in_use) > >> - return NULL; > >> + uint32_t i; > >> + struct rte_kni_memzone_slot* it; > >> + struct rte_kni* kni; > >> + > >> + /* Note: could be improved perf-wise if necessary */ > >> + for(i=3D0; i >> + it =3D &kni_memzone_pool.slots[i]; > >> + if(it->in_use =3D=3D 0) > >> + continue; > >> + kni =3D it->m_ctx->addr; > >> + if(strncmp(kni->name, name, RTE_KNI_NAMESIZE) =3D=3D 0) > >> + return kni; > >> + } > >> > >> - return kni; > >> + return NULL; > >> } > >> > >> /* > >> diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h > >> index 1a0b004..0159a1d 100644 > >> --- a/lib/librte_kni/rte_kni.h > >> +++ b/lib/librte_kni/rte_kni.h > >> @@ -90,11 +90,27 @@ struct rte_kni_conf { }; > >> > >> /** > >> + * Initialize and preallocate KNI subsystem > >> + * > >> + * This function is to be executed on the MASTER lcore only, after > >> +EAL > >> + * initialization and before any KNI interface is attempted to be > >> + * allocated > >> + * > >> + * @param max_kni_ifaces > >> + * The maximum number of KNI interfaces that can coexist > >> +concurrently */ extern void rte_kni_init(unsigned int > >> +max_kni_ifaces); > >> + > >> + > >> +/** > >> * Allocate KNI interface according to the port id, mbuf size, mbuf = pool, > >> * configurations and callbacks for kernel requests.The KNI interfac= e > created > >> * in the kernel space is the net interface the traditional Linux ap= plication > >> * talking to. > >> * > >> + * The rte_kni_alloc shall not be called before rte_kni_init() has > >> + been > >> + * called. rte_kni_alloc is thread safe. > >> + * > >> * @param pktmbuf_pool > >> * The mempool for allocting mbufs for packets. > >> * @param conf > >> @@ -138,6 +154,8 @@ extern struct rte_kni *rte_kni_create(uint8_t > port_id, > >> * Release KNI interface according to the context. It will also rele= ase the > >> * paired KNI interface in kernel space. All processing on the speci= fic KNI > >> * context need to be stopped before calling this interface. > >> + * > >> + * rte_kni_release is thread safe. > >> * > >> * @param kni > >> * The pointer to the context of an existent KNI interface. > >> -- > >> 1.7.10.4 > > Regards, > > Helin