From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41555) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YAxRg-0001Pw-0r for qemu-devel@nongnu.org; Tue, 13 Jan 2015 04:07:28 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YAxRa-00068d-V3 for qemu-devel@nongnu.org; Tue, 13 Jan 2015 04:07:27 -0500 Received: from mail.ispras.ru ([83.149.199.45]:34158) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YAxRa-00068L-Mj for qemu-devel@nongnu.org; Tue, 13 Jan 2015 04:07:22 -0500 From: "Pavel Dovgaluk" References: <20150112115944.3504.66763.stgit@PASHA-ISP> <20150112120111.3504.51955.stgit@PASHA-ISP> <54B3BA70.3080408@redhat.com> In-Reply-To: <54B3BA70.3080408@redhat.com> Date: Tue, 13 Jan 2015 12:07:21 +0300 Message-ID: <000301d02f10$56ec3fd0$04c4bf70$@Dovgaluk@ispras.ru> MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Content-Language: ru Subject: Re: [Qemu-devel] [RFC PATCH v7 15/21] replay: checkpoints List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: 'Paolo Bonzini' , qemu-devel@nongnu.org Cc: peter.maydell@linaro.org, peter.crosthwaite@xilinx.com, alex.bennee@linaro.org, mark.burton@greensocs.com, real@ispras.ru, batuzovk@ispras.ru, maria.klimushenkova@ispras.ru, afaerber@suse.de, fred.konrad@greensocs.com > From: Paolo Bonzini [mailto:pbonzini@redhat.com] > On 12/01/2015 13:01, Pavel Dovgalyuk wrote: > > + default: > > + case QEMU_CLOCK_VIRTUAL: > > + if ((replay_mode != REPLAY_MODE_NONE && !runstate_is_running()) > > + || !replay_checkpoint(run_all ? 2 : 3)) { > > + return false; > > + } > > + break; > > Please document the meaning of the numbers by making an enum. The numbers have no meaning. They just have to be distinct in different places. > Why do you have to distinguish run_all? It seems to be early versions artifact. I'll remove it. Pavel Dovgalyuk