From: Prashant Gupta <[email protected]>

Add RTE_PROC_PRIMARY checks in device initialization paths for
net/dpaa, crypto/dpaa_sec and dma/dpaa drivers. Secondary
processes should skip hardware initialization to prevent
segfaults when accessing hardware registers that are only
mapped in the primary process.

Signed-off-by: Prashant Gupta <[email protected]>
---
 drivers/crypto/dpaa_sec/dpaa_sec.c |  3 ---
 drivers/dma/dpaa/dpaa_qdma.c       | 23 +++++++++++++++++++++++
 drivers/net/dpaa/dpaa_ethdev.c     |  3 +++
 3 files changed, 26 insertions(+), 3 deletions(-)

diff --git a/drivers/crypto/dpaa_sec/dpaa_sec.c 
b/drivers/crypto/dpaa_sec/dpaa_sec.c
index c53ee70853..5caf6b18dc 100644
--- a/drivers/crypto/dpaa_sec/dpaa_sec.c
+++ b/drivers/crypto/dpaa_sec/dpaa_sec.c
@@ -3782,9 +3782,6 @@ cryptodev_dpaa_sec_probe(struct rte_dpaa_driver *dpaa_drv 
__rte_unused,
                        RTE_DPAA_MAX_NB_SEC_QPS,
        };
 
-       if (rte_eal_process_type() != RTE_PROC_PRIMARY)
-               return 0;
-
        cryptodev = rte_cryptodev_pmd_create(dpaa_dev->name, &dpaa_dev->device, 
&init_params);
        if (cryptodev == NULL) {
                DPAA_SEC_ERR("failed to create cryptodev vdev");
diff --git a/drivers/dma/dpaa/dpaa_qdma.c b/drivers/dma/dpaa/dpaa_qdma.c
index a695f58bc5..3e1f60eab2 100644
--- a/drivers/dma/dpaa/dpaa_qdma.c
+++ b/drivers/dma/dpaa/dpaa_qdma.c
@@ -1346,11 +1346,34 @@ dpaa_qdma_init(struct rte_dma_dev *dmadev)
        int ret;
        uint32_t i, j, k;
 
+       if (rte_eal_process_type() != RTE_PROC_PRIMARY)
+               return -ENOTSUP;
+
        if (dpaa_get_devargs(dmadev->device->devargs, DPAA_DMA_ERROR_CHECK)) {
                s_hw_err_check = true;
                DPAA_QDMA_INFO("Enable DMA error checks");
        }
 
+       if (getenv("DPAA_QDMA_DATA_VALIDATION"))
+               s_data_validation = 1;
+
+       if (getenv("DPAA_QDMA_HW_ERR_CHECK"))
+               s_hw_err_check = 1;
+
+       char *penv = getenv("DPAA_QDMA_SG_ENABLE");
+       if (penv)
+               s_sg_enable = atoi(penv);
+
+       penv = getenv("DPAA_QDMA_SG_MAX_ENTRY_SIZE");
+       if (penv)
+               s_sg_max_entry_sz = atoi(penv);
+
+#ifdef RTE_DMA_DPAA_ERRATA_ERR050757
+       penv = getenv("DPAA_QDMA_PCI_READ");
+       if (penv)
+               s_pci_read = atoi(penv);
+#endif
+
        fsl_qdma->n_queues = QDMA_QUEUES * QDMA_BLOCKS;
        fsl_qdma->num_blocks = QDMA_BLOCKS;
        fsl_qdma->block_offset = QDMA_BLOCK_OFFSET;
diff --git a/drivers/net/dpaa/dpaa_ethdev.c b/drivers/net/dpaa/dpaa_ethdev.c
index b8d1ec8cbb..8b58e16ec5 100644
--- a/drivers/net/dpaa/dpaa_ethdev.c
+++ b/drivers/net/dpaa/dpaa_ethdev.c
@@ -2685,6 +2685,9 @@ rte_dpaa_remove(struct rte_dpaa_device *dpaa_dev)
 
        PMD_INIT_FUNC_TRACE();
 
+       if (rte_eal_process_type() != RTE_PROC_PRIMARY)
+               return 0;
+
        eth_dev = rte_eth_dev_allocated(dpaa_dev->device.name);
        dpaa_eth_dev_close(eth_dev);
        ret = rte_eth_dev_release_port(eth_dev);
-- 
2.25.1

Reply via email to