RE: [PATCHv3 0/6] Crashdump Accepting Active IOMMU

2014-04-25 Thread Sumner, William
1 Hi Don, 2 It seems that we agree in the area of keeping the iommu active and using it to 3 isolate the legacy DMA. We also agree that the device's IO driver should be 4 the software that deals with the device (e.g.: resets it, etc.) 5 6 It also seems that

Re: [PATCHv3 0/6] Crashdump Accepting Active IOMMU

2014-04-08 Thread Don Dutile
On 04/08/2014 12:14 PM, David Woodhouse wrote: On Mon, 2014-04-07 at 16:43 -0400, Don Dutile wrote: Additionally, a tidbit of information like some servers force NMI's on DMAR faults, and cause a system reset, thereby, preventing a kdump to occur should have been included as one reason to stop

Re: [PATCHv3 0/6] Crashdump Accepting Active IOMMU

2014-03-12 Thread Sumner, William
On Mon, Mar 10, 2014 at 04:43PM, Vivek Goyal wrote: On Fri, Jan 10, 2014 at 03:07:26PM -0700, Bill Sumner wrote: [..] This patch set modifies the behavior of the iommu in the (new) crashdump kernel: 1. to accept the iommu hardware in an active state, 2. to leave the current translations

FW: Re: [PATCHv3 0/6] Crashdump Accepting Active IOMMU

2014-03-12 Thread Sumner, William
...@vger.kernel.org; iommu@lists.linux-foundation.org; ddut...@redhat.com; Hatch, Douglas B (HPS Linux PM); ishii.hiron...@jp.fujitsu.com; bhelg...@google.com; Li, Zhen-Hua Subject: Re: [PATCHv3 0/6] Crashdump Accepting Active IOMMU On Mon, Mar 10, 2014 at 04:43PM, Vivek Goyal wrote: On Fri, Jan 10

RE: [PATCHv3 0/6] Crashdump Accepting Active IOMMU

2014-03-10 Thread Sumner, William
Hi Joerg, On Tue 3/4/2014 9:06 AM, Joerg Roedel wrote: On Fri, Jan 10, 2014 at 03:07:26PM -0700, Bill Sumner wrote: Bill Sumner (6): Crashdump-Accepting-Active-IOMMU-Flags-and-Prototype Crashdump-Accepting-Active-IOMMU-Utility-functions

Re: [PATCHv3 0/6] Crashdump Accepting Active IOMMU

2014-03-10 Thread Vivek Goyal
On Fri, Jan 10, 2014 at 03:07:26PM -0700, Bill Sumner wrote: [..] This patch set modifies the behavior of the iommu in the (new) crashdump kernel: 1. to accept the iommu hardware in an active state, 2. to leave the current translations in-place so that legacy DMA will continue using

Re: [PATCHv3 0/6] Crashdump Accepting Active IOMMU

2014-03-04 Thread Joerg Roedel
Hi Bill, On Fri, Jan 10, 2014 at 03:07:26PM -0700, Bill Sumner wrote: Bill Sumner (6): Crashdump-Accepting-Active-IOMMU-Flags-and-Prototype Crashdump-Accepting-Active-IOMMU-Utility-functions Crashdump-Accepting-Active-IOMMU-Domain-Interfaces

Re: [PATCHv3 0/6] Crashdump Accepting Active IOMMU

2014-01-24 Thread Baoquan He
Tested this patchset on my local HP Z420 workstation, and it works very well. Hi Bill, Thanks for your effort. There are several concerns from me. Firstly, I think the patch log need be rearanged. Patchset cover letter can contain information to express why, how briefly. If you think this is

[PATCHv3 0/6] Crashdump Accepting Active IOMMU

2014-01-10 Thread Bill Sumner
v2-v3: 1. Commented-out #define DEBUG 1 to eliminate debug messages 2. Updated the comments about changes in each version in all patches in the set. 3. Fixed: one-line added to Copy-Translations patch to initialize the iovad struct as recommended by Baoquan He [b...@redhat.com]