>From: [email protected] [mailto:[email protected]] >On Behalf Of Ilya Maximets >Sent: Tuesday, December 12, 2017 8:41 AM >To: [email protected]; Stokes, Ian <[email protected]>; Ben Pfaff ><[email protected]> >Subject: Re: [ovs-dev] OVS DPDK: dpdk_merge pull request > >Hi Ian and Ben. > >One more thing that is broken is the travis build. >I've send a patch-set to fix it here: > https://mail.openvswitch.org/pipermail/ovs-dev/2017-December/342005.html
Thanks for this Ilya. The DPDK v17.11 upgrade patch failed to remove once instance of the word 'stable', since I simply performed a 'find-and-replace' in most instances for '17.05', removing the reference to 'stable' where the latter was found. However, the travis script in question uses parameter substitution, hence it was missed in the 17.11 patch - apologies for this oversight on my part. I fully agree with Ilya's proposal to enable travis on the dpdk_merge branch, which should help identify such errors prior to mainline merge in future. Thanks and best, Mark > >My general suggestion is to enable travis builds for 'dpdk_merge' branch and >wait >for successful build before making the pull request. What do you think? > >Best regards, Ilya Maximets. > >> Hi Ben, >> >> The following changes since commit 65d9759c4fc433dbda89ad3d7225c1f5eac274ca: >> >> ovsdb-data: Add OVS_WARN_UNUSED_RESULT annotations to function >definitions. (2017-12-08 13:39:29 -0800) >> >> are available in the git repository at: >> >> https://github.com/istokes/ovs dpdk_merge >> >> for you to fetch changes up to 3eb8d4fa0db3159a8ffc8f52223417b3417263b3: >> >> netdev-dpdk: extend netdev_dpdk_get_status to include if_type and if_descr >(2017-12-08 21:42:54 +0000) >> >> ---------------------------------------------------------------- >> Ilya Maximets (3): >> netdev-dpdk: Fix variables naming in set_admin_state function. >> netdev-dpdk: Add comment about variables naming convention. >> Revert "dpif_netdev: Refactor dp_netdev_pmd_thread structure." >> >> Kevin Traynor (4): >> netdev-dpdk: Remove uneeded call to rte_eth_dev_count(). >> dpif-netdev: Add port/queue tiebreaker to rxq_cycle_sort. >> dpif-netdev: Rename rxq_cycle_sort to compare_rxq_cycles. >> dpif-netdev: Calculate rxq cycles prior to compare_rxq_cycles calls. >> >> Mark Kavanagh (2): >> netdev-dpdk: DPDK v17.11 upgrade >> netdev-dpdk: vHost IOMMU support >> >> Michal Weglicki (1): >> netdev-dpdk: extend netdev_dpdk_get_status to include if_type and >if_descr >> >> Yifeng Sun (1): >> dpif-netdev: Fix memory leak >> >> .travis/linux-build.sh | 2 +- >> Documentation/faq/releases.rst | 1 + >> Documentation/intro/install/dpdk.rst | 10 +++++----- >> Documentation/topics/dpdk/ring.rst | 2 +- >> Documentation/topics/dpdk/vhost-user.rst | 60 >+++++++++++++++++++++++++++++++++++++++++++++++++++----- >> NEWS | 3 +++ >> lib/dpdk-stub.c | 6 ++++++ >> lib/dpdk.c | 12 ++++++++++++ >> lib/dpdk.h | 3 +++ >> lib/dpif-netdev.c | 202 >++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ >++++++++++++------------------------------------------------------------------ >-------------------------------- >> lib/netdev-dpdk.c | 70 >+++++++++++++++++++++++++++++++++++++++++++++++------------------ >> vswitchd/vswitch.xml | 79 >++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ >> 12 files changed, 314 insertions(+), 136 deletions(-) >> >> Thanks >> Ian > >_______________________________________________ >dev mailing list >[email protected] >https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
