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