All of lore.kernel.org
 help / color / mirror / Atom feed
From: Niklas Cassel <cassel@kernel.org>
To: snoep <snoepke@live.nl>
Cc: "alvinwylim@gmail.com" <alvinwylim@gmail.com>,
	"artmoty@gmail.com" <artmoty@gmail.com>,
	"david.laight.linux@gmail.com" <david.laight.linux@gmail.com>,
	"kernel@wantstofly.org" <kernel@wantstofly.org>,
	"linux-ide@vger.kernel.org" <linux-ide@vger.kernel.org>,
	"mario.limonciello@amd.com" <mario.limonciello@amd.com>,
	"mikael1022bzh@gmail.com" <mikael1022bzh@gmail.com>,
	"roland.waltersson@netinsight.net"
	<roland.waltersson@netinsight.net>
Subject: Re: [PATCH] ahci: force 32-bit DMA for JMicron JMB582/JMB585
Date: Mon, 7 Sep 2026 12:28:24 +0200	[thread overview]
Message-ID: <ap6RyP28fxXNjRWS@ryzen> (raw)
In-Reply-To: <AM8P193MB2659454F98AD53DEB002CF8AA9B42@AM8P193MB2659.EURP193.PROD.OUTLOOK.COM>

Hello Sven,

On Sat, Sep 05, 2026 at 09:04:57PM +0000, snoep wrote:
> So I was unable to reproduce the corruption on this Raspberry Pi 5
> with either the normal 64-bit DMA configuration or the Raspberry Pi
> 32-bit PCIe DMA overlay.
> 
> Again, since this JMB585 has no IOMMU group, I do not consider this a
> test of the Intel/ARM IOMMU enforcing-mode case.

Thank you for your testing!

Considering that Rasberry Pi seems to hand out 64-bit PCI addresses by
default, which is why they even have the (from JMB585's point of view)
completely meaningless pcie-32bit-dma-pi5 DT overlay.

Considering that you could run the reproducer for several iterations
without any issues, I am confident that JMB585 does indeed handle
64-bit DMA addresses perfectly fine, and we will drop the JMB585 quirk
as soon as there is fix for the AMD IOMMU issue.


> 
> However, I am still trying to determine what caused the repeated EXT4
> metadata corruption in the original RAID1 setup, and I would appreciate
> guidance on what would be most useful to test next.
> 
> Would repeating a sustained checksumming workload with the original
> two-SSD md RAID1 setup be useful, or is there another workload, tracing
> option, kernel configuration or test that would better help distinguish
> between the JMB585/PCIe path, md RAID1, EXT4, or another component?

I have no idea why you saw corruption on ext4 using 6.12.75+rpt-rpi-2712
downstream kernel.

Considering that they seem okay with carrying dirty workarounds for issues
for several months, that they don't even bother reporting to upstream, I
would strongly recommend running an upstream kernel instead.


I did see some reports from people that the JMB585 could overheat:
https://github.com/geerlingguy/raspberry-pi-pcie-devices/issues/354
https://forums.truenas.com/t/jmb585-m-2-to-sata-card-doesnt-see-ssd/8231

If you don't have a heatsink, perhaps it could be something to try.


Kind regards,
Niklas

       reply	other threads:[~2026-09-07 10:28 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <AM8P193MB2659454F98AD53DEB002CF8AA9B42@AM8P193MB2659.EURP193.PROD.OUTLOOK.COM>
2026-09-07 10:28 ` Niklas Cassel [this message]
2026-09-06 11:48 [PATCH] ahci: force 32-bit DMA for JMicron JMB582/JMB585 snoep
  -- strict thread matches above, loose matches on Subject: below --
2026-09-03 13:35 Roland Waltersson
2026-09-03 21:40 ` Niklas Cassel
2026-09-03 22:36   ` Mario Limonciello
2026-09-04  5:20     ` Roland Waltersson
2026-09-04 11:14       ` Niklas Cassel
2026-09-04 12:38         ` Mario Limonciello
2026-09-04 12:50           ` Niklas Cassel
2026-09-04 16:55             ` Mikael Etienne
2026-09-05 11:38               ` Alvin Lim
2026-09-05 12:26                 ` Mario Limonciello
2026-09-06  5:05                   ` Alvin Lim
2026-09-06 19:27                     ` John Smith
2026-09-05 14:13               ` Lennert Buytenhek
2026-04-03  5:04 Arthur Husband
2026-04-03  7:02 ` Damien Le Moal
2026-04-03  8:12 ` Niklas Cassel
2026-04-03  8:19 ` Niklas Cassel
2026-04-03  5:02 Arthur Husband

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ap6RyP28fxXNjRWS@ryzen \
    --to=cassel@kernel.org \
    --cc=alvinwylim@gmail.com \
    --cc=artmoty@gmail.com \
    --cc=david.laight.linux@gmail.com \
    --cc=kernel@wantstofly.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=mario.limonciello@amd.com \
    --cc=mikael1022bzh@gmail.com \
    --cc=roland.waltersson@netinsight.net \
    --cc=snoepke@live.nl \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.