Skip to content

fix: acs override to separate iommu groups#1711

Merged
brianmcgillion merged 1 commit intotiiuae:mainfrom
enesoztrk:fix/eth-passthrough
Jan 27, 2026
Merged

fix: acs override to separate iommu groups#1711
brianmcgillion merged 1 commit intotiiuae:mainfrom
enesoztrk:fix/eth-passthrough

Conversation

@enesoztrk
Copy link
Copy Markdown
Contributor

@enesoztrk enesoztrk commented Jan 26, 2026

Description of Changes

On System76 darp11-b and Dell 7330, the Ethernet controller (00:1f.6) is in the same IOMMU group as audio devices (00:1f.0, 00:1f.3, 00:1f.4, 00:1f.5). This prevents passing the Ethernet to netvm while audio goes to audiovm, since VFIO requires entire IOMMU groups. Adding a kernel patch (pcie_acs_override) that overrides ACS capabilities for specific PCI devices by vendor:device ID. This allows devices to be isolated into separate IOMMU groups.

Type of Change

  • New Feature
  • Bug Fix
  • Improvement / Refactor

Related Issues / Tickets

SSRCSP-7725

Checklist

  • Clear summary in PR description
  • Detailed and meaningful commit message(s)
  • Commits are logically organized and squashed if appropriate
  • Contribution guidelines followed
  • Ghaf documentation updated with the commit - https://tiiuae.github.io/ghaf/
  • Author has run make-checks and it passes
  • All automatic GitHub Action checks pass - see actions
  • Author has added reviewers and removed PR draft status

Testing Instructions

Applicable Targets

  • Orin AGX aarch64
  • Orin NX aarch64
  • Lenovo X1 x86_64
  • Dell Latitude 7330 x86_64
  • System 76 x86_64

Installation Method

  • Requires full re-installation
  • Can be updated with nixos-rebuild ... switch
  • Other:

Test Steps To Verify:

  1. All the VMs must be up and running.
  2. audio-vm must only have the ethint0 Ethernet interface (no other Ethernet interfaces present).
  3. net-vm must have an eth0 network interface and eth0 has an IP address after connected to the network.
  4. Connect to internet through built-in ethernet port (Do not connect wifi)

Signed-off-by: Enes Öztürk <enes.ozturk@unikie.com>
@enesoztrk enesoztrk force-pushed the fix/eth-passthrough branch from f41c87c to ab1e602 Compare January 27, 2026 07:22
@enesoztrk enesoztrk marked this pull request as ready for review January 27, 2026 07:23
@enesoztrk enesoztrk added the Needs Testing CI Team to pre-verify label Jan 27, 2026
@enesoztrk enesoztrk requested a review from mbssrc January 27, 2026 07:39
@milva-unikie
Copy link
Copy Markdown

Tested on Darter Pro and Dell 7330 (new images)

  • Ethernet is passed through to the net-vm
  • Ethernet passthrough to the other VMs work

Pre-merge pipeline is still running, we had to restart the run because a robot finger ran out of battery. All tests are good, I just re-triggered it to get the green check in Git as well.

@brianmcgillion brianmcgillion merged commit 11ebeae into tiiuae:main Jan 27, 2026
32 checks passed
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.

3 participants