[Pacemaker] [Question] About the rotation of the pe-file.

Andrew Beekhof andrew at beekhof.net
Mon Jan 16 06:36:03 CET 2012


On Mon, Jan 16, 2012 at 11:48 AM,  <renayama19661014 at ybb.ne.jp> wrote:
> Hi Andrew,
> Hi Lars,
>
>> >> If you want it to be between [0, max-1],
>> >> obviously that should be
>> >>         while(max > 0 && sequence >= max) {
>> >>                 sequence -= max;
>> >>         }
>
> The rotation was carried out definitely from 0 to max-1.
>
>
>> >> Though I wonder why not simply:
>> >>     if (max == 0)
>> >>         return;
>> >>     if (sequence > max)
>> >>         sequence = 0;
>
> The rotation was carried out definitely from 0 to max.
>
>> I was probably trying to get too fancy when dealing with run-time
>> reduction of max.
>> Lets go with your way :-)
>
> I think that the application of the patch of the second simple correction is good.

Its in my private tree so far:
  https://github.com/beekhof/pacemaker/commit/bfbb73c

It will make its way to clusterlabs when I merge next.



More information about the Pacemaker mailing list