From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Date: Thu, 13 May 2010 12:03:58 +0000 Subject: Re: [PATCH 0/7] Consolidate vcpu ioctl locking Message-Id: <4BEBEAAE.9030502@redhat.com> List-Id: References: <1273749459-622-1-git-send-email-avi@redhat.com> In-Reply-To: <1273749459-622-1-git-send-email-avi@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: kvm-ia64@vger.kernel.org On 05/13/2010 03:03 PM, Avi Kivity wrote: > On 05/13/2010 03:01 PM, Avi Kivity wrote: >> On 05/13/2010 02:57 PM, Alexander Graf wrote: >>> >>> Mind to give a high level overview on where you're moving which locks? >>> >> >> Um, looks like I forgot to fill in the patchset header. Sorry. > > Gar, I actually wrote it but forgot to save the file. > And it had some useful info: [PATCH 0/7] Consolidate vcpu ioctl locking In general, all vcpu ioctls need to take the vcpu mutex, but each one does it (or not) individually. This is cumbersome and error prone. This patchset moves all locking to a central place. This is complicated by the fact that ppc's KVM_INTERRUPT and s390's KVM_S390_INTERRUPT break the convention and need to run unlocked. -- error compiling committee.c: too many arguments to function