[Xorp-cvs] XORP cvs commit: xorp/libxorp
Andrea Bittau
abittau at icir.org
Tue Sep 23 12:55:42 PDT 2008
CVSROOT: /usr/local/www/data/cvs
Module name: xorp
Changes by: abittau at chum.icir.org 2008-09-23 19:55:42 UTC
XORP CVS repository
Modified files:
libxorp eventloop.cc timer.cc
Log message:
Force commit to correct CVS log message. Message /tmp/log.r456 should read:
Get rid of 3 system calls to time() - 6% improvement in XRL performance. Before
we would call time:
1) Top of event loop for sanity check
2) Top of event loop for time infrastructure.
3) When checking for expired timers.
4) After select.
5) Bottom of event loop for sanity check.
The first three calls return the same time and can be simplified to one call.
The 5th call isn't strictly necessary - we can either use reading #1 or #4. We
now call time twice:
1) Top of event loop.
2) After select.
We can probably get away by killing #1, or in some cases, #2 (e.g., small select
timeout).
Revision Changes Path
1.35 +1 -1; commitid: 1c3948d949bc41a7; xorp/libxorp/eventloop.cc
1.46 +1 -1; commitid: 1c3948d949bc41a7; xorp/libxorp/timer.cc
More information about the Xorp-cvs
mailing list