From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34934) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZuZHw-0007Vm-M0 for qemu-devel@nongnu.org; Fri, 06 Nov 2015 00:10:13 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZuZHs-0000ck-LR for qemu-devel@nongnu.org; Fri, 06 Nov 2015 00:10:12 -0500 Received: from mail.ispras.ru ([83.149.199.45]:55267) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZuZHs-0000ZZ-Di for qemu-devel@nongnu.org; Fri, 06 Nov 2015 00:10:08 -0500 From: "Pavel Dovgaluk" References: <1446725643-82458-1-git-send-email-pbonzini@redhat.com> <563B6298.8050304@redhat.com> In-Reply-To: <563B6298.8050304@redhat.com> Date: Fri, 6 Nov 2015 08:10:08 +0300 Message-ID: <001201d11851$6893a860$39baf920$@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] [PULL 00/18] Record/replay core for 2.5-rc1 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: 'Paolo Bonzini' , 'Peter Maydell' Cc: 'QEMU Developers' > From: Paolo Bonzini [mailto:pbonzini@redhat.com] > On 05/11/2015 15:00, Peter Maydell wrote: > > On 5 November 2015 at 12:13, Paolo Bonzini wrote: > >> The following changes since commit 6c5f30cad290c745f910481d0e890b3f4fad1f00: > >> > >> Merge remote-tracking branch 'remotes/juanquintela/tags/migration/20151104' into staging > (2015-11-05 10:10:57 +0000) > >> > >> are available in the git repository at: > >> > >> > >> git://github.com/bonzini/qemu.git tags/for-upstream-replay > >> > >> for you to fetch changes up to 88dcb341cd6163a47346de83d4206591da6a9959: > >> > >> replay: recording of the user input (2015-11-05 12:40:48 +0100) > >> > >> ---------------------------------------------------------------- > >> So here it is, let's see what happens. > > > > Hi. This got through most of the merge tests, but running linux-user > > tests failed: Thanks for the report! > > /home/petmay01/linaro/qemu-for-merges/build/all-linux-static/i386-linux-user/qemu-i386 > > -L ./gnemul/qemu-i386 i386/ls -l dummyfile > > qemu-i386: /home/petmay01/linaro/qemu-for-merges/translate-all.c:146: > > tb_unlock: Assertion `have_tb_lock' failed. > > > > Seems to assert the first time we call tb_unlock(). Here's a backtrace: > > Program received signal SIGABRT, Aborted. > > 0x00000000601fa5d9 in raise () > > (gdb) bt > > #0 0x00000000601fa5d9 in raise () > > #1 0x000000006019a608 in abort () > > #2 0x0000000060194584 in __assert_fail_base () > > #3 0x00000000601945de in __assert_fail () > > #4 0x0000000060003d5b in tb_unlock () at > > /home/petmay01/linaro/qemu-for-merges/translate-all.c:146 > > #5 0x00000000600073fa in tb_find_slow (cpu=0x62605710, pc=4135504272, > > cs_base=0, flags=4194483) > > at /home/petmay01/linaro/qemu-for-merges/cpu-exec.c:279 > > #6 0x0000000060007581 in tb_find_fast (cpu=0x62605710) at > > /home/petmay01/linaro/qemu-for-merges/cpu-exec.c:316 > > #7 0x0000000060007868 in cpu_x86_exec (cpu=0x62605710) at > > /home/petmay01/linaro/qemu-for-merges/cpu-exec.c:423 > > #8 0x00000000600372b6 in cpu_loop (env=0x6260d990) at > > /home/petmay01/linaro/qemu-for-merges/linux-user/main.c:281 > > #9 0x0000000060039370 in main (argc=6, argv=0x7fffffffe438, > > envp=0x7fffffffe470) > > at /home/petmay01/linaro/qemu-for-merges/linux-user/main.c:4666 > > > > thanks > > -- PMM > > > > Pavel, can you look at it? I'll take care of squashing the change. Here is the patch: diff --git a/replay/replay-user.c b/replay/replay-user.c index eeaa41d..7d4f9fb 100755 --- a/replay/replay-user.c +++ b/replay/replay-user.c @@ -18,7 +18,7 @@ bool replay_exception(void) bool replay_has_exception(void) { - return true; + return false; } bool replay_interrupt(void) @@ -28,7 +28,7 @@ bool replay_interrupt(void) bool replay_has_interrupt(void) { - return true; + return false; } Pavel Dovgalyuk