[ovs-dev] [PATCH] ofproto-dpif: Reconfigure when IPFIX is enabled/disabled.

Justin Pettit jpettit at nicira.com
Wed Jun 19 07:32:39 UTC 2013


The presense or absence of IPFIX affects wildcarding, and there needs to
cause a revalidation when changed.

Found by inspecting Ethan's similar patch on NetFlow.

Signed-off-by: Justin Pettit <jpettit at nicira.com>
---
Since I just fixed IPFIX wildcarding in the previous patch, I'll
probably fold this patch into that one.
---
 ofproto/ofproto-dpif.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index 5766861..0d7481d 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -1633,12 +1633,14 @@ set_ipfix(
     if (bridge_exporter_options || flow_exporters_options) {
         if (!di) {
             di = ofproto->ipfix = dpif_ipfix_create();
+            ofproto->backer->need_revalidate = REV_RECONFIGURE;
         }
         dpif_ipfix_set_options(
             di, bridge_exporter_options, flow_exporters_options,
             n_flow_exporters_options);
     } else {
         if (di) {
+            ofproto->backer->need_revalidate = REV_RECONFIGURE;
             dpif_ipfix_destroy(di);
             ofproto->ipfix = NULL;
         }
-- 
1.7.5.4




More information about the dev mailing list