Sync up with master branch - #1024
Open
jkottiku wants to merge 391 commits into
Open
Conversation
jkottiku
requested review from
a team,
frepaul and
manoj-freyr
as code owners
September 30, 2025 17:30
yugang-amd
approved these changes
Sep 30, 2025
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
1 similar comment
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
Add time-based Babel run
Add action select (-s) cli. option.
This will ensure new ROCm is not causing issue in building a known good RVS mainline.
Fixed an issue on upload to release branch even when build is not from release branch. Also simplified the condition check for each type of job and removed artifact upload to github storage.
No need to check for branch/event based Release summary.
Add module (-m) cli option
Update RVS version (1.4.0) and change logs.
With the new proposal packages generated by RVS need to stick to a new standard to match with compatible ROCm Core version.Install location also changes to /opt/rocm/extras-<ROCm Major>. Inside the install location file layout remain same as before. Within a major release only in-place upgrade with compatibility contract to older and newer ROCm minor releases. Side-by-side install across major version releases.
Ubuntu job need to set directory as safe to fetch tag and count for patch count
Keep the old name for when triggered directly from CMake and use build_package_local file to control the naming in build workflow
File layout should be flat with FHS spec inside install dir https://refspecs.linuxfoundation.org/FHS_3.0/fhs/index.html
Keep RVS command backward combatible with old version config files need to be in same place as before, share/rocm-validation-suite. Same time copyright need to be share/package-name. In .tgz pkg empty dirs present and need to be removed.
subfolder cmake files have differen project(name). This overrides expected behaviour of installing files. Bring back CPACK_SET_DESTDIR=ON, this is needed to create tgz file
Need to get rid of empty extras folder and ensure no double slash
…keep in sync with master.
Add version string that are incrementing and meaningful to explain the image rel* branch: Ver: rvs-<major.minor.patch>-buildnumber master branch: Merge,Daily build Ver: rvs-<major.minor.patch>.r<rocmmajor<rocmminor>.yyyymmdd PR build: Ver.branch.commit
TGZ file did not have any release version string.It is good to keep all format having same version format as well.
TransferBench v1.69+ adds `third-party/ibverbs/` headers (`IbvDynLoad.hpp`). RVS auto-detects that directory at configure time and adds it to the PEBB/PBQT include path (`TRANSFERBENCH_IBVERBS_INC_DIR` in the root `CMakeLists.txt`). When the folder is absent (older submodule pins), no extra includes or `libdl` link are applied. Runtime `libibverbs` is loaded dynamically only if present; PEBB/PBQT gfx/dma tests do not require it.
This is to cleanup the workspace after build is done.This helps to reclaim disk space while running parallel jobs.
Make libpci static and cleanup.
Reformatting Babel json logging
Sync-up with master
Co-authored-by: yugang-amd <yugang.wang@amd.com>
User-guide update
Update Transferbench to v1.69.00
Sync-up with master
Add tolerance for iet stress.
MI450X support
Add support of domain while resolving the pcie location id to avoid non-GPUs to be reporting wrong values
Signed-off-by: Urvashi Tiwari <urtiwari.com>
Signed-off-by: Urvashi Tiwari <urtiwari.com>
Signed-off-by: Urvashi Tiwari <urtiwari.com>
pbqt: fix test_bandwidth as false reporting failure.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Technical Details
Test Plan
Test Result
Submission Checklist