Skip to content

Conversation

@systems-assistant
Copy link
Contributor

@systems-assistant systems-assistant bot commented Aug 19, 2025

This can occur on platforms that operates in WGP (work group processor) mode in which the CUs are grouped in pairs so there are effectively half as many multiprocessing units.

Associated JIRA ticket number/Github issue number

Fixes SWDEV-537865

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Continuous Integration

What were the changes?

Clamp the results from GenerateThreadDimensions, GenerateBlockDimensions, GenerateThreadDimensionsForShuffle and GenerateBlockDimensionsForShuffle to prevent invalid zero values when multiProcessorCount returns 1.

Why are these changes needed?

Fixes divide by zero errors and invalid parameter errors.

Updated CHANGELOG?

  • Yes
  • No, Does not apply to this PR.

Added/Updated documentation?

  • Yes
  • No, Does not apply to this PR.

Additional Checks

  • I have added tests relevant to the introduced functionality, and the unit tests are passing locally.
  • Any dependent changes have been merged.

🔁 Imported from ROCm/hip-tests#477
🧑‍💻 Originally authored by @gnifAMD

gnifAMD and others added 2 commits July 29, 2025 16:26
This can occur on platforms that operates in WGP (work group processor) mode in
which the CUs are grouped in pairs so there are effectively half as many
multiprocessing units.
ammallya pushed a commit that referenced this pull request Nov 17, 2025
…625)

Increased the AMDSMI_MAX_DEVICES to 64 to accomodate all
devices in CPX mode. The link type has been modified in
amd-smi to match with rocm-smi types, updated the same
for drm tests.

---------

Signed-off-by: Bindhiya Kanangot Balakrishnan <[email protected]>
ammallya pushed a commit that referenced this pull request Nov 18, 2025
…625)

Increased the AMDSMI_MAX_DEVICES to 64 to accomodate all
devices in CPX mode. The link type has been modified in
amd-smi to match with rocm-smi types, updated the same
for drm tests.

---------

Signed-off-by: Bindhiya Kanangot Balakrishnan <[email protected]>

[ROCm/amdsmi commit: 6715c5a]
ammallya pushed a commit that referenced this pull request Nov 21, 2025
…625)

Increased the AMDSMI_MAX_DEVICES to 64 to accomodate all
devices in CPX mode. The link type has been modified in
amd-smi to match with rocm-smi types, updated the same
for drm tests.

---------

Signed-off-by: Bindhiya Kanangot Balakrishnan <[email protected]>

[ROCm/amdsmi commit: 6715c5a]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants