Jarno Rajahalme <ja...@ovn.org> writes:

> diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
> index 83dcc9c..5b42b7e 100644
> --- a/ofproto/ofproto-dpif.c
> +++ b/ofproto/ofproto-dpif.c
> @@ -864,7 +864,7 @@ close_dpif_backer(struct dpif_backer *backer)
>      free(backer->type);
>      free(backer->dp_version_string);
>      dpif_close(backer->dpif);
> -    free(backer);
> +    ovsrcu_postpone(free, backer);
>  }
>  
>  /* Datapath port slated for removal from datapath. */
>
> Maybe this would solve the problem you found?

It should, and as a bonus, we don't really mind leaking a bit of memory
if the process is about to exit anyway, so this is a much better
solution.

Thanks,
Petr
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to