Red Hat Bugzilla – Bug 1268412
Please pull patches for Fedora DPDK package
Last modified: 2015-10-07 09:59:04 EDT
Git address: git://bytheb.org/dpdk.git
All patches on mainline; SHA sums as follows:
I just noticed something while backporting this. One of your commits (aee079) modifies the config file in such a way that kni and igb_uio kernel modules are built. I understand that we're not including those in the package, but we are including the pmd that uses it with this change, which means that pmd will not be functional. That will need to be backed out, both from Fedora and RHEL.
Apologies, I must have done something wrong then.
Just so I don't get any modifications incorrect, on master of that repository, commit ID 549fcba... is the commit which is being discussed (the one you reference is on the Proposal_for_Neil branch, which I've deleted). It is almost completely identical to the aee079c: I added acked-by lines to the commit message.
As for the PMD, is that the IGB and EM PMDs? Will we need to enable the IGB_UIO kmod for this set of changes? I'll do some additional investigation and exchange mails, thanks.
hmm, let me look closer, the commit I referenced is accurate (I didn't do individual commits, I just merged the Proposal for Neil branch, which appears to not be deleted, as I have access to it from a fresh repository. Perhaps you forgot to push the delete?).
Regardless, I'm referring to the igb_uio.ko module, which I think is required for the em pmd to work properly (the em pmd I think encompasses both the e1000 and igb hardware).
Huh? The last version where igb_uio.ko was required for anything was DPDK 1.7,
since then VFIO support has been available and since 2.0 there's also generic PCI UIO. That half the PMDs are disabled in the Fedora package is just a remnant from the 1.7 days (commit fe6107da05c0db442c7ace1e87b1520f30aded67)
ok, apologies, it looks like the previous branch I pulled was doing something flaky and allowed the uio code to build. Pulling the master branch (as indicated by comment 2) resolves the issue. I'll commit this today. Thanks!