Hi Hung,

Jut trying to syncup the ligic of unsets the mIsStale flag of the clients before immnd do invoking immnd_proc_imma_discard_connection() , immnd in multiple other cases also invoking immnd_proc_imma_discard_connection() for exaple immnd_proc_imma_down() as well, in addition to
immnd_proc_imma_discard_stales() & immnd_proc_discard_other_nodes().

But after this patch immnd is unsets the mIsStale flag of the clients, olnly immnd_proc_imma_discard_stales() & immnd_proc_discard_other_nodes(),
so that the clients can be discarded.

I if releate the above logic, that immnd need to unsets the mIsStale flag of the clients only for immnd_proc_imma_discard_stales() & immnd_proc_discard_other_nodes() , because of these two function being invoked
in HEADLESS case, is my under stating correct?

-AVM

On 6/9/2017 12:10 PM, Hung Nguyen wrote:
Just like immnd_proc_imma_discard_stales(), in immnd_proc_discard_other_nodes() 
we should unset the mIsStale flag of the clients.
So that the clients can be discarded.
---
  src/imm/immnd/immnd_proc.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/src/imm/immnd/immnd_proc.c b/src/imm/immnd/immnd_proc.c
index 2bba717..22f8737 100644
--- a/src/imm/immnd/immnd_proc.c
+++ b/src/imm/immnd/immnd_proc.c
@@ -2824,6 +2824,7 @@ void immnd_proc_discard_other_nodes(IMMND_CB *cb)
        while (cl_node) {
                LOG_NO("Removing client id:%llx sv_id:%u", cl_node->imm_app_hdl,
                       cl_node->sv_id);
+               cl_node->mIsStale = false;
                osafassert(
                    immnd_proc_imma_discard_connection(cb, cl_node, true));
                osafassert(immnd_client_node_del(cb, cl_node) ==


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to