[PATCH 0/5] amd_iommu fixes staged for 11.1

Alejandro Jimenez posted 5 patches 3 weeks, 4 days ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260630220806.1758748-1-alejandro.j.jimenez@oracle.com
Maintainers: Alejandro Jimenez <alejandro.j.jimenez@oracle.com>, Sairaj Kodilkar <sarunkod@amd.com>, "Michael S. Tsirkin" <mst@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>
hw/i386/amd_iommu.c | 168 +++++++++++++++++++++++++++++++-------------
hw/i386/amd_iommu.h |  63 -----------------
2 files changed, 120 insertions(+), 111 deletions(-)
[PATCH 0/5] amd_iommu fixes staged for 11.1
Posted by Alejandro Jimenez 3 weeks, 4 days ago
Hi Michael,

I have staged these recently reviewed AMD vIOMMU fixes for 11.1:

https://github.com/aljimenezb/qemu/commits/amdvi-next

Base: 30e8a06b64aa ("Merge tag 'net-pull-request' of https://github.com/jasowang/qemu into staging")
Tip:  018c1dfb3e8c ("amd_iommu: Fix endianness handling for command buffer entries")

Patch 1 is an independent fix previously posted at:
https://lore.kernel.org/qemu-devel/20260616174620.438468-1-alejandro.j.jimenez@oracle.com/

Patches 2-5 address a Coverity finding and big-endian portability
issues. They were reviewed in:
https://lore.kernel.org/qemu-devel/20260624195925.1254462-1-alejandro.j.jimenez@oracle.com/

I did basic smoke testing on an AMD Zen4 host using the AMD vIOMMU device.
Completed several guest boot and reboot cycles with xtsup both enabled and
disabled. No regressions were observed in guest startup or reboot behavior.

There are two other outstanding series pending review, but I am unlikely to
complete review/testing by myself for those before the upcoming soft-freeze
deadline.

Thank you,
Alejandro

Alejandro Jimenez (4):
  amd_iommu: Return int from page walk status helpers
  amd_iommu: Decode XT interrupt control register without bitfields
  amd_iommu: Decode IRTEs without bitfields
  amd_iommu: Fix endianness handling for command buffer entries

David Hoppenbrouwers (1):
  amd_iommu: Fix opcode reported in invalid command handling

 hw/i386/amd_iommu.c | 168 +++++++++++++++++++++++++++++++-------------
 hw/i386/amd_iommu.h |  63 -----------------
 2 files changed, 120 insertions(+), 111 deletions(-)


base-commit: 30e8a06b64aa58a3990ba39cb5d09531e7d265e0
-- 
2.47.3
Re: [PATCH 0/5] amd_iommu fixes staged for 11.1
Posted by Michael S. Tsirkin 3 weeks, 4 days ago
On Tue, Jun 30, 2026 at 10:08:01PM +0000, Alejandro Jimenez wrote:
> Hi Michael,
> 
> I have staged these recently reviewed AMD vIOMMU fixes for 11.1:
> 
> https://github.com/aljimenezb/qemu/commits/amdvi-next
> Base: 30e8a06b64aa ("Merge tag 'net-pull-request' of https://github.com/jasowang/qemu into staging")
> Tip:  018c1dfb3e8c ("amd_iommu: Fix endianness handling for command buffer entries")
> 
> Patch 1 is an independent fix previously posted at:
> https://lore.kernel.org/qemu-devel/20260616174620.438468-1-alejandro.j.jimenez@oracle.com/
> 
> Patches 2-5 address a Coverity finding and big-endian portability
> issues. They were reviewed in:
> https://lore.kernel.org/qemu-devel/20260624195925.1254462-1-alejandro.j.jimenez@oracle.com/

Great, I'm applying these. Thanks!


> 
> I did basic smoke testing on an AMD Zen4 host using the AMD vIOMMU device.
> Completed several guest boot and reboot cycles with xtsup both enabled and
> disabled. No regressions were observed in guest startup or reboot behavior.
> 
> There are two other outstanding series pending review, but I am unlikely to
> complete review/testing by myself for those before the upcoming soft-freeze
> deadline.
> 
> Thank you,
> Alejandro


To be frank, these are bugfixes and such changes are okay after soft freeze. Thanks!

> Alejandro Jimenez (4):
>   amd_iommu: Return int from page walk status helpers
>   amd_iommu: Decode XT interrupt control register without bitfields
>   amd_iommu: Decode IRTEs without bitfields
>   amd_iommu: Fix endianness handling for command buffer entries
> 
> David Hoppenbrouwers (1):
>   amd_iommu: Fix opcode reported in invalid command handling
> 
>  hw/i386/amd_iommu.c | 168 +++++++++++++++++++++++++++++++-------------
>  hw/i386/amd_iommu.h |  63 -----------------
>  2 files changed, 120 insertions(+), 111 deletions(-)
> 
> 
> base-commit: 30e8a06b64aa58a3990ba39cb5d09531e7d265e0
> -- 
> 2.47.3