summary refs log tree commit diff
diff options
context:
space:
mode:
authorFlorian Weimer <fweimer@redhat.com>2016-02-19 17:07:04 +0100
committerFlorian Weimer <fweimer@redhat.com>2016-02-19 17:07:04 +0100
commitca135f824b1dbaf43e4a673de7725db76a51b714 (patch)
tree487c36e49482479f9f9a88094cf8896827dadc88
parent00d4e2ea3503e6de0f198cd65343f287a51f04db (diff)
downloadglibc-ca135f824b1dbaf43e4a673de7725db76a51b714.tar.gz
glibc-ca135f824b1dbaf43e4a673de7725db76a51b714.tar.xz
glibc-ca135f824b1dbaf43e4a673de7725db76a51b714.zip
malloc: Remove max_total_mem member form struct malloc_par
Also note that sumblks in struct mallinfo is always 0.
No functional change.
-rw-r--r--ChangeLog11
-rw-r--r--malloc/hooks.c2
-rw-r--r--malloc/malloc.c8
-rw-r--r--malloc/malloc.h2
-rw-r--r--manual/memory.texi2
5 files changed, 16 insertions, 9 deletions
diff --git a/ChangeLog b/ChangeLog
index 52bf5c70c0..6658cb5692 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,16 @@
 2016-02-19  Florian Weimer  <fweimer@redhat.com>
 
+	* malloc/malloc.c (__libc_mallinfo): Update comment.
+	(struct malloc_par): Remove max_total_mem member, it was always 0.
+	(int_mallinfo): Store 0 into usmblks (no functional change).
+	* malloc/hooks.c (struct malloc_state): Document that
+	max_total_mem is always 0.
+	* malloc/malloc.h (struct mallinfo): Dcoument that usmblks is
+	always 0.
+	* manual/memory.texi (Statistics of Malloc): Likewise.
+
+2016-02-19  Florian Weimer  <fweimer@redhat.com>
+
 	* malloc/malloc.c (sysmalloc): Do not update arena_max.
 	* malloc/arena.c (arena_max): Remove.
 	(heap_trim, _int_new_arena): Do not update arena_max.
diff --git a/malloc/hooks.c b/malloc/hooks.c
index 7a85136b45..0d2bb96c03 100644
--- a/malloc/hooks.c
+++ b/malloc/hooks.c
@@ -480,7 +480,7 @@ struct malloc_save_state
   unsigned long mmap_threshold;
   int check_action;
   unsigned long max_sbrked_mem;
-  unsigned long max_total_mem;
+  unsigned long max_total_mem;	/* Always 0, for backwards compatibility.  */
   unsigned int n_mmaps;
   unsigned int max_n_mmaps;
   unsigned long mmapped_mem;
diff --git a/malloc/malloc.c b/malloc/malloc.c
index 86e2a03452..1038df4864 100644
--- a/malloc/malloc.c
+++ b/malloc/malloc.c
@@ -649,8 +649,7 @@ libc_hidden_proto (__libc_mallopt)
 	       have been freed but not use resused or consolidated)
   hblks:     current number of mmapped regions
   hblkhd:    total bytes held in mmapped regions
-  usmblks:   the maximum total allocated space. This will be greater
-		than current total if trimming has occurred.
+  usmblks:   always 0
   fsmblks:   total bytes held in fastbin blocks
   uordblks:  current total allocated space (normal or mmapped)
   fordblks:  total free space
@@ -1743,10 +1742,7 @@ struct malloc_par
 
   /* Statistics */
   INTERNAL_SIZE_T mmapped_mem;
-  /*INTERNAL_SIZE_T  sbrked_mem;*/
-  /*INTERNAL_SIZE_T  max_sbrked_mem;*/
   INTERNAL_SIZE_T max_mmapped_mem;
-  INTERNAL_SIZE_T max_total_mem;  /* only kept for NO_THREADS */
 
   /* First address handed out by MORECORE/sbrk.  */
   char *sbrk_base;
@@ -4665,7 +4661,7 @@ int_mallinfo (mstate av, struct mallinfo *m)
     {
       m->hblks = mp_.n_mmaps;
       m->hblkhd = mp_.mmapped_mem;
-      m->usmblks = mp_.max_total_mem;
+      m->usmblks = 0;
       m->keepcost = chunksize (av->top);
     }
 }
diff --git a/malloc/malloc.h b/malloc/malloc.h
index f65e18aa82..d95a3157a3 100644
--- a/malloc/malloc.h
+++ b/malloc/malloc.h
@@ -83,7 +83,7 @@ struct mallinfo
   int smblks;   /* number of fastbin blocks */
   int hblks;    /* number of mmapped regions */
   int hblkhd;   /* space in mmapped regions */
-  int usmblks;  /* maximum total allocated space */
+  int usmblks;  /* always 0, preserved for backwards compatibility */
   int fsmblks;  /* space available in freed fastbin blocks */
   int uordblks; /* total allocated space */
   int fordblks; /* total free space */
diff --git a/manual/memory.texi b/manual/memory.texi
index 700555ed46..3d99147592 100644
--- a/manual/memory.texi
+++ b/manual/memory.texi
@@ -1520,7 +1520,7 @@ This is the total number of chunks allocated with @code{mmap}.
 This is the total size of memory allocated with @code{mmap}, in bytes.
 
 @item int usmblks
-This field is unused.
+This field is unused and always 0.
 
 @item int fsmblks
 This field is unused.