Ack, code review only.
Thanks,
Praveen
On 29-Jun-16 6:17 AM, Minh Hon Chau wrote:
> osaf/services/saf/amf/amfd/ndfsm.cc | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>
> If more than 2 payloads are joining from headless, amfd will think all
> nodes have already been synced even there's still one being in headless
> sync period.
>
> The patch corrects the conditional statement that fixes the size of nodes
> being synchronized.
>
> diff --git a/osaf/services/saf/amf/amfd/ndfsm.cc
> b/osaf/services/saf/amf/amfd/ndfsm.cc
> --- a/osaf/services/saf/amf/amfd/ndfsm.cc
> +++ b/osaf/services/saf/amf/amfd/ndfsm.cc
> @@ -298,7 +298,7 @@ void avd_node_up_evh(AVD_CL_CB *cb, AVD_
> uint32_t rc_node_up;
> avnd->node_up_msg_count++;
> rc_node_up = avd_count_node_up(cb);
> - if (rc_node_up == sync_nd_size-1) {
> + if (rc_node_up == sync_nd_size) {
> if (cb->node_sync_tmr.is_active) {
> avd_stop_tmr(cb, &cb->node_sync_tmr);
> TRACE("stop NodeSync timer");
>
------------------------------------------------------------------------------
Attend Shape: An AT&T Tech Expo July 15-16. Meet us at AT&T Park in San
Francisco, CA to explore cutting-edge tech and listen to tech luminaries
present their vision of the future. This family event has something for
everyone, including kids. Get more information and register today.
http://sdm.link/attshape
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel
