Skip to content

Conversation

@ronlieb
Copy link
Contributor

@ronlieb ronlieb commented Nov 22, 2025

gfx1200 memory limits milestone3-babel (#1671)

swdev-565113

swdev-565113

 Please enter the commit message for your changes. Lines starting
Copy link
Contributor

@carlobertolli carlobertolli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small nits can be fixed post merge

save_status
./milestone-3-babel -n 20 -s 100000000 # 2400.0 MB
save_status
if [[ "$AOMP_GPU" =~ ^gfx12.* ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would limit it to gfx1200 and gfx1201


$PRECMD ./milestone-3-babel -n 20 -s 100000000 # 2400.0 MB
save_status
if [[ "$AOMP_GPU" =~ ^gfx12.* ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ditto

Copy link
Contributor

@gregrodgers gregrodgers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 30, 2025
@ronlieb ronlieb reopened this Nov 30, 2025
@ronlieb ronlieb closed this Nov 30, 2025
@ronlieb ronlieb reopened this Nov 30, 2025
@ronlieb ronlieb closed this Nov 30, 2025
@ronlieb ronlieb reopened this Nov 30, 2025
@ronlieb ronlieb closed this Nov 30, 2025
@ronlieb ronlieb reopened this Nov 30, 2025
@rahulc-gh
Copy link

re-running test seems infra issue - http://rocm-ci.amd.com/job/rocm-tests/34924/

@geomin12
Copy link

geomin12 commented Dec 1, 2025

Landing this as PSDB rocm-ci failures are unrelated, as this is a fortran update. Failures in rocgdb, ocl, rocprof are irrelevant

@geomin12 geomin12 merged commit 740c9bc into release/rocm-rel-7.2 Dec 1, 2025
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants