On Wed, Jul 29, 2015 at 5:55 PM, Daniel Borkmann <dan...@iogearbox.net> wrote: > Hm, so this seems not correct: if we only ever increase tcfc_refcnt > when there's bind=1, and only ever decrease when bind=1, then we > will never free the action as we do start out from ref=1 in case > it has been added without initial binding, if I see this correctly. >
Right, I think your patch should be fine for net. The code is kinda messy, but we can always clean up the logic for net-next. Reviewed-by: Cong Wang <cw...@twopensource.com> (It looks like mirred doesn't handle bind == true case correctly, I will send a separated for it after your patch.) Thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
