Skip to content

[linux-nvidia-6.18] Use device ID range for DGX Spark iGPU#444

Open
arighi wants to merge 2 commits into
NVIDIA:linux-nvidia-6.18-nextfrom
arighi:linux-nvidia-6.18-dgx-spark-device-id
Open

[linux-nvidia-6.18] Use device ID range for DGX Spark iGPU#444
arighi wants to merge 2 commits into
NVIDIA:linux-nvidia-6.18-nextfrom
arighi:linux-nvidia-6.18-dgx-spark-device-id

Conversation

@arighi
Copy link
Copy Markdown
Collaborator

@arighi arighi commented May 27, 2026

Add the DGX Spark iGPU device ID list with a range check covering 0x2E00-0x2E3F to accommodate all possible DGX Spark iGPU PCI device IDs without requiring individual additions.

See also #390


LP: https://bugs.launchpad.net/ubuntu/+source/linux-nvidia-6.17/+bug/2150487

nirmoy and others added 2 commits May 27, 2026 19:54
… existing iommu quirk

BugLink: https://bugs.launchpad.net/bugs/2132033

Add two more device IDs for the existing Spark iommu quirk.

Link: https://bugs.launchpad.net/ubuntu/+source/linux-nvidia-6.14/+bug/2132033

Signed-off-by: Nirmoy Das <nirmoyd@nvidia.com>
Acked-by: Jamie Nguyen <jamien@nvidia.com>
Acked-by: Carol L Soto <csoto@nvidia.com>
Acked-by: Abdur Rahman <abdur.rahman@canonical.com>
Acked-by: Noah Wager <noah.wager@canonical.com>
Signed-off--by: Brad Figg <bfigg@nvidia.com>
(backported from commit cf6aaef linux-nvidia-6.17)
Signed-off-by: Andrea Righi <arighi@nvidia.com>
…GPU iommu quirk

BugLink: https://bugs.launchpad.net/bugs/2150487

Replace the explicit DGX Spark iGPU device ID list with a range check
covering 0x2E00-0x2E3F to accommodate all possible DGX Spark iGPU PCI
device IDs without requiring individual additions.

The original quirk was introduced in
commit ab85863 ("NVIDIA: SAUCE: iommu/arm-smmu-v3:
Set DGX Spark iGPU default domain type to DMA") and
extended with two more IDs in
commit 8dc61ab ("NVIDIA: SAUCE: iommu/arm-smmu-v3:
Add two more DGX Spark iGPU IDs for existing iommu quirk").
Using a range avoids further per-ID additions as new DGX Spark
variants are introduced.

Signed-off-by: Abhishek Sahu <abhsahu@nvidia.com>
Acked-by: Jamie Nguyen <jamien@nvidia.com>
Acked-by: Matthew R. Ochs <mochs@nvidia.com>
Acked-by: Nirmoy Das <nirmoyd@nvidia.com>
Signed-off-by: Brad Figg <bfigg@nvidia.com>
(cherry picked from commit b406505 linux-nvidia-6.17)
Signed-off-by: Andrea Righi <arighi@nvidia.com>
@arighi arighi requested review from ltrager and sforshee May 27, 2026 18:01
@nirmoy nirmoy added the help wanted Extra attention is needed label May 27, 2026
@nirmoy
Copy link
Copy Markdown
Collaborator

nirmoy commented May 27, 2026

Boro review

Latest watcher review: open review

Head: 9aef9944c7bb

This comment is maintained by nv-pr-bot. It is updated when the GitHub watcher publishes a newer review.

Copy link
Copy Markdown
Collaborator

@sforshee sforshee left a comment

Choose a reason for hiding this comment

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

Acked-by: Seth Forshee <sforshee@nvidia.com>

@nvmochs
Copy link
Copy Markdown
Collaborator

nvmochs commented May 27, 2026

@nirmoy Is this needed for Vera? I thought these commits were only for Spark?

@ltrager
Copy link
Copy Markdown
Collaborator

ltrager commented May 27, 2026

@nirmoy Is this needed for Vera? I thought these commits were only for Spark?

I think there may have been some confusion. We started down the list of patches tracked in your spreadsheet. We're refocusing on what is currently available in 6.17.

@nvmochs
Copy link
Copy Markdown
Collaborator

nvmochs commented May 27, 2026

@nirmoy Is this needed for Vera? I thought these commits were only for Spark?

I think there may have been some confusion. We started down the list of patches tracked in your spreadsheet. We're refocusing on what is currently available in 6.17.

These patches will also be in 6.17 as it supports the Spark platform. AFAIK, the linux-nvidia-6.18 reference kernel does not need to support Spark (i.e. these are not needed on linux-nvidia-6.18 if they are Spark-only).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants