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 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.