[ovs-build] Fixed: openvswitch/ovs#2383 (branch-2.5 - 3b70d8f)

Travis CI builds at travis-ci.org
Wed Mar 23 02:31:35 UTC 2016


Build Update for openvswitch/ovs
-------------------------------------

Build: #2383
Status: Fixed

Duration: 22 minutes and 5 seconds
Commit: 3b70d8f (branch-2.5)
Author: Ilya Maximets
Message: netdev-dpdk: Fix crash when changing the vhost-user port.

According to netdev-provider API:
	'The "destruct" function is not allowed to fail.'

netdev-dpdk breaks this restriction for vhost-user ports.
This leads to SIGABRT or SIGSEGV in dpdk_watchdog thread
because 'dealloc' will be called anyway indifferently
to result of 'destruct'.

For example, if we call
	# ovs-vsctl set interface vhost1 ofport_request=5
while QEMU still attached, we'll get:
------------------[cut]------------------
|dpdk|ERR|Can not remove port, vhost device still attached
VHOST_CONFIG: socket created, fd:98
VHOST_CONFIG: fail to bind fd:98, remove file:/home/vhost1 and try again.
|dpdk|ERR|vhost-user socket device setup failure for socket /home/vhost1
|bridge|WARN|could not open network device vhost1 (Unknown error -1)
ovs-vswitchd(dpdk_watchdog1): lib/netdev-dpdk.c:532: ovs_mutex_lock_at()
        passed uninitialized ovs_mutex

Program received signal SIGABRT, Aborted.
------------------[cut]------------------

Fix that by removing port anyway even when guest is still
attached. Guest becomes an orphan in that case but OVS
will not crash and will continue forwarding for other ports.
VM restart required to restore connectivity.

Fixes: 58397e6c1e6c ("netdev-dpdk: add dpdk vhost-cuse ports")
Signed-off-by: Ilya Maximets <i.maximets at samsung.com>
Acked-by: Kevin Traynor <kevin.traynor at intel.com>
Acked-by: Daniele Di Proietto <diproiettod at vmware.com>

View the changeset: https://github.com/openvswitch/ovs/compare/e25f5e78e775...3b70d8fcca23

View the full build log and details: https://travis-ci.org/openvswitch/ovs/builds/117865253

--

You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.openvswitch.org/pipermail/ovs-build/attachments/20160323/b3d3bf0b/attachment-0002.html>


More information about the build mailing list