On 9/17/26 21:08, Stefano Brivio wrote:
+ clock_gettime(CLOCK_MONOTONIC, &now); + debug("route dependency handling time: %f s", + (now.tv_sec - start.tv_sec) + (now.tv_nsec - start.tv_nsec)/1.0e9);
And anyway, I'm not sure I see the value of this. Which other functions should we profile? Does it help at all to do this once you're done developing it?
I would suggest to simply drop this patch. I understand you needed that for development but you already explain in the commit message for 3/3 how it improves thing, and that's all the documentation we possibly need for the future, I think.
For this reason I placed these profiling related changes in a separate patch. They can be easily ignored. Nevertheless, it's still very important to optimize and profile the actual handling of huge routing tables. The original implementation was really horrible in this regard.