Re: [Qemu-devel] [PATCH v2 0/9] synchronization profiler

2018-08-18 Thread Emilio G. Cota
On Sat, Aug 18, 2018 at 14:45:54 +0800, Fam Zheng wrote: > On Fri, 08/17 19:14, no-re...@patchew.org wrote: (snip) > > /tmp/qemu-test/src/include/qemu/thread.h:101:1: error: all paths through > > this function will call itself [-Werror,-Winfinite-recursion] (snip) > > subprocess.CalledProcessError

Re: [Qemu-devel] [PATCH v2 0/9] synchronization profiler

2018-08-17 Thread Fam Zheng
20180817051853.23792-1-c...@braap.org > Subject: [Qemu-devel] [PATCH v2 0/9] synchronization profiler > > === TEST SCRIPT BEGIN === > #!/bin/bash > time make docker-test-clang@ubuntu SHOW_ENV=1 J=8 > === TEST SCRIPT END === > > Updating 3c8cf5a9c21ff8782164d1def7f44bd88871338

Re: [Qemu-devel] [PATCH v2 0/9] synchronization profiler

2018-08-17 Thread Emilio G. Cota
On Fri, Aug 17, 2018 at 12:38:05 +0200, Paolo Bonzini wrote: > Queued, I'll wait for more comments before sending a pull request. Thanks! Patchew reported some build errors on mingw, the most important of which is that I didn't handle !CONFIG_ATOMIC64. I have a v3 that fixes this by using a seqlo

Re: [Qemu-devel] [PATCH v2 0/9] synchronization profiler

2018-08-17 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20180817051853.23792-1-c...@braap.org Subject: [Qemu-devel] [PATCH v2 0/9] synchronization profiler === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log

Re: [Qemu-devel] [PATCH v2 0/9] synchronization profiler

2018-08-17 Thread Paolo Bonzini
On 17/08/2018 07:18, Emilio G. Cota wrote: > v1: https://lists.gnu.org/archive/html/qemu-devel/2018-08/msg02186.html > > Changes since v1: > > - Rebase on current master. > - Update copyright to 2018. > - Add -m option to the HMP info command to sort by average wait time, > as suggested by Paol

[Qemu-devel] [PATCH v2 0/9] synchronization profiler

2018-08-16 Thread Emilio G. Cota
v1: https://lists.gnu.org/archive/html/qemu-devel/2018-08/msg02186.html Changes since v1: - Rebase on current master. - Update copyright to 2018. - Add -m option to the HMP info command to sort by average wait time, as suggested by Paolo. - Add -n option to the HMP info command to NOT merge cal