Skip to content

Conversation

@13r0ck
Copy link

@13r0ck 13r0ck commented Nov 14, 2022

This fixes the pang sd card issue

@13r0ck 13r0ck marked this pull request as ready for review November 14, 2022 22:29
@XV-02 XV-02 self-assigned this Nov 15, 2022
@XV-02
Copy link

XV-02 commented Nov 15, 2022

This isn't resolving the core issue of detection on hot-plug. However, it does seem to be having some kind of effect.
Previously:

  • Booting with a micro-SD card present would result in detection and mounting.
  • Hot plugging does not work, regardless of whether a card was present at boot.

With this reversion:

  • Booting with a micro-SD card present results in detection and mounting.
  • Hot plugging works if card was present at boot.
  • Hot plugging does not work if card was not present at boot.

As a note: Bouncing the kernel module related to SD card reading had the same effect - enabling hot plug detection - both with and without this PR.

@13r0ck
Copy link
Author

13r0ck commented Nov 15, 2022

This issue is unrelated to this. Now that we know the issue is micro sd hot plug we need to find the point release that broke it. 5.18 seems broken, 5.13 seems fine. Need to narrow that gap.

@13r0ck 13r0ck closed this Nov 15, 2022
@13r0ck 13r0ck deleted the revert-08ebf90 branch December 14, 2022 23:32
mmstick pushed a commit that referenced this pull request Oct 22, 2025
[ Upstream commit fbe6070 ]

In legacy mode, SSPTPTR is ignored if TT is not 00b or 01b. SSPTPTR
maybe uninitialized or zero in that case and may cause oops like:

 Oops: general protection fault, probably for non-canonical address
       0xf00087d3f000f000: 0000 [#1] SMP NOPTI
 CPU: 2 UID: 0 PID: 786 Comm: cat Not tainted 6.16.0 #191 PREEMPT(voluntary)
 Hardware name: QEMU Standard PC (Q35 + ICH9, 2009), BIOS 1.17.0-5.fc42 04/01/2014
 RIP: 0010:pgtable_walk_level+0x98/0x150
 RSP: 0018:ffffc90000f279c0 EFLAGS: 00010206
 RAX: 0000000040000000 RBX: ffffc90000f27ab0 RCX: 000000000000001e
 RDX: 0000000000000003 RSI: f00087d3f000f000 RDI: f00087d3f0010000
 RBP: ffffc90000f27a00 R08: ffffc90000f27a98 R09: 0000000000000002
 R10: 0000000000000000 R11: 0000000000000000 R12: f00087d3f000f000
 R13: 0000000000000000 R14: 0000000040000000 R15: ffffc90000f27a98
 FS:  0000764566dcb740(0000) GS:ffff8881f812c000(0000) knlGS:0000000000000000
 CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
 CR2: 0000764566d44000 CR3: 0000000109d81003 CR4: 0000000000772ef0
 PKRU: 55555554
 Call Trace:
  <TASK>
  pgtable_walk_level+0x88/0x150
  domain_translation_struct_show.isra.0+0x2d9/0x300
  dev_domain_translation_struct_show+0x20/0x40
  seq_read_iter+0x12d/0x490
...

Avoid walking the page table if TT is not 00b or 01b.

Fixes: 2b437e8 ("iommu/vt-d: debugfs: Support dumping a specified page table")
Signed-off-by: Vineeth Pillai (Google) <vineeth@bitbyteword.org>
Reviewed-by: Kevin Tian <kevin.tian@intel.com>
Link: https://lore.kernel.org/r/20250814163153.634680-1-vineeth@bitbyteword.org
Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
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.

4 participants