diff options
| author | Mike Gerwitz <mike.gerwitz@ryansg.com> | 2022-04-06 11:50:07 -0400 |
|---|---|---|
| committer | Mike Gerwitz <mike.gerwitz@ryansg.com> | 2022-04-06 11:50:07 -0400 |
| commit | 2e386f1baf516fa0ce300ba3e42d6076c179c87f (patch) | |
| tree | d76da9fd605fdc9bd97f18c21160adc1a588db72 /build-aux | |
| parent | 6871a0cdc75e9140c8f8ae6116aae5695df0b387 (diff) | |
| download | tame-2e386f1baf516fa0ce300ba3e42d6076c179c87f.tar.gz tame-2e386f1baf516fa0ce300ba3e42d6076c179c87f.tar.bz2 tame-2e386f1baf516fa0ce300ba3e42d6076c179c87f.zip | |
tamer: xir::reader::XmlXirReader::refill_buf: Clear read buffer
This was done in the old reader many months ago, but I somehow forgot to do
it here (or forgot to). The new reader was using substantially more memory.
Here's how this change affects the memory profile for one of our
systems (output from `ms_print`):
Before:
MB
79.75^ #
| #
| # @
| @@@@ # @
| @@@ # @@
| @@@ @@@#@ @@@@@
| @@@ @@ #@@@@@@@@@@
| @@@@@@ @@@@ #@@@@@@@@@@
| @@ @@ @@@ @@ @ @@ #@@@@@@@@@@
| @@ @@ @@@ @@@@@ @@ #@@@@@@@@@@
| @@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
| @@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
| @@ @@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
| @ @@ @@ @ @@@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
| @ @ @@@ @@ @@@ @@@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
| @ @@@@ @@@@@@@@@@@@@@@@@@@@@ @@@@@@@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
| @@@ @@@@@@ @@@@@@@@@ @@@@@ @@@@@ @@ @@@@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
| @@@ @ @@@@ @@@@@@@@@ @@@@@ @@@@@ @@ @@@@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
| @@@ @@@ @@@@ @@@@@@@@@ @@@@@ @@@@@ @@ @@@@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
| @@@ @@@ @@@@ @@@@@@@@@ @@@@@ @@@@@ @@ @@@@@@@ @@@ @@@@@@ @@ #@@@@@@@@@@
0 +----------------------------------------------------------------------->Gi
0 15.20
After:
MB
63.25^ #
| #
| @@@@@@@@@#@
| @@@@@@ @@#@
| @@@@@@ @@#@
| @@@@@@ @@#@
| @@@@@@ @@#@
| @@@@@@@@@@@@ @@#@
| @@@@@@@@@ @@ @@@@@@ @@#@
| @@@@@@@@ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@@@@@@@@@ @@@@@@@@ @@@@@@@@@@@@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@@@@@@@@@@@ @@@@@@@@ @@@@@@@@@@@@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@@@@@@@@@@@@@ @@@@@@@@ @@@@@@@@@@@@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@@@@@@@@@@@@@@@@ @@@@@@@@ @@@@@@@@@@@@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
| @@@@@@@@@@@@@@@@@@@@ @@@@@@@@ @@@@@@@@@@@@@@@ @ @@@ @@@ @@ @@@@@@ @@#@
0 +----------------------------------------------------------------------->Gi
0 15.20
The bottom graph is virtually identical to the memory profile of the old
reader, just with the exception that it's interning a bit more data than
before, because we're reading more comprehensively.
That's (potentially) the subject of future changes.
DEV-12038
Diffstat (limited to 'build-aux')
0 files changed, 0 insertions, 0 deletions