An upcoming commit will make dpif_poll_threads_set() record the requested configuration and dpif_run() apply it, so it makes sense to change the order.
Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com> --- ofproto/ofproto-dpif.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index b963ff2..efc27a4 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto/ofproto-dpif.c @@ -536,6 +536,8 @@ type_run(const char *type) return 0; } + /* This must be called before dpif_run() */ + dpif_poll_threads_set(backer->dpif, pmd_cpu_mask); if (dpif_run(backer->dpif)) { backer->need_revalidate = REV_RECONFIGURE; @@ -564,8 +566,6 @@ type_run(const char *type) udpif_set_threads(backer->udpif, n_handlers, n_revalidators); } - dpif_poll_threads_set(backer->dpif, pmd_cpu_mask); - if (backer->need_revalidate) { struct ofproto_dpif *ofproto; struct simap_node *node; -- 2.1.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev