Ack with comments:

1) you have a too complicated summary of the changelog. Should be something 
like:
"amfnd: reset HC state at tmr start [#568]"

2) please provide a longer detailed description in the changelog of the issue 
and how the patch solves the problem.

For good info about changelogs see linux kernel source under 
Documentation/development-process/5.Posting chapter 5.4

Thanks,
Hans

> -----Original Message-----
> From: [email protected] [mailto:[email protected]]
> Sent: den 17 september 2013 09:19
> To: [email protected]; [email protected]; Praveen Malviya; 
> Mathivanan Naickan Palanivelu
> Cc: [email protected]
> Subject: [devel] [PATCH 1 of 1] Amfnd: Mark Health Check Status as Stable in 
> CLC resp if HC Send Timer already expired [#568]
> 
>  osaf/services/saf/avsv/avnd/avnd_chc.c |  2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> 
> diff --git a/osaf/services/saf/avsv/avnd/avnd_chc.c 
> b/osaf/services/saf/avsv/avnd/avnd_chc.c
> --- a/osaf/services/saf/avsv/avnd/avnd_chc.c
> +++ b/osaf/services/saf/avsv/avnd/avnd_chc.c
> @@ -781,7 +781,7 @@ void avnd_comp_hc_cmd_restart(AVND_COMP
>       if (rec->status == AVND_COMP_HC_STATUS_SND_TMR_EXPD) {
>               uint32_t rc = avnd_start_tmr(avnd_cb, &rec->tmr, AVND_TMR_HC, 
> rec->period, rec->opq_hdl);
>               osafassert(rc == NCSCC_RC_SUCCESS);
> -             rec->status = AVND_COMP_HC_STATUS_WAIT_FOR_RESP;
> +             rec->status = AVND_COMP_HC_STATUS_STABLE;
>       } else
>               rec->status = AVND_COMP_HC_STATUS_STABLE;
>  }
> 
> ------------------------------------------------------------------------------
> LIMITED TIME SALE - Full Year of Microsoft Training For Just $49.99!
> 1,500+ hours of tutorials including VisualStudio 2012, Windows 8, SharePoint
> 2013, SQL 2012, MVC 4, more. BEST VALUE: New Multi-Library Power Pack includes
> Mobile, Cloud, Java, and UX Design. Lowest price ever! Ends 9/20/13.
> http://pubads.g.doubleclick.net/gampad/clk?id=58041151&iu=/4140/ostg.clktrk
> _______________________________________________
> Opensaf-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/opensaf-devel

------------------------------------------------------------------------------
LIMITED TIME SALE - Full Year of Microsoft Training For Just $49.99!
1,500+ hours of tutorials including VisualStudio 2012, Windows 8, SharePoint
2013, SQL 2012, MVC 4, more. BEST VALUE: New Multi-Library Power Pack includes
Mobile, Cloud, Java, and UX Design. Lowest price ever! Ends 9/20/13. 
http://pubads.g.doubleclick.net/gampad/clk?id=58041151&iu=/4140/ostg.clktrk
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to