Reviewed-by: Jan Friesse <[email protected]>

Steven Dake napsal(a):
> 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;

_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to