[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v1 5/6] intel_iommu: Let vtd_iommu_notify_flag_cha
From: |
Peter Xu |
Subject: |
Re: [Qemu-arm] [PATCH v1 5/6] intel_iommu: Let vtd_iommu_notify_flag_changed() fail |
Date: |
Mon, 16 Sep 2019 11:33:08 +0800 |
User-agent: |
Mutt/1.11.4 (2019-03-13) |
On Fri, Sep 13, 2019 at 10:36:14AM +0200, Eric Auger wrote:
> In case a MAP notifier is attempted to be registered without
> caching mode, let's simply return an error. This latter now is
> handled in the VFIO code.
>
> Signed-off-by: Eric Auger <address@hidden>
> ---
> hw/i386/intel_iommu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
> index 7a89ea9ba1..2f66d6882c 100644
> --- a/hw/i386/intel_iommu.c
> +++ b/hw/i386/intel_iommu.c
> @@ -2931,7 +2931,7 @@ static int
> vtd_iommu_notify_flag_changed(IOMMUMemoryRegion *iommu,
> if (!s->caching_mode && new & IOMMU_NOTIFIER_MAP) {
> error_report("We need to set caching-mode=on for intel-iommu to
> enable "
> "device assignment with IOMMU protection.");
> - exit(1);
> + return -EINVAL;
This might be conflicting with the other series because that's going
to drop these lines:
https://lists.gnu.org/archive/html/qemu-devel/2019-08/msg01968.html
Though current change looks ok to me, and I don't even know which one
will reach master earlier after all so let's see. :)
The rest patches besides patch 1 all look sane to me. If we're going
to have "Error **" then the rest patches might need trivial touch-ups
of course. And, IMHO squashing the whole series into patch 1 could be
better, but it's a personal preference only.
Thanks,
--
Peter Xu
- [Qemu-arm] [PATCH v1 0/6] Allow memory_region_register_iommu_notifier() to fail, Eric Auger, 2019/09/13
- [Qemu-arm] [PATCH v1 1/6] memory: allow memory_region_register_iommu_notifier() to fail, Eric Auger, 2019/09/13
- [Qemu-arm] [PATCH v1 2/6] vfio/common: Handle memory_region_register_iommu_notifier() failure, Eric Auger, 2019/09/13
- [Qemu-arm] [PATCH v1 3/6] exec: assert on memory_region_register_iommu_notifier() failure, Eric Auger, 2019/09/13
- [Qemu-arm] [PATCH v1 4/6] vhost: assert on memory_region_register_iommu_notifier() failure, Eric Auger, 2019/09/13
- [Qemu-arm] [PATCH v1 5/6] intel_iommu: Let vtd_iommu_notify_flag_changed() fail, Eric Auger, 2019/09/13
- Re: [Qemu-arm] [PATCH v1 5/6] intel_iommu: Let vtd_iommu_notify_flag_changed() fail,
Peter Xu <=
- [Qemu-arm] [PATCH v1 6/6] amd_iommu: Let amdvi_iommu_notify_flag_changed() fail, Eric Auger, 2019/09/13