Re: [PATCH] Add no_addrs field in srp_addr structure

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

 



On 03/20/2012 07:35 AM, Jan Friesse wrote:
> This should allow us future change to dynamic number of rings without
> breaking wire compatibility.
> 
> Signed-off-by: Jan Friesse <jfriesse@xxxxxxxxxx>
> ---
>  exec/totemsrp.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/exec/totemsrp.c b/exec/totemsrp.c
> index c262d98..dc93cc2 100644
> --- a/exec/totemsrp.c
> +++ b/exec/totemsrp.c
> @@ -153,6 +153,7 @@ enum encapsulation_type {
>   * New membership algorithm local variables
>   */
>  struct srp_addr {
> +	uint8_t no_addrs;
>  	struct totem_ip_address addr[INTERFACE_MAX];
>  };
>  
> @@ -677,6 +678,8 @@ static void totemsrp_instance_initialize (struct totemsrp_instance *instance)
>  	instance->orf_token_discard = 0;
>  
>  	instance->commit_token = (struct memb_commit_token *)instance->commit_token_storage;
> +
> +	instance->my_id.no_addrs = INTERFACE_MAX;
>  }
>  
>  static void main_token_seqid_get (

Did you test that all received messages that use the srp_addr struct
have no_addrs set to 2?  If so, then

Reviewed-by: Steven Dake <sdake@xxxxxxxxxx>

Regards
-steve
_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss


[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux