From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:53493) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T3WRx-0000wS-Aw for qemu-devel@nongnu.org; Mon, 20 Aug 2012 14:11:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1T3WRv-00085A-Gy for qemu-devel@nongnu.org; Mon, 20 Aug 2012 14:11:41 -0400 Received: from thoth.sbs.de ([192.35.17.2]:16540) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T3WRv-00084u-6k for qemu-devel@nongnu.org; Mon, 20 Aug 2012 14:11:39 -0400 Message-ID: <50327DD8.8070205@siemens.com> Date: Mon, 20 Aug 2012 20:11:36 +0200 From: Jan Kiszka MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] [PATCH] Drop redundant resume_all_vcpus from main List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel Cc: Paolo Bonzini VCPUs are either resumed directly via vm_start, after the incoming migration is done, or when a continue command is issued. We don't need the explicit resume before entering main_loop. Signed-off-by: Jan Kiszka --- I was adding nesting support to pause/resume_all_vcpus, and that stumbled over the imbalance below. vl.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/vl.c b/vl.c index ebee867..231d3ab 100644 --- a/vl.c +++ b/vl.c @@ -3757,7 +3757,6 @@ int main(int argc, char **argv, char **envp) os_setup_post(); - resume_all_vcpus(); main_loop(); bdrv_close_all(); pause_all_vcpus(); -- 1.7.3.4