From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37659) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZzRGE-0004JC-BZ for qemu-devel@nongnu.org; Thu, 19 Nov 2015 10:36:35 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZzRG9-0005V3-7K for qemu-devel@nongnu.org; Thu, 19 Nov 2015 10:36:34 -0500 Received: from e38.co.us.ibm.com ([32.97.110.159]:54489) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZzRG8-0005Ug-VU for qemu-devel@nongnu.org; Thu, 19 Nov 2015 10:36:29 -0500 Received: from localhost by e38.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 19 Nov 2015 08:36:26 -0700 Received: from b03cxnp08027.gho.boulder.ibm.com (b03cxnp08027.gho.boulder.ibm.com [9.17.130.19]) by d03dlp01.boulder.ibm.com (Postfix) with ESMTP id BEFCF1FF0045 for ; Thu, 19 Nov 2015 08:24:34 -0700 (MST) Received: from d03av04.boulder.ibm.com (d03av04.boulder.ibm.com [9.17.195.170]) by b03cxnp08027.gho.boulder.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id tAJFaLW21704194 for ; Thu, 19 Nov 2015 08:36:23 -0700 Received: from d03av04.boulder.ibm.com (loopback [127.0.0.1]) by d03av04.boulder.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id tAJFaLO1010408 for ; Thu, 19 Nov 2015 08:36:21 -0700 References: <1447945814-15765-1-git-send-email-mjrosato@linux.vnet.ibm.com> <1447945814-15765-3-git-send-email-mjrosato@linux.vnet.ibm.com> <564DE9CE.6080303@redhat.com> From: Matthew Rosato Message-ID: <564DEC73.6080904@linux.vnet.ibm.com> Date: Thu, 19 Nov 2015 10:36:19 -0500 MIME-Version: 1.0 In-Reply-To: <564DE9CE.6080303@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 2/9] cpus: Add a sync version of cpu_remove() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini , qemu-devel@nongnu.org Cc: agraf@suse.de, borntraeger@de.ibm.com, Bharata B Rao , cornelia.huck@de.ibm.com, afaerber@suse.de, rth@twiddle.net On 11/19/2015 10:25 AM, Paolo Bonzini wrote: > > > On 19/11/2015 16:10, Matthew Rosato wrote: >> From: Bharata B Rao >> >> This sync API will be used by the CPU hotplug code to wait for the CPU to >> completely get removed before flagging the failure to the device_add >> command. >> >> Sync version of this call is needed to correctly recover from CPU >> realization failures when ->plug() handler fails. >> >> Signed-off-by: Bharata B Rao >> Signed-off-by: Matthew Rosato >> [Call cpu_remove() directly from cpu_remove_sync()] > > Are you using it in these patches? > Ugh, nice catch -- I am not using it anymore, I decided it was fine to just use cpu_remove in patch 8 directly. So, this patch can be dropped from the set. Matt