[ovs-build] Passed: openvswitch/ovs#538 (master - 4c75aaa)
notifications at travis-ci.org
Thu Jan 8 02:40:07 UTC 2015
Build Update for openvswitch/ovs
Duration: 21 minutes and 54 seconds
Commit: 4c75aaa (master)
Author: Ethan Jackson
Message: dpif-netdev: Fix rare flow add race condition.
Before this patch, dp_netdev_flow_add() inserted newly minted flows in
the "flow_table" cmap before inserting them into the per core "dpcls"
classifier. Since dpcls_insert() initializes 'flow->cr.mask', there's
a brief window where the flow is accessible from the cmap, but has a
bogus mask value.
In my testing, under rare instances (i.e. once every 20 minutes with a
very specific flow table and traffic pattern), revalidators core dump
when they call dpif_netdev_flow_dump_next(), which accesses this bogus
mask value from dp_netdev_flow_to_dpif_flow().
By inserting into the per core classifier before the cmap, all the
values are guaranteed to be initialized during flow dumps. With this
patch, I can no longer reproduce the crash.
Signed-off-by: Ethan Jackson <ethan at nicira.com>
Acked-by: Jarno Rajahalme <jrajahalme at nicira.com>
View the changeset: https://github.com/openvswitch/ovs/compare/8ae8176fd0d8...4c75aaabb153
View the full build log and details: https://travis-ci.org/openvswitch/ovs/builds/46270410
You can configure recipients for build notifications in your .travis.yml file. See http://docs.travis-ci.com/user/notifications
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the build