qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] COLO: remove the operation to clear connection


From: Darren Kenny
Subject: Re: [Qemu-devel] [PATCH] COLO: remove the operation to clear connection list
Date: Mon, 23 Oct 2017 12:25:50 +0100
User-agent: NeoMutt/20171013

That's consistent with the documented behaviour of
g_hash_table_remove_all().


  Reviewed-by: Darren Kenny <address@hidden>

Thanks,

Darren.

On Tue, Oct 24, 2017 at 03:27:59AM +0800, address@hidden wrote:
From: Zhi Yong Wu <address@hidden>

When hash table is created with g_hash_table_new_full(),
the free function has been registered. So it isn't necessary
since g_hash_table_remove_all() will call connection_destroy()
for each connection struct automatically.

Signed-off-by: Zhi Yong Wu <address@hidden>
---
net/colo.c |    6 ------
1 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/net/colo.c b/net/colo.c
index 28ce7c8..668abb7 100644
--- a/net/colo.c
+++ b/net/colo.c
@@ -197,12 +197,6 @@ Connection *connection_get(GHashTable 
*connection_track_table,
            trace_colo_proxy_main("colo proxy connection hashtable full,"
                                  " clear it");
            connection_hashtable_reset(connection_track_table);
-            /*
-             * clear the conn_list
-             */
-            while (!g_queue_is_empty(conn_list)) {
-                connection_destroy(g_queue_pop_head(conn_list));
-            }
        }

        g_hash_table_insert(connection_track_table, new_key, conn);
--
1.7.1





reply via email to

[Prev in Thread] Current Thread [Next in Thread]