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
On Fri, 08/17 19:14, no-re...@patchew.org wrote: > This series failed docker-clang@ubuntu build test. Please find the testing > commands and > their output below. If you have Docker installed, you can probably reproduce > it > locally. > > Type: series > Message-id: 20180817051853.23792-1-c...@b

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 --one

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