[Openais] [PATCH]: openais/trunk: Initial patch for OpenAIS TMR service
Steven Dake
sdake at redhat.com
Wed Feb 11 11:02:27 PST 2009
First excellent work!
The IPC messages should be mar_XXX and all parameters including the data
type should be __attribute((aligned(8)))) or however the code goes.
tmr_time_now should be deleted and instead the api call
api->timer_time_get should be used (and also implemented.:)
tmr_timer_create defines no function. Is that intentional?
entering and exiting of a function should be marked with ENTER() and
LEAVE() instead of printfs.
revert the changes in openaisserviceenable relating to the openais_clm
and openais_evt lines (adding an extra space).
what can I say - great work!
On Wed, 2009-02-11 at 12:19 -0600, Ryan O'Hara wrote:
> This patch contains the initial implementation of the OpenAIS TMR
> service.
>
> Most of the service API calls are implemented. The exception is
> saTmrTimerRemainingTimeGet. This call should return the time until the
> next expiration of a periodic timer. To implement this in a sane
> manner will require changes to the corosync API, which are currently
> being designed and will be included in a separate patch.
>
> Another missing bit of code is the code to check is an absolute timer
> has an initial expire time in the past. This is invalid, and this
> check should be done in the corosync timer API code. Until that code
> is completed, you should be careful when adding absolute timers.
>
> This patch contains many debugging printf's and log_printf's. These
> will be removed in the future.
>
> A test program is included (testtmr.c).
>
>
>
> _______________________________________________
> Openais mailing list
> Openais at lists.linux-foundation.org
> https://lists.linux-foundation.org/mailman/listinfo/openais
More information about the Openais
mailing list