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 the NVProfilerService and ProcessCallGraph [12.4.x] #39399

Merged
merged 3 commits into from Sep 17, 2022
Merged

Fix the NVProfilerService and ProcessCallGraph [12.4.x] #39399

merged 3 commits into from Sep 17, 2022

Commits on Sep 15, 2022

  1. Check that the Source is added exactly once to the ProcessCallGraph

    @fwyzard
    fwyzard committed Sep 15, 2022
    Copy the full SHA
    fd95924 View commit details
    Browse the repository at this point in the history

  2. Avoid static variables 

    Using a static local variable results in invalid ids when multiple
ProcessCallGraph instances are used e.g. by different Services.
    @fwyzard
    fwyzard committed Sep 15, 2022
    Copy the full SHA
    0eeced9 View commit details
    Browse the repository at this point in the history

  3. Fix the NVProfilerService 

    Use a ProcessCallGraph to get the highest possible module id, instead of
relaying on the modules count.
    @fwyzard
    fwyzard committed Sep 15, 2022
    Copy the full SHA
    496330b View commit details
    Browse the repository at this point in the history