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

Fix ksymbols mem consumption #4092

Closed
wants to merge 2 commits into from
Closed

Fix ksymbols mem consumption #4092

wants to merge 2 commits into from

Commits on May 30, 2024

  1. ksymbols: reintroduce lazy symbol queries 

    Add new required symbol and addresses lists to the KSymbolTable helper.
When either list is given, the data structure will only save required
symbols during scanning. This should reduce memory usage for users
who can mostly guarentee that symbols are known ahead of time.

Dynamic querying is still allowed in this mode, though it will incur
a penalty as rescanning is required.
    @NDStrahilevitz @yanivagman
    NDStrahilevitz authored and yanivagman committed May 30, 2024
    Configuration menu
    Copy the full SHA
    7dc9917 View commit details
    Browse the repository at this point in the history

  2. feat(ksymbols): restore lazy ksyms implementtion 

    Use modified ksymbols implementation. The new implementation may take
a list of required symbols and addresses to track. If the list is given,
symbol scanning will only save those symbols or addresses which were
given in the list. If a new symbol is queried, then a rescan is needed.

Refactor tracee initialization to find all necessary symbols to track
ahead of runtime.
    @NDStrahilevitz @yanivagman
    NDStrahilevitz authored and yanivagman committed May 30, 2024
    Configuration menu
    Copy the full SHA
    af1daed View commit details
    Browse the repository at this point in the history