Previously, dpif_offload_set_config() was skipped if the dpif
provider did not implement the corresponding class function.
This was incorrect: the offload configuration function should be
called regardless of whether the dpif-provider-specific function
exists.

Fixes: bd6543466dc6 ("dpif-offload: Add offload provider set_config API.")
Signed-off-by: Eelco Chaudron <[email protected]>
---
 lib/dpif.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/dpif.c b/lib/dpif.c
index 88b5edfad5..cab5884254 100644
--- a/lib/dpif.c
+++ b/lib/dpif.c
@@ -1609,8 +1609,8 @@ dpif_set_config(struct dpif *dpif, const struct smap *cfg)
         if (error) {
             log_operation(dpif, "set_config", error);
         }
-        dpif_offload_set_config(dpif, cfg);
     }
+    dpif_offload_set_config(dpif, cfg);
 
     return error;
 }
-- 
2.52.0

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to