>>> "Xu, Quan" <quan...@intel.com> 06/13/16 5:22 PM >>> >From: Quan Xu <quan...@intel.com> > >When IOMMU mapping is failed, we issue a best effort rollback, stopping >IOMMU mapping, unmapping the previous IOMMU maps and then reporting the >error up to the call trees. When rollback is not feasible (in early >initialization phase or trade-off of complexity) for the hardware domain, >we do things on a best effort basis, only throwing out an error message. > >IOMMU unmapping should perhaps continue despite an error, in an attempt >to do best effort cleanup. > >Signed-off-by: Quan Xu <quan...@intel.com> >Reviewed-by: Jan Beulich <jbeul...@suse.com> >Reviewed-by: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com> >Acked-by: Kevin Tian <kevin.t...@intel.com> > >CC: Jan Beulich <jbeul...@suse.com> >CC: Andrew Cooper <andrew.coop...@citrix.com> >CC: Jun Nakajima <jun.nakaj...@intel.com> >CC: Kevin Tian <kevin.t...@intel.com> >CC: George Dunlap <george.dun...@eu.citrix.com> >CC: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com> >CC: Feng Wu <feng...@intel.com> > >v8: >1. add missing blank >2. add a brief comment (Jan, if you have a better one, could you help me >enhance it upon commit?)
This _still_ sits above the first --- separator, despite you having been asked more than once to move it down. As to question 2, I'll see what I can do (I'm specifically not very happy that this comment which you added in more than one place isn't really "brief"). And before the patch can be committed, it'll need George's ack anyway. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
