The check of 'RTE_ETH_DEV_SWITCH_DOMAIN_ID_INVALID' cause the following call of 'rte_eth_switch_domain_alloc()' never trigger.
Fix this by just remove the check logic. Fixes: e1124c4f8a45 ("net/nfp: add flower representor framework") Cc: sta...@dpdk.org Signed-off-by: Chaoyong He <chaoyong...@corigine.com> Reviewed-by: Long Wu <long...@corigine.com> Reviewed-by: Peng Zhang <peng.zh...@corigine.com> --- drivers/net/nfp/flower/nfp_flower_representor.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/net/nfp/flower/nfp_flower_representor.c b/drivers/net/nfp/flower/nfp_flower_representor.c index 700957f945..e7550ce9ef 100644 --- a/drivers/net/nfp/flower/nfp_flower_representor.c +++ b/drivers/net/nfp/flower/nfp_flower_representor.c @@ -907,10 +907,9 @@ nfp_flower_repr_create(struct nfp_app_fw_flower *app_fw_flower, pci_dev = pf_dev->pci_dev; /* Allocate a switch domain for the flower app */ - if (app_fw_flower->switch_domain_id == RTE_ETH_DEV_SWITCH_DOMAIN_ID_INVALID && - rte_eth_switch_domain_alloc(&app_fw_flower->switch_domain_id) != 0) { + ret = rte_eth_switch_domain_alloc(&app_fw_flower->switch_domain_id); + if (ret != 0) PMD_INIT_LOG(WARNING, "failed to allocate switch domain for device"); - } /* Now parse PCI device args passed for representor info */ if (pci_dev->device.devargs != NULL) { -- 2.39.1