Hi Aaron I refreshed my stack that combines your current series with the vhost-user permission/ownership work of december.
quilt applied patches/ovs-dev-v7-1-5-netdev-dpdk-Restore-thread-affinity-after-DPDK-init.patch patches/ovs-dev-v7-2-5-netdev-dpdk-Convert-initialization-from-cmdline-to-db.patch patches/ovs-dev-v7-3-5-netdev-dpdk-Autofill-lcore-coremask-if-absent.patch patches/ovs-dev-v7-4-5-netdev-dpdk-Allow-arbitrary-eal-arguments.patch patches/ovs-dev-v7-5-5-NEWS-Announce-the-DPDK-EAL-configuration-change.patch patches/ovs-dev-4-5-lib-daemon-Move-the-user-group-code-up-one-level.patch patches/ovs-dev-5-5-netdev-dpdk-Support-user-cfg-vhost-socket-perms.patch It passes all the tests I had yesterday dpdk|INFO|User-provided vhost-sock-dir in use: /usr/local/var/run/openvswitch-vhost dpdk|INFO|User-provided vhost_sock_owners in use: :kvm dpdk|INFO|User-provided vhost_sock_permissions in use: 0660 dpdk|INFO|Socket /usr/local/var/run/openvswitch-vhost/vhost-user-1 created for vhost-user port vhost-user-1 dpdk|INFO|Socket /usr/local/var/run/openvswitch-vhost/vhost-user-1 changed permissions to 0660 dpdk|INFO|Socket /usr/local/var/run/openvswitch-vhost/vhost-user-1 changed ownership to :kvm So the issue with dpdk-alloc-mem is fixed - thanks. The remaining issue with the range check for the permissions doesn't belong to this series :-) While I agree to the communities assessment in December that they should be discussed split, so the second series doesn't stall the former one. I'd consider it great to submit them always together, but as two series. So one 0/5 and one 0/2. That way we keep the people aware and avoid people doing the same work over and over again (me and just today e.g. by Robert Wojciechowicz) It would simplify testing both, but still keep discussions as separate as needed. I might seem to miss something obvious (feels that way), but fyi with the series various parts of "make check" fail. After realizing this I more or less blindly ran a loop of "quilt push; make -j12 && make check TESTSUITEFLAGS=-j12" to see which of the patches triggers this behavior. It is #2 "ovs-dev-v7-2-5-netdev-dpdk-Convert-initialization-from-cmdline-to-db.patch" A lot of tests report fail, and it keeps "hanging" since some seem no more to end correctly. Here a log of my "make check TESTSUITEFLAGS=-j12" http://paste.ubuntu.com/14688004/ I didn't look into it in detail yet - is that a real issue, or am I missing just some sort of DB upgrade or so? Christian Ehrhardt Software Engineer, Ubuntu Server Canonical Ltd On Wed, Jan 27, 2016 at 10:33 PM, Aaron Conole <acon...@redhat.com> wrote: > Currently, configuration of DPDK parameters is done via the command line > through a --dpdk **OPTIONS** -- command line argument. This has a number of > challenges, including: > * It must be the first option passed to ovs-vswitchd > * It breaks from the way most other things are configured in OVS > * It doesn't allow an easy way to populate defaults > > > This series brings the following changes to openvswitch: > * All DPDK options are taken from the ovs database rather than the > command line > * DPDK lcores are optionally auto-assigned to a single core based on the > bridge coremask. > * Updated documentation > > v2: > * Dropped the vhost-user socket configuration options. Those can be > re-added > as an extension > * Incorporated feedback from Kevin Traynor. > > v3: > * Went back to a global dpdk-init > * Language cleanup and various minor fixes > > v4: > * Added a way to pass arbitrary eal arguments > > v5: > * Restore the socket-mem default, and fix up the ovs-dev.py script, along > with the manpage for ovsdb-server > > v6: > * Correct a documentation issue with INSTALL.DPDK.md > * Correct a non-dpdk enabled OVS incorrect warning variable > * Remove an excess whitespace > > v7: > * After testing by Christian with dpdk-alloc-mem > > Aaron Conole (5): > netdev-dpdk: Restore thread affinity after DPDK init > netdev-dpdk: Convert initialization from cmdline to db > netdev-dpdk: Autofill lcore coremask if absent > netdev-dpdk: Allow arbitrary eal arguments > NEWS: Announce the DPDK EAL configuration change > > FAQ.md | 6 +- > INSTALL.DPDK.md | 90 ++++++++++--- > NEWS | 5 + > lib/netdev-dpdk.c | 327 > ++++++++++++++++++++++++++++++++++++++------- > lib/netdev-dpdk.h | 22 ++- > utilities/ovs-dev.py | 7 +- > vswitchd/bridge.c | 3 + > vswitchd/ovs-vswitchd.8.in | 5 +- > vswitchd/ovs-vswitchd.c | 25 +--- > vswitchd/vswitch.xml | 128 +++++++++++++++++- > 10 files changed, 513 insertions(+), 105 deletions(-) > > -- > 2.5.0 > > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
