From: "Renato Machelett" <renato@machelett.com>
To: "'Baolu Lu'" <baolu.lu@linux.intel.com>
Cc: <robin.murphy@arm.com>, <joro@8bytes.org>,
<iommu@lists.linux.dev>, <kernel-team@lists.ubuntu.com>
Subject: RE: iommu/dma: SATA RMRR reservation omission in shared group on HP Gen8
Date: Mon, 19 Oct 2026 12:46:23 +0200 [thread overview]
Message-ID: <034b01dd5fb7$159f53c0$40ddfb40$@machelett.com> (raw)
In-Reply-To: <ea131a99-d3f0-4d52-a076-d80d84d690e8@linux.intel.com>
On 10/19/2026, Baolu Lu wrote:
> The early return is still there. Have you tried reserving the second
> device's RMRR in the IOVA allocator as well? Something like:
>
> diff --git a/drivers/iommu/dma-iommu.c b/drivers/iommu/dma-iommu.c
> index 68ee57800fef..e9b39009f109 100644
> --- a/drivers/iommu/dma-iommu.c
> +++ b/drivers/iommu/dma-iommu.c
> @@ -704,7 +704,7 @@ static int iommu_dma_init_domain(struct iommu_domain *domain, struct device *dev
> return -EFAULT;
> }
>
> - return 0;
> + return iova_reserve_iommu_regions(dev, domain);
> }
>
> init_iova_domain(iovad, 1UL << order, base_pfn);
>
> This would at least confirm your analysis.
Hi Baolu,
I have not tested a patched kernel because this is a production server.
In the October 2024 thread, Jerry Snitselaar proposed the equivalent
change, and Robin Murphy pointed out why calling
iova_reserve_iommu_regions() in the already-initialized branch is unsafe
as a general fix (https://lkml.iu.edu/2411.1/00575.html):
> It's also possible for the second device to get here after the first
> device has already started using the domain, so at that point it's no
> longer safe to use reserve_iova() due to how it merges overlapping and
> adjacent rbtree nodes, which could really screw things up once there
> are normal allocations in the tree as well.
Best regards,
Renato
prev parent reply other threads:[~2026-10-19 10:46 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-18 1:50 iommu/dma: SATA RMRR reservation omission in shared group on HP Gen8 Renato Machelett
2026-10-19 7:27 ` Baolu Lu
2026-10-19 10:46 ` Renato Machelett [this message]
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='034b01dd5fb7$159f53c0$40ddfb40$@machelett.com' \
--to=renato@machelett.com \
--cc=baolu.lu@linux.intel.com \
--cc=iommu@lists.linux.dev \
--cc=joro@8bytes.org \
--cc=kernel-team@lists.ubuntu.com \
--cc=robin.murphy@arm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox