Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-04-17 Thread Xu, Quan
On April 15, 2016 10:03pm, wrote: > On 31/03/16 10:06, Xu, Quan wrote: > > All, > > > > Here is a summary of my investigation of the abstract model: > > > > Below policies are adopted when deciding whether to rollback a callchain: > > > > 1. Domain will be crashed immediately within iommu_{,un}map

Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-04-15 Thread George Dunlap
On 31/03/16 10:06, Xu, Quan wrote: > All, > > Here is a summary of my investigation of the abstract model: > > Below policies are adopted when deciding whether to rollback a callchain: > > 1. Domain will be crashed immediately within iommu_{,un}map_page, treated as > a fatal error (with the exc

Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-04-15 Thread George Dunlap
On 12/04/16 04:30, Xu, Quan wrote: > George, > > In this discussion, most of them are memory-related problems. Your comments > are valuable. > If you have read this thread, could you give me some feedback? I really > appreciate your precious advice. Sorry -- I was trying to get all the stuff fo

Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-04-11 Thread Xu, Quan
George, In this discussion, most of them are memory-related problems. Your comments are valuable. If you have read this thread, could you give me some feedback? I really appreciate your precious advice. Quan On March 31, 2016 5:06pm, Quan, Xu wrote: > All, > > Here is a summary of my invest

Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-04-08 Thread Xu, Quan
On April 08, 2016 6:44am, wrote: > >>> On 06.04.16 at 09:38, wrote: > > On April 01, 2016 7:57pm, wrote: > >> >>> On 31.03.16 at 11:06, wrote: > >> > 4. __gnttab_unmap_common():rollback (no change) > >> > > >> > (Existing code) > >> >>>... > >> > if ( !kind ) > >> >

Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-04-07 Thread Jan Beulich
>>> On 06.04.16 at 09:38, wrote: > On April 01, 2016 7:57pm, wrote: >> >>> On 31.03.16 at 11:06, wrote: >> > 4. __gnttab_unmap_common():rollback (no change) >> > >> > (Existing code) >> >>>... >> > if ( !kind ) >> > err = iommu_unmap_page(ld, op->frame); >> >

Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-04-06 Thread Xu, Quan
On April 01, 2016 7:57pm, wrote: > >>> On 31.03.16 at 11:06, wrote: > > 4. __gnttab_unmap_common():rollback (no change) > > > > (Existing code) > >>>... > > if ( !kind ) > > err = iommu_unmap_page(ld, op->frame); > > else if ( !(kind & MAPKIND_WRITE) ) > >

Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-04-01 Thread Jan Beulich
>>> On 31.03.16 at 11:06, wrote: > 4. __gnttab_unmap_common():rollback (no change) > > (Existing code) >>>... > if ( !kind ) > err = iommu_unmap_page(ld, op->frame); > else if ( !(kind & MAPKIND_WRITE) ) > err = iommu_map_page(ld, op->frame, op

Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-03-31 Thread Tian, Kevin
Jan and others, I've reviewed below with Quan internally. Basically I'm OK with the proposal, but there does be several opens (8-11) which we'd like to know your opinions. Once they are cleared, next version of the patch should be made quickly... > From: Xu, Quan > Sent: Thursday, March 31, 2016

Re: [Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-03-31 Thread Xu, Quan
+to 2 key maintainers, On March 31, 2016 5:06pm, Xu, Quan wrote: > All, > > Here is a summary of my investigation of the abstract model: > > Below policies are adopted when deciding whether to rollback a callchain: > > 1. Domain will be crashed immediately within iommu_{,un}map_page, treated

[Xen-devel] abstract model of IOMMU unmaping/mapping failures

2016-03-31 Thread Xu, Quan
All, Here is a summary of my investigation of the abstract model: Below policies are adopted when deciding whether to rollback a callchain: 1. Domain will be crashed immediately within iommu_{,un}map_page, treated as a fatal error (with the exception of the hardware one). Whether to rollback d