From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:36933)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1UGjsB-0006DJ-6I
	for qemu-devel@nongnu.org; Sat, 16 Mar 2013 01:41:40 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1UGjsA-0007Cl-5I
	for qemu-devel@nongnu.org; Sat, 16 Mar 2013 01:41:39 -0400
Message-ID: <1363412487.1244.21.camel@pasglop>
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: Sat, 16 Mar 2013 06:41:27 +0100
In-Reply-To: <DEFE944E-0603-460B-9382-AD35B791E50E@suse.de>
References: <1363226008-26639-1-git-send-email-david@gibson.dropbear.id.au>
	<1363226008-26639-6-git-send-email-david@gibson.dropbear.id.au>
	<89E98310-17EE-4BFD-B8F6-A8FCC23FE5F1@suse.de>
	<1363403667.1244.0.camel@pasglop>
	<DEFE944E-0603-460B-9382-AD35B791E50E@suse.de>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH 5/5] pseries: Move XICS initialization
 before cpu initialization
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/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: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Alexander Graf <agraf@suse.de>
Cc: Michael Ellerman <michael@ellerman.id.au>, Paul Mackerras <paulus@samba.org>, "qemu-ppc@nongnu.org
	list:PowerPC" <qemu-ppc@nongnu.org>, qemu-devel qemu-devel <qemu-devel@nongnu.org>, David Gibson <david@gibson.dropbear.id.au>

On Sat, 2013-03-16 at 06:33 +0100, Alexander Graf wrote:
> >> We're changing that notion in the in-kernel XICS discussions.  The
> flow will look like this:
> >> 
> >>  * create vcpus
> >>  * create XICS
> >>  * foreach (vcpu)
> >>      * enable_cap(vcpu, CAP_XICS_SERVER, xics_handle)
> > 
> > This is stupid. Why have the VCPU initialize itself for non-kernel
> > interrupts and *then* switch it over ?
> 
> Because non-kernel initialization is a nop.

And ?

Ben.