From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Zhang, Yang Z" <yang.z.zhang@intel.com>
Subject: Re: Please revert / review
 077fc1c04d70ef1748ac2daa6622b3320a1a004c
Date: Fri, 20 Jun 2014 00:20:48 +0000
Message-ID: <A9667DDFB95DB7438FA9D7D576C3D87E0AAFFE73@SHSMSX104.ccr.corp.intel.com>
References: <CABoYbGpyFWPhbA52u7Bys93dVwSsYuHv72LZWh4BPLJW5fH89Q@mail.gmail.com>
	<A9667DDFB95DB7438FA9D7D576C3D87E0AAF65C3@SHSMSX104.ccr.corp.intel.com>
	<5399828D02000078000B63B7@mail.emea.novell.com>
	<CABoYbGpm_Mfa+t=mZn=GAsGrJKtptydZ9T1Q9Vj-pKHtOY9i7g@mail.gmail.com>
	<A9667DDFB95DB7438FA9D7D576C3D87E0AAF9ABB@SHSMSX104.ccr.corp.intel.com>
	<539EC228020000780001A7FC@mail.emea.novell.com>
	<CABoYbGpgMpvGSw38ijKxVW7m9FmD-FssQe-_hQ9g+xS5W2jr=Q@mail.gmail.com>
	<A9667DDFB95DB7438FA9D7D576C3D87E0AAFC869@SHSMSX104.ccr.corp.intel.com>
	<CABoYbGpQRKYrp5rEE=rGbs=Xrvbvt4B33QDV9mSHx-GBstWK-Q@mail.gmail.com>
	<A9667DDFB95DB7438FA9D7D576C3D87E0AAFF48C@SHSMSX104.ccr.corp.intel.com>
	<CABoYbGoVPy9R=DD98xD4oYV88vAW=44-SUbFP1EbNWfFCR_DWw@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <CABoYbGoVPy9R=DD98xD4oYV88vAW=44-SUbFP1EbNWfFCR_DWw@mail.gmail.com>
Content-Language: en-US
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Matthias <matthias.kannenberg@googlemail.com>
Cc: "tim@xen.org" <tim@xen.org>, Jan Beulich <JBeulich@suse.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

Matthias wrote on 2014-06-20:
> Did some testing with your patches:
> 
> 
> Test 1:
> 
> 
> Xen-Version: Compiled a xen with the 077 commit, your first patch and your 3.
> (latest) patch for testing.
> 
> Passed-through devices: only vga
> 
> Result: At some late point in the boot of the domU, the dom0 (the 
> server itself) restarted itself. This is repeatable. I added the 
> qemu-log and what I was able to capture from the xl dmesg (I only hit 
> 'xl dmesg' numerous times until the server restarted and might not 
> have gotten the end of it, but since there is no xl dmesg log i'm 
> aware off, I couldn't think of another way. If you have any, please 
> tell me). From the qemu log  I would guess the restart was caused by some problem in the vram mapping?

I am a little confused: Previously, you said the failure is caused by the IOMMU fault:

(XEN) AMD-Vi: IO_PAGE_FAULT: domain = 1, device id = 0x500, fault address = 0x1d27d0000, flags = 0

But now, it seems you are encountering another problem. What's the difference between the two testings? The patch 1 and patch 3 only add some debug codes. They would not affect your testing. 

Also, I would suggest you to enable the serial console and get the log from console.

> 
> 
> Test 2:
> 
> 
> Xen-Version: Compiled a normal 077 commit version without any of your 
> patches
> 
> Xen cmd line: added immo=no-sharept as suggested
> 
> Passed-through devices: vga, 2 usb controller (ohci+ehci), sound card
> Result: works fine. domU boots, no BSOD, no other problems, server 
> seems to be stable. So not sharing the VT-d page tables seems to do 
> the trick / the problem is in the page table sharing.
>

Yes, this is what I am expecting.

Best regards,
Yang