drivers: remove direct access to interrupt handle
[dpdk.git] / drivers / bus / fslmc / portal / dpaa2_hw_dpio.c
index c626c12..2210a0f 100644 (file)
@@ -1,34 +1,8 @@
-/*-
- *   BSD LICENSE
+/* SPDX-License-Identifier: BSD-3-Clause
  *
  *   Copyright (c) 2016 Freescale Semiconductor, Inc. All rights reserved.
- *   Copyright (c) 2016 NXP. All rights reserved.
+ *   Copyright 2016-2019 NXP
  *
- *   Redistribution and use in source and binary forms, with or without
- *   modification, are permitted provided that the following conditions
- *   are met:
- *
- *     * Redistributions of source code must retain the above copyright
- *       notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above copyright
- *       notice, this list of conditions and the following disclaimer in
- *       the documentation and/or other materials provided with the
- *       distribution.
- *     * Neither the name of Freescale Semiconductor, Inc nor the names of its
- *       contributors may be used to endorse or promote products derived
- *       from this software without specific prior written permission.
- *
- *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 #include <unistd.h>
 #include <stdio.h>
 #include <sys/ioctl.h>
 #include <sys/stat.h>
 #include <sys/mman.h>
+#include <sys/epoll.h>
+#include <sys/eventfd.h>
 #include <sys/syscall.h>
 
 #include <rte_mbuf.h>
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 #include <rte_malloc.h>
 #include <rte_memcpy.h>
 #include <rte_string_fns.h>
 #include <rte_cycles.h>
 #include <rte_kvargs.h>
 #include <rte_dev.h>
-#include <rte_ethdev.h>
 
 #include <fslmc_logs.h>
-#include <fslmc_vfio.h>
+#include <rte_fslmc.h>
 #include "dpaa2_hw_pvt.h"
 #include "dpaa2_hw_dpio.h"
 #include <mc/fsl_dpmng.h>
@@ -75,6 +50,17 @@ static struct dpio_dev_list dpio_dev_list
        = TAILQ_HEAD_INITIALIZER(dpio_dev_list); /*!< DPIO device list */
 static uint32_t io_space_count;
 
+/* Variable to store DPAA2 platform type */
+uint32_t dpaa2_svr_family;
+
+/* Variable to store DPAA2 DQRR size */
+uint8_t dpaa2_dqrr_size;
+/* Variable to store DPAA2 EQCR size */
+uint8_t dpaa2_eqcr_size;
+
+/* Variable to hold the portal_key, once created.*/
+static pthread_key_t dpaa2_portal_key;
+
 /*Stashing Macros default for LS208x*/
 static int dpaa2_core_cluster_base = 0x04;
 static int dpaa2_cluster_sz = 2;
@@ -89,140 +75,196 @@ static int dpaa2_cluster_sz = 2;
  * Cluster 1 (ID = x02) : CPU0, CPU1, CPU2, CPU3;
  * Cluster 2 (ID = x03) : CPU4, CPU5, CPU6, CPU7;
  */
-
-/* Set the STASH Destination depending on Current CPU ID.
- * e.g. Valid values of SDEST are 4,5,6,7. Where,
- * CPU 0-1 will have SDEST 4
- * CPU 2-3 will have SDEST 5.....and so on.
+/* For LX2160 platform There are four clusters with following mapping:
+ * Cluster 1 (ID = x00) : CPU0, CPU1;
+ * Cluster 2 (ID = x01) : CPU2, CPU3;
+ * Cluster 3 (ID = x02) : CPU4, CPU5;
+ * Cluster 4 (ID = x03) : CPU6, CPU7;
+ * Cluster 1 (ID = x04) : CPU8, CPU9;
+ * Cluster 2 (ID = x05) : CPU10, CP11;
+ * Cluster 3 (ID = x06) : CPU12, CPU13;
+ * Cluster 4 (ID = x07) : CPU14, CPU15;
  */
+
 static int
-dpaa2_core_cluster_sdest(int cpu_id)
+dpaa2_get_core_id(void)
 {
-       int x = cpu_id / dpaa2_cluster_sz;
+       rte_cpuset_t cpuset;
+       int i, ret, cpu_id = -1;
 
-       if (x > 3)
-               x = 3;
+       ret = pthread_getaffinity_np(pthread_self(), sizeof(cpu_set_t),
+               &cpuset);
+       if (ret) {
+               DPAA2_BUS_ERR("pthread_getaffinity_np() failed");
+               return ret;
+       }
 
-       return dpaa2_core_cluster_base + x;
+       for (i = 0; i < RTE_MAX_LCORE; i++) {
+               if (CPU_ISSET(i, &cpuset)) {
+                       if (cpu_id == -1)
+                               cpu_id = i;
+                       else
+                               /* Multiple cpus are affined */
+                               return -1;
+               }
+       }
+
+       return cpu_id;
 }
 
 static int
