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

Fdavid/mi300 fixes #2482

Merged
merged 2 commits into from
Sep 17, 2024
Merged

Fdavid/mi300 fixes #2482

merged 2 commits into from
Sep 17, 2024

Commits on Sep 16, 2024

  1. plugins/amdgpu: Zero ib_info on initialization 

    This struct was being used un-initialized, meaning it
was filled with random garbage.

Mea culpa.

Signed-off-by: David Francis <[email protected]>
    @fdavid-amd
    fdavid-amd committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    bd476d7 View commit details
    Browse the repository at this point in the history

  2. plugins/amdgpu - Increase maximum parameter length 

    The topology parsing assumed that all parameter names were
30 characters or fewer, but

recommended_sdma_engine_id_mask

is 31 characters.

Make the maximum length a macro, and set it to 64.

Signed-off-by: David Francis <[email protected]>
    @fdavid-amd
    fdavid-amd committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    d14b5c2 View commit details
    Browse the repository at this point in the history