[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 3/8] memory: rename iommu_notifier_init()
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [RFC PATCH 3/8] memory: rename iommu_notifier_init() |
Date: |
Mon, 1 May 2017 14:53:35 +1000 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Thu, Apr 27, 2017 at 05:34:15PM +0800, Peter Xu wrote:
> It's new name is iommu_mr_notifier_init(). Again, literal changes only.
>
> Signed-off-by: Peter Xu <address@hidden>
Again, I think this could be folded with the previous two patches.
Signed-off-by: David Gibson <address@hidden>
I also note that these patches will conflict with Alexey's proposed
changes to make IOMMU MR's a QOM subtype of the ordinary MR (thus
removing some of the MR specific fields from other MRs.
> ---
> hw/vfio/common.c | 8 ++++----
> hw/virtio/vhost.c | 8 ++++----
> include/exec/memory.h | 7 ++++---
> 3 files changed, 12 insertions(+), 11 deletions(-)
>
> diff --git a/hw/vfio/common.c b/hw/vfio/common.c
> index bd113b7..6b3953b 100644
> --- a/hw/vfio/common.c
> +++ b/hw/vfio/common.c
> @@ -481,10 +481,10 @@ static void vfio_listener_region_add(MemoryListener
> *listener,
> llend = int128_add(int128_make64(section->offset_within_region),
> section->size);
> llend = int128_sub(llend, int128_one());
> - iommu_notifier_init(&giommu->n, vfio_iommu_map_notify,
> - IOMMU_MR_EVENT_ALL,
> - section->offset_within_region,
> - int128_get64(llend));
> + iommu_mr_notifier_init(&giommu->n, vfio_iommu_map_notify,
> + IOMMU_MR_EVENT_ALL,
> + section->offset_within_region,
> + int128_get64(llend));
> QLIST_INSERT_HEAD(&container->giommu_list, giommu, giommu_next);
>
> memory_region_register_iommu_notifier(giommu->iommu, &giommu->n);
> diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
> index 81a8aea..7449cf8 100644
> --- a/hw/virtio/vhost.c
> +++ b/hw/virtio/vhost.c
> @@ -746,10 +746,10 @@ static void vhost_iommu_region_add(MemoryListener
> *listener,
> end = int128_add(int128_make64(section->offset_within_region),
> section->size);
> end = int128_sub(end, int128_one());
> - iommu_notifier_init(&iommu->n, vhost_iommu_unmap_notify,
> - IOMMU_MR_EVENT_UNMAP,
> - section->offset_within_region,
> - int128_get64(end));
> + iommu_mr_notifier_init(&iommu->n, vhost_iommu_unmap_notify,
> + IOMMU_MR_EVENT_UNMAP,
> + section->offset_within_region,
> + int128_get64(end));
> iommu->mr = section->mr;
> iommu->iommu_offset = section->offset_within_address_space -
> section->offset_within_region;
> diff --git a/include/exec/memory.h b/include/exec/memory.h
> index 6be0c02..8d8dcb2 100644
> --- a/include/exec/memory.h
> +++ b/include/exec/memory.h
> @@ -93,9 +93,10 @@ struct IOMMUMRNotifier {
> };
> typedef struct IOMMUMRNotifier IOMMUMRNotifier;
>
> -static inline void iommu_notifier_init(IOMMUMRNotifier *n, IOMMUNotify fn,
> - IOMMUMREventFlags flags,
> - hwaddr start, hwaddr end)
> +static inline void iommu_mr_notifier_init(IOMMUMRNotifier *n,
> + IOMMUNotify fn,
> + IOMMUMREventFlags flags,
> + hwaddr start, hwaddr end)
> {
> n->notify = fn;
> n->notifier_flags = flags;
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- Re: [Qemu-devel] [RFC PATCH 3/8] memory: rename iommu_notifier_init(),
David Gibson <=