Bug 6
Summary: | dpdk-pdump leads to ovs-vswitchd crash | ||
---|---|---|---|
Product: | DPDK | Reporter: | wangzhike |
Component: | other | Assignee: | Vipin Varghese (vipin.varghese) |
Status: | RESOLVED INVALID | ||
Severity: | normal | CC: | ajit.khaparde, david.marchand, qian.q.xu, vipin.varghese, zhongdahulinfan |
Priority: | Normal | ||
Version: | 17.11 | ||
Target Milestone: | --- | ||
Hardware: | All | ||
OS: | All |
Description
wangzhike
2018-01-04 07:19:35 CET
hi, I have seen similar issue when running in shared library mode. The solution that worked for me is to pass same shared PMD as primary. Example- Primary: -d librte_pmd_ixgbe.so -d librte_pmd_i40e.so Secondary (PDUMP): -d librte_pmd_ixgbe.so -d librte_pmd_i40e.so Missing to pass the same drivers causes crash. A quick fix was to check either before or in rte_eth_dev_attch the device count. Possible Fix: --- a/lib/librte_ether/rte_ethdev.c +++ b/lib/librte_ether/rte_ethdev.c @@ -376,6 +376,12 @@ rte_eth_dev_attach(const char *devargs, uint16_t *port_id) goto err; } + if (current <= 0) { + RTE_LOG(ERR, EAL, "No port found for device (%s)\n", name); + ret = -EINVAL; + goto err; + } + /* parse devargs, then retrieve device name and args */ if (rte_eal_parse_devargs_str(devargs, &name, &args)) goto err; @Vipin, could you submit the patch? thanks. Vipin, is this still an issue? Thanks Hi Ajith, I think you have mistakenly updated with me as assignee. The right owner is `Wang Zhike`. So humbly requesting for doing the needful. Vipin, I asked you because you said you saw a similar issue. Will check with Wang as well. Thanks Ajitm sure no issues. The suggestion was shared on DPDK 17.11. I have not notices any issues from 18.11 LTS onwards. Ajitm sure no issues. The suggestion was shared on DPDK 17.11. I have not notices any issues from 18.11 LTS onwards. This problem saw no activity for more than a year and is probably a configuration issue. Besides, note that OVS dropped pdump support in favor of its own packet capture mechanism. So closing this bz. Feel free to reopen if you think there is still an issue. |