Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

track heap usage #196

Merged
merged 51 commits into from
Jun 16, 2024
Merged

track heap usage #196

merged 51 commits into from
Jun 16, 2024

Conversation

yamt
Copy link
Owner

@yamt yamt commented Jun 13, 2024 •
edited
Loading

todo

  • bitmap.h
  • jump table
  • type annotation
  • cellidxes
  • localchunks
  • resulttype_alloc
  • waitlist
  • functype_to_string (only used by callgraph example)
  • fileio.c
  • struct validation_context
  • add cli options to specify limit
  • adapt dyld
  • host functions. libwasi tables have been adapted. maybe other structures? but it's a bit moot as we can't account host resources (eg libc internal consumption) anyway.
  • adapt examples
  • benchmark the overhead

yamt added 4 commits June 15, 2024 13:01
@yamt
mem_context wip
e713e1d 
introduce mem_context to track/limit heap allocations
@yamt
mem_context wip
19ec8a8
@yamt
format
1b85bfb
@yamt
mem_contexnt wip
cfe5ca0
yamt added 25 commits June 15, 2024 13:06
@yamt
typo
41badae
@yamt
mem_context: TOYWASM_ENABLE_DYLD
c6f5832
@yamt
mem.c: add a debug code
b9fe96b
@yamt
module.c: fix a mem_free with a wrong size
2cc2627
@yamt
mem_context: adapt dyld
6ead71c
@yamt
lib/mem.c: include stdint.h for SIZE_MAX
1418551
@yamt
lib/mem.c: more debug code
eb28007
@yamt
_read_vec_with_ctx_impl: fix mem_context accounting on error
57ecec6 
also, revert the array size.
@yamt
mem_context: fix accounting on br_table failure
e090d23
@yamt
read_expr_common: fix mem_context accounting
34ff965
@yamt
read_const_expr: assertions and a comment
daf5dc0
@yamt
lib: expose mem.h
3b8a325
@yamt
mem.h: extern C
6682afd
@yamt
examples/fuzz: adapt to mem_context
3ae7050
@yamt
lib/mem.c: fix reservation failure handling
1819772
@yamt
module.c: fix a crash on a broken module
13b9ef9 
found by examples/fuzz
@yamt
examples/fuzz: limit memory consumption to 1MB
0b95bf4
@yamt
module_funcs_clear: fix an assertion
784e5e5
@yamt
fix TOYWASM_ENABLE_WASI_LITTLEFS build after mem_context changes
7f26777
@yamt
name.c: add a missing mem_context_init
81c17dc
@yamt
repl: fix load-hex command after mem_context changes
1554363
@yamt
lib/mem.h: multiple inclusion protection
685583f
@yamt
examples/app: adapt to mem_context
86ad173
@yamt
examples/runwasi: adapt to mem_context
abfe679
@yamt
examples/hostfunc: adapt to mem_context
b343be3
@yamt yamt marked this pull request as ready for review June 16, 2024 12:16
@yamt yamt changed the title WIP mem context track heap usage Jun 16, 2024
@yamt yamt merged commit 83a249c into master Jun 16, 2024
212 checks passed
@yamt yamt deleted the mem-context branch June 16, 2024 13:44
@yamt yamt mentioned this pull request Jun 16, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yamt