Comment by pajko

3 hours ago

Timestamps should NOT be compared like this. Exactly this is why time_before() or time_after() exist.

https://elixir.bootlin.com/linux/v6.15.7/source/include/linu...

3 comments

pajko

Reply
AceJohnny2  21 minutes ago

Offtopic...

    * Do this with "<0" and ">=0" to only test the sign of the result. A
    * good compiler would generate better code (and a really good compiler
    * wouldn't care). Gcc is currently neither.

It's funny the love-hate relationship the Linux kernel has with GCC. It's the only supported compiler[1], and yet...

[1] can Clang fully compile Linux yet? I haven't followed the updates in a while.

Joker_vD  1 hour ago

Do I understand it correctly that the logic is that if timestamp B is above timestamp A, but the difference is more than half of the unsigned range, B is considered to happen before A?