diff mbox series

[ovs-dev] ovn-controller: pending_ct_zones should be destroyed

Message ID 1506317056-13394-1-git-send-email-xu.rong@zte.com.cn
State Accepted
Delegated to: Russell Bryant
Headers show
Series [ovs-dev] ovn-controller: pending_ct_zones should be destroyed | expand

Commit Message

xurong00037997 Sept. 25, 2017, 5:24 a.m. UTC
pending_ct_zones in ovn-controller main should be destroyed when exit.

Signed-off-by: xu rong <xu.rong@zte.com.cn>
---
 ovn/controller/ovn-controller.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Miguel Angel Ajo Sept. 25, 2017, 9:01 a.m. UTC | #1
Acked-by: Miguel Angel Ajo <majopela@redhat.com>

(Somehow I thought I had acked this, but apparently I only checked it was
ok and never responded)

On Mon, Sep 25, 2017 at 7:24 AM, 00037997 <xu.rong@zte.com.cn> wrote:

> pending_ct_zones in ovn-controller main should be destroyed when exit.
>
> Signed-off-by: xu rong <xu.rong@zte.com.cn>
> ---
>  ovn/controller/ovn-controller.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/ovn/controller/ovn-controller.c b/ovn/controller/ovn-
> controller.c
> index a935a79..32cdc9f 100644
> --- a/ovn/controller/ovn-controller.c
> +++ b/ovn/controller/ovn-controller.c
> @@ -843,6 +843,7 @@ main(int argc, char *argv[])
>      pinctrl_destroy();
>
>      simap_destroy(&ct_zones);
> +    shash_destroy(&pending_ct_zones);
>
>      bitmap_free(group_table.group_ids);
>      hmap_destroy(&group_table.desired_groups);
> --
> 1.8.3.1
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Russell Bryant Sept. 25, 2017, 1 p.m. UTC | #2
On Mon, Sep 25, 2017 at 5:01 AM, Miguel Angel Ajo Pelayo
<majopela@redhat.com> wrote:
> Acked-by: Miguel Angel Ajo <majopela@redhat.com>
>
> (Somehow I thought I had acked this, but apparently I only checked it was
> ok and never responded)

You had acked a previous version of the patch.

I applied this to master and also updated the AUTHORS file.  Thanks!
Miguel Angel Ajo Sept. 25, 2017, 1:45 p.m. UTC | #3
ooh thanks ':D

On Mon, Sep 25, 2017 at 3:00 PM, Russell Bryant <russell@ovn.org> wrote:

> On Mon, Sep 25, 2017 at 5:01 AM, Miguel Angel Ajo Pelayo
> <majopela@redhat.com> wrote:
> > Acked-by: Miguel Angel Ajo <majopela@redhat.com>
> >
> > (Somehow I thought I had acked this, but apparently I only checked it was
> > ok and never responded)
>
> You had acked a previous version of the patch.
>
> I applied this to master and also updated the AUTHORS file.  Thanks!
>
diff mbox series

Patch

diff --git a/ovn/controller/ovn-controller.c b/ovn/controller/ovn-controller.c
index a935a79..32cdc9f 100644
--- a/ovn/controller/ovn-controller.c
+++ b/ovn/controller/ovn-controller.c
@@ -843,6 +843,7 @@  main(int argc, char *argv[])
     pinctrl_destroy();
 
     simap_destroy(&ct_zones);
+    shash_destroy(&pending_ct_zones);
 
     bitmap_free(group_table.group_ids);
     hmap_destroy(&group_table.desired_groups);