From: Jason Gunthorpe <jgg@nvidia.com>
To: Josua Mayer <josua@solid-run.com>
Cc: "iommu@lists.linux.dev" <iommu@lists.linux.dev>,
Joerg Roedel <joro@8bytes.org>,
Robin Murphy <robin.murphy@arm.com>,
Will Deacon <will@kernel.org>, Kevin Tian <kevin.tian@intel.com>,
"patches@lists.linux.dev" <patches@lists.linux.dev>,
Samiullah Khawaja <skhawaja@google.com>,
Rabeeh Khoury <rabeeh@solid-run.com>,
Yazan Shhady <yazan.shhady@solid-run.com>,
Jon Nettleton <jon@solid-run.com>,
Mikhail Anikin <mikhail.anikin@solid-run.com>
Subject: Re: [PATCH v3 2/2] iommupt: Avoid rewalking during map
Date: Sat, 9 May 2026 16:40:53 -0300 [thread overview]
Message-ID: <20260509194053.GM9254@nvidia.com> (raw)
In-Reply-To: <321c2e57-6a17-4aef-ba42-d2ebd577e472@solid-run.com>
On Sat, May 09, 2026 at 05:41:02PM +0000, Josua Mayer wrote:
> This patch has landed in v7.1-rc1 and introduced a regression on the
> arm64 SolidRun LX2160A Clearfog-CX board.
>
> Since d6c65b0fd621 ("iommupt: Avoid rewalking during map"),
> boot now stalls - printing many smmu errors.
>
> Please find attached full boot logs for reference:
>
> - good.txt @ 99fb8afa16ad
> - bad.txt @ d6c65b0fd621
> - config.txt: .config from the build
>
> Is there any advice how to deal with this?
Does this help?
--- a/drivers/iommu/iommu.c
+++ b/drivers/iommu/iommu.c
@@ -2669,7 +2669,7 @@ int iommu_map_nosync(struct iommu_domain *domain, unsigned long iova,
return 0;
}
ret = __iommu_map_domain_pgtbl(domain, iova, paddr, size, prot, gfp);
- if (!ret)
+ if (ret)
return ret;
trace_map(iova, paddr, size);
Although it would be surprising your system triggers a ret!=0 while
booting successfully, that could by why it wasn't noticed in any
testing so far..
Jason
next prev parent reply other threads:[~2026-05-09 19:43 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-27 19:30 [PATCH v3 0/2] Let iommupt manage changes in page size internally Jason Gunthorpe
2026-02-27 19:30 ` [PATCH v3 1/2] iommupt: Directly call iommupt's unmap_range() Jason Gunthorpe
2026-02-27 19:30 ` [PATCH v3 2/2] iommupt: Avoid rewalking during map Jason Gunthorpe
2026-05-09 17:41 ` Josua Mayer
2026-05-09 19:40 ` Jason Gunthorpe [this message]
2026-05-09 20:25 ` Josua Mayer
2026-05-09 23:39 ` Jason Gunthorpe
2026-03-03 3:55 ` [PATCH v3 0/2] Let iommupt manage changes in page size internally Baolu Lu
2026-03-17 12:58 ` Joerg Roedel
-- strict thread matches above, loose matches on Subject: below --
2026-02-28 1:00 [PATCH v3 2/2] iommupt: Avoid rewalking during map kernel test robot
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=20260509194053.GM9254@nvidia.com \
--to=jgg@nvidia.com \
--cc=iommu@lists.linux.dev \
--cc=jon@solid-run.com \
--cc=joro@8bytes.org \
--cc=josua@solid-run.com \
--cc=kevin.tian@intel.com \
--cc=mikhail.anikin@solid-run.com \
--cc=patches@lists.linux.dev \
--cc=rabeeh@solid-run.com \
--cc=robin.murphy@arm.com \
--cc=skhawaja@google.com \
--cc=will@kernel.org \
--cc=yazan.shhady@solid-run.com \
/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.