On Thu, Mar 24, 2022 at 4:27 PM Tian, Kevin <kevin.t...@intel.com> wrote: > > > From: Jason Wang > > Sent: Monday, March 21, 2022 1:54 PM > > @@ -1724,6 +1713,19 @@ out: > > trace_vtd_pt_enable_fast_path(source_id, success); > > } > > > > +static void vtd_qualify_report_fault(IntelIOMMUState *s, > > + int err, bool is_fpd_set, > > + uint16_t source_id, > > + hwaddr addr, > > + bool is_write) > > vtd_report_qualified_fault() is clearer.
Fine. Thanks > > > +{ > > + if (is_fpd_set && vtd_is_qualified_fault(err)) { > > + trace_vtd_fault_disabled(); > > + } else { > > + vtd_report_dmar_fault(s, source_id, addr, err, is_write); > > + } > > +} > > + > > /* Map dev to context-entry then do a paging-structures walk to do a iommu > > * translation. > > * >