aboutsummaryrefslogtreecommitdiff
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
glibc-ca135f824b1dbaf43e4a673de7725db76a51b714.tar.gz
glibc-ca135f824b1dbaf43e4a673de7725db76a51b714.tar.bz2
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.