Wei Wang2
2009-Oct-28 16:31 UTC
[Xen-devel] [PATCH] AMD IOMMU: Parse IVRS table early to prevent iommu_enabled from being disabled
This patch moves IVRS table parsing functions into an early position, which prevent iommu_enabled from being disabled by failure dmar table detection. Thanks, Wei Signed-off-by: Wei Wang <wei.wang2@amd.com> -- AMD GmbH, Germany Operating System Research Center Legal Information: Advanced Micro Devices GmbH Karl-Hammerschmidt-Str. 34 85609 Dornach b. München Geschäftsführer: Andrew Bowd, Thomas M. McCoy, Giuliano Meroni Sitz: Dornach, Gemeinde Aschheim, Landkreis München Registergericht München, HRB Nr. 43632 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Keir Fraser
2009-Oct-28 17:30 UTC
Re: [Xen-devel] [PATCH] AMD IOMMU: Parse IVRS table early to prevent iommu_enabled from being disabled
On 28/10/2009 16:31, "Wei Wang2" <wei.wang2@amd.com> wrote:> This patch moves IVRS table parsing functions into an early position, which > prevent iommu_enabled from being disabled by failure dmar table detection.Not nice. See if c/s 20381 fixes your problem. It''s justa cleanup of some VT-d code which should avoid the problem altogether. -- Keir _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Wei Wang2
2009-Oct-28 17:46 UTC
Re: [Xen-devel] [PATCH] AMD IOMMU: Parse IVRS table early to prevent iommu_enabled from being disabled
On Wednesday 28 October 2009 18:30:31 Keir Fraser wrote:> On 28/10/2009 16:31, "Wei Wang2" <wei.wang2@amd.com> wrote: > > This patch moves IVRS table parsing functions into an early position, > > which prevent iommu_enabled from being disabled by failure dmar table > > detection. > > Not nice. See if c/s 20381 fixes your problem. It''s justa cleanup of some > VT-d code which should avoid the problem altogether. > > -- KeirThanks, I have not checked 20381 while creating this patch. I will test it again and send something new if necessary. -Wei _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Keir Fraser
2009-Oct-28 18:03 UTC
Re: [Xen-devel] [PATCH] AMD IOMMU: Parse IVRS table early to prevent iommu_enabled from being disabled
On 28/10/2009 17:46, "Wei Wang2" <wei.wang2@amd.com> wrote:>> Not nice. See if c/s 20381 fixes your problem. It''s justa cleanup of some >> VT-d code which should avoid the problem altogether. >> >> -- Keir > > Thanks, I have not checked 20381 while creating this patch.That''s because I created it in response to your patch! :-) K.> I will test it again and send something new if necessary._______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Wei Wang2
2009-Oct-29 13:51 UTC
Re: [Xen-devel] [PATCH] AMD IOMMU: Parse IVRS table early to prevent iommu_enabled from being disabled
On Wednesday 28 October 2009 19:03:17 Keir Fraser wrote:> On 28/10/2009 17:46, "Wei Wang2" <wei.wang2@amd.com> wrote: > >> Not nice. See if c/s 20381 fixes your problem. It''s justa cleanup of > >> some VT-d code which should avoid the problem altogether. > >> > >> -- Keir > > > > Thanks, I have not checked 20381 while creating this patch. > > That''s because I created it in response to your patch! :-) > > K. > > > I will test it again and send something new if necessary.Keir, Attached patch removes a BUG_ON condition, to boot amd iommu system correctly. Thanks, Wei Signed-off-by: Wei Wang <wei.wang2@amd.com> -- AMD GmbH, Germany Operating System Research Center Legal Information: Advanced Micro Devices GmbH Karl-Hammerschmidt-Str. 34 85609 Dornach b. München Geschäftsführer: Andrew Bowd, Thomas M. McCoy, Giuliano Meroni Sitz: Dornach, Gemeinde Aschheim, Landkreis München Registergericht München, HRB Nr. 43632 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel