From 7bab1576309d18850d9cb07a3397476010831513 Mon Sep 17 00:00:00 2001 From: Ethan Jackson Date: Tue, 18 Jun 2013 18:03:32 -0700 Subject: [PATCH] ofproto-dpif: Reconfigure when netflow is enabled/disabled. The presence or absence of netflow affects wildcarding, and therefore need to cause a revalidation when changed. Found by inspection. Signed-off-by: Ethan Jackson Acked-by: Ben Pfaff --- ofproto/ofproto-dpif.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index 1b3b1e3bb..52d22774d 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto/ofproto-dpif.c @@ -5614,13 +5614,16 @@ set_netflow(struct ofproto *ofproto_, if (netflow_options) { if (!ofproto->netflow) { ofproto->netflow = netflow_create(); + ofproto->backer->need_revalidate = REV_RECONFIGURE; } return netflow_set_options(ofproto->netflow, netflow_options); - } else { + } else if (ofproto->netflow) { + ofproto->backer->need_revalidate = REV_RECONFIGURE; netflow_destroy(ofproto->netflow); ofproto->netflow = NULL; - return 0; } + + return 0; } static void -- 2.43.0