patchmldonkey, a multi-networks file-sharing client - Patches: patch #7204, dump memory statistics to file

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #7204: dump memory statistics to file

Submitter:  ygrek <ygrek>
Submitted:  Mon 24 May 2010 09:24:30 PM UTC
   
 
Category:  Improved interfaces Priority:  5 - Normal
Status:  Postponed Assigned to:  ygrek
Open/Closed:  Open

Sat 23 Oct 2010 06:51:46 PM UTC, comment #4: 

I think we can leave it uncommited, it was used simply as a quick hack to view memory usage and I didn't want to throw away code - maybe it will be needed again some day.

ygrek <ygrek>
Group Member
Sat 23 Oct 2010 06:46:08 PM UTC, comment #3: 

At least one compilation error is still left:

File "src/daemon/common/commonSources.ml", line 2030, characters 32-35:
This expression has type
  string -> [ `F of float | `I of int | `S of string | `X ] -> unit
but is here used with type Buffer.t

spiralvoice <spiralvoice>
Group administrator
Sat 23 Oct 2010 06:45:32 PM UTC, comment #2: 

Yet another patch to fix compilation error:

--- src/daemon/common/commonHosts.ml 2010-10-23 20:45:06.000000000 +0200
+++ src/daemon/common/commonHosts.ml 2005-09-19 12:09:05.000000000 +0200
@@ -93,9 +93,9 @@
 
     let hosts_by_key = Hashtbl.create 103
 
-    let () =
-     Heap.add_memstat "CommonHosts" (fun _level k ->
-      k "hosts_by_key" (`I (Hashtbl.length hosts_by_key));
+    let _ =
+     Heap.add_memstat "CommonHosts" (fun level buf ->
+      Printf.bprintf buf "  hosts_by_key: %d\n" (Hashtbl.length hosts_by_key);
      )
 
 

spiralvoice <spiralvoice>
Group administrator
Sat 23 Oct 2010 06:44:24 PM UTC, comment #1: 

this is patch is needed to allow GUI compilation:

--- config/Makefile.in<>2010-10-19 19:30:54.000000000 +0200
+++ config/Makefile.in<>2010-10-23 20:37:50.000000000 +0200
@@ -148,9 +148,9 @@
.
.
 CDK_SRCS+= $(LIB)/fifo.ml $(CDK)/arg2.ml $(LIB)/syslog.ml $(CDK)/printf2.ml \
-  $(CDK)/heap.ml \
   $(CDK)/printexc2.ml $(CDK)/genlex2.ml \
   $(CDK)/filepath.ml $(CDK)/string2.ml \
+  $(CDK)/heap.ml \
   $(CDK)/filename2.ml $(CDK)/list2.ml $(CDK)/hashtbl2.ml \
   $(CDK)/unix2.ml $(CDK)/file.ml \
   $(CDK)/heap_c.c $(CDK)/array2.ml

spiralvoice <spiralvoice>
Group administrator
Mon 24 May 2010 09:24:30 PM UTC, original submission:  
ygrek <ygrek>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #20609:  mem_stats.diff added by ygrek (32KiB - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by spiralvoice (Posted a comment)
  • -email is unavailable- added by ygrek (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-10-23 spiralvoice StatusIn Progress Postponed
    2010-10-23 spiralvoice CategoryNone Improved interfaces
        StatusNone In Progress
    2010-05-24 ygrek Attached File- Added mem_stats.diff, #20609

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code