From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from [140.186.70.92] (port=56936 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1Polsq-000874-Gx
	for qemu-devel@nongnu.org; Sun, 13 Feb 2011 19:01:41 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <dgibson@ozlabs.org>) id 1Polsp-0007fV-6a
	for qemu-devel@nongnu.org; Sun, 13 Feb 2011 19:01:40 -0500
Received: from ozlabs.org ([203.10.76.45]:47079)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <dgibson@ozlabs.org>) id 1Polso-0007eq-P5
	for qemu-devel@nongnu.org; Sun, 13 Feb 2011 19:01:39 -0500
Date: Mon, 14 Feb 2011 10:33:26 +1100
From: David Gibson <david@gibson.dropbear.id.au>
Subject: Re: [Qemu-devel] Re: [PATCH 15/15] Implement the bus structure for
	PAPR virtual IO
Message-ID: <20110213233326.GB2821@yookeroo>
References: <1297522467-5975-16-git-send-email-david@gibson.dropbear.id.au>
	<1BA55C28-0AAF-46A5-A14F-04B0B61419DB@suse.de>
	<AANLkTimYxXeBuKmXKdv=ENu4ZZyDK=CEv+et568ga1Qk@mail.gmail.com>
	<1297544424.14982.628.camel@pasglop>
	<AANLkTikKov5tHRiNwU+CxzRmWU2obZDURPUp3w+jsuB-@mail.gmail.com>
	<1297552503.14982.637.camel@pasglop>
	<AANLkTikTZrcvcRE9msA+1HF1xPDQgHpRugtAn9CzC3Or@mail.gmail.com>
	<20110213111245.GC18294@yookeroo> <4D57F3E6.10504@codemonkey.ws>
	<AANLkTik8gfBDrMwWmVTshopHKiuO6CQQge605dxDJwvt@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <AANLkTik8gfBDrMwWmVTshopHKiuO6CQQge605dxDJwvt@mail.gmail.com>
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Blue Swirl <blauwirbel@gmail.com>
Cc: Alexander Graf <agraf@suse.de>, Paul Mackerras <paulus@samba.org>, "qemu-devel@nongnu.org List" <qemu-devel@nongnu.org>, anton@samba.org

On Sun, Feb 13, 2011 at 08:29:05PM +0200, Blue Swirl wrote:
> On Sun, Feb 13, 2011 at 5:08 PM, Anthony Liguori <anthony@codemonkey.ws> wrote:
> > On 02/13/2011 05:12 AM, David Gibson wrote:
[snip]
> > The arguments will have to be extracted from the CPU state but I don't think
> > we'll really ever have common hypercall implementations anyway so that's not
> > a huge problem.
> 
> Nice idea. Then the part handling CPUState probably should belong to
> target-ppc/ rather than hw/.

Doesn't work.  Different hypervisors may have arguments - even the
hcall number itself - arranged differently in the registers.  My
earlier drafts had this in target-ppc/; I moved it to hw/ for a
reason.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson