From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1NchBA-0001fv-19 for qemu-devel@nongnu.org; Wed, 03 Feb 2010 10:30:08 -0500 Received: from [199.232.76.173] (port=60633 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NchB9-0001eq-Fl for qemu-devel@nongnu.org; Wed, 03 Feb 2010 10:30:07 -0500 Received: from Debian-exim by monty-python.gnu.org with spam-scanned (Exim 4.60) (envelope-from ) id 1NchB5-0004AX-9u for qemu-devel@nongnu.org; Wed, 03 Feb 2010 10:30:07 -0500 Received: from mx1.redhat.com ([209.132.183.28]:46931) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1NchB4-0004AC-7n for qemu-devel@nongnu.org; Wed, 03 Feb 2010 10:30:02 -0500 Subject: Re: [Qemu-devel] [Patch] Support translating Guest physical address to Host virtual address. From: Lucas Meneghel Rodrigues In-Reply-To: <4B6986F6.5000808@linux.vnet.ibm.com> References: <4B60B28A.40400@linux.vnet.ibm.com> <1264631460.29051.35.camel@w-amax.beaverton.ibm.com> <4B697D04.7070507@linux.vnet.ibm.com> <46D41A8912DCCF4FB93FA509BD00C63101E79CA0@irsmsx002.ger.corp.intel.com> <4B6986F6.5000808@linux.vnet.ibm.com> Content-Type: text/plain; charset="UTF-8" Date: Wed, 03 Feb 2010 13:29:43 -0200 Message-ID: <1265210983.2295.15.camel@localhost.localdomain> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: "Li, Haicheng" , Max Asbock , "qemu-devel@nongnu.org" , "Zheng, Jiajia" , "You, Yongkang" , "avi@redhat.com" , "Kleen, Andi" On Wed, 2010-02-03 at 08:23 -0600, Anthony Liguori wrote: > On 02/03/2010 08:11 AM, Kleen, Andi wrote: > > > >> If you want to integrate MCE testing into qemu using the host > >> to inject > >> MCEs, you'll need to figure out a way to do it that allows the whole > >> thing to be done without potentially killing the wrong process or > >> > > In the test it's very simple. The process is kept running during > > the injection. With that its addresses do not change. > > > > So that problem is already solved. > > > > The feature doesn't aim to be a general feature for normal users, > > it's merely for QA. > > > > Yeah, but if we put a feature in qemu, we need to be able to support it > for anyone who wants to use it. > > Adding something for a very particular test suite that won't work in > normal circumstances is just asking for trouble IMHO. Perhaps we can enable it at compile time, so in production builds that is disabled. It's not *that* much of a problem. The purpose for this patch is that we can integrate the whole test suite into kvm autotest, and we have ways to conveniently build qemu source code, passing special flags to configure and everything. So having this patch upstream at some point would make things easier, since the folks at intel won't have to keep maintaining that patch for every little change on qemu code, and we'll have better qa for qemu and kvm.