From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from fgwmail6.fujitsu.co.jp ([192.51.44.36]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1URfHo-0004za-Ih for kexec@lists.infradead.org; Mon, 15 Apr 2013 09:01:19 +0000 Received: from m4.gw.fujitsu.co.jp (unknown [10.0.50.74]) by fgwmail6.fujitsu.co.jp (Postfix) with ESMTP id 567583EE0B6 for ; Mon, 15 Apr 2013 18:01:06 +0900 (JST) Received: from smail (m4 [127.0.0.1]) by outgoing.m4.gw.fujitsu.co.jp (Postfix) with ESMTP id 13C3F45DE5C for ; Mon, 15 Apr 2013 18:01:06 +0900 (JST) Received: from s4.gw.fujitsu.co.jp (s4.gw.fujitsu.co.jp [10.0.50.94]) by m4.gw.fujitsu.co.jp (Postfix) with ESMTP id D7BE645DE52 for ; Mon, 15 Apr 2013 18:01:05 +0900 (JST) Received: from s4.gw.fujitsu.co.jp (localhost.localdomain [127.0.0.1]) by s4.gw.fujitsu.co.jp (Postfix) with ESMTP id BDAA81DB802F for ; Mon, 15 Apr 2013 18:01:05 +0900 (JST) Received: from m1001.s.css.fujitsu.com (m1001.s.css.fujitsu.com [10.240.81.139]) by s4.gw.fujitsu.co.jp (Postfix) with ESMTP id 6BBB91DB803B for ; Mon, 15 Apr 2013 18:01:05 +0900 (JST) Message-ID: <516BC19D.4090303@jp.fujitsu.com> Date: Mon, 15 Apr 2013 18:00:13 +0900 From: Takao Indoh MIME-Version: 1.0 Subject: Re: [PATCH] intel-iommu: Synchronize gcmd value with global command register References: <1363829556-2128-1-git-send-email-indou.takao@jp.fujitsu.com> <20130326144629.GB2727@8bytes.org> <51527D74.9080209@jp.fujitsu.com> <20130327103122.GK30540@8bytes.org> <51591EEE.60401@jp.fujitsu.com> <20130402140546.GA15687@8bytes.org> <515BD638.8070307@jp.fujitsu.com> <1364977479.28127.15.camel@i7.infradead.org> <20130405110616.GB6858@8bytes.org> <5164EEE6.8000707@jp.fujitsu.com> In-Reply-To: <5164EEE6.8000707@jp.fujitsu.com> 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" Errors-To: kexec-bounces+dwmw2=twosheds.infradead.org@lists.infradead.org To: joro@8bytes.org, dwmw2@infradead.org Cc: iommu@lists.linux-foundation.org, kexec@lists.infradead.org, linux-kernel@vger.kernel.org, tindoh@gmail.com (2013/04/10 13:47), Takao Indoh wrote: > (2013/04/05 20:06), Joerg Roedel wrote: >> On Wed, Apr 03, 2013 at 09:24:39AM +0100, David Woodhouse wrote: >>> On Wed, 2013-04-03 at 16:11 +0900, Takao Indoh wrote: >>>> Yeah, you are right. I forgot such a case. >>> >>> If you disable translation and there's some device still doing DMA, it's >>> going to scribble over random areas of memory. You really want to have >>> translation enabled and all the page tables *cleared*, during kexec. I >>> think it's fair to insist that the secondary kernel should use the IOMMU >>> if the first one did. >> >> Do we really need to insist on that? The IOMMU initialization on x86 >> happens after the kernel scanned and enumerated the PCI bus. While doing >> this the kernel (at least it should) disables all devices it finds. So >> when the IOMMU init code runs we should be safe from any in-flight DMA >> and can either disable translation or re-initialize it for the kdump >> kernel. Until then translation needs to stay enabled of course, so that >> the old page-tables are still used and in-flight DMA doesn't corrupt >> any data. > > So we should do in this order, right? > (1) PCI initialization. Stop all ongoing DMA here. > (2) Disable translation if already enable. > (3) Make pgtable and enable translation. Joerg, David, On DMAR initialization during kdump boot, do you guys agree to change order like this? Current order: (1) Disable translation (2) PCI initialization (3) Make pgtable and enable translation. Order I'm proposing: (1) PCI initialization (2) Disable translation (3) Make pgtable and enable translation. The purpose to change the behavior is to stop DMA in PCI layer. As Joerg said, if we need to consider the case that kdump kernel is compiled without dma-remapping(CONFIG_INTEL_IOMMU is off?), I can update my patch to handle such a case properly by adding ifdef CONFIG_INTEL_IOMMU. About how to stop the DMA on PCI layer, I'll post another mail to discuss it to iommu and pci list. Thanks, Takao Indoh _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec