Skip to content

更新 amdgpu.ids 文件 #5316

Merged
Glavo merged 4 commits intoHMCL-dev:mainfrom
CiiLu:amdgpu
Mar 29, 2026
Merged

更新 amdgpu.ids 文件 #5316
Glavo merged 4 commits intoHMCL-dev:mainfrom
CiiLu:amdgpu

Conversation

@CiiLu
Copy link
Copy Markdown
Contributor

@CiiLu CiiLu commented Jan 24, 2026

No description provided.

7480, C3, AMD Radeon RX 7600S
7480, C7, AMD Radeon RX 7600M XT
7480, CF, AMD Radeon RX 7600
7481, C7, AMD Steam Machine
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

为什么这里不是制表符?

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the bundled amdgpu.ids database (sourced from Mesa/libdrm) used by HMCLCore’s Linux GPU detection to map AMD PCI device/revision IDs to human-friendly GPU names.

Changes:

  • Update upstream reference URL for amdgpu.ids and refresh entries to a newer libdrm snapshot.
  • Add new AMD GPU ID/revision mappings and adjust some existing product names.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +652 to +655
75A0, 00, AMD Instinct MI350X
75A3, 00, AMD Instinct MI355X
75B0, 00, AMD Instinct MI350X VF
75B3, 00, AMD Instinct MI355X VF
Copy link

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

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

LinuxGPUDetector builds the lookup prefix with String.format("%X,\t%X,\t", deviceId, revision) (no zero-padding). If a device reports revision 0, the prefix will contain \t0\t, which will not match these newly added \t00\t entries, so the name lookup will fail and fall back to PCI IDs. Consider adding equivalent entries with revision 0 (or updating the lookup to use %02X in a follow-up) to avoid this mismatch for revision-zero devices.

Copilot uses AI. Check for mistakes.
@Glavo Glavo merged commit 356df5d into HMCL-dev:main Mar 29, 2026
5 checks passed
@CiiLu CiiLu deleted the amdgpu branch April 5, 2026 06:20
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.

3 participants