Skip to content

touchbar: missing include#175

Merged
marcan merged 1 commit intoAsahiLinux:bits/230-touchbarfrom
Perpixel:tb_missing_inc
Aug 21, 2023
Merged

touchbar: missing include#175
marcan merged 1 commit intoAsahiLinux:bits/230-touchbarfrom
Perpixel:tb_missing_inc

Conversation

@Perpixel
Copy link
Copy Markdown

No description provided.

@jannau
Copy link
Copy Markdown
Member

jannau commented Aug 18, 2023

Please squash both commits. We usually provide such simple fixes as fixup commits which can be easily squashed into the "broken" commit on the next rebase. git commit -s --amend --fixup 51e31ee6a775 will format the commit message automatically so that git rebase --autosquash ... will result in fix being squashed into the "broken" commit.
Please use bits/230-touchbar as base branch for the pull request. asahi and asahi-wip are just integration branches which will be rewritten as merges from the bits/* branches.

@Perpixel Perpixel changed the base branch from asahi-wip to bits/230-touchbar August 18, 2023 12:34
@Perpixel Perpixel marked this pull request as draft August 18, 2023 12:40
Signed-off-by: Guillaume Plourde <gplourde@protonmail.com>
@Perpixel Perpixel marked this pull request as ready for review August 18, 2023 13:05
@marcan marcan merged commit 91fe761 into AsahiLinux:bits/230-touchbar Aug 21, 2023
asdfugil pushed a commit to HoolockLinux/linux that referenced this pull request May 10, 2025
Commit <5518f239aff1> ("iommu/vt-d: Move scalable mode ATS enablement to
probe path") changed the PCI ATS enablement logic to run earlier,
specifically before the default domain attachment.

On some client platforms, this change resulted in boot failures, causing
the kernel to panic with the following message and call trace:

 Kernel panic - not syncing: DMAR hardware is malfunctioning
 CPU: 0 UID: 0 PID: 1 Comm: swapper/0 Not tainted 6.14.0-rc3+ AsahiLinux#175
 Call Trace:
  <TASK>
  dump_stack_lvl+0x6f/0xb0
  dump_stack+0x10/0x16
  panic+0x10a/0x2b7
  iommu_enable_translation.cold+0xc/0xc
  intel_iommu_init+0xe39/0xec0
  ? trace_hardirqs_on+0x1e/0xd0
  ? __pfx_pci_iommu_init+0x10/0x10
  pci_iommu_init+0xd/0x40
  do_one_initcall+0x5b/0x390
  kernel_init_freeable+0x26d/0x2b0
  ? __pfx_kernel_init+0x10/0x10
  kernel_init+0x15/0x120
  ret_from_fork+0x35/0x60
  ? __pfx_kernel_init+0x10/0x10
  ret_from_fork_asm+0x1a/0x30
 RIP: 1f0f:0x0
 Code: Unable to access opcode bytes at 0xffffffffffffffd6.
 RSP: 0000:0000000000000000 EFLAGS: 841f0f2e66 ORIG_RAX:
      1f0f2e6600000000
 RAX: 0000000000000000 RBX: 1f0f2e6600000000 RCX:
      2e66000000000084
 RDX: 0000000000841f0f RSI: 000000841f0f2e66 RDI:
      00841f0f2e660000
 RBP: 00841f0f2e660000 R08: 00841f0f2e660000 R09:
      000000841f0f2e66
 R10: 0000000000841f0f R11: 2e66000000000084 R12:
      000000841f0f2e66
 R13: 0000000000841f0f R14: 2e66000000000084 R15:
      1f0f2e6600000000
  </TASK>
 ---[ end Kernel panic - not syncing: DMAR hardware is malfunctioning ]---

Fix this by reverting the timing change for ATS enablement introduced by
the offending commit and restoring the previous behavior.

Fixes: 5518f23 ("iommu/vt-d: Move scalable mode ATS enablement to probe path")
Reported-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Closes: https://lore.kernel.org/linux-iommu/01b9c72f-460d-4f77-b696-54c6825babc9@linux.intel.com/
Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Tested-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Reviewed-by: Kevin Tian <kevin.tian@intel.com>
Link: https://lore.kernel.org/r/20250416073608.1799578-1-baolu.lu@linux.intel.com
Signed-off-by: Joerg Roedel <jroedel@suse.de>
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