[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 09/12] intel_iommu: allow dev-iotlb context e
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v3 09/12] intel_iommu: allow dev-iotlb context entry conditionally |
Date: |
Thu, 11 May 2017 11:51:43 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Thu, May 11, 2017 at 10:56:42AM +0800, Jason Wang wrote:
>
>
> On 2017年05月10日 16:01, Peter Xu wrote:
> >When device-iotlb is not specified, we should fail this check.
> >
> >Signed-off-by: Peter Xu <address@hidden>
> >---
> > hw/i386/intel_iommu.c | 13 ++++++++++++-
> > 1 file changed, 12 insertions(+), 1 deletion(-)
> >
> >diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
> >index 05ae631..1a7eba2 100644
> >--- a/hw/i386/intel_iommu.c
> >+++ b/hw/i386/intel_iommu.c
> >@@ -836,6 +836,8 @@ static int vtd_dev_to_context_entry(IntelIOMMUState *s,
> >uint8_t bus_num,
> > {
> > VTDRootEntry re;
> > int ret_fr;
> >+ bool type_fail = false;
> >+ X86IOMMUState *x86_iommu = X86_IOMMU_DEVICE(s);
> > ret_fr = vtd_get_root_entry(s, bus_num, &re);
> > if (ret_fr) {
> >@@ -872,10 +874,19 @@ static int vtd_dev_to_context_entry(IntelIOMMUState
> >*s, uint8_t bus_num,
> > } else {
> > switch (vtd_ce_get_type(ce)) {
> > case VTD_CONTEXT_TT_MULTI_LEVEL:
> >- /* fall through */
> >+ /* Always supported */
> >+ break;
> > case VTD_CONTEXT_TT_DEV_IOTLB:
> >+ if (!x86_iommu->dt_supported) {
> >+ type_fail = true;
> >+ }
> > break;
> > default:
> >+ /* Unknwon type */
> >+ type_fail = true;
> >+ break;
> >+ }
> >+ if (type_fail) {
> > trace_vtd_ce_invalid(ce->hi, ce->lo);
> > return -VTD_FR_CONTEXT_ENTRY_INV;
> > }
>
> How about e.g exclude the type if not supported in vtd_ce_get_type()? This
> looks better than using something like type_fail.
(after a quick discussion with Jason offlist)
I'll keep current vtd_ce_get_type() since there are other places that
used it, and introduce another vtd_ce_type_check() to make codes more
elegant (and get rid of type_fail variable). Thanks.
--
Peter Xu
- [Qemu-devel] [PATCH v3 03/12] memory: remove the last param in memory_region_iommu_replay(), (continued)
- [Qemu-devel] [PATCH v3 03/12] memory: remove the last param in memory_region_iommu_replay(), Peter Xu, 2017/05/10
- [Qemu-devel] [PATCH v3 04/12] memory: fix address_space_get_iotlb_entry(), Peter Xu, 2017/05/10
- [Qemu-devel] [PATCH v3 05/12] x86-iommu: use DeviceClass properties, Peter Xu, 2017/05/10
- [Qemu-devel] [PATCH v3 07/12] intel_iommu: provide vtd_ce_get_type(), Peter Xu, 2017/05/10
- [Qemu-devel] [PATCH v3 06/12] intel_iommu: renaming context entry helpers, Peter Xu, 2017/05/10
- [Qemu-devel] [PATCH v3 08/12] intel_iommu: use IOMMU_ACCESS_FLAG(), Peter Xu, 2017/05/10
- [Qemu-devel] [PATCH v3 09/12] intel_iommu: allow dev-iotlb context entry conditionally, Peter Xu, 2017/05/10
- [Qemu-devel] [PATCH v3 10/12] intel_iommu: support passthrough (PT), Peter Xu, 2017/05/10
- [Qemu-devel] [PATCH v3 11/12] intel_iommu: turn off pt before 2.9, Peter Xu, 2017/05/10
- [Qemu-devel] [PATCH v3 12/12] vhost: iommu: cache static mapping if there is, Peter Xu, 2017/05/10
- Re: [Qemu-devel] [PATCH v3 00/12] VT-d: PT (passthrough) mode support and misc fixes, no-reply, 2017/05/10