Xen devel - Oct 2009 - Re: RE: [Xen-changelog] [xen-unstable]vt-d: Fixpanic in msi_msg_read_remap_rte with acpi=off

----- "Dexuan Cui" <dexuan.cui@intel.com> wrote:
> From: "Dexuan Cui" <dexuan.cui@intel.com>
> To: "Miroslav Rezanina" <mrezanin@redhat.com>
> Cc: Keir@redhat.com, xen-devel@lists.xensource.com, "Fraser"
<keir.fraser@eu.citrix.com>
> Sent: Monday, October 19, 2009 10:49:35 AM GMT +01:00 Amsterdam / Berlin /
Bern / Rome / Stockholm / Vienna
> Subject: RE: [Xen-devel] RE: [Xen-changelog] [xen-unstable]vt-d: Fixpanic
in msi_msg_read_remap_rte with acpi=off
>
> Miroslav Rezanina wrote:
> > Hi Dexuan,
> > you''re right. We should print warning. In your patch, I do
not
> > understand 
> > why you put comment only in setup_dom0_devices function. There is
> more
> > calling of domain_context_mapping and we check NULL also in
> In other places, the retuen value of domain_context_mapping() has been
> checked properly, e.g., passing to the caller, so we wouldn''t
ignore
> the failure. :-)
> 
> > domain_context_unmap and reassign_device_ownership. We should put
> > warning in there too, shouldn''t we? 
> domain_context_unmap() is invoked in 2 places:
> 1) in intel_iommu_remove_device(), the return value has been
> propagated properly;
> 2) in reassign_device_ownership(), invoking reassign_device_ownership
> implies the device has been successfully assigned and the
> domain_context_mapping() returned success, so here the
> domain_context_unmap() can''t fail.
> 
> The other returing place in reassign_device_ownership()  has been
> propagated properly to the caller.
> 
> Thanks,
> -- Dexuan
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
Ok, I understand. I just need to clarify this. Your v2 is all right.
-- 
Miroslav Rezanina
Software Engineer - Virtualization Team - XEN kernel


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel