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.

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

Add the ability to determine GPU model from Chip ID #423

Merged

Conversation

xuchen-amd
Copy link
Contributor

@xuchen-amd xuchen-amd commented Sep 10, 2024

Add the ability to determine GPU model from Chip ID to distinguish MI300 systems by using a built-in dictionary.
The built-in dictionary should be populated to include known chip id to system pairs.

Copy link
Collaborator

@coleramos425 coleramos425 left a comment

Choose a reason for hiding this comment

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

Nice start, but I think this PR is still incomplete. To finish the PR, you'll need to update the populate_mspec() function which sets gpu_model for gfx942 archs to consider your new chip id map
https://github.com/ROCm/omniperf/blob/6bc1a3a1a1bec44d0c3fbbe3993199c071efcecc/src/omniperf_soc/soc_base.py#L187-L200

Once these changes are made, re-request my review, and I can add the original Jira ticket filer as a reviewer

src/omniperf_base.py Outdated Show resolved Hide resolved
src/omniperf_soc/soc_base.py Outdated Show resolved Hide resolved
@xuchen-amd xuchen-amd force-pushed the determine_gpu_model_from_chip_id branch 2 times, most recently from 11297ac to 623c2f9 Compare September 18, 2024 16:11
@xuchen-amd xuchen-amd requested a review from a team as a code owner September 18, 2024 16:17
@xuchen-amd xuchen-amd force-pushed the determine_gpu_model_from_chip_id branch from bc1b6c1 to 95e58bf Compare September 18, 2024 19:29
Copy link
Collaborator

@coleramos425 coleramos425 left a comment

Choose a reason for hiding this comment

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

LGTM. Requesting a review from @skyreflectedinmirrors to see if this satisfies his original request:
https://ontrack-internal.amd.com/browse/SWDEV-461358

docs/sphinx/requirements.txt Outdated Show resolved Hide resolved
readthedocs.yaml Outdated Show resolved Hide resolved
src/omniperf_soc/soc_base.py Show resolved Hide resolved
src/omniperf_soc/soc_base.py Outdated Show resolved Hide resolved
src/omniperf_soc/soc_base.py Outdated Show resolved Hide resolved
@xuchen-amd xuchen-amd force-pushed the determine_gpu_model_from_chip_id branch from d7b9807 to 6857b2b Compare September 19, 2024 19:46
@coleramos425
Copy link
Collaborator

Looks good. @xuchen-amd once you fix Python formatting, I will squash and merge.

@xuchen-amd xuchen-amd force-pushed the determine_gpu_model_from_chip_id branch 2 times, most recently from b8ac619 to efc70df Compare September 24, 2024 17:28
@coleramos425 coleramos425 merged commit 0496a46 into ROCm:amd-staging Sep 24, 2024
20 of 22 checks passed
coleramos425 pushed a commit that referenced this pull request Sep 25, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
coleramos425 pushed a commit that referenced this pull request Sep 27, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 16, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 17, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 19, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 19, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 19, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 22, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 22, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 22, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 22, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 22, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 23, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 23, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 23, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 23, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 23, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 23, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 24, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 24, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 24, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 24, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
dgaliffiAMD pushed a commit to dgaliffiAMD/omniperf that referenced this pull request Oct 24, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
dgaliffiAMD pushed a commit to dgaliffiAMD/omniperf that referenced this pull request Oct 24, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
dgaliffiAMD pushed a commit to dgaliffiAMD/omniperf that referenced this pull request Oct 24, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
dgaliffiAMD pushed a commit to dgaliffiAMD/omniperf that referenced this pull request Oct 24, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 29, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 29, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 29, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 29, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 29, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 29, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
xuchen-amd added a commit to xuchen-amd/rocprofiler-compute that referenced this pull request Oct 29, 2024
* Add the ability to determine GPU model from Chip ID for distinguishing MI300 systems by using a built-in dictionary.

Signed-off-by: xuchen-amd <[email protected]>

* Add support for MI300X_A1

Signed-off-by: xuchen-amd <[email protected]>

* Remove MI308X identification using num CUs, and format Python using black.

Signed-off-by: xuchen-amd <[email protected]>

* Add Read the Docs

Signed-off-by: xuchen-amd <[email protected]>

* Add sphinx requirement

Signed-off-by: xuchen-amd <[email protected]>

* Remove gpu_model identification using gpu_arch

Signed-off-by: xuchen-amd <[email protected]>

* Remove OMNIPERF_ARCH_OVERRIDE and its usage. Determining MI300 gpu model solely based on chip id.

Signed-off-by: xuchen-amd <[email protected]>

* Fix Python formatting using black.

Signed-off-by: xuchen-amd <[email protected]>

---------

Signed-off-by: xuchen-amd <[email protected]>
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.

4 participants