Re: [PATCH v5 0/8] In-kernel XICS interrupt controller emulation

2013-04-30 Thread Alexander Graf
On 28.04.2013, at 14:05, Paul Mackerras wrote: > On Sun, Apr 28, 2013 at 11:50:49AM +0200, Alexander Graf wrote: >> >> On 27.04.2013, at 10:36, Paul Mackerras wrote: >> >>> On Fri, Apr 26, 2013 at 04:30:37PM +0200, Alexander Graf wrote: Very nice patch set. I've applie 1-7 of it to kvm-ppc

Re: [PATCH v5 0/8] In-kernel XICS interrupt controller emulation

2013-04-28 Thread Paul Mackerras
On Sun, Apr 28, 2013 at 11:50:49AM +0200, Alexander Graf wrote: > > On 27.04.2013, at 10:36, Paul Mackerras wrote: > > > On Fri, Apr 26, 2013 at 04:30:37PM +0200, Alexander Graf wrote: > >> Very nice patch set. I've applie 1-7 of it to kvm-ppc-queue. So they will > >> hopefully make it to 3.10.

Re: [PATCH v5 0/8] In-kernel XICS interrupt controller emulation

2013-04-28 Thread Alexander Graf
On 27.04.2013, at 10:36, Paul Mackerras wrote: > On Fri, Apr 26, 2013 at 04:30:37PM +0200, Alexander Graf wrote: >> Very nice patch set. I've applie 1-7 of it to kvm-ppc-queue. So they will >> hopefully make it to 3.10. >> >> Please check for 8/8 whether >> >> a) You want to have a released k

Re: [PATCH v5 0/8] In-kernel XICS interrupt controller emulation

2013-04-27 Thread Paul Mackerras
On Fri, Apr 26, 2013 at 04:30:37PM +0200, Alexander Graf wrote: > Very nice patch set. I've applie 1-7 of it to kvm-ppc-queue. So they will > hopefully make it to 3.10. > > Please check for 8/8 whether > > a) You want to have a released kernel version without irq routing (irqfd) > support. It

Re: [PATCH v5 0/8] In-kernel XICS interrupt controller emulation

2013-04-26 Thread Scott Wood
On 04/26/2013 06:52:47 PM, Paul Mackerras wrote: On Fri, Apr 26, 2013 at 12:13:33PM -0500, Scott Wood wrote: > Also please note that we no longer hold kvm->lock during device > creation, so your EEXIST check looks racy. Huh? Patch 2/8 adds this code: + mutex_lock(&kvm->lock); + if

Re: [PATCH v5 0/8] In-kernel XICS interrupt controller emulation

2013-04-26 Thread Paul Mackerras
On Fri, Apr 26, 2013 at 12:13:33PM -0500, Scott Wood wrote: > Also please note that we no longer hold kvm->lock during device > creation, so your EEXIST check looks racy. Huh? Patch 2/8 adds this code: + mutex_lock(&kvm->lock); + if (kvm->arch.xics) + ret = -EEXIST; +

Re: [PATCH v5 0/8] In-kernel XICS interrupt controller emulation

2013-04-26 Thread Scott Wood
On 04/26/2013 09:30:37 AM, Alexander Graf wrote: On 18.04.2013, at 08:29, Paul Mackerras wrote: > This is a repost of my patch series implementing in-kernel emulation > of the XICS interrupt controller architecture defined in PAPR (Power > Architecture Platform Requirements, the document that d

Re: [PATCH v5 0/8] In-kernel XICS interrupt controller emulation

2013-04-26 Thread Alexander Graf
On 18.04.2013, at 08:29, Paul Mackerras wrote: > This is a repost of my patch series implementing in-kernel emulation > of the XICS interrupt controller architecture defined in PAPR (Power > Architecture Platform Requirements, the document that defines IBM's > pSeries platform architecture). Thi

[PATCH v5 0/8] In-kernel XICS interrupt controller emulation

2013-04-17 Thread Paul Mackerras
This is a repost of my patch series implementing in-kernel emulation of the XICS interrupt controller architecture defined in PAPR (Power Architecture Platform Requirements, the document that defines IBM's pSeries platform architecture). This version of the patch series uses the latest device API