-configure_dpio_qbman_swp(struct dpaa2_dpio_dev *dpio_dev)
+dpaa2_core_cluster_sdest(int cpu_id)
 {
-       struct qbman_swp_desc p_des;
-       struct dpio_attr attr;
+       int x = cpu_id / dpaa2_cluster_sz;
 
-       dpio_dev->dpio = malloc(sizeof(struct fsl_mc_io));
-       if (!dpio_dev->dpio) {
-               PMD_INIT_LOG(ERR, "Memory allocation failure\n");
-               return -1;
-       }
+       return dpaa2_core_cluster_base + x;
+}
 
-       PMD_DRV_LOG(DEBUG, "\t Allocated  DPIO Portal[%p]", dpio_dev->dpio);
-       dpio_dev->dpio->regs = dpio_dev->mc_portal;
-       if (dpio_open(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->hw_id,
-                     &dpio_dev->token)) {
-               PMD_INIT_LOG(ERR, "Failed to allocate IO space\n");
-               free(dpio_dev->dpio);
-               return -1;
+#ifdef RTE_EVENT_DPAA2
+static void
+dpaa2_affine_dpio_intr_to_respective_core(int32_t dpio_id, int cpu_id)
+{
+#define STRING_LEN     28
+#define COMMAND_LEN    50
+       uint32_t cpu_mask = 1;
+       int ret;
+       size_t len = 0;
+       char *temp = NULL, *token = NULL;
+       char string[STRING_LEN], command[COMMAND_LEN];
+       FILE *file;
+
+       snprintf(string, STRING_LEN, "dpio.%d", dpio_id);
+       file = fopen("/proc/interrupts", "r");
+       if (!file) {
+               DPAA2_BUS_WARN("Failed to open /proc/interrupts file");
+               return;
        }
-
-       if (dpio_reset(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->token)) {
-               PMD_INIT_LOG(ERR, "Failed to reset dpio\n");
-               dpio_close(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->token);
-               free(dpio_dev->dpio);
-               return -1;
+       while (getline(&temp, &len, file) != -1) {
+               if ((strstr(temp, string)) != NULL) {
+                       token = strtok(temp, ":");
+                       break;
+               }
        }
 
-       if (dpio_enable(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->token)) {
-               PMD_INIT_LOG(ERR, "Failed to Enable dpio\n");
-               dpio_close(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->token);
-               free(dpio_dev->dpio);
-               return -1;
+       if (!token) {
+               DPAA2_BUS_WARN("Failed to get interrupt id for dpio.%d",
+                              dpio_id);
+               if (temp)
+                       free(temp);
+               fclose(file);
+               return;
        }
 
-       if (dpio_get_attributes(dpio_dev->dpio, CMD_PRI_LOW,
-                               dpio_dev->token, &attr)) {
-               PMD_INIT_LOG(ERR, "DPIO Get attribute failed\n");
-               dpio_disable(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->token);
-               dpio_close(dpio_dev->dpio, CMD_PRI_LOW,  dpio_dev->token);
-               free(dpio_dev->dpio);
+       cpu_mask = cpu_mask << cpu_id;
+       snprintf(command, COMMAND_LEN, "echo %X > /proc/irq/%s/smp_affinity",
+                cpu_mask, token);
+       ret = system(command);
+       if (ret < 0)
+               DPAA2_BUS_DEBUG(
+                       "Failed to affine interrupts on respective core");
+       else
+               DPAA2_BUS_DEBUG(" %s command is executed", command);
+
+       free(temp);
+       fclose(file);
+}
+
+static int dpaa2_dpio_intr_init(struct dpaa2_dpio_dev *dpio_dev)
+{
+       struct epoll_event epoll_ev;
+       int eventfd, dpio_epoll_fd, ret;
+       int threshold = 0x3, timeout = 0xFF;
+
+       dpio_epoll_fd = epoll_create(1);
+       ret = rte_dpaa2_intr_enable(dpio_dev->intr_handle, 0);
+       if (ret) {
+               DPAA2_BUS_ERR("Interrupt registeration failed");
                return -1;
        }
 
-       PMD_INIT_LOG(DEBUG, "Qbman Portal ID %d", attr.qbman_portal_id);
+       if (getenv("DPAA2_PORTAL_INTR_THRESHOLD"))
+               threshold = atoi(getenv("DPAA2_PORTAL_INTR_THRESHOLD"));
 
-       /* Configure & setup SW portal */
-       p_des.block = NULL;
-       p_des.idx = attr.qbman_portal_id;
-       p_des.cena_bar = (void *)(dpio_dev->qbman_portal_ce_paddr);
-       p_des.cinh_bar = (void *)(dpio_dev->qbman_portal_ci_paddr);
-       p_des.irq = -1;
-       p_des.qman_version = attr.qbman_version;
+       if (getenv("DPAA2_PORTAL_INTR_TIMEOUT"))
+               sscanf(getenv("DPAA2_PORTAL_INTR_TIMEOUT"), "%x", &timeout);
 
-       dpio_dev->sw_portal = qbman_swp_init(&p_des);
-       if (dpio_dev->sw_portal == NULL) {
-               PMD_DRV_LOG(ERR, " QBMan SW Portal Init failed\n");
-               dpio_close(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->token);
-               free(dpio_dev->dpio);
+       qbman_swp_interrupt_set_trigger(dpio_dev->sw_portal,
+                                       QBMAN_SWP_INTERRUPT_DQRI);
+       qbman_swp_interrupt_clear_status(dpio_dev->sw_portal, 0xffffffff);
+       qbman_swp_interrupt_set_inhibit(dpio_dev->sw_portal, 0);
+       qbman_swp_dqrr_thrshld_write(dpio_dev->sw_portal, threshold);
+       qbman_swp_intr_timeout_write(dpio_dev->sw_portal, timeout);
+
+       eventfd = rte_intr_fd_get(dpio_dev->intr_handle);
+       epoll_ev.events = EPOLLIN | EPOLLPRI | EPOLLET;
+       epoll_ev.data.fd = eventfd;
+
+       ret = epoll_ctl(dpio_epoll_fd, EPOLL_CTL_ADD, eventfd, &epoll_ev);
+       if (ret < 0) {
+               DPAA2_BUS_ERR("epoll_ctl failed");
                return -1;
        }
+       dpio_dev->epoll_fd = dpio_epoll_fd;
 
        return 0;
 }
 
-static int
-dpaa2_configure_stashing(struct dpaa2_dpio_dev *dpio_dev)
+static void dpaa2_dpio_intr_deinit(struct dpaa2_dpio_dev *dpio_dev)
 {
-       int sdest;
-       int cpu_id, ret;
-       static int first_time;
+       int ret;
 
-       /* find the SoC type for the first time */
-       if (!first_time) {
-               struct mc_soc_version mc_plat_info = {0};
+       ret = rte_dpaa2_intr_disable(dpio_dev->intr_handle, 0);
+       if (ret)
+               DPAA2_BUS_ERR("DPIO interrupt disable failed");
 
-               if (mc_get_soc_version(dpio_dev->dpio,
-                                      CMD_PRI_LOW, &mc_plat_info)) {
-                       PMD_INIT_LOG(ERR, "\tmc_get_soc_version failed\n");
-               } else if ((mc_plat_info.svr & 0xffff0000) == SVR_LS1080A) {
-                       dpaa2_core_cluster_base = 0x02;
-                       dpaa2_cluster_sz = 4;
-                       PMD_INIT_LOG(DEBUG, "\tLS108x (A53) Platform Detected");
-               }
-               first_time = 1;
-       }
+       close(dpio_dev->epoll_fd);
+}
+#endif
+
+static int
+dpaa2_configure_stashing(struct dpaa2_dpio_dev *dpio_dev, int cpu_id)
+{
+       int sdest, ret;
 
-       /* Set the Stashing Destination */
-       cpu_id = rte_lcore_id();
-       if (cpu_id < 0) {
-               cpu_id = rte_get_master_lcore();
-               if (cpu_id < 0) {
-                       RTE_LOG(ERR, PMD, "\tGetting CPU Index failed\n");
-                       return -1;
-               }
-       }
        /* Set the STASH Destination depending on Current CPU ID.
         * Valid values of SDEST are 4,5,6,7. Where,
         */
-
        sdest = dpaa2_core_cluster_sdest(cpu_id);
-       PMD_DRV_LOG(DEBUG, "Portal= %d  CPU= %u SDEST= %d",
-                   dpio_dev->index, cpu_id, sdest);
+       DPAA2_BUS_DEBUG("Portal= %d  CPU= %u SDEST= %d",
+                       dpio_dev->index, cpu_id, sdest);
 
        ret = dpio_set_stashing_destination(dpio_dev->dpio, CMD_PRI_LOW,
                                            dpio_dev->token, sdest);
        if (ret) {
-               PMD_DRV_LOG(ERR, "%d ERROR in SDEST\n",  ret);
+               DPAA2_BUS_ERR("%d ERROR in SDEST",  ret);
+               return -1;
+       }
+
+#ifdef RTE_EVENT_DPAA2
+       if (dpaa2_dpio_intr_init(dpio_dev)) {
+               DPAA2_BUS_ERR("Interrupt registration failed for dpio");
                return -1;
        }
+       dpaa2_affine_dpio_intr_to_respective_core(dpio_dev->hw_id, cpu_id);
+#endif
 
        return 0;
 }
 
-static inline struct dpaa2_dpio_dev *dpaa2_get_qbman_swp(void)
+static void dpaa2_put_qbman_swp(struct dpaa2_dpio_dev *dpio_dev)
+{
+       if (dpio_dev) {
+#ifdef RTE_EVENT_DPAA2
+               dpaa2_dpio_intr_deinit(dpio_dev);
+#endif
+               rte_atomic16_clear(&dpio_dev->ref_count);
+       }
+}
+
+static struct dpaa2_dpio_dev *dpaa2_get_qbman_swp(void)
 {
        struct dpaa2_dpio_dev *dpio_dev = NULL;
+       int cpu_id;
        int ret;
 
        /* Get DPIO dev handle from list using index */
@@ -230,15 +272,35 @@ static inline struct dpaa2_dpio_dev *dpaa2_get_qbman_swp(void)
                if (dpio_dev && rte_atomic16_test_and_set(&dpio_dev->ref_count))
                        break;
        }
-       if (!dpio_dev)
+       if (!dpio_dev) {
+               DPAA2_BUS_ERR("No software portal resource left");
                return NULL;
+       }
 
-       PMD_DRV_LOG(DEBUG, "New Portal=0x%x (%d) affined thread - %lu",
-                   dpio_dev, dpio_dev->index, syscall(SYS_gettid));
+       DPAA2_BUS_DEBUG("New Portal %p (%d) affined thread - %u",
+                       dpio_dev, dpio_dev->index, rte_gettid());
 
-       ret = dpaa2_configure_stashing(dpio_dev);
-       if (ret)
-               PMD_DRV_LOG(ERR, "dpaa2_configure_stashing failed");
+       /* Set the Stashing Destination */
+       cpu_id = dpaa2_get_core_id();
+       if (cpu_id < 0) {
+               DPAA2_BUS_WARN("Thread not affined to a single core");
+               if (dpaa2_svr_family != SVR_LX2160A)
+                       qbman_swp_update(dpio_dev->sw_portal, 1);
+       } else {
+               ret = dpaa2_configure_stashing(dpio_dev, cpu_id);
+               if (ret) {
+                       DPAA2_BUS_ERR("dpaa2_configure_stashing failed");
+                       rte_atomic16_clear(&dpio_dev->ref_count);
+                       return NULL;
+               }
+       }
+
+       ret = pthread_setspecific(dpaa2_portal_key, (void *)dpio_dev);
+       if (ret) {
+               DPAA2_BUS_ERR("pthread_setspecific failed with ret: %d", ret);
+               dpaa2_put_qbman_swp(dpio_dev);
+               return NULL;
+       }
 
        return dpio_dev;
 }
@@ -246,170 +308,264 @@ static inline struct dpaa2_dpio_dev *dpaa2_get_qbman_swp(void)
 int
 dpaa2_affine_qbman_swp(void)
 {
-       unsigned int lcore_id = rte_lcore_id();
-       uint64_t tid = syscall(SYS_gettid);
-
-       if (lcore_id == LCORE_ID_ANY)
-               lcore_id = rte_get_master_lcore();
-       /* if the core id is not supported */
-       else if (lcore_id >= RTE_MAX_LCORE)
-               return -1;
-
-       if (dpaa2_io_portal[lcore_id].dpio_dev) {
-               PMD_DRV_LOG(INFO, "DPAA Portal=0x%x (%d) is being shared"
-                           " between thread %lu and current  %lu",
-                           dpaa2_io_portal[lcore_id].dpio_dev,
-                           dpaa2_io_portal[lcore_id].dpio_dev->index,
-                           dpaa2_io_portal[lcore_id].net_tid,
-                           tid);
-               RTE_PER_LCORE(_dpaa2_io).dpio_dev
-                       = dpaa2_io_portal[lcore_id].dpio_dev;
-               rte_atomic16_inc(&dpaa2_io_portal
-                                [lcore_id].dpio_dev->ref_count);
-               dpaa2_io_portal[lcore_id].net_tid = tid;
-
-               PMD_DRV_LOG(DEBUG, "Old Portal=0x%x (%d) affined thread - %lu",
-                           dpaa2_io_portal[lcore_id].dpio_dev,
-                           dpaa2_io_portal[lcore_id].dpio_dev->index,
-                           tid);
-               return 0;
-       }
+       struct dpaa2_dpio_dev *dpio_dev;
+       uint64_t tid = rte_gettid();
 
        /* Populate the dpaa2_io_portal structure */
-       dpaa2_io_portal[lcore_id].dpio_dev = dpaa2_get_qbman_swp();
-
-       if (dpaa2_io_portal[lcore_id].dpio_dev) {
-               RTE_PER_LCORE(_dpaa2_io).dpio_dev
-                       = dpaa2_io_portal[lcore_id].dpio_dev;
-               dpaa2_io_portal[lcore_id].net_tid = tid;
+       if (!RTE_PER_LCORE(_dpaa2_io).dpio_dev) {
+               dpio_dev = dpaa2_get_qbman_swp();
+               if (!dpio_dev) {
+                       DPAA2_BUS_ERR("Error in software portal allocation");
+                       return -1;
+               }
+               RTE_PER_LCORE(_dpaa2_io).dpio_dev = dpio_dev;
 
-               return 0;
-       } else {
-               return -1;
+               DPAA2_BUS_INFO(
+                       "DPAA Portal=%p (%d) is affined to thread %" PRIu64,
+                       dpio_dev, dpio_dev->index, tid);
        }
+       return 0;
 }
 
 int
-dpaa2_affine_qbman_swp_sec(void)
+dpaa2_affine_qbman_ethrx_swp(void)
 {
-       unsigned int lcore_id = rte_lcore_id();
-       uint64_t tid = syscall(SYS_gettid);
+       struct dpaa2_dpio_dev *dpio_dev;
+       uint64_t tid = rte_gettid();
 
-       if (lcore_id == LCORE_ID_ANY)
-               lcore_id = rte_get_master_lcore();
-       /* if the core id is not supported */
-       else if (lcore_id >= RTE_MAX_LCORE)
-               return -1;
+       /* Populate the dpaa2_io_portal structure */
+       if (!RTE_PER_LCORE(_dpaa2_io).ethrx_dpio_dev) {
+               dpio_dev = dpaa2_get_qbman_swp();
+               if (!dpio_dev) {
+                       DPAA2_BUS_ERR("Error in software portal allocation");
+                       return -1;
+               }
+               RTE_PER_LCORE(_dpaa2_io).ethrx_dpio_dev = dpio_dev;
 
-       if (dpaa2_io_portal[lcore_id].sec_dpio_dev) {
-               PMD_DRV_LOG(INFO, "DPAA Portal=0x%x (%d) is being shared"
-                           " between thread %lu and current  %lu",
-                           dpaa2_io_portal[lcore_id].sec_dpio_dev,
-                           dpaa2_io_portal[lcore_id].sec_dpio_dev->index,
-                           dpaa2_io_portal[lcore_id].sec_tid,
-                           tid);
-               RTE_PER_LCORE(_dpaa2_io).sec_dpio_dev
-                       = dpaa2_io_portal[lcore_id].sec_dpio_dev;
-               rte_atomic16_inc(&dpaa2_io_portal
-                                [lcore_id].sec_dpio_dev->ref_count);
-               dpaa2_io_portal[lcore_id].sec_tid = tid;
-
-               PMD_DRV_LOG(DEBUG, "Old Portal=0x%x (%d) affined thread - %lu",
-                           dpaa2_io_portal[lcore_id].sec_dpio_dev,
-                           dpaa2_io_portal[lcore_id].sec_dpio_dev->index,
-                           tid);
-               return 0;
+               DPAA2_BUS_INFO(
+                       "DPAA Portal=%p (%d) is affined for eth rx to thread %"
+                       PRIu64, dpio_dev, dpio_dev->index, tid);
        }
+       return 0;
+}
 
-       /* Populate the dpaa2_io_portal structure */
-       dpaa2_io_portal[lcore_id].sec_dpio_dev = dpaa2_get_qbman_swp();
+static void dpaa2_portal_finish(void *arg)
+{
+       RTE_SET_USED(arg);
 
-       if (dpaa2_io_portal[lcore_id].sec_dpio_dev) {
-               RTE_PER_LCORE(_dpaa2_io).sec_dpio_dev
-                       = dpaa2_io_portal[lcore_id].sec_dpio_dev;
-               dpaa2_io_portal[lcore_id].sec_tid = tid;
-               return 0;
-       } else {
-               return -1;
-       }
+       dpaa2_put_qbman_swp(RTE_PER_LCORE(_dpaa2_io).dpio_dev);
+       dpaa2_put_qbman_swp(RTE_PER_LCORE(_dpaa2_io).ethrx_dpio_dev);
+
+       pthread_setspecific(dpaa2_portal_key, NULL);
 }
 
 static int
-dpaa2_create_dpio_device(struct fslmc_vfio_device *vdev,
+dpaa2_create_dpio_device(int vdev_fd,
                         struct vfio_device_info *obj_info,
                         int object_id)
 {
-       struct dpaa2_dpio_dev *dpio_dev;
+       struct dpaa2_dpio_dev *dpio_dev = NULL;
        struct vfio_region_info reg_info = { .argsz = sizeof(reg_info)};
+       struct qbman_swp_desc p_des;
+       struct dpio_attr attr;
+       int ret;
 
        if (obj_info->num_regions < NUM_DPIO_REGIONS) {
-               PMD_INIT_LOG(ERR, "ERROR, Not sufficient number "
-                               "of DPIO regions.\n");
+               DPAA2_BUS_ERR("Not sufficient number of DPIO regions");
                return -1;
        }
 
-       dpio_dev = rte_malloc(NULL, sizeof(struct dpaa2_dpio_dev),
+       dpio_dev = rte_zmalloc(NULL, sizeof(struct dpaa2_dpio_dev),
                              RTE_CACHE_LINE_SIZE);
        if (!dpio_dev) {
-               PMD_INIT_LOG(ERR, "Memory allocation failed for DPIO Device\n");
+               DPAA2_BUS_ERR("Memory allocation failed for DPIO Device");
                return -1;
        }
 
        dpio_dev->dpio = NULL;
        dpio_dev->hw_id = object_id;
-       dpio_dev->vfio_fd = vdev->fd;
        rte_atomic16_init(&dpio_dev->ref_count);
        /* Using single portal  for all devices */
-       dpio_dev->mc_portal = rte_mcp_ptr_list[MC_PORTAL_INDEX];
+       dpio_dev->mc_portal = dpaa2_get_mcp_ptr(MC_PORTAL_INDEX);
+
+       /* Allocate interrupt instance */
+       dpio_dev->intr_handle =
+               rte_intr_instance_alloc(RTE_INTR_INSTANCE_F_SHARED);
+       if (!dpio_dev->intr_handle) {
+               DPAA2_BUS_ERR("Failed to allocate intr handle");
+               goto err;
+       }
 
-       reg_info.index = 0;
-       if (ioctl(dpio_dev->vfio_fd, VFIO_DEVICE_GET_REGION_INFO, &reg_info)) {
-               PMD_INIT_LOG(ERR, "vfio: error getting region info\n");
-               rte_free(dpio_dev);
-               return -1;
+       dpio_dev->dpio = rte_zmalloc(NULL, sizeof(struct fsl_mc_io),
+                                    RTE_CACHE_LINE_SIZE);
+       if (!dpio_dev->dpio) {
+               DPAA2_BUS_ERR("Memory allocation failure");
+               goto err;
        }
 
-       dpio_dev->ce_size = reg_info.size;
-       dpio_dev->qbman_portal_ce_paddr = (uint64_t)mmap(NULL, reg_info.size,
-                               PROT_WRITE | PROT_READ, MAP_SHARED,
-                               dpio_dev->vfio_fd, reg_info.offset);
+       dpio_dev->dpio->regs = dpio_dev->mc_portal;
+       if (dpio_open(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->hw_id,
+                     &dpio_dev->token)) {
+               DPAA2_BUS_ERR("Failed to allocate IO space");
+               goto err;
+       }
 
-       /* Create Mapping for QBMan Cache Enabled area. This is a fix for
-        * SMMU fault for DQRR statshing transaction.
-        */
-       if (vfio_dmamap_mem_region(dpio_dev->qbman_portal_ce_paddr,
-                                  reg_info.offset, reg_info.size)) {
-               PMD_INIT_LOG(ERR, "DMAMAP for Portal CE area failed.\n");
-               rte_free(dpio_dev);
-               return -1;
+       if (dpio_reset(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->token)) {
+               DPAA2_BUS_ERR("Failed to reset dpio");
+               goto err;
        }
 
-       reg_info.index = 1;
-       if (ioctl(dpio_dev->vfio_fd, VFIO_DEVICE_GET_REGION_INFO, &reg_info)) {
-               PMD_INIT_LOG(ERR, "vfio: error getting region info\n");
-               rte_free(dpio_dev);
-               return -1;
+       if (dpio_enable(dpio_dev->dpio, CMD_PRI_LOW, dpio_dev->token)) {
+               DPAA2_BUS_ERR("Failed to Enable dpio");
+               goto err;
+       }
+
+       if (dpio_get_attributes(dpio_dev->dpio, CMD_PRI_LOW,
+                               dpio_dev->token, &attr)) {
+               DPAA2_BUS_ERR("DPIO Get attribute failed");
+               goto err;
+       }
+
+       /* find the SoC type for the first time */
+       if (!dpaa2_svr_family) {
+               struct mc_soc_version mc_plat_info = {0};
+
+               if (mc_get_soc_version(dpio_dev->dpio,
+                                      CMD_PRI_LOW, &mc_plat_info)) {
+                       DPAA2_BUS_ERR("Unable to get SoC version information");
+               } else if ((mc_plat_info.svr & 0xffff0000) == SVR_LS1080A) {
+                       dpaa2_core_cluster_base = 0x02;
+                       dpaa2_cluster_sz = 4;
+                       DPAA2_BUS_DEBUG("LS108x (A53) Platform Detected");
+               } else if ((mc_plat_info.svr & 0xffff0000) == SVR_LX2160A) {
+                       dpaa2_core_cluster_base = 0x00;
+                       dpaa2_cluster_sz = 2;
+                       DPAA2_BUS_DEBUG("LX2160 Platform Detected");
+               }
+               dpaa2_svr_family = (mc_plat_info.svr & 0xffff0000);
+
+               if (dpaa2_svr_family == SVR_LX2160A) {
+                       dpaa2_dqrr_size = DPAA2_LX2_DQRR_RING_SIZE;
+                       dpaa2_eqcr_size = DPAA2_LX2_EQCR_RING_SIZE;
+               } else {
+                       dpaa2_dqrr_size = DPAA2_DQRR_RING_SIZE;
+                       dpaa2_eqcr_size = DPAA2_EQCR_RING_SIZE;
+               }
+       }
+
+       if (dpaa2_svr_family == SVR_LX2160A)
+               reg_info.index = DPAA2_SWP_CENA_MEM_REGION;
+       else
+               reg_info.index = DPAA2_SWP_CENA_REGION;
+
+       if (ioctl(vdev_fd, VFIO_DEVICE_GET_REGION_INFO, &reg_info)) {
+               DPAA2_BUS_ERR("vfio: error getting region info");
+               goto err;
+       }
+
+       dpio_dev->ce_size = reg_info.size;
+       dpio_dev->qbman_portal_ce_paddr = (size_t)mmap(NULL, reg_info.size,
+                               PROT_WRITE | PROT_READ, MAP_SHARED,
+                               vdev_fd, reg_info.offset);
+
+       reg_info.index = DPAA2_SWP_CINH_REGION;
+       if (ioctl(vdev_fd, VFIO_DEVICE_GET_REGION_INFO, &reg_info)) {
+               DPAA2_BUS_ERR("vfio: error getting region info");
+               goto err;
        }
 
        dpio_dev->ci_size = reg_info.size;
-       dpio_dev->qbman_portal_ci_paddr = (uint64_t)mmap(NULL, reg_info.size,
+       dpio_dev->qbman_portal_ci_paddr = (size_t)mmap(NULL, reg_info.size,
                                PROT_WRITE | PROT_READ, MAP_SHARED,
-                               dpio_dev->vfio_fd, reg_info.offset);
+                               vdev_fd, reg_info.offset);
 
-       if (configure_dpio_qbman_swp(dpio_dev)) {
-               PMD_INIT_LOG(ERR,
-                            "Fail to configure the dpio qbman portal for %d\n",
-                            dpio_dev->hw_id);
-               rte_free(dpio_dev);
-               return -1;
+       /* Configure & setup SW portal */
+       p_des.block = NULL;
+       p_des.idx = attr.qbman_portal_id;
+       p_des.cena_bar = (void *)(dpio_dev->qbman_portal_ce_paddr);
+       p_des.cinh_bar = (void *)(dpio_dev->qbman_portal_ci_paddr);
+       p_des.irq = -1;
+       p_des.qman_version = attr.qbman_version;
+       p_des.eqcr_mode = qman_eqcr_vb_ring;
+       p_des.cena_access_mode = qman_cena_fastest_access;
+
+       dpio_dev->sw_portal = qbman_swp_init(&p_des);
+       if (dpio_dev->sw_portal == NULL) {
+               DPAA2_BUS_ERR("QBMan SW Portal Init failed");
+               goto err;
        }
 
        io_space_count++;
        dpio_dev->index = io_space_count;
+
+       if (rte_dpaa2_vfio_setup_intr(dpio_dev->intr_handle, vdev_fd, 1)) {
+               DPAA2_BUS_ERR("Fail to setup interrupt for %d",
+                             dpio_dev->hw_id);
+               goto err;
+       }
+
+       dpio_dev->eqresp = rte_zmalloc(NULL, MAX_EQ_RESP_ENTRIES *
+                                    (sizeof(struct qbman_result) +
+                                    sizeof(struct eqresp_metadata)),
+                                    RTE_CACHE_LINE_SIZE);
+       if (!dpio_dev->eqresp) {
+               DPAA2_BUS_ERR("Memory allocation failed for eqresp");
+               goto err;
+       }
+       dpio_dev->eqresp_meta = (struct eqresp_metadata *)(dpio_dev->eqresp +
+                               MAX_EQ_RESP_ENTRIES);
+
+
        TAILQ_INSERT_TAIL(&dpio_dev_list, dpio_dev, next);
-       PMD_INIT_LOG(DEBUG, "DPAA2: Added [dpio-%d]", object_id);
+
+       if (!dpaa2_portal_key) {
+               /* create the key, supplying a function that'll be invoked
+                * when a portal affined thread will be deleted.
+                */
+               ret = pthread_key_create(&dpaa2_portal_key,
+                                        dpaa2_portal_finish);
+               if (ret) {
+                       DPAA2_BUS_DEBUG("Unable to create pthread key (%d)",
+                                       ret);
+                       goto err;
+               }
+       }
 
        return 0;
+
+err:
+       if (dpio_dev->dpio) {
+               if (dpio_dev->token) {
+                       dpio_disable(dpio_dev->dpio, CMD_PRI_LOW,
+                                    dpio_dev->token);
+                       dpio_close(dpio_dev->dpio, CMD_PRI_LOW,
+                                  dpio_dev->token);
+               }
+
+               rte_free(dpio_dev->eqresp);
+               rte_free(dpio_dev->dpio);
+       }
+
+       rte_intr_instance_free(dpio_dev->intr_handle);
+       rte_free(dpio_dev);
+
+       /* For each element in the list, cleanup */
+       TAILQ_FOREACH(dpio_dev, &dpio_dev_list, next) {
+               if (dpio_dev->dpio) {
+                       dpio_disable(dpio_dev->dpio, CMD_PRI_LOW,
+                               dpio_dev->token);
+                       dpio_close(dpio_dev->dpio, CMD_PRI_LOW,
+                               dpio_dev->token);
+                       rte_free(dpio_dev->dpio);
+               }
+               rte_intr_instance_free(dpio_dev->intr_handle);
+               rte_free(dpio_dev);
+       }
+
+       /* Preventing re-use of the list with old entries */
+       TAILQ_INIT(&dpio_dev_list);
+
+       return -1;
 }
 
 void
@@ -430,7 +586,7 @@ dpaa2_alloc_dq_storage(struct queue_storage_info_t *q_storage)
 
        for (i = 0; i < NUM_DQS_PER_QUEUE; i++) {
                q_storage->dq_storage[i] = rte_malloc(NULL,
-                       DPAA2_DQRR_RING_SIZE * sizeof(struct qbman_result),
+                       dpaa2_dqrr_size * sizeof(struct qbman_result),
                        RTE_CACHE_LINE_SIZE);
                if (!q_storage->dq_storage[i])
                        goto fail;
@@ -443,8 +599,43 @@ fail:
        return -1;
 }
 
+uint32_t
+dpaa2_free_eq_descriptors(void)
+{
+       struct dpaa2_dpio_dev *dpio_dev = DPAA2_PER_LCORE_DPIO;
+       struct qbman_result *eqresp;
+       struct eqresp_metadata *eqresp_meta;
+       struct dpaa2_queue *txq;
+
+       while (dpio_dev->eqresp_ci != dpio_dev->eqresp_pi) {
+               eqresp = &dpio_dev->eqresp[dpio_dev->eqresp_ci];
+               eqresp_meta = &dpio_dev->eqresp_meta[dpio_dev->eqresp_ci];
+
+               if (!qbman_result_eqresp_rspid(eqresp))
+                       break;
+
+               if (qbman_result_eqresp_rc(eqresp)) {
+                       txq = eqresp_meta->dpaa2_q;
+                       txq->cb_eqresp_free(dpio_dev->eqresp_ci);
+               }
+               qbman_result_eqresp_set_rspid(eqresp, 0);
+
+               dpio_dev->eqresp_ci + 1 < MAX_EQ_RESP_ENTRIES ?
+                       dpio_dev->eqresp_ci++ : (dpio_dev->eqresp_ci = 0);
+       }
+
+       /* Return 1 less entry so that PI and CI are never same in a
+        * case there all the EQ responses are in use.
+        */
+       if (dpio_dev->eqresp_ci > dpio_dev->eqresp_pi)
+               return dpio_dev->eqresp_ci - dpio_dev->eqresp_pi - 1;
+       else
+               return dpio_dev->eqresp_ci - dpio_dev->eqresp_pi +
+                       MAX_EQ_RESP_ENTRIES - 1;
+}
+
 static struct rte_dpaa2_object rte_dpaa2_dpio_obj = {
-       .object_id = DPAA2_MC_DPIO_DEVID,
+       .dev_type = DPAA2_IO,
        .create = dpaa2_create_dpio_device,
 };