diff options
author | Florian Weimer <fweimer@redhat.com> | 2016-02-19 17:07:04 +0100 |
---|---|---|
committer | Florian Weimer <fweimer@redhat.com> | 2016-02-19 17:07:04 +0100 |
commit | ca135f824b1dbaf43e4a673de7725db76a51b714 (patch) | |
tree | 487c36e49482479f9f9a88094cf8896827dadc88 | |
parent | 00d4e2ea3503e6de0f198cd65343f287a51f04db (diff) | |
download | glibc-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-- | ChangeLog | 11 | ||||
-rw-r--r-- | malloc/hooks.c | 2 | ||||
-rw-r--r-- | malloc/malloc.c | 8 | ||||
-rw-r--r-- | malloc/malloc.h | 2 | ||||
-rw-r--r-- | manual/memory.texi | 2 |
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. |