>From: [email protected] [mailto:[email protected]] >On Behalf Of [email protected] >Sent: Wednesday, October 11, 2017 5:01 PM >To: [email protected] >Subject: [ovs-dev] [PATCH v5 3/6] netdev-dpdk: skip init for existing >mempools. > >Skip initialization of mempool packet areas if this was already >done in a previous call to dpdk_mp_create. >
Hi Antonio, As per my previous comment, this should be squashed with the first patch in the series. Apart from that, LGTM. -Mark >CC: Darrell Ball <[email protected]> >CC: Ciara Loftus <[email protected]> >CC: Kevin Traynor <[email protected]> >CC: Aaron Conole <[email protected]> >Signed-off-by: Antonio Fischetti <[email protected]> >--- > lib/netdev-dpdk.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > >diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c >index e664931..2aa4a55 100644 >--- a/lib/netdev-dpdk.c >+++ b/lib/netdev-dpdk.c >@@ -550,6 +550,11 @@ dpdk_mp_create(struct netdev_dpdk *dev, int mtu, bool >*mp_exists) > if (dmp->mp) { > VLOG_DBG("Allocated \"%s\" mempool with %u mbufs", mp_name, > dmp->mp_size); >+ /* rte_pktmbuf_pool_create has done some initialization of the >+ * rte_mbuf part of each dp_packet. Some OvS specific fields >+ * of the packet still need to be initialized by >+ * ovs_rte_pktmbuf_init. */ >+ rte_mempool_obj_iter(dmp->mp, ovs_rte_pktmbuf_init, NULL); > } else if (rte_errno == EEXIST) { > /* A mempool with the same name already exists. We just > * retrieve its pointer to be returned to the caller. */ >@@ -566,11 +571,6 @@ dpdk_mp_create(struct netdev_dpdk *dev, int mtu, bool >*mp_exists) > } > free(mp_name); > if (dmp->mp) { >- /* rte_pktmbuf_pool_create has done some initialization of the >- * rte_mbuf part of each dp_packet, while ovs_rte_pktmbuf_init >- * initializes some OVS specific fields of dp_packet. >- */ >- rte_mempool_obj_iter(dmp->mp, ovs_rte_pktmbuf_init, NULL); > return dmp; > } > } while (!(*mp_exists) && >-- >2.4.11 > >_______________________________________________ >dev mailing list >[email protected] >https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
