I sent a fix: http://openvswitch.org/pipermail/dev/2014-December/049412.html
There's still another, more rare race condition with the controller role test but I don't think it's related to these changes and has most likely been possible for quite some time. I don't have a good solution for that one, so I'm inclined to defer it and only address it if it becomes more problematic. On 2 December 2014 at 18:50, Joe Stringer <[email protected]> wrote: > This seems to be some race condition that sometimes occurs now that ukey > entries are created in handler threads. I've got a solution in mind, but > I'd like to run an overnight test to check that it works reliably. > > On 2 December 2014 at 15:34, Travis CI <[email protected]> > wrote: > >> *openvswitch / ovs >> <http://clicks.travis-ci.org/track/click/30007208/travis-ci.org?p=eyJzIjoiQ24xU2tvRFp5TzRUcXo2cnBqY3lnbm1SN3ZZIiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwczpcXFwvXFxcL3RyYXZpcy1jaS5vcmdcXFwvb3BlbnZzd2l0Y2hcXFwvb3ZzXCIsXCJpZFwiOlwiZDAxOWQ5ZDUwNWZkNGVkM2JlZDVhNGZjZTlhYTA4ODlcIixcInVybF9pZHNcIjpbXCJjMDViODE3ZjVlN2ZjYTlmMTE3ZTBhOWE1OWZmMjc0OThmN2NjN2QwXCJdfSJ9>* >> (master >> <http://clicks.travis-ci.org/track/click/30007208/github.com?p=eyJzIjoiWm5ZQk1icUZqOU5rWTQtdVR6UXN5T2FGUGdBIiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwczpcXFwvXFxcL2dpdGh1Yi5jb21cXFwvb3BlbnZzd2l0Y2hcXFwvb3ZzXFxcL3RyZWVcXFwvbWFzdGVyXCIsXCJpZFwiOlwiZDAxOWQ5ZDUwNWZkNGVkM2JlZDVhNGZjZTlhYTA4ODlcIixcInVybF9pZHNcIjpbXCIzMDVjM2I2NTk5ZDZhMDI1NDk4NGQ1MGVjMGUyZTc5Y2UxZTdmODM2XCJdfSJ9> >> ) >> Build #458 was broken. >> <http://clicks.travis-ci.org/track/click/30007208/travis-ci.org?p=eyJzIjoiczdvc1dudDNnSlB3YmRwX3Nxc1FJaUhRYWU4IiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwczpcXFwvXFxcL3RyYXZpcy1jaS5vcmdcXFwvb3BlbnZzd2l0Y2hcXFwvb3ZzXFxcL2J1aWxkc1xcXC80MjgwMTY5MVwiLFwiaWRcIjpcImQwMTlkOWQ1MDVmZDRlZDNiZWQ1YTRmY2U5YWEwODg5XCIsXCJ1cmxfaWRzXCI6W1wiNWNhNzZhYzk3ZDliY2FhNjdjYjExZjRlOGRiNDdjZjc5NDg0OWE0YlwiXX0ifQ> >> 31 minutes and 39 seconds *Joe Stringer* 64bb477 >> <http://clicks.travis-ci.org/track/click/30007208/github.com?p=eyJzIjoiRXNGVWo2MFVZZGI1VXlRM2M2SXJUY1hSQTVvIiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwczpcXFwvXFxcL2dpdGh1Yi5jb21cXFwvb3BlbnZzd2l0Y2hcXFwvb3ZzXFxcL2NvbW1pdFxcXC82NGJiNDc3ZjA1Njg4ZTYxY2YxY2EwNjJlMmU0N2M2OWM1YzQ5NzM1XCIsXCJpZFwiOlwiZDAxOWQ5ZDUwNWZkNGVkM2JlZDVhNGZjZTlhYTA4ODlcIixcInVybF9pZHNcIjpbXCI1MGRiMjliMDcyM2ZhZjhjNTcwZDcyZTNjNDZmZjdkYjcwODMxYTE3XCJdfSJ9> >> Changeset >> → >> <http://clicks.travis-ci.org/track/click/30007208/github.com?p=eyJzIjoiVXlfLVpjM291M28zV2JLQ2lvb2VhOFlmSHhVIiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwczpcXFwvXFxcL2dpdGh1Yi5jb21cXFwvb3BlbnZzd2l0Y2hcXFwvb3ZzXFxcL2NvbXBhcmVcXFwvYTEyMzVlZTVmZDYzLi4uNjRiYjQ3N2YwNTY4XCIsXCJpZFwiOlwiZDAxOWQ5ZDUwNWZkNGVkM2JlZDVhNGZjZTlhYTA4ODlcIixcInVybF9pZHNcIjpbXCJhMjc2YWFhMTE5NDdiZjY4MTY1Yzg0YWQ0ZDFjZDg2Y2YwZDBjMTc2XCJdfSJ9> >> dpif: Minimize memory copy for revalidation. >> >> One of the limiting factors on the number of flows that can be supported >> in the datapath is the overhead of assembling flow dump messages in the >> datapath. This patch modifies the dpif to allow revalidators to skip >> dumping the key, mask and actions from the datapath, by making use of >> the unique flow identifiers introduced in earlier patches. >> >> For each flow dump, the dpif user specifies whether to skip these >> attributes, allowing the common case to only dump a pair of 128-bit ID >> and flow stats. With datapath support, this increases the number of >> flows that a revalidator can handle per second by 50% or more. Support >> in dpif-netdev and dpif-netlink is added in this patch; kernel support >> is left for future patches. >> >> Signed-off-by: Joe Stringer <[email protected]> >> Acked-by: Ben Pfaff <[email protected]> >> *System message:* >> >> Currently we have a long Linux build back log. We are processing them as >> fast as we can. See http://status.travis-ci.com/incidents/hlwthjm93n9n >> <http://clicks.travis-ci.org/track/click/30007208/status.travis-ci.com?p=eyJzIjoic2VDLXIyUDUybFU1NlRNaWxsX09rVG9UaVRZIiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwOlxcXC9cXFwvc3RhdHVzLnRyYXZpcy1jaS5jb21cXFwvaW5jaWRlbnRzXFxcL2hsd3Roam05M245blwiLFwiaWRcIjpcImQwMTlkOWQ1MDVmZDRlZDNiZWQ1YTRmY2U5YWEwODg5XCIsXCJ1cmxfaWRzXCI6W1wiMjdhNTVkZTUzYTA5MzYwZDcyNWNiMjVlMjFmM2YxOWIyMWRmMmRiN1wiXX0ifQ> >> >> We just switched the default image for OS X jobs to our new image >> including Xcode 6.1. See our blog post >> <http://clicks.travis-ci.org/track/click/30007208/blog.travis-ci.com?p=eyJzIjoibW96TGZUNWVZMXpnRDNueEw3cHItVU1vTXFZIiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwOlxcXC9cXFwvYmxvZy50cmF2aXMtY2kuY29tXFxcLzIwMTQtMTEtMDMteGNvZGUtNjEtYmV0YVxcXC9cIixcImlkXCI6XCJkMDE5ZDlkNTA1ZmQ0ZWQzYmVkNWE0ZmNlOWFhMDg4OVwiLFwidXJsX2lkc1wiOltcIjBhZjA2M2JmMDdhOWQ0MzZhOGRiMzdlOWY4MGQ2Njk0NDJiMmU4MzBcIl19In0> >> for a full list of changes. >> Documentation >> <http://clicks.travis-ci.org/track/click/30007208/docs.travis-ci.com?p=eyJzIjoiU3VlRUdIc3MyV1lld0xReTRNYkFtSWpWbnFRIiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwOlxcXC9cXFwvZG9jcy50cmF2aXMtY2kuY29tXCIsXCJpZFwiOlwiZDAxOWQ5ZDUwNWZkNGVkM2JlZDVhNGZjZTlhYTA4ODlcIixcInVybF9pZHNcIjpbXCI1ZmE3OTJiOGY5Nzk0YTkzNmU0OWZiNzcwNDI0NzJkNjU5N2ZmMDhjXCJdfSJ9> >> about Travis CI >> For help please join our IRC channel irc.freenode.net#travis. >> Choose who receives these build notification emails in your configuration >> file >> <http://clicks.travis-ci.org/track/click/30007208/docs.travis-ci.com?p=eyJzIjoib1JfZlU2WllUa2NYYUZVNUlLUkE2SmsxeGo4IiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwOlxcXC9cXFwvZG9jcy50cmF2aXMtY2kuY29tXFxcL3VzZXJcXFwvbm90aWZpY2F0aW9uc1wiLFwiaWRcIjpcImQwMTlkOWQ1MDVmZDRlZDNiZWQ1YTRmY2U5YWEwODg5XCIsXCJ1cmxfaWRzXCI6W1wiNGY1YzYwZDBkY2IxOGQ5ZGQzYTMwMzFkYjNjMjc1MTk4NmIyMmNhMFwiXX0ifQ>. >> >> >> *Would you like to test your private code?* >> >> Travis Pro >> <http://clicks.travis-ci.org/track/click/30007208/travis-ci.com?p=eyJzIjoiTm5tb1ppRklUc0stQWlPWWphdmNKQ0lMNVFzIiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwOlxcXC9cXFwvdHJhdmlzLWNpLmNvbT91dG1fc291cmNlPWJ1aWxkX2VtYWlsX2Zvb3RlciZ1dG1fY2FtcGFpZ249dHJhdmlzLWNpLm9yZyZ1dG1fbWVkaXVtPWVtYWlsXCIsXCJpZFwiOlwiZDAxOWQ5ZDUwNWZkNGVkM2JlZDVhNGZjZTlhYTA4ODlcIixcInVybF9pZHNcIjpbXCIyMDAwYzI0MjkxOWZkYWYzNzVlNDhhM2Q5OTM4ZGY4MDc3YzZlMTk4XCJdfSJ9> >> could be your new best friend! >> >> Travis CI is powered by >> <http://clicks.travis-ci.org/track/click/30007208/bluebox.net?p=eyJzIjoiZTRjbGtVVlByQ250Y1o0b2hqZHZEVW16WUFFIiwidiI6MSwicCI6IntcInVcIjozMDAwNzIwOCxcInZcIjoxLFwidXJsXCI6XCJodHRwOlxcXC9cXFwvYmx1ZWJveC5uZXRcIixcImlkXCI6XCJkMDE5ZDlkNTA1ZmQ0ZWQzYmVkNWE0ZmNlOWFhMDg4OVwiLFwidXJsX2lkc1wiOltcIjBmZTEzMmY2NGJkMjQ2ODhlMGM5MDM2MDcyNDA2NGY2NDRlMWVjMzFcIl19In0> >> >> _______________________________________________ >> build mailing list >> [email protected] >> http://www.openvswitch.org/mailman/listinfo/build >> >> > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
