[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC v8 2/5] memory: Add IOMMUTLBEvent
From: |
Eugenio Perez Martin |
Subject: |
Re: [RFC v8 2/5] memory: Add IOMMUTLBEvent |
Date: |
Wed, 2 Sep 2020 10:14:51 +0200 |
On Tue, Sep 1, 2020 at 10:55 PM Peter Xu <peterx@redhat.com> wrote:
>
> On Tue, Sep 01, 2020 at 04:26:05PM +0200, Eugenio Pérez wrote:
> > This way we can tell between regulars IOMMURLBEntries (entry of IOMMU
> > hardware) and notifications.
>
> s/regulars IOMMURLBEntries/regular IOMMUTLBEntry/
>
> >
> > In the notifications, we set explicitly if it is a MAPs or an UNMAP,
> > instead of trusting in entry permissions to differenciate them.
> >
> > Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
>
> [...]
>
> > struct IOMMUTLBEntry {
> > - AddressSpace *target_as;
> > - hwaddr iova;
> > - hwaddr translated_addr;
> > - hwaddr addr_mask; /* 0xfff = 4k translation */
> > - IOMMUAccessFlags perm;
> > + AddressSpace *target_as;
> > + hwaddr iova;
> > + hwaddr translated_addr;
> > + hwaddr addr_mask; /* 0xfff = 4k translation */
> > + IOMMUAccessFlags perm;
> > };
>
> If these lines are identical, then we can avoid touching the spaces.
>
Thanks for the catches! Didn't notice them rebasing.
> With above changes, please feel free to add:
>
> Reviewed-by: Peter Xu <peterx@redhat.com>
>
> --
> Peter Xu
>
[RFC v8 4/5] intel_iommu: Do not notify regular iotlb to device-iotlb notifiers, Eugenio Pérez, 2020/09/01