From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from 8bytes.org ([88.198.83.132]) by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux)) id 1NxcuP-0000bb-0V for kexec@lists.infradead.org; Fri, 02 Apr 2010 09:11:22 +0000 Date: Fri, 2 Apr 2010 11:11:18 +0200 From: Joerg Roedel Subject: Re: [PATCH 1/2] x86/amd-iommu: enable iommu before attaching devices Message-ID: <20100402091118.GM24846@8bytes.org> References: <20100331202745.GE13406@hmsreliant.think-freely.org> <20100401142902.GF24846@8bytes.org> <20100401144736.GA14069@shamino.rdu.redhat.com> <20100401155643.GG24846@8bytes.org> <20100401171149.GH13603@shamino.rdu.redhat.com> <20100401201433.GK24846@8bytes.org> <20100402000012.GA8930@hmsreliant.think-freely.org> <20100402003034.GX29241@sequoia.sous-sol.org> <20100402012353.GY29241@sequoia.sous-sol.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20100402012353.GY29241@sequoia.sous-sol.org> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: kexec-bounces@lists.infradead.org Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: Chris Wright Cc: Neil Horman , Neil Horman , Joerg Roedel , kexec@lists.infradead.org, linux-kernel@vger.kernel.org, hbabu@us.ibm.com, iommu@lists.linux-foundation.org, "Eric W. Biederman" , Vivek Goyal On Thu, Apr 01, 2010 at 06:23:53PM -0700, Chris Wright wrote: > Hit another kdump problem as reported by Neil Horman. When initializaing > the IOMMU, we attach devices to their domains before the IOMMU is > fully (re)initialized. Attaching a device will issue some important > invalidations. In the context of the newly kexec'd kdump kernel, the > IOMMU may have stale cached data from the original kernel. Because we > do the attach too early, the invalidation commands are placed in the new > command buffer before the IOMMU is updated w/ that buffer. This leaves > the stale entries in the kdump context and can renders device unusable. > Simply enable the IOMMU before we do the attach. > > Cc: Neil Horman > Cc: Vivek Goyal > Signed-off-by: Chris Wright > --- > arch/x86/kernel/amd_iommu_init.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > --- a/arch/x86/kernel/amd_iommu_init.c > +++ b/arch/x86/kernel/amd_iommu_init.c > @@ -1288,6 +1288,8 @@ static int __init amd_iommu_init(void) > if (ret) > goto free; > > + enable_iommus(); > + > if (iommu_pass_through) > ret = amd_iommu_init_passthrough(); > else > @@ -1300,8 +1302,6 @@ static int __init amd_iommu_init(void) > > amd_iommu_init_notifier(); > > - enable_iommus(); > - > if (iommu_pass_through) > goto out; Ok, good to know this fixes the problem. One issue: If the initialization of the domains fails the iommu hardware needs to be disabled again in the free path. Thanks, Joerg _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec