Re: [PATCH 0/8] Various Book3s HV fixes that haven't been picked up yet

2012-10-16 Thread Alexander Graf
On 16.10.2012, at 05:08, Paul Mackerras wrote: > On Mon, Oct 15, 2012 at 02:00:54PM +0200, Alexander Graf wrote: >> >> Sorry, I can't accept patches that haven't shown up on kvm@vger. Please send >> this patch set again with CC to kvm@vger. > > Done; I didn't cc kvm-ppc this time since the p

Re: [PATCH 1/5] KVM: Provide mmu notifier retry test based on struct kvm

2012-10-16 Thread Avi Kivity
On 10/16/2012 05:59 AM, Paul Mackerras wrote: > The mmu_notifier_retry() function, used to test whether any page > invalidations are in progress, currently takes a vcpu pointer, though > the code only needs the VM's struct kvm pointer. Forthcoming patches > to the powerpc Book3S HV code will need

Re: [PATCH 1/5] KVM: Provide mmu notifier retry test based on struct kvm

2012-10-16 Thread Alexander Graf
On 16.10.2012, at 11:44, Avi Kivity wrote: > On 10/16/2012 05:59 AM, Paul Mackerras wrote: >> The mmu_notifier_retry() function, used to test whether any page >> invalidations are in progress, currently takes a vcpu pointer, though >> the code only needs the VM's struct kvm pointer. Forthcoming

Re: [PATCH 5/5] KVM: PPC: Book3S HV: Provide a method for userspace to read and write the HPT

2012-10-16 Thread Avi Kivity
On 10/16/2012 06:01 AM, Paul Mackerras wrote: > A new ioctl, KVM_PPC_GET_HTAB_FD, returns a file descriptor. Reads on > this fd return the contents of the HPT (hashed page table), writes > create and/or remove entries in the HPT. There is a new capability, > KVM_CAP_PPC_HTAB_FD, to indicate the p

Re: [PATCH 0/2] KVM: PPC: Support ioeventfd

2012-10-16 Thread Avi Kivity
On 10/15/2012 02:02 PM, Alexander Graf wrote: > In order to support vhost, we need to be able to support ioeventfd. > > This patch set adds support for ioeventfd to PPC and makes it possible to > do so without implementing irqfd along the way, as it requires an in-kernel > irqchip which we don't h

Re: [PATCH 1/2] KVM: Distangle eventfd code from irqchip

2012-10-16 Thread Avi Kivity
On 10/15/2012 02:02 PM, Alexander Graf wrote: > The current eventfd code assumes that when we have eventfd, we also have > irqfd for in-kernel interrupt delivery. This is not necessarily true. On > PPC we don't have an in-kernel irqchip yet, but we can still support easily > support eventfd. > Do

Re: [PATCH 0/2] KVM: PPC: Support ioeventfd

2012-10-16 Thread Alexander Graf
On 16.10.2012, at 12:56, Avi Kivity wrote: > On 10/15/2012 02:02 PM, Alexander Graf wrote: >> In order to support vhost, we need to be able to support ioeventfd. >> >> This patch set adds support for ioeventfd to PPC and makes it possible to >> do so without implementing irqfd along the way, as

Re: [PATCH 0/2] KVM: PPC: Support ioeventfd

2012-10-16 Thread Avi Kivity
On 10/16/2012 12:59 PM, Alexander Graf wrote: > > On 16.10.2012, at 12:56, Avi Kivity wrote: > >> On 10/15/2012 02:02 PM, Alexander Graf wrote: >>> In order to support vhost, we need to be able to support ioeventfd. >>> >>> This patch set adds support for ioeventfd to PPC and makes it possible t

Re: [PATCH 1/2] KVM: Distangle eventfd code from irqchip

2012-10-16 Thread Alexander Graf
On 16.10.2012, at 12:57, Avi Kivity wrote: > On 10/15/2012 02:02 PM, Alexander Graf wrote: >> The current eventfd code assumes that when we have eventfd, we also have >> irqfd for in-kernel interrupt delivery. This is not necessarily true. On >> PPC we don't have an in-kernel irqchip yet, but we

Re: [PATCH 0/2] KVM: PPC: Support ioeventfd

2012-10-16 Thread Alexander Graf
On 16.10.2012, at 13:01, Avi Kivity wrote: > On 10/16/2012 12:59 PM, Alexander Graf wrote: >> >> On 16.10.2012, at 12:56, Avi Kivity wrote: >> >>> On 10/15/2012 02:02 PM, Alexander Graf wrote: In order to support vhost, we need to be able to support ioeventfd. This patch set add

