Hi Peter,
On 06/04/2017 09:08, Peter Xu wrote:
> Reviewed-by: David Gibson <da...@gibson.dropbear.id.au>
> Signed-off-by: Peter Xu <pet...@redhat.com>
Even if the commit message is obvious it may be requested?
Reviewed-by: Eric Auger <eric.au...@redhat.com>
> ---
> include/exec/memory.h | 3 +++
> memory.c | 4 ++--
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/include/exec/memory.h b/include/exec/memory.h
> index 0840c89..07e43da 100644
> --- a/include/exec/memory.h
> +++ b/include/exec/memory.h
> @@ -239,6 +239,9 @@ struct MemoryRegion {
> IOMMUNotifierFlag iommu_notify_flags;
> };
>
> +#define IOMMU_NOTIFIER_FOREACH(n, mr) \
> + QLIST_FOREACH((n), &(mr)->iommu_notify, node)
> +
> /**
> * MemoryListener: callbacks structure for updates to the physical memory map
> *
> diff --git a/memory.c b/memory.c
> index 75ac595..7496b3d 100644
> --- a/memory.c
> +++ b/memory.c
> @@ -1583,7 +1583,7 @@ static void
> memory_region_update_iommu_notify_flags(MemoryRegion *mr)
> IOMMUNotifierFlag flags = IOMMU_NOTIFIER_NONE;
> IOMMUNotifier *iommu_notifier;
>
> - QLIST_FOREACH(iommu_notifier, &mr->iommu_notify, node) {
> + IOMMU_NOTIFIER_FOREACH(iommu_notifier, mr) {
> flags |= iommu_notifier->notifier_flags;
> }
>
> @@ -1667,7 +1667,7 @@ void memory_region_notify_iommu(MemoryRegion *mr,
> request_flags = IOMMU_NOTIFIER_UNMAP;
> }
>
> - QLIST_FOREACH(iommu_notifier, &mr->iommu_notify, node) {
> + IOMMU_NOTIFIER_FOREACH(iommu_notifier, mr) {
> /*
> * Skip the notification if the notification does not overlap
> * with registered range.
>
