From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Mackerras Subject: Re: [PATCH v3 2/4] KVM: use simple waitqueue for vcpu->wq Date: Wed, 21 Oct 2015 20:24:11 +1100 Message-ID: <20151021092411.GD15591@fergus.ozlabs.ibm.com> References: <1445326090-1698-1-git-send-email-daniel.wagner@bmw-carit.de> <1445326090-1698-3-git-send-email-daniel.wagner@bmw-carit.de> <20151020140031.GG17308@twins.programming.kicks-ass.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Daniel Wagner , linux-kernel@vger.kernel.org, linux-rt-users@vger.kernel.org, Marcelo Tosatti , Paolo Bonzini , "Paul E. McKenney" , Paul Gortmaker , Thomas Gleixner To: Peter Zijlstra Return-path: Content-Disposition: inline In-Reply-To: <20151020140031.GG17308@twins.programming.kicks-ass.net> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-rt-users.vger.kernel.org On Tue, Oct 20, 2015 at 04:00:31PM +0200, Peter Zijlstra wrote: > > Should we not take this opportunity to get rid of these open-coded wait > loops? > > > Does this work? No, on Book3S HV (POWER8) the VM hangs immediately after the kernel brings up all the secondary vCPUs, and is then unkillable. I'm not sure what's wrong, although I wonder why you have TASK_IDLE rather than TASK_INTERRUPTIBLE in the ___wait_event call. Paul.