Re: [PATCH 5/5] KVM: PPC: Book3S HV: Provide a method for userspace to read and write the HPT

2012-10-16 Thread Paul Mackerras
On Tue, Oct 16, 2012 at 12:06:58PM +0200, Avi Kivity wrote: > On 10/16/2012 06:01 AM, Paul Mackerras wrote: > > +4.78 KVM_PPC_GET_HTAB_FD > > + > > +Capability: KVM_CAP_PPC_HTAB_FD > > +Architectures: powerpc > > +Type: vm ioctl > > +Parameters: Pointer to struct kvm_get_htab_fd (in) > > +Returns:

Re: [PATCH 5/5] KVM: PPC: Book3S HV: Provide a method for userspace to read and write the HPT

2012-10-16 Thread Avi Kivity
On 10/16/2012 01:58 PM, Paul Mackerras wrote: > On Tue, Oct 16, 2012 at 12:06:58PM +0200, Avi Kivity wrote: >> On 10/16/2012 06:01 AM, Paul Mackerras wrote: >> > +4.78 KVM_PPC_GET_HTAB_FD >> > + >> > +Capability: KVM_CAP_PPC_HTAB_FD >> > +Architectures: powerpc >> > +Type: vm ioctl >> > +Parameters

Re: [PATCH 0/2] KVM: PPC: Support ioeventfd

2012-10-16 Thread Avi Kivity
On 10/16/2012 01:06 PM, Alexander Graf wrote: > > On 16.10.2012, at 13:01, Avi Kivity wrote: > >> On 10/16/2012 12:59 PM, Alexander Graf wrote: >>> >>> On 16.10.2012, at 12:56, Avi Kivity wrote: >>> On 10/15/2012 02:02 PM, Alexander Graf wrote: > In order to support vhost, we need to b

Re: [PATCH 0/2] KVM: PPC: Support ioeventfd

2012-10-16 Thread Alexander Graf
On 10/16/2012 03:47 PM, Avi Kivity wrote: On 10/16/2012 01:06 PM, Alexander Graf wrote: On 16.10.2012, at 13:01, Avi Kivity wrote: On 10/16/2012 12:59 PM, Alexander Graf wrote: On 16.10.2012, at 12:56, Avi Kivity wrote: On 10/15/2012 02:02 PM, Alexander Graf wrote: In order to support vhos

Re: [PATCH] kvm/powerpc: Handle errors in secondary thread grabbing

2012-10-16 Thread Benjamin Herrenschmidt
On Tue, 2012-10-16 at 17:00 +1100, Michael Ellerman wrote: > > Thanks for looking at this - but in fact this is fixed by my patch > > entitled "KVM: PPC: Book3S HV: Fix some races in starting secondary > > threads" submitted back on August 28. > > OK thanks. It seems that patch didn't make 3.7 ? >

Re: [PATCH 5/5] KVM: PPC: Book3S HV: Provide a method for userspace to read and write the HPT

2012-10-16 Thread Anthony Liguori
Avi Kivity writes: > On 10/16/2012 01:58 PM, Paul Mackerras wrote: >> On Tue, Oct 16, 2012 at 12:06:58PM +0200, Avi Kivity wrote: >>> On 10/16/2012 06:01 AM, Paul Mackerras wrote: >>> > +4.78 KVM_PPC_GET_HTAB_FD >>> > + >>> > +Capability: KVM_CAP_PPC_HTAB_FD >>> > +Architectures: powerpc >>> > +T

Re: [PATCH 5/5] KVM: PPC: Book3S HV: Provide a method for userspace to read and write the HPT

2012-10-16 Thread Paul Mackerras
On Tue, Oct 16, 2012 at 03:06:33PM +0200, Avi Kivity wrote: > On 10/16/2012 01:58 PM, Paul Mackerras wrote: > > On Tue, Oct 16, 2012 at 12:06:58PM +0200, Avi Kivity wrote: > >> Does/should the fd support O_NONBLOCK and poll? (=waiting for an entry > >> to change). > > > > No. > > This forces user

Re: [PATCH] kvm/powerpc: Handle errors in secondary thread grabbing

2012-10-16 Thread Alexander Graf
On 16.10.2012, at 21:33, Benjamin Herrenschmidt wrote: > On Tue, 2012-10-16 at 17:00 +1100, Michael Ellerman wrote: >>> Thanks for looking at this - but in fact this is fixed by my patch >>> entitled "KVM: PPC: Book3S HV: Fix some races in starting secondary >>> threads" submitted back on August