On Sam, 2013-05-18 at 10:58 -0400, Jim Rees wrote:
> Bernd Petrovitsch wrote:
>
> Hmm, what is actually wrong with plain-old
> ---- snip ----
> static bool stateid_generation_after(stateid_t *a, stateid_t *b)
> {
> return a->si_generation > b->si_generation;
> }
> ---- snip ----
> ?
>
> It doesn't account for wraparound. Try this:
>
> unsigned int a = 0xffff0000, b = 0x7fff0000;
> printf("%d %d\n", a > b, (int)(a - b) > 0);
>
> I just realized that I said this needed a comment, then didn't provide one.
Ooops, yes, thx.
Sry for the noise;-)
Bernd
--
Bernd Petrovitsch Email : bernd@xxxxxxxxxxxxxxxxxxx
LUGA : http://www.luga.at
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
