Re: [ovs-dev] [PATCH] ofproto-dpif-upcall: prevent memory leak on log message
On Wed, Jun 08, 2016 at 09:58:51AM -0700, Joe Stringer wrote: > On 8 June 2016 at 09:04, Thadeu Lima de Souza Cascardo >wrote: > > When DPIF does not support UFID (like old kernels), it may print this > > message > > quite frequently, if using an OVS version that does not include the > > upstream fix > > af50de8 ("ofproto-dpif-upcall: Pass key to dpif_flow_get()."). > > > > Signed-off-by: Thadeu Lima de Souza Cascardo > > Fixes: af50de8 ("ofproto-dpif-upcall: Pass key to dpif_flow_get().") > > Thanks for the fix. > > It seems to me that af50de8 ("ofproto-dpif-upcall: Pass key to > dpif_flow_get().") makes this problem less likely, while 64bb477f0568 > ("dpif: Minimize memory copy for revalidation.") is the commit which > actually introduced the bug. I updated the "Fixes:" tag and used this > commit to determine which branches the fix should apply to. > > Applied to master, branch-2.5 and branch-2.4. Yes, I meant 64bb477f0568. My bad while copy and pasting the commit reference. Thanks. Cascardo. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
Re: [ovs-dev] [PATCH] ofproto-dpif-upcall: prevent memory leak on log message
On 8 June 2016 at 09:04, Thadeu Lima de Souza Cascardowrote: > When DPIF does not support UFID (like old kernels), it may print this message > quite frequently, if using an OVS version that does not include the upstream > fix > af50de8 ("ofproto-dpif-upcall: Pass key to dpif_flow_get()."). > > Signed-off-by: Thadeu Lima de Souza Cascardo > Fixes: af50de8 ("ofproto-dpif-upcall: Pass key to dpif_flow_get().") Thanks for the fix. It seems to me that af50de8 ("ofproto-dpif-upcall: Pass key to dpif_flow_get().") makes this problem less likely, while 64bb477f0568 ("dpif: Minimize memory copy for revalidation.") is the commit which actually introduced the bug. I updated the "Fixes:" tag and used this commit to determine which branches the fix should apply to. Applied to master, branch-2.5 and branch-2.4. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
[ovs-dev] [PATCH] ofproto-dpif-upcall: prevent memory leak on log message
When DPIF does not support UFID (like old kernels), it may print this message quite frequently, if using an OVS version that does not include the upstream fix af50de8 ("ofproto-dpif-upcall: Pass key to dpif_flow_get()."). Signed-off-by: Thadeu Lima de Souza CascardoFixes: af50de8 ("ofproto-dpif-upcall: Pass key to dpif_flow_get().") --- ofproto/ofproto-dpif-upcall.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ofproto/ofproto-dpif-upcall.c b/ofproto/ofproto-dpif-upcall.c index 1374950..a18fc5a 100644 --- a/ofproto/ofproto-dpif-upcall.c +++ b/ofproto/ofproto-dpif-upcall.c @@ -2056,6 +2056,7 @@ log_unexpected_flow(const struct dpif_flow *flow, int error) "unexpected flow (%s): ", ovs_strerror(error)); odp_format_ufid(>ufid, ); VLOG_WARN_RL(, "%s", ds_cstr()); +ds_destroy(); } static void -- 2.5.5 ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev