Bug 57342 - Network timing statis now configurable
Summary: Network timing statis now configurable
Alias: None
Product: eCos
Classification: Retired
Component: Patches and contributions   
(Show other bugs)
Version: 1.5.2
Hardware: i386
OS: Linux
Target Milestone: ---
Assignee: ecc-bugs-int
QA Contact: ecc-bugs-int
Depends On:
TreeView+ depends on / blocked
Reported: 2001-12-10 17:26 UTC by Andrew Lunn
Modified: 2007-04-18 16:38 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2001-12-10 17:27:10 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Here is the patch (7.92 KB, patch)
2001-12-10 17:27 UTC, Andrew Lunn
no flags Details | Diff

Description Andrew Lunn 2001-12-10 17:26:25 UTC
Description of Patch:

At the moment the network timing statistics are not configurable. For some
calls the represent quite an overhead. This patch makes the collection of
the statistics and the functions used to print them out configurable at
compile time. By default they are not compiled. The patch also ammends the
tests so they link with our without the statistical information function
being present.  It also makes the tests linkable when the SNMP library is
present which has its own implementation of gettimeofday!

Version-Release number of selected component (if applicable):1.5.2

Comment 1 Andrew Lunn 2001-12-10 17:27:02 UTC
Created attachment 40290 [details]
Here is the patch

Comment 2 Jonathan Larmour 2001-12-12 01:09:55 UTC
I've applied the patch, thanks! The ChangeLog had a few missing bits, which I've
added. FYI here's the new entry. As I've written here, who supplies
gettimeofday() should be resolved "properly" really, but this will do for now.

Thanks again for the contrib.

2001-12-10  Andrew Lunn <andrew.lunn@ascom.ch>

	* cdl/net.cdl: Add configury to eliminate network timing stats.
	* src/include/machine/param.h: Timing statistics for things like 
	memcpy, mbuf_alloc etc now use configury to determine if the
	statistics code should be compiled.
	* src/ecos/support.c: Ditto.
	* tests/nc_test_slave.c: Only print timing stats when they
	are enabled. 
	* tests/tcp_echo.c: Ditto.
	* tests/nc_test_master.c: Allow building with SNMP lib which has
	its own gettimeofday(). This should be fixed better a different
	way but will do for now.

Comment 3 Jonathan Larmour 2001-12-12 01:12:25 UTC
One more thing: I removed the CYGPKG_NET_TIMING component. It didn't appear to
serve any purpose as a distinct option. We can always add it later if we add
more timing related options.

Comment 4 Andrew Lunn 2001-12-12 08:26:59 UTC
OK. I just cut&paste the CYGPKG_NET_DEBUG from above to make the new entry.
Maybe you should also take that out :-)

Comment 5 Jonathan Larmour 2001-12-12 16:42:20 UTC
Ah, no in that case CYGPKG_NET_DEBUG _*is* used (in src/ecos/support.c)
separately from CYGDBG_NET_DHCP_CHATTER. Sorry :-).

Note You need to log in before you can comment on or make changes to this bug.