Looks good. Reviewed-by: Liu, Yi L <[email protected]>
On Fri, Dec 08, 2017 at 12:26:54PM +0800, Peter Xu wrote: > It should be caching-mode. It may confuse people when it pops up. > > Signed-off-by: Peter Xu <[email protected]> > --- > 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 c1fa08d205..fe15d3ba84 100644 > --- a/hw/i386/intel_iommu.c > +++ b/hw/i386/intel_iommu.c > @@ -2327,7 +2327,7 @@ static void > vtd_iommu_notify_flag_changed(IOMMUMemoryRegion *iommu, > IntelIOMMUNotifierNode *next_node = NULL; > > if (!s->caching_mode && new & IOMMU_NOTIFIER_MAP) { > - error_report("We need to set cache_mode=1 for intel-iommu to enable " > + error_report("We need to set caching-mode=1 for intel-iommu to > enable " > "device assignment with IOMMU protection."); > exit(1); > } > -- > 2.14.3 > >
