Re: [PATCH v3 0/7] amd_iommu: Fixes to align with AMDVi specification

2025-06-16 Thread Alejandro Jimenez
Hi Phil, On 6/16/25 2:59 AM, Philippe Mathieu-Daudé wrote: Hi Alejandro, On 12/6/25 22:59, Alejandro Jimenez wrote: Hi Ethan, On 6/12/25 4:36 AM, Ethan MILON wrote: Hi, Is this series the right place to include the following minor fix? I would defer this change for two reasons: 1) This

Re: [PATCH v3 0/7] amd_iommu: Fixes to align with AMDVi specification

2025-06-16 Thread Philippe Mathieu-Daudé
Hi Alejandro, On 12/6/25 22:59, Alejandro Jimenez wrote: Hi Ethan, On 6/12/25 4:36 AM, Ethan MILON wrote: Hi, Is this series the right place to include the following minor fix? I would defer this change for two reasons: 1) This series has been reviewed and tested already. I was hoping it

Re: [PATCH v3 0/7] amd_iommu: Fixes to align with AMDVi specification

2025-06-12 Thread Alejandro Jimenez
Hi Ethan, On 6/12/25 4:36 AM, Ethan MILON wrote: Hi, Is this series the right place to include the following minor fix? I would defer this change for two reasons: 1) This series has been reviewed and tested already. I was hoping it would be included on the Jun 1st pull but I sent v3 too la

Re: [PATCH v3 0/7] amd_iommu: Fixes to align with AMDVi specification

2025-06-12 Thread Ethan MILON
Hi, Is this series the right place to include the following minor fix? diff --git a/hw/i386/amd_iommu.c b/hw/i386/amd_iommu.c index 0775c..18d30e1 100644 --- a/hw/i386/amd_iommu.c +++ b/hw/i386/amd_iommu.c @@ -140,7 +140,7 @@ static void amdvi_writeq(AMDVIState *s, hwaddr addr, uint64_t val) {

[PATCH v3 0/7] amd_iommu: Fixes to align with AMDVi specification

2025-05-29 Thread Alejandro Jimenez
The main reason for sending this new revision so soon is that v2 included a duplicated [PATCH 5/7]. I fixed a typo in the commit subject and missed removing the old patch. Apologies for the mistake. Additional changes in v3: - Fixed typo on [PATCH 1/7] subject line (s/Miscellanous/Miscellaneous/).