Mengxin Liu <[email protected]> writes:

> Signed-off-by: Mengxin Liu <[email protected]>
> ---
>  ofproto/ofproto.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index 3a527683c..7c6cb1c56 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -2429,7 +2429,7 @@ static void
>  dealloc_ofp_port(struct ofproto *ofproto, ofp_port_t ofp_port)
>  {
>      if (ofp_to_u16(ofp_port) < ofproto->max_ports) {
> -        ofport_set_usage(ofproto, ofp_port, time_msec());
> +        ofport_remove_usage(ofproto, ofp_port);
>      }
>  }
>
> --

This interferes with ofp port recycling code.  Can you describe the
leak?  I don't really understand where / what the leak is, and the
commit message is very sparse here.  Please give details under when the
memory leaks, how you found it, and whether/how this changes the
recycling code.

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

Reply via email to