Re: [PATCH v9 07/10] iommu/vt-d: enable kdump support in iommu module

2015-04-03 Thread Li, ZhenHua
Hi Joerg, Thinking about it carefully, I think you suggestions are very helpful, and the checks should be: * All these things should be done in the second kernel, not only the kdump kernel. Sometimes user may use kexec manually start a new kernel. * Copying those tables should only happen in the

Re: [PATCH v9 07/10] iommu/vt-d: enable kdump support in iommu module

2015-04-02 Thread Joerg Roedel
On Thu, Mar 19, 2015 at 01:36:25PM +0800, Li, Zhen-Hua wrote: > +#ifdef CONFIG_CRASH_DUMP > + if (is_kdump_kernel()) > + __iommu_update_old_root_entry(iommu, bus); > +#endif All the is_kdump_kernel checks in this patch (and maybe in other patches too) should really

[PATCH v9 07/10] iommu/vt-d: enable kdump support in iommu module

2015-03-18 Thread Li, Zhen-Hua
Modify the operation of the following functions when called during crash dump: device_to_context_entry free_context_table get_domain_for_dev init_dmars intel_iommu_init Bill Sumner: Original version. Zhenhua: The name of new calling functions. Do not disable and re