[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