[dpdk-dev] [PATCH 0/3] remove RTE_EAL_UNBIND_PORTS related code

2014-04-15 Thread Thomas Monjalon
Hi Anatoly, 2014-04-14 14:39, Burakov, Anatoly: > > This functionality is at least used by virtio-net-pmd. > > So we cannot remove this without a fix for virtio-net-pmd. > > It appears that virtio-net-pmd hasn't been merged in yet? At least I can't > see it in the git tree for 1.6 release. Maybe

[dpdk-dev] [PATCH 0/3] remove RTE_EAL_UNBIND_PORTS related code

2014-04-15 Thread Burakov, Anatoly
Hi Thomas, David, Thanks for your detailed explanations and insights, I now have a better picture. I still have to do a lot of catch up on dpdk.org tree and ecosystem, so please forgive my ignorance about some dpdk.org-related subjects at times :-) > Do you agree that the David's version is a g

[dpdk-dev] [PATCH 0/3] remove RTE_EAL_UNBIND_PORTS related code

2014-04-14 Thread David Marchand
On 04/14/2014 03:12 PM, Burakov, Anatoly wrote: >> For now, I have a problem with your patch as it breaks drivers that use >> RTE_PCI_DRV_FORCE_UNBIND flag, since it removes the part where this flag >> is handled. Besides, I can see some remaining parts mentioning >> pci_switch_module() in comment

[dpdk-dev] [PATCH 0/3] remove RTE_EAL_UNBIND_PORTS related code

2014-04-14 Thread David Marchand
Hello Anatoly, I think we might want to start a discussion on what should be done by the eal and what should be done by the drivers themselves. But this can be done later. For now, I have a problem with your patch as it breaks drivers that use RTE_PCI_DRV_FORCE_UNBIND flag, since it removes the

[dpdk-dev] [PATCH 0/3] remove RTE_EAL_UNBIND_PORTS related code

2014-04-14 Thread Burakov, Anatoly
Hi David, > This functionality is at least used by virtio-net-pmd. > So we cannot remove this without a fix for virtio-net-pmd. It appears that virtio-net-pmd hasn't been merged in yet? At least I can't see it in the git tree for 1.6 release. > Yet, even if we remove this from the eal, I think

[dpdk-dev] [PATCH 0/3] remove RTE_EAL_UNBIND_PORTS related code

2014-04-14 Thread Burakov, Anatoly
Hi David, > For now, I have a problem with your patch as it breaks drivers that use > RTE_PCI_DRV_FORCE_UNBIND flag, since it removes the part where this flag > is handled. Besides, I can see some remaining parts mentioning > pci_switch_module() in comments. I wonder if this functionality is need