Re: [PATCH] [1/4] x86: MCE: Make polling timer interval per CPU

Hidetoshi Seto
2009-04-08T11:54:57Z

Andi Kleen wrote:
>>> + if (!*n)
>>> + *n = check_interval * HZ;
>>> + if (!*n)
>>> return;
>
> The !*n will return for check_interval == 0 because 0*HZ is still 0 so it should be
> equivalent. Did I miss something?

At First, *n is 0 on boot.
And then soon it will be initialized to non-zero (check_interval * HZ)
by the first if-statement.

After that if check_interval is changed via sysfs while *n is non-zero,
which if-statement runs?

Thanks,
H.Seto