net/liquidio: fix uninitialized variable
[dpdk.git] / drivers / mempool / dpaa2 / dpaa2_hw_mempool.c
index 973b9d4..6df203f 100644 (file)
@@ -47,7 +47,6 @@
 #include <rte_cycles.h>
 #include <rte_kvargs.h>
 #include <rte_dev.h>
-#include <rte_ethdev.h>
 
 #include <fslmc_logs.h>
 #include <mc/fsl_dpbp.h>
@@ -63,6 +62,7 @@ rte_hw_mbuf_create_pool(struct rte_mempool *mp)
 {
        struct dpaa2_bp_list *bp_list;
        struct dpaa2_dpbp_dev *avail_dpbp;
+       struct dpaa2_bp_info *bp_info;
        struct dpbp_attr dpbp_attr;
        uint32_t bpid;
        int ret, p_ret;
@@ -127,7 +127,12 @@ rte_hw_mbuf_create_pool(struct rte_mempool *mp)
        rte_dpaa2_bpid_info[bpid].bp_list = bp_list;
        rte_dpaa2_bpid_info[bpid].bpid = bpid;
 
-       mp->pool_data = (void *)&rte_dpaa2_bpid_info[bpid];
+       bp_info = rte_malloc(NULL,
+                            sizeof(struct dpaa2_bp_info),
+                            RTE_CACHE_LINE_SIZE);
+       rte_memcpy(bp_info, (void *)&rte_dpaa2_bpid_info[bpid],
+                  sizeof(struct dpaa2_bp_info));
+       mp->pool_data = (void *)bp_info;
 
        PMD_INIT_LOG(DEBUG, "BP List created for bpid =%d", dpbp_attr.bpid);
 
@@ -161,7 +166,7 @@ rte_hw_mbuf_free_pool(struct rte_mempool *mp)
                while (temp) {
                        if (temp == bp) {
                                prev->next = temp->next;
-                               free(bp);
+                               rte_free(bp);
                                break;
                        }
                        prev = temp;
@@ -169,6 +174,7 @@ rte_hw_mbuf_free_pool(struct rte_mempool *mp)
                }
        }
 
+       rte_free(mp->pool_data);
        dpaa2_free_dpbp_dev(dpbp_node);
 }
 
@@ -294,7 +300,7 @@ rte_dpaa2_mbuf_alloc_bulk(struct rte_mempool *pool,
                        /* Releasing all buffers allocated */
                        rte_dpaa2_mbuf_release(pool, obj_table, bpid,
                                           bp_info->meta_data_size, n);
-                       return ret;
+                       return -ENOBUFS;
                }
                /* assigning mbuf from the acquired objects */
                for (i = 0; (i < ret) && bufs[i]; i++) {