From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Wilson Subject: Re: [RFC PATCH 1/5] Support for running secondary emulators Date: Mon, 3 Mar 2014 14:41:19 -0800 Message-ID: <20140303224117.GA6867@u109add4315675089e695.ant.amazon.com> References: <1391091590-5454-1-git-send-email-paul.durrant@citrix.com> <20140301222442.GA24562@u109add4315675089e695.ant.amazon.com> <9AAE0902D5BC7E449B7C8E4E778ABCD02612E0@AMSPEX01CL01.citrite.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <9AAE0902D5BC7E449B7C8E4E778ABCD02612E0@AMSPEX01CL01.citrite.net> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Paul Durrant Cc: "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org On Mon, Mar 03, 2014 at 01:34:41PM +0000, Paul Durrant wrote: > > -----Original Message----- > > From: Matt Wilson [mailto:mswilson@gmail.com] On Behalf Of Matt Wilson > > Sent: 01 March 2014 22:25 > > To: Paul Durrant > > Cc: xen-devel@lists.xen.org > > Subject: Re: [Xen-devel] [RFC PATCH 1/5] Support for running secondary > > emulators > > > > On Thu, Jan 30, 2014 at 02:19:45PM +0000, Paul Durrant wrote: > > > This patch series adds the ioreq server interface which I mentioned in > > > my talk at the Xen developer summit in Edinburgh at the end of last year. > > > The code is based on work originally done by Julien Grall but has been > > > re-written to allow existing versions of QEMU to work unmodified. > > > > > [...] > > > > Hi Paul, > > > > I'm coming back to play with this after a few weeks and I'm having > > trouble getting things going. It seems that I'm crashing early when > > hvmloader is programming the PCI-ISA bridge link routes. [...] > > Any quick ideas before I go instrumenting things? > > > > > Matt, > > I've re-worked the patches a bit and am about to submit them non-RFC > this time. I had a look at the shared page setup given what you saw > and I can't find a memset anywhere to init the shared ioreq state so > I suspect your problem is just uninitialized mem (and I guess I may > not have seen it as I usually only run up a couple of VMs before I > need to do a host reboot so I'm probably getting freshly scrubbed > pages every time). So, I'll stick a memset in my new patch and > hopefully you can give the new code a spin once I've posted it. I'm not sure... I tossed a scrub_one_page(page); in hvm_set_ioreq_page() but that didn't seem to help. I'll give your next series a spin when you post them (or push to your git repository). --msw