[oprof-cvs] CVS: oprofile ChangeLog,1.710,1.711 TODO,1.140,1.141

[Philippe E. <ph...@us...>]

Update of /cvsroot/oprofile/oprofile
In directory sc8-pr-cvs1:/tmp/cvs-serv28058

Modified Files:
	ChangeLog TODO 
Log Message:

merge from db-branch-1 to head removing module hash table and change
implementation of samples files to use a hash table.

regards,
Phil


Index: ChangeLog
===================================================================
RCS file: /cvsroot/oprofile/oprofile/ChangeLog,v
retrieving revision 1.710
retrieving revision 1.711
diff -u -d -r1.710 -r1.711
--- ChangeLog	2 Dec 2002 23:13:46 -0000	1.710
+++ ChangeLog	3 Dec 2002 01:57:56 -0000	1.711
@@ -1,3 +1,53 @@
+2002-12-02  Philippe Elie  <ph...@wa...>
+
+	* TODO: update
+
+	* dae/opd_image.h:
+	* dae/opd_kernel.c:
+	* dae/opd_kernel.h:
+	* dae/opd_parse_proc.c:
+	* dae/opd_proc.c:
+	* dae/opd_proc.h:
+	* dae/opd_sample_files.c:
+	* dae/oprofiled.c:
+	* daemon/opd_image.h:
+	* pp/op_merge.cpp:
+	* pp/samples_file.h:
+	* pp/samples_file.cpp:
+	* pp/Makefile.am:
+	* libabi/Makefile.am:
+	* libabi/abi.cpp:
+	* libabi/abi_test.cpp:
+	* libabi/op_import.cpp: use hash table implementation of libdb
+
+	* gui/oprof_start.cpp:
+	* gui/oprof_start_config.cpp:
+	* gui/oprof_start_config.h:
+	* gui/ui/oprof_start.base.ui:
+	* utils/op_start:
+	* doc/oprofile.1.in:
+	* doc/oprofile.xml: remove module hash table size
+
+	* libdb/.cvsignore: update
+	* libdb/Makefile.am: build libdb-hash.a
+	* libdb/db-hash.h:
+	* libdb/db-hash-debug.c:
+	* libdb/db-hash-manage.c:
+	* libdb/db-hash-test.c:
+	* libdb/db-hash-insert.c:
+	* libdb/db-hash-stat.c:
+	* libdb/db-hash-travel.c: new files implementing a growable hash table
+	* libdb/Makefile.am: update
+
+	* libop/op_config_24.h: remove module hash table. Increase
+	  OP_PRE_WATERMARK, OP_DEFAULT_BUF_SIZE.
+	* libop/op_interface.h: struct op_sample update.
+
+	* module/oprofile.h:
+	* module/oprofile.c: remove hash table
+	* module/x86/opc,model_ppro.c: break overflowed counter checking loop
+	  sooner
+
 2002-12-02  John Levon  <le...@mo...>
 
 	* configure.in: remove annoying AC_REVISION()

Index: TODO
===================================================================
RCS file: /cvsroot/oprofile/oprofile/TODO,v
retrieving revision 1.140
retrieving revision 1.141
diff -u -d -r1.140 -r1.141
--- TODO	15 Nov 2002 11:12:43 -0000	1.140
+++ TODO	3 Dec 2002 01:57:56 -0000	1.141
@@ -25,7 +25,6 @@
  o document somewhere that oprofpp -s blah will only find the first symbol
    called "blah", or fix it
  o There is probably place of post profile tools where looking at errno will give better error messages.
- o don't make hash table entries dependent on cache line size ?? 
  o op_to_source --source-dir=~moz/src/oprofile/ --output-dir=op /usr/bin/oprofiled
    will fail because the ~ is not expanded (no space around it) (popt bug I say)
  o move op_start to simple C program ??
@@ -33,6 +32,8 @@
  o audit oprof_start for security + document sudo
  o coherent naming class conventions
  o use standard C integer type <stdint.h> int32_t int16_t etc.
+ o tune libdb/db-hash.h do_hash() function
+ o check for module buffer with smp (and up with high load/memory pressure)
 
 General checks to make
 ----------------------
@@ -48,8 +49,6 @@
  o use nm and a little script to track unused function
  o test it to hell and back
  o compile all C++ programs with STL_port and test them
- o tune memory use for module hash table use for 64 bits architecture, perhaps
-  making the hash table size independant of the cache line size 
 
 Later
 -----