On April 27, 2016 11:03 PM, Jan Beulich wrote:
> >>> On 27.04.16 at 16:26, wrote:
> > On April 25, 2016 5:27 PM, Jan Beulich wrote:
> >> >>> On 18.04.16 at 16:00, wrote:
> I.e. I continue to think that
>
> if ( is_hardware_domain() )
> printk();
> else
> domain_crash();
>>> On 27.04.16 at 16:26, wrote:
> On April 25, 2016 5:27 PM, Jan Beulich wrote:
>> >>> On 18.04.16 at 16:00, wrote:
>> > --- a/xen/drivers/passthrough/iommu.c
>> > +++ b/xen/drivers/passthrough/iommu.c
>> > @@ -243,21 +243,33 @@ int iommu_map_page(struct domain *d,
>> unsigned long gfn, unsigne
On April 25, 2016 5:27 PM, Jan Beulich wrote:
> >>> On 18.04.16 at 16:00, wrote:
> > --- a/xen/drivers/passthrough/iommu.c
> > +++ b/xen/drivers/passthrough/iommu.c
> > @@ -243,21 +243,33 @@ int iommu_map_page(struct domain *d,
> unsigned long gfn, unsigned long mfn,
> > unsig
>>> On 18.04.16 at 16:00, wrote:
> --- a/xen/drivers/passthrough/iommu.c
> +++ b/xen/drivers/passthrough/iommu.c
> @@ -243,21 +243,33 @@ int iommu_map_page(struct domain *d, unsigned long gfn,
> unsigned long mfn,
> unsigned int flags)
> {
> struct hvm_iommu *hd = domain
On April 19, 2016 2:37pm, Tian, Kevin wrote:
> > From: Quan Xu
> > Sent: Monday, April 18, 2016 10:00 PM
> >
> > Now IOMMU mapping and unmapping failures are treated as a fatal to the
> > domain (with the exception of the hardware domain).
>
> 'Now' is more about eixsting state, while it's actual
> From: Quan Xu
> Sent: Monday, April 18, 2016 10:00 PM
>
> Now IOMMU mapping and unmapping failures are treated as a fatal to
> the domain (with the exception of the hardware domain).
'Now' is more about eixsting state, while it's actually what you want
to change towards. Better directly say "T
Now IOMMU mapping and unmapping failures are treated as a fatal to
the domain (with the exception of the hardware domain).
If IOMMU mapping and unmapping failed, crash the domain (with the
exception of the hardware domain) and propagate the error up to the
call trees.
Signed-off-by: Quan Xu
CC:
