[v2] hw/riscv/riscv-iommu.c: check for misaligned IOHGATP_PPN

[PATCH v2 0/1] hw/riscv/riscv-iommu.c: check for misaligned IOHGATP_PPN

Daniel Henrique Barboza posted 1 patch 3 weeks, 2 days ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260702203616.1795588-1-daniel.barboza@oss.qualcomm.com

Maintainers: Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Weiwei Li <liwei1518@gmail.com>, Daniel Henrique Barboza <daniel.barboza@oss.qualcomm.com>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Chao Liu <chao.liu.zevorn@gmail.com>

hw/riscv/riscv-iommu.c | 15 +++++++++++++++
1 file changed, 15 insertions(+)

Expand all Fold all

[PATCH v2 0/1] hw/riscv/riscv-iommu.c: check for misaligned IOHGATP_PPN

Posted by Daniel Henrique Barboza 3 weeks, 2 days ago

Hi,

This new version fixes a blunder I made in v1 pointed by Chao and
Nutty - I was using the PPN value, instead of the page table base
address, to check for the 16kb alignment. 

We no longer need to change qos-riscv-iommu.h because now we're doing
the right thing.

Changes from v1:
- use PPN_PHYS(iohgatp_ppn), i.e. the page table base address, to 
  check for the 16kb alignment
- v1 link: https://lore.kernel.org/qemu-devel/20260630203155.2103084-1-daniel.barboza@oss.qualcomm.com/


Daniel Henrique Barboza (1):
  hw/riscv/riscv-iommu.c: check for misaligned IOHGATP_PPN

 hw/riscv/riscv-iommu.c | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

-- 
2.43.0

Re: [PATCH v2 0/1] hw/riscv/riscv-iommu.c: check for misaligned IOHGATP_PPN

Posted by Alistair Francis 2 weeks, 6 days ago

On Fri, Jul 3, 2026 at 6:37 AM Daniel Henrique Barboza
<daniel.barboza@oss.qualcomm.com> wrote:
>
> Hi,
>
> This new version fixes a blunder I made in v1 pointed by Chao and
> Nutty - I was using the PPN value, instead of the page table base
> address, to check for the 16kb alignment.
>
> We no longer need to change qos-riscv-iommu.h because now we're doing
> the right thing.
>
> Changes from v1:
> - use PPN_PHYS(iohgatp_ppn), i.e. the page table base address, to
>   check for the 16kb alignment
> - v1 link: https://lore.kernel.org/qemu-devel/20260630203155.2103084-1-daniel.barboza@oss.qualcomm.com/

Thanks!

Applied to riscv-to-apply.next

Alistair

>
>
> Daniel Henrique Barboza (1):
>   hw/riscv/riscv-iommu.c: check for misaligned IOHGATP_PPN
>
>  hw/riscv/riscv-iommu.c | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
>
> --
> 2.43.0
>
>