Fixes a problem where there are gaps in the recovery queue. Example my_aru = 5, but there are messages at 7,8. 8 = my_high_seq_received which results in data slots taken up in new message queue. What should really happen is these last messages should be delivered after a transitional configuration to maintain SAFE agreement. We don't have support for SAFE atm, so it is probably safe just to throw these messages away. Without this change, the new message queue on a new configuraton change is out of sync.
Signed-off-by: Steven Dake <[email protected]> Tested-by: Tim Beale <[email protected]> --- exec/totemsrp.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/exec/totemsrp.c b/exec/totemsrp.c index 3dcc05e..16de74d 100644 --- a/exec/totemsrp.c +++ b/exec/totemsrp.c @@ -1809,7 +1809,7 @@ static void memb_state_operational_enter (struct totemsrp_instance *instance) sizeof (struct srp_addr) * instance->my_memb_entries); instance->my_failed_list_entries = 0; - instance->my_high_delivered = instance->my_aru; + instance->my_high_delivered = instance->my_high_seq_received; for (i = 0; i <= instance->my_high_delivered; i++) { void *ptr; -- 1.7.4.4 _______________________________________________ Openais mailing list [email protected] https://lists.linux-foundation.org/mailman/listinfo/openais
