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 support for RX 7900 XT #153

Open
deadmeu opened this issue Mar 4, 2023 · 12 comments
Open

Add support for RX 7900 XT #153

deadmeu opened this issue Mar 4, 2023 · 12 comments

Comments

@deadmeu
Copy link

deadmeu commented Mar 4, 2023

Running version 1.4 seems to work just fine with the Sapphire RX 7900 XT.

This might help:

> cat /usr/share/hwdata/pci.ids | grep "RX 7900"
744c  Navi 31 [Radeon RX 7900 XT/7900 XTX]
	1da2 e471  NITRO+ RX 7900 XTX Vapor-X

Happy to provide more info or even a PR if someone could let me know the steps required.

@clbr
Copy link
Owner

clbr commented Mar 4, 2023

Sorry? Did you mean this as a reply somewhere else?

@deadmeu
Copy link
Author

deadmeu commented Mar 4, 2023

Sorry? Did you mean this as a reply somewhere else?

It seems like my card isn't officially supported?
Unknown Radeon card. <= R500 won't work, new cards might.
I'm just raising a ticket to make it officially supported.

@clbr
Copy link
Owner

clbr commented Mar 4, 2023

You wrote above it works just fine?

@deadmeu
Copy link
Author

deadmeu commented Mar 4, 2023

I assume it's not preferred for "Unknown Radeon card" and "... running on UNKNOWN_CHIP ..." to be presented to the user?

@clbr
Copy link
Owner

clbr commented Mar 4, 2023

The pci ids for new cards will eventually be added. It needs a new source as they're no longer updated in the kernel where they were gotten from til now.

@Umio-Yasuno
Copy link

The AMDGPU chip can be detected from chip_rev and family_id in amdgpu_gpu_info. The pci id is not required.

@Umio-Yasuno
Copy link

Umio-Yasuno commented Mar 5, 2023

Draft: Umio-Yasuno@88e2bb5

Detection of GFX1036/Raphael, GFX1037/Mendocino, GFX1100/Navi31, GFX1101/Navi32, GFX1102/Navi33, GFX1103_R1/Phoenix, GFX1103_R2/Phoenix .

@deadmeu
Copy link
Author

deadmeu commented Mar 6, 2023

Draft: Umio-Yasuno@88e2bb5

Detection of GFX1036/Raphael, GFX1037/Mendocino, GFX1100/Navi31, GFX1101/Navi31, GFX1102/Navi33, GFX1103_R1/Phoenix, GFX1103_R2/Phoenix .

Nice! This is picking my card up as running on "GFX1101".

@Umio-Yasuno
Copy link

Umio-Yasuno commented Mar 6, 2023

Draft: Umio-Yasuno@88e2bb5
Detection of GFX1036/Raphael, GFX1037/Mendocino, GFX1100/Navi31, GFX1101/Navi31, GFX1102/Navi33, GFX1103_R1/Phoenix, GFX1103_R2/Phoenix .

Nice! This is picking my card up as running on "GFX1101".

Oops, GFX1100 was missing in family_str.c.

Fixed: Umio-Yasuno@accd6ff

@deadmeu
Copy link
Author

deadmeu commented Mar 7, 2023

Oops, GFX1100 was missing in family_str.c.

Fixed: Umio-Yasuno@accd6ff

I'm not sure what the correct code should be, but it is reporting GFX1100 now 🙂. The version seems to be missing: https://i.imgur.com/sL10Zx2.png

@Umio-Yasuno
Copy link

AFAIK, AMD Radeon RX 7900 XT is Navi31 based.

	GFX1100, // Navi31
	GFX1101, // Navi32
	GFX1102, // Navi33
	GFX1103_R1,  // Phoenix
	GFX1103_R2, // Phoenix
	GFX1036, // Raphael
	GFX1037, // Mendocino

@Umio-Yasuno
Copy link

Umio-Yasuno commented Mar 15, 2023

I am currently developing software in the Rust language that will be the successor to radeontop.
https://github.com/Umio-Yasuno/amdgpu_top

@clbr is busy and radeontop has issues such as reporting deleted values on Vega/GFX9/soc15 and Navi/GFX10.

#93

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 a pull request may close this issue.

3 participants