[{"id":1767418,"web_url":"http://patchwork.ozlabs.org/comment/1767418/","msgid":"<150525566622.321.14749941518071789021@6d89bd104fc3>","list_archive_url":null,"date":"2017-09-12T22:34:26","subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","submitter":{"id":69632,"url":"http://patchwork.ozlabs.org/api/people/69632/","name":null,"email":"no-reply@patchew.org"},"content":"Hi,\n\nThis series seems to have some coding style problems. See output below for\nmore information:\n\nSubject: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event instrumentation\nMessage-id: 150525010239.15988.8172586618197849619.stgit@frigg.lan\nType: series\n\n=== TEST SCRIPT BEGIN ===\n#!/bin/bash\n\nBASE=base\nn=1\ntotal=$(git log --oneline $BASE.. | wc -l)\nfailed=0\n\ngit config --local diff.renamelimit 0\ngit config --local diff.renames True\n\ncommits=\"$(git log --format=%H --reverse $BASE..)\"\nfor c in $commits; do\n    echo \"Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)...\"\n    if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then\n        failed=1\n        echo\n    fi\n    n=$((n+1))\ndone\n\nexit $failed\n=== TEST SCRIPT END ===\n\nUpdating 3c8cf5a9c21ff8782164d1def7f44bd888713384\nFrom https://github.com/patchew-project/qemu\n * [new tag]               patchew/150525010239.15988.8172586618197849619.stgit@frigg.lan -> patchew/150525010239.15988.8172586618197849619.stgit@frigg.lan\n t [tag update]            patchew/20170912144459.11359-1-pbonzini@redhat.com -> patchew/20170912144459.11359-1-pbonzini@redhat.com\nSwitched to a new branch 'test'\n1ab48ae9b7 instrument: Add API to manipulate guest memory\n7e0bd2cad7 instrument: Add event 'guest_user_syscall_ret'\n334caef899 instrument: Add event 'guest_user_syscall'\n09a1773791 instrument: Add event 'guest_mem_before_exec'\n2bd64563d3 instrument: Add event 'guest_mem_before_trans'\n5b344ec1c3 trace: Introduce a proper structure to describe memory accesses\n04e5b883b1 instrument: Add event 'guest_cpu_reset'\n7971d0f2a4 instrument: Add event 'guest_cpu_exit'\n53dbc9ad88 exec: Add function to synchronously flush TB on a stopped vCPU\nd8b51515d2 instrument: Support synchronous modification of vCPU state\n08d492e35f instrument: Add event 'guest_cpu_enter'\n0be52b1bbd instrument: Track vCPUs\n7ab01f20f5 instrument: Add support for tracing events\n78676cff2d instrument: Add basic control interface\n00172972ae instrument: [hmp] Add library loader\n34ccf831e6 instrument: [qapi] Add library loader\nd1ab648b00 instrument: [softmmu] Add command line library loader\n150ad4a651 instrument: [bsd-user] Add command line library loader\na064b1621a instrument: [linux-user] Add command line library loader\naa78ee9f5a instrument: Add generic library loader\nf10357e313 instrument: Add configure-time flag\n4d324ad619 instrument: Add documentation\n\n=== OUTPUT BEGIN ===\nChecking PATCH 1/22: instrument: Add documentation...\nChecking PATCH 2/22: instrument: Add configure-time flag...\nChecking PATCH 3/22: instrument: Add generic library loader...\nChecking PATCH 4/22: instrument: [linux-user] Add command line library loader...\nChecking PATCH 5/22: instrument: [bsd-user] Add command line library loader...\nChecking PATCH 6/22: instrument: [softmmu] Add command line library loader...\nChecking PATCH 7/22: instrument: [qapi] Add library loader...\nERROR: externs should be avoided in .c files\n#254: FILE: stubs/instrument.c:40:\n+void qmp_instr_unload(const char *id, Error **errp);\n\ntotal: 1 errors, 0 warnings, 204 lines checked\n\nYour patch has style problems, please review.  If any of these errors\nare false positives report them to the maintainer, see\nCHECKPATCH in MAINTAINERS.\n\nChecking PATCH 8/22: instrument: [hmp] Add library loader...\nChecking PATCH 9/22: instrument: Add basic control interface...\nWARNING: architecture specific defines should be avoided\n#52: FILE: include/qemu/compiler.h:119:\n+#if defined _WIN32 || defined __CYGWIN__\n\nWARNING: architecture specific defines should be avoided\n#53: FILE: include/qemu/compiler.h:120:\n+  #ifdef __GNUC__\n\nWARNING: architecture specific defines should be avoided\n#59: FILE: include/qemu/compiler.h:126:\n+  #if __GNUC__ >= 4\n\nWARNING: architecture specific defines should be avoided\n#343: FILE: instrument/qemu-instr/control.h:13:\n+#ifdef __cplusplus\n\nWARNING: architecture specific defines should be avoided\n#372: FILE: instrument/qemu-instr/control.h:42:\n+#ifdef __cplusplus\n\ntotal: 0 errors, 5 warnings, 309 lines checked\n\nYour patch has style problems, please review.  If any of these errors\nare false positives report them to the maintainer, see\nCHECKPATCH in MAINTAINERS.\nChecking PATCH 10/22: instrument: Add support for tracing events...\nWARNING: architecture specific defines should be avoided\n#77: FILE: instrument/qemu-instr/types.h:13:\n+#ifdef __cplusplus\n\nWARNING: architecture specific defines should be avoided\n#111: FILE: instrument/qemu-instr/types.h:47:\n+#ifdef __cplusplus\n\ntotal: 0 errors, 2 warnings, 225 lines checked\n\nYour patch has style problems, please review.  If any of these errors\nare false positives report them to the maintainer, see\nCHECKPATCH in MAINTAINERS.\nChecking PATCH 11/22: instrument: Track vCPUs...\nChecking PATCH 12/22: instrument: Add event 'guest_cpu_enter'...\nChecking PATCH 13/22: instrument: Support synchronous modification of vCPU state...\nWARNING: line over 80 characters\n#73: FILE: instrument/control.c:85:\n+        async_run_on_cpu(cpu, instr_cpu_stop_all__cb, RUN_ON_CPU_HOST_PTR(info));\n\ntotal: 0 errors, 1 warnings, 127 lines checked\n\nYour patch has style problems, please review.  If any of these errors\nare false positives report them to the maintainer, see\nCHECKPATCH in MAINTAINERS.\nChecking PATCH 14/22: exec: Add function to synchronously flush TB on a stopped vCPU...\nChecking PATCH 15/22: instrument: Add event 'guest_cpu_exit'...\nChecking PATCH 16/22: instrument: Add event 'guest_cpu_reset'...\nChecking PATCH 17/22: trace: Introduce a proper structure to describe memory accesses...\nERROR: spaces prohibited around that ':' (ctx:WxW)\n#244: FILE: trace/mem.h:29:\n+            uint8_t size_shift : 2;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:VxW)\n#245: FILE: trace/mem.h:30:\n+            bool    sign_extend: 1;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:WxW)\n#246: FILE: trace/mem.h:31:\n+            uint8_t endianness : 1;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:WxW)\n#247: FILE: trace/mem.h:32:\n+            bool    store      : 1;\n                                ^\n\ntotal: 4 errors, 0 warnings, 227 lines checked\n\nYour patch has style problems, please review.  If any of these errors\nare false positives report them to the maintainer, see\nCHECKPATCH in MAINTAINERS.\n\nChecking PATCH 18/22: instrument: Add event 'guest_mem_before_trans'...\nERROR: spaces prohibited around that ':' (ctx:WxW)\n#302: FILE: instrument/qemu-instr/types.h:64:\n+            uint8_t size_shift : 2;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:VxW)\n#303: FILE: instrument/qemu-instr/types.h:65:\n+            bool    sign_extend: 1;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:WxW)\n#304: FILE: instrument/qemu-instr/types.h:66:\n+            uint8_t endianness : 1;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:WxW)\n#305: FILE: instrument/qemu-instr/types.h:67:\n+            bool    store      : 1;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:WxW)\n#430: FILE: trace/control.h:37:\n+            uint8_t size_shift : 2;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:VxW)\n#431: FILE: trace/control.h:38:\n+            bool    sign_extend: 1;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:WxW)\n#432: FILE: trace/control.h:39:\n+            uint8_t endianness : 1;\n                                ^\n\nERROR: spaces prohibited around that ':' (ctx:WxW)\n#433: FILE: trace/control.h:40:\n+            bool    store      : 1;\n                                ^\n\ntotal: 8 errors, 0 warnings, 389 lines checked\n\nYour patch has style problems, please review.  If any of these errors\nare false positives report them to the maintainer, see\nCHECKPATCH in MAINTAINERS.\n\nChecking PATCH 19/22: instrument: Add event 'guest_mem_before_exec'...\nERROR: externs should be avoided in .c files\n#337: FILE: stubs/instrument.c:61:\n+void helper_instr_guest_mem_before_exec(\n\ntotal: 1 errors, 0 warnings, 258 lines checked\n\nYour patch has style problems, please review.  If any of these errors\nare false positives report them to the maintainer, see\nCHECKPATCH in MAINTAINERS.\n\nChecking PATCH 20/22: instrument: Add event 'guest_user_syscall'...\nChecking PATCH 21/22: instrument: Add event 'guest_user_syscall_ret'...\nChecking PATCH 22/22: instrument: Add API to manipulate guest memory...\nWARNING: architecture specific defines should be avoided\n#41: FILE: instrument/qemu-instr/state.h:13:\n+#ifdef __cplusplus\n\nWARNING: architecture specific defines should be avoided\n#128: FILE: instrument/qemu-instr/state.h:100:\n+#ifdef __cplusplus\n\ntotal: 0 errors, 2 warnings, 181 lines checked\n\nYour patch has style problems, please review.  If any of these errors\nare false positives report them to the maintainer, see\nCHECKPATCH in MAINTAINERS.\n=== OUTPUT END ===\n\nTest command exited with code: 1\n\n\n---\nEmail generated automatically by Patchew [http://patchew.org/].\nPlease send your feedback to patchew-devel@freelists.org","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xsKPF4DWJz9t4X\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 13 Sep 2017 08:35:13 +1000 (AEST)","from localhost ([::1]:39013 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1drtlv-0007S5-Ov\n\tfor incoming@patchwork.ozlabs.org; Tue, 12 Sep 2017 18:35:11 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:56966)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <no-reply@patchew.org>) id 1drtlV-0007RU-Vs\n\tfor qemu-devel@nongnu.org; Tue, 12 Sep 2017 18:34:48 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <no-reply@patchew.org>) id 1drtlS-0002lc-3c\n\tfor qemu-devel@nongnu.org; Tue, 12 Sep 2017 18:34:46 -0400","from sender-of-o52.zoho.com ([135.84.80.217]:21344)\n\tby eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from <no-reply@patchew.org>)\n\tid 1drtlR-0002kS-Rn\n\tfor qemu-devel@nongnu.org; Tue, 12 Sep 2017 18:34:42 -0400","from [172.17.0.2] (23.253.156.214 [23.253.156.214]) by\n\tmx.zohomail.com with SMTPS id 1505255666812480.87153545026;\n\tTue, 12 Sep 2017 15:34:26 -0700 (PDT)"],"Resent-Date":"Tue, 12 Sep 2017 18:34:46 -0400","Resent-Message-Id":"<E1drtlS-0002lc-3c@eggs.gnu.org>","In-Reply-To":"<150525010239.15988.8172586618197849619.stgit@frigg.lan>","Message-ID":"<150525566622.321.14749941518071789021@6d89bd104fc3>","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Resent-From":"","From":"no-reply@patchew.org","To":"vilanova@ac.upc.edu","Date":"Tue, 12 Sep 2017 15:34:26 -0700 (PDT)","X-ZohoMailClient":"External","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 3.x [fuzzy]","X-Received-From":"135.84.80.217","Subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Reply-To":"qemu-devel@nongnu.org","Cc":"cota@braap.org, famz@redhat.com, qemu-devel@nongnu.org,\n\tstefanha@redhat.com, armbru@redhat.com","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1767420,"web_url":"http://patchwork.ozlabs.org/comment/1767420/","msgid":"<150525577025.321.3001704927093785609@6d89bd104fc3>","list_archive_url":null,"date":"2017-09-12T22:36:10","subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","submitter":{"id":69632,"url":"http://patchwork.ozlabs.org/api/people/69632/","name":null,"email":"no-reply@patchew.org"},"content":"Hi,\n\nThis series failed automatic build test. Please find the testing commands and\ntheir output below. If you have docker installed, you can probably reproduce it\nlocally.\n\nSubject: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event instrumentation\nMessage-id: 150525010239.15988.8172586618197849619.stgit@frigg.lan\nType: series\n\n=== TEST SCRIPT BEGIN ===\n#!/bin/bash\nset -e\ngit submodule update --init dtc\n# Let docker tests dump environment info\nexport SHOW_ENV=1\nexport J=8\ntime make docker-test-quick@centos6\ntime make docker-test-build@min-glib\ntime make docker-test-mingw@fedora\n=== TEST SCRIPT END ===\n\nUpdating 3c8cf5a9c21ff8782164d1def7f44bd888713384\nSwitched to a new branch 'test'\n1ab48ae9b7 instrument: Add API to manipulate guest memory\n7e0bd2cad7 instrument: Add event 'guest_user_syscall_ret'\n334caef899 instrument: Add event 'guest_user_syscall'\n09a1773791 instrument: Add event 'guest_mem_before_exec'\n2bd64563d3 instrument: Add event 'guest_mem_before_trans'\n5b344ec1c3 trace: Introduce a proper structure to describe memory accesses\n04e5b883b1 instrument: Add event 'guest_cpu_reset'\n7971d0f2a4 instrument: Add event 'guest_cpu_exit'\n53dbc9ad88 exec: Add function to synchronously flush TB on a stopped vCPU\nd8b51515d2 instrument: Support synchronous modification of vCPU state\n08d492e35f instrument: Add event 'guest_cpu_enter'\n0be52b1bbd instrument: Track vCPUs\n7ab01f20f5 instrument: Add support for tracing events\n78676cff2d instrument: Add basic control interface\n00172972ae instrument: [hmp] Add library loader\n34ccf831e6 instrument: [qapi] Add library loader\nd1ab648b00 instrument: [softmmu] Add command line library loader\n150ad4a651 instrument: [bsd-user] Add command line library loader\na064b1621a instrument: [linux-user] Add command line library loader\naa78ee9f5a instrument: Add generic library loader\nf10357e313 instrument: Add configure-time flag\n4d324ad619 instrument: Add documentation\n\n=== OUTPUT BEGIN ===\nSubmodule 'dtc' (git://git.qemu-project.org/dtc.git) registered for path 'dtc'\nCloning into '/var/tmp/patchew-tester-tmp-lf5qg9x6/src/dtc'...\nSubmodule path 'dtc': checked out '558cd81bdd432769b59bff01240c44f82cfb1a9d'\n  BUILD   centos6\nmake[1]: Entering directory '/var/tmp/patchew-tester-tmp-lf5qg9x6/src'\n  ARCHIVE qemu.tgz\n  ARCHIVE dtc.tgz\n  COPY    RUNNER\n    RUN test-quick in qemu:centos6 \nPackages installed:\nSDL-devel-1.2.14-7.el6_7.1.x86_64\nbison-2.4.1-5.el6.x86_64\nbzip2-devel-1.0.5-7.el6_0.x86_64\nccache-3.1.6-2.el6.x86_64\ncsnappy-devel-0-6.20150729gitd7bc683.el6.x86_64\nflex-2.5.35-9.el6.x86_64\ngcc-4.4.7-18.el6.x86_64\ngit-1.7.1-8.el6.x86_64\nglib2-devel-2.28.8-9.el6.x86_64\nlibepoxy-devel-1.2-3.el6.x86_64\nlibfdt-devel-1.4.0-1.el6.x86_64\nlibrdmacm-devel-1.0.21-0.el6.x86_64\nlzo-devel-2.03-3.1.el6_5.1.x86_64\nmake-3.81-23.el6.x86_64\nmesa-libEGL-devel-11.0.7-4.el6.x86_64\nmesa-libgbm-devel-11.0.7-4.el6.x86_64\npackage g++ is not installed\npixman-devel-0.32.8-1.el6.x86_64\nspice-glib-devel-0.26-8.el6.x86_64\nspice-server-devel-0.12.4-16.el6.x86_64\ntar-1.23-15.el6_8.x86_64\nvte-devel-0.25.1-9.el6.x86_64\nxen-devel-4.6.3-15.el6.x86_64\nzlib-devel-1.2.3-29.el6.x86_64\n\nEnvironment variables:\nPACKAGES=bison     bzip2-devel     ccache     csnappy-devel     flex     g++     gcc     git     glib2-devel     libepoxy-devel     libfdt-devel     librdmacm-devel     lzo-devel     make     mesa-libEGL-devel     mesa-libgbm-devel     pixman-devel     SDL-devel     spice-glib-devel     spice-server-devel     tar     vte-devel     xen-devel     zlib-devel\nHOSTNAME=fd698ce7a83b\nTERM=xterm\nMAKEFLAGS= -j8\nHISTSIZE=1000\nJ=8\nUSER=root\nLS_COLORS=rs=0:di=01;34:ln=01;36:mh=00:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:mi=01;05;37;41:su=37;41:sg=30;43:ca=30;41:tw=30;42:ow=34;42:st=37;44:ex=01;32:*.tar=01;31:*.tgz=01;31:*.arj=01;31:*.taz=01;31:*.lzh=01;31:*.lzma=01;31:*.tlz=01;31:*.txz=01;31:*.zip=01;31:*.z=01;31:*.Z=01;31:*.dz=01;31:*.gz=01;31:*.lz=01;31:*.xz=01;31:*.bz2=01;31:*.tbz=01;31:*.tbz2=01;31:*.bz=01;31:*.tz=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.rar=01;31:*.ace=01;31:*.zoo=01;31:*.cpio=01;31:*.7z=01;31:*.rz=01;31:*.jpg=01;35:*.jpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.svg=01;35:*.svgz=01;35:*.mng=01;35:*.pcx=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.m2v=01;35:*.mkv=01;35:*.ogm=01;35:*.mp4=01;35:*.m4v=01;35:*.mp4v=01;35:*.vob=01;35:*.qt=01;35:*.nuv=01;35:*.wmv=01;35:*.asf=01;35:*.rm=01;35:*.rmvb=01;35:*.flc=01;35:*.avi=01;35:*.fli=01;35:*.flv=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.yuv=01;35:*.cgm=01;35:*.emf=01;35:*.axv=01;35:*.anx=01;35:*.ogv=01;35:*.ogx=01;35:*.aac=01;36:*.au=01;36:*.flac=01;36:*.mid=01;36:*.midi=01;36:*.mka=01;36:*.mp3=01;36:*.mpc=01;36:*.ogg=01;36:*.ra=01;36:*.wav=01;36:*.axa=01;36:*.oga=01;36:*.spx=01;36:*.xspf=01;36:\nCCACHE_DIR=/var/tmp/ccache\nEXTRA_CONFIGURE_OPTS=\nV=\nSHOW_ENV=1\nMAIL=/var/spool/mail/root\nPATH=/usr/lib/ccache:/usr/lib64/ccache:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin\nPWD=/\nLANG=en_US.UTF-8\nTARGET_LIST=\nHISTCONTROL=ignoredups\nSHLVL=1\nHOME=/root\nTEST_DIR=/tmp/qemu-test\nLOGNAME=root\nLESSOPEN=||/usr/bin/lesspipe.sh %s\nFEATURES= dtc\nDEBUG=\nG_BROKEN_FILENAMES=1\nCCACHE_HASHDIR=\n_=/usr/bin/env\n\nConfigure options:\n--enable-werror --target-list=x86_64-softmmu,aarch64-softmmu --prefix=/var/tmp/qemu-build/install\nNo C++ compiler available; disabling C++ specific optional code\nInstall prefix    /var/tmp/qemu-build/install\nBIOS directory    /var/tmp/qemu-build/install/share/qemu\nbinary directory  /var/tmp/qemu-build/install/bin\nlibrary directory /var/tmp/qemu-build/install/lib\nmodule directory  /var/tmp/qemu-build/install/lib/qemu\nlibexec directory /var/tmp/qemu-build/install/libexec\ninclude directory /var/tmp/qemu-build/install/include\nconfig directory  /var/tmp/qemu-build/install/etc\nlocal state directory   /var/tmp/qemu-build/install/var\nManual directory  /var/tmp/qemu-build/install/share/man\nELF interp prefix /usr/gnemul/qemu-%M\nSource path       /tmp/qemu-test/src\nC compiler        cc\nHost C compiler   cc\nC++ compiler      \nObjective-C compiler cc\nARFLAGS           rv\nCFLAGS            -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -g \nQEMU_CFLAGS       -I/usr/include/pixman-1   -I$(SRC_PATH)/dtc/libfdt -pthread -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include   -DNCURSES_WIDECHAR   -fPIE -DPIE -m64 -mcx16 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv  -Wendif-labels -Wno-missing-include-dirs -Wempty-body -Wnested-externs -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration -Wold-style-definition -Wtype-limits -fstack-protector-all  -I/usr/include/libpng12   -I/usr/include/libdrm     -I/usr/include/spice-server -I/usr/include/cacard -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/nss3 -I/usr/include/nspr4 -I/usr/include/spice-1   -I/usr/include/cacard -I/usr/include/nss3 -I/usr/include/nspr4  \nLDFLAGS           -Wl,--warn-common -Wl,-z,relro -Wl,-z,now -pie -m64 -g \nmake              make\ninstall           install\npython            python -B\nsmbd              /usr/sbin/smbd\nmodule support    no\nhost CPU          x86_64\nhost big endian   no\ntarget list       x86_64-softmmu aarch64-softmmu\ngprof enabled     no\nsparse enabled    no\nstrip binaries    yes\nprofiler          no\nstatic build      no\npixman            system\nSDL support       yes (1.2.14)\nGTK support       yes (2.24.23)\nGTK GL support    no\nVTE support       yes (0.25.1)\nTLS priority      NORMAL\nGNUTLS support    no\nGNUTLS rnd        no\nlibgcrypt         no\nlibgcrypt kdf     no\nnettle            no \nnettle kdf        no\nlibtasn1          no\ncurses support    yes\nvirgl support     no\ncurl support      no\nmingw32 support   no\nAudio drivers     oss\nBlock whitelist (rw) \nBlock whitelist (ro) \nVirtFS support    no\nVNC support       yes\nVNC SASL support  no\nVNC JPEG support  yes\nVNC PNG support   yes\nxen support       yes\nxen ctrl version  40600\npv dom build      no\nbrlapi support    no\nbluez  support    no\nDocumentation     no\nPIE               yes\nvde support       no\nnetmap support    no\nLinux AIO support no\nATTR/XATTR support yes\nInstall blobs     yes\nKVM support       yes\nHAX support       no\nTCG support       yes\nTCG debug enabled no\nTCG interpreter   no\nRDMA support      yes\nfdt support       yes\npreadv support    yes\nfdatasync         yes\nmadvise           yes\nposix_madvise     yes\nlibcap-ng support no\nvhost-net support yes\nvhost-scsi support yes\nvhost-vsock support yes\nvhost-user support yes\nTrace backends    log\ninstrumentation   no\nspice support     yes (0.12.6/0.12.4)\nrbd support       no\nxfsctl support    no\nsmartcard support yes\nlibusb            no\nusb net redir     no\nOpenGL support    yes\nOpenGL dmabufs    no\nlibiscsi support  no\nlibnfs support    no\nbuild guest agent yes\nQGA VSS support   no\nQGA w32 disk info no\nQGA MSI support   no\nseccomp support   no\ncoroutine backend ucontext\ncoroutine pool    yes\ndebug stack usage no\ncrypto afalg      no\nGlusterFS support no\ngcov              gcov\ngcov enabled      no\nTPM support       yes\nlibssh2 support   no\nTPM passthrough   yes\nQOM debugging     yes\nLive block migration yes\nlzo support       yes\nsnappy support    no\nbzip2 support     yes\nNUMA host support no\ntcmalloc support  no\njemalloc support  no\navx2 optimization no\nreplication support yes\nVxHS block device no\nmkdir -p dtc/libfdt\nmkdir -p dtc/tests\n  GEN     aarch64-softmmu/config-devices.mak.tmp\n  GEN     x86_64-softmmu/config-devices.mak.tmp\n  GEN     config-host.h\n  GEN     qemu-options.def\n  GEN     qmp-commands.h\n  GEN     qapi-types.h\n  GEN     qapi-visit.h\n  GEN     qapi-event.h\n  GEN     x86_64-softmmu/config-devices.mak\n  GEN     qmp-marshal.c\n  GEN     aarch64-softmmu/config-devices.mak\n  GEN     qapi-types.c\n  GEN     qapi-visit.c\n  GEN     qapi-event.c\n  GEN     qmp-introspect.h\n  GEN     qmp-introspect.c\n  GEN     trace/generated-tcg-tracers.h\n  GEN     trace/generated-helpers-wrappers.h\n  GEN     trace/generated-helpers.h\n  GEN     trace/generated-helpers.c\n  GEN     module_block.h\n  GEN     tests/test-qapi-types.h\n  GEN     tests/test-qapi-visit.h\n  GEN     tests/test-qmp-commands.h\n  GEN     tests/test-qapi-event.h\n  GEN     tests/test-qmp-introspect.h\n  GEN     trace-root.h\n  GEN     util/trace.h\n  GEN     crypto/trace.h\n  GEN     io/trace.h\n  GEN     migration/trace.h\n  GEN     block/trace.h\n  GEN     chardev/trace.h\n  GEN     hw/block/trace.h\n  GEN     hw/block/dataplane/trace.h\n  GEN     hw/char/trace.h\n  GEN     hw/intc/trace.h\n  GEN     hw/net/trace.h\n  GEN     hw/virtio/trace.h\n  GEN     hw/audio/trace.h\n  GEN     hw/misc/trace.h\n  GEN     hw/usb/trace.h\n  GEN     hw/scsi/trace.h\n  GEN     hw/nvram/trace.h\n  GEN     hw/display/trace.h\n  GEN     hw/input/trace.h\n  GEN     hw/timer/trace.h\n  GEN     hw/dma/trace.h\n  GEN     hw/sparc/trace.h\n  GEN     hw/sd/trace.h\n  GEN     hw/isa/trace.h\n  GEN     hw/mem/trace.h\n  GEN     hw/i386/trace.h\n  GEN     hw/i386/xen/trace.h\n  GEN     hw/9pfs/trace.h\n  GEN     hw/ppc/trace.h\n  GEN     hw/pci/trace.h\n  GEN     hw/s390x/trace.h\n  GEN     hw/vfio/trace.h\n  GEN     hw/acpi/trace.h\n  GEN     hw/arm/trace.h\n  GEN     hw/alpha/trace.h\n  GEN     hw/xen/trace.h\n  GEN     ui/trace.h\n  GEN     audio/trace.h\n  GEN     net/trace.h\n  GEN     target/arm/trace.h\n  GEN     target/i386/trace.h\n  GEN     target/mips/trace.h\n  GEN     target/sparc/trace.h\n  GEN     target/s390x/trace.h\n  GEN     target/ppc/trace.h\n  GEN     qom/trace.h\n  GEN     linux-user/trace.h\n  GEN     qapi/trace.h\n  GEN     accel/tcg/trace.h\n  GEN     accel/kvm/trace.h\n  GEN     nbd/trace.h\n  GEN     trace-root.c\n  GEN     util/trace.c\n  GEN     crypto/trace.c\n  GEN     io/trace.c\n  GEN     migration/trace.c\n  GEN     block/trace.c\n  GEN     chardev/trace.c\n  GEN     hw/block/trace.c\n  GEN     hw/block/dataplane/trace.c\n  GEN     hw/char/trace.c\n  GEN     hw/intc/trace.c\n  GEN     hw/net/trace.c\n  GEN     hw/virtio/trace.c\n  GEN     hw/audio/trace.c\n  GEN     hw/misc/trace.c\n  GEN     hw/usb/trace.c\n  GEN     hw/scsi/trace.c\n  GEN     hw/nvram/trace.c\n  GEN     hw/display/trace.c\n  GEN     hw/input/trace.c\n  GEN     hw/timer/trace.c\n  GEN     hw/dma/trace.c\n  GEN     hw/sparc/trace.c\n  GEN     hw/sd/trace.c\n  GEN     hw/isa/trace.c\n  GEN     hw/mem/trace.c\n  GEN     hw/i386/trace.c\n  GEN     hw/i386/xen/trace.c\n  GEN     hw/9pfs/trace.c\n  GEN     hw/ppc/trace.c\n  GEN     hw/pci/trace.c\n  GEN     hw/s390x/trace.c\n  GEN     hw/vfio/trace.c\n  GEN     hw/acpi/trace.c\n  GEN     hw/arm/trace.c\n  GEN     hw/alpha/trace.c\n  GEN     hw/xen/trace.c\n  GEN     ui/trace.c\n  GEN     audio/trace.c\n  GEN     net/trace.c\n  GEN     target/arm/trace.c\n  GEN     target/i386/trace.c\n  GEN     target/mips/trace.c\n  GEN     target/sparc/trace.c\n  GEN     target/s390x/trace.c\n  GEN     target/ppc/trace.c\n  GEN     qom/trace.c\n  GEN     linux-user/trace.c\n  GEN     qapi/trace.c\n  GEN     accel/tcg/trace.c\n  GEN     accel/kvm/trace.c\n  GEN     nbd/trace.c\n  GEN     config-all-devices.mak\n\t DEP /tmp/qemu-test/src/dtc/tests/dumptrees.c\n\t DEP /tmp/qemu-test/src/dtc/tests/trees.S\n\t DEP /tmp/qemu-test/src/dtc/tests/testutils.c\n\t DEP /tmp/qemu-test/src/dtc/tests/value-labels.c\n\t DEP /tmp/qemu-test/src/dtc/tests/asm_tree_dump.c\n\t DEP /tmp/qemu-test/src/dtc/tests/truncated_property.c\n\t DEP /tmp/qemu-test/src/dtc/tests/check_path.c\n\t DEP /tmp/qemu-test/src/dtc/tests/overlay_bad_fixup.c\n\t DEP /tmp/qemu-test/src/dtc/tests/overlay.c\n\t DEP /tmp/qemu-test/src/dtc/tests/subnode_iterate.c\n\t DEP /tmp/qemu-test/src/dtc/tests/property_iterate.c\n\t DEP /tmp/qemu-test/src/dtc/tests/integer-expressions.c\n\t DEP /tmp/qemu-test/src/dtc/tests/utilfdt_test.c\n\t DEP /tmp/qemu-test/src/dtc/tests/path_offset_aliases.c\n\t DEP /tmp/qemu-test/src/dtc/tests/add_subnode_with_nops.c\n\t DEP /tmp/qemu-test/src/dtc/tests/dtbs_equal_unordered.c\n\t DEP /tmp/qemu-test/src/dtc/tests/dtb_reverse.c\n\t DEP /tmp/qemu-test/src/dtc/tests/dtbs_equal_ordered.c\n\t DEP /tmp/qemu-test/src/dtc/tests/extra-terminating-null.c\n\t DEP /tmp/qemu-test/src/dtc/tests/incbin.c\n\t DEP /tmp/qemu-test/src/dtc/tests/boot-cpuid.c\n\t DEP /tmp/qemu-test/src/dtc/tests/phandle_format.c\n\t DEP /tmp/qemu-test/src/dtc/tests/path-references.c\n\t DEP /tmp/qemu-test/src/dtc/tests/references.c\n\t DEP /tmp/qemu-test/src/dtc/tests/string_escapes.c\n\t DEP /tmp/qemu-test/src/dtc/tests/propname_escapes.c\n\t DEP /tmp/qemu-test/src/dtc/tests/appendprop2.c\n\t DEP /tmp/qemu-test/src/dtc/tests/appendprop1.c\n\t DEP /tmp/qemu-test/src/dtc/tests/del_node.c\n\t DEP /tmp/qemu-test/src/dtc/tests/del_property.c\n\t DEP /tmp/qemu-test/src/dtc/tests/setprop.c\n\t DEP /tmp/qemu-test/src/dtc/tests/set_name.c\n\t DEP /tmp/qemu-test/src/dtc/tests/rw_tree1.c\n\t DEP /tmp/qemu-test/src/dtc/tests/open_pack.c\n\t DEP /tmp/qemu-test/src/dtc/tests/nopulate.c\n\t DEP /tmp/qemu-test/src/dtc/tests/mangle-layout.c\n\t DEP /tmp/qemu-test/src/dtc/tests/move_and_save.c\n\t DEP /tmp/qemu-test/src/dtc/tests/sw_tree1.c\n\t DEP /tmp/qemu-test/src/dtc/tests/nop_node.c\n\t DEP /tmp/qemu-test/src/dtc/tests/nop_property.c\n\t DEP /tmp/qemu-test/src/dtc/tests/setprop_inplace.c\n\t DEP /tmp/qemu-test/src/dtc/tests/stringlist.c\n\t DEP /tmp/qemu-test/src/dtc/tests/addr_size_cells.c\n\t DEP /tmp/qemu-test/src/dtc/tests/notfound.c\n\t DEP /tmp/qemu-test/src/dtc/tests/sized_cells.c\n\t DEP /tmp/qemu-test/src/dtc/tests/get_alias.c\n\t DEP /tmp/qemu-test/src/dtc/tests/char_literal.c\n\t DEP /tmp/qemu-test/src/dtc/tests/node_offset_by_compatible.c\n\t DEP /tmp/qemu-test/src/dtc/tests/node_check_compatible.c\n\t DEP /tmp/qemu-test/src/dtc/tests/node_offset_by_phandle.c\n\t DEP /tmp/qemu-test/src/dtc/tests/node_offset_by_prop_value.c\n\t DEP /tmp/qemu-test/src/dtc/tests/parent_offset.c\n\t DEP /tmp/qemu-test/src/dtc/tests/supernode_atdepth_offset.c\n\t DEP /tmp/qemu-test/src/dtc/tests/get_path.c\n\t DEP /tmp/qemu-test/src/dtc/tests/get_phandle.c\n\t DEP /tmp/qemu-test/src/dtc/tests/getprop.c\n\t DEP /tmp/qemu-test/src/dtc/tests/get_name.c\n\t DEP /tmp/qemu-test/src/dtc/tests/path_offset.c\n\t DEP /tmp/qemu-test/src/dtc/tests/subnode_offset.c\n\t DEP /tmp/qemu-test/src/dtc/tests/find_property.c\n\t DEP /tmp/qemu-test/src/dtc/tests/root_node.c\n\t DEP /tmp/qemu-test/src/dtc/tests/get_mem_rsv.c\n\t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_overlay.c\n\t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_addresses.c\n\t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_empty_tree.c\n\t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_strerror.c\n\t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_rw.c\n\t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_sw.c\n\t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_ro.c\n\t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_wip.c\n\t DEP /tmp/qemu-test/src/dtc/libfdt/fdt.c\n\t DEP /tmp/qemu-test/src/dtc/util.c\n\t DEP /tmp/qemu-test/src/dtc/fdtput.c\n\t DEP /tmp/qemu-test/src/dtc/fdtget.c\n\t DEP /tmp/qemu-test/src/dtc/fdtdump.c\n\t LEX convert-dtsv0-lexer.lex.c\n\t DEP /tmp/qemu-test/src/dtc/srcpos.c\n\t BISON dtc-parser.tab.c\n\t LEX dtc-lexer.lex.c\n\t DEP /tmp/qemu-test/src/dtc/treesource.c\n\t DEP /tmp/qemu-test/src/dtc/livetree.c\n\t DEP /tmp/qemu-test/src/dtc/flattree.c\n\t DEP /tmp/qemu-test/src/dtc/fstree.c\n\t DEP /tmp/qemu-test/src/dtc/dtc.c\n\t DEP /tmp/qemu-test/src/dtc/data.c\n\t DEP /tmp/qemu-test/src/dtc/checks.c\n\t DEP convert-dtsv0-lexer.lex.c\n\t DEP dtc-parser.tab.c\n\t DEP dtc-lexer.lex.c\n\tCHK version_gen.h\n\tUPD version_gen.h\n\t DEP /tmp/qemu-test/src/dtc/util.c\n\t CC libfdt/fdt.o\n\t CC libfdt/fdt_ro.o\n\t CC libfdt/fdt_wip.o\n\t CC libfdt/fdt_sw.o\n\t CC libfdt/fdt_rw.o\n\t CC libfdt/fdt_empty_tree.o\n\t CC libfdt/fdt_strerror.o\n\t CC libfdt/fdt_addresses.o\n\t CC libfdt/fdt_overlay.o\n\t AR libfdt/libfdt.a\nar: creating libfdt/libfdt.a\na - libfdt/fdt.o\na - libfdt/fdt_ro.o\na - libfdt/fdt_wip.o\na - libfdt/fdt_sw.o\na - libfdt/fdt_rw.o\na - libfdt/fdt_strerror.o\na - libfdt/fdt_empty_tree.o\na - libfdt/fdt_addresses.o\na - libfdt/fdt_overlay.o\n  CC      tests/qemu-iotests/socket_scm_helper.o\n  GEN     qga/qapi-generated/qga-qapi-types.h\n  GEN     qga/qapi-generated/qga-qapi-visit.h\n  GEN     qga/qapi-generated/qga-qapi-types.c\n  GEN     qga/qapi-generated/qga-qmp-marshal.c\n  GEN     qga/qapi-generated/qga-qapi-visit.c\n  GEN     qga/qapi-generated/qga-qmp-commands.h\n  CC      qmp-introspect.o\n  CC      qapi-types.o\n  CC      qapi-visit.o\n  CC      qapi-event.o\n  CC      qapi/qapi-visit-core.o\n  CC      qapi/qapi-dealloc-visitor.o\n  CC      qapi/qobject-input-visitor.o\n  CC      qapi/qobject-output-visitor.o\n  CC      qapi/qmp-registry.o\n  CC      qapi/qmp-dispatch.o\n  CC      qapi/string-input-visitor.o\n  CC      qapi/string-output-visitor.o\n  CC      qapi/opts-visitor.o\n  CC      qapi/qapi-clone-visitor.o\n  CC      qapi/qmp-event.o\n  CC      qapi/qapi-util.o\n  CC      qobject/qnull.o\n  CC      qobject/qnum.o\n  CC      qobject/qstring.o\n  CC      qobject/qdict.o\n  CC      qobject/qlist.o\n  CC      qobject/qbool.o\n  CC      qobject/qlit.o\n  CC      qobject/qjson.o\n  CC      qobject/qobject.o\n  CC      qobject/json-lexer.o\n  CC      qobject/json-streamer.o\n  CC      qobject/json-parser.o\n  CC      trace/control.o\n  CC      trace/qmp.o\n  CC      util/osdep.o\n  CC      util/cutils.o\n  CC      util/unicode.o\n  CC      util/qemu-timer-common.o\n  CC      util/bufferiszero.o\n  CC      util/lockcnt.o\n  CC      util/aiocb.o\n  CC      util/async.o\n  CC      util/thread-pool.o\n  CC      util/qemu-timer.o\n  CC      util/main-loop.o\n  CC      util/iohandler.o\n  CC      util/aio-posix.o\n  CC      util/compatfd.o\n  CC      util/event_notifier-posix.o\n  CC      util/mmap-alloc.o\n  CC      util/oslib-posix.o\n  CC      util/qemu-openpty.o\n  CC      util/qemu-thread-posix.o\n  CC      util/memfd.o\n  CC      util/envlist.o\n  CC      util/path.o\n  CC      util/module.o\n  CC      util/host-utils.o\n  CC      util/bitmap.o\n  CC      util/bitops.o\n  CC      util/hbitmap.o\n  CC      util/fifo8.o\n  CC      util/acl.o\n  CC      util/cacheinfo.o\n  CC      util/error.o\n  CC      util/qemu-error.o\n  CC      util/id.o\n  CC      util/iov.o\n  CC      util/qemu-config.o\n  CC      util/qemu-sockets.o\n  CC      util/uri.o\n  CC      util/notify.o\n  CC      util/qemu-option.o\n  CC      util/qemu-progress.o\n  CC      util/keyval.o\n  CC      util/hexdump.o\n  CC      util/crc32c.o\n  CC      util/uuid.o\n  CC      util/throttle.o\n  CC      util/getauxval.o\n  CC      util/readline.o\n  CC      util/rcu.o\n  CC      util/qemu-coroutine.o\n  CC      util/qemu-coroutine-lock.o\n  CC      util/qemu-coroutine-io.o\n  CC      util/qemu-coroutine-sleep.o\n  CC      util/coroutine-ucontext.o\n  CC      util/buffer.o\n  CC      util/timed-average.o\n  CC      util/base64.o\n  CC      util/log.o\n  CC      util/qdist.o\n  CC      util/qht.o\n  CC      util/range.o\n  CC      util/stats64.o\n  CC      util/systemd.o\n  CC      trace-root.o\n  CC      util/trace.o\n  CC      crypto/trace.o\n  CC      io/trace.o\n  CC      migration/trace.o\n  CC      block/trace.o\n  CC      chardev/trace.o\n  CC      hw/block/trace.o\n  CC      hw/block/dataplane/trace.o\n  CC      hw/char/trace.o\n  CC      hw/intc/trace.o\n  CC      hw/net/trace.o\n  CC      hw/virtio/trace.o\n  CC      hw/audio/trace.o\n  CC      hw/misc/trace.o\n  CC      hw/usb/trace.o\n  CC      hw/scsi/trace.o\n  CC      hw/nvram/trace.o\n  CC      hw/display/trace.o\n  CC      hw/input/trace.o\n  CC      hw/timer/trace.o\n  CC      hw/dma/trace.o\n  CC      hw/sparc/trace.o\n  CC      hw/sd/trace.o\n  CC      hw/isa/trace.o\n  CC      hw/mem/trace.o\n  CC      hw/i386/trace.o\n  CC      hw/i386/xen/trace.o\n  CC      hw/9pfs/trace.o\n  CC      hw/ppc/trace.o\n  CC      hw/pci/trace.o\n  CC      hw/s390x/trace.o\n  CC      hw/vfio/trace.o\n  CC      hw/acpi/trace.o\n  CC      hw/arm/trace.o\n  CC      hw/alpha/trace.o\n  CC      hw/xen/trace.o\n  CC      ui/trace.o\n  CC      audio/trace.o\n  CC      net/trace.o\n  CC      target/arm/trace.o\n  CC      target/i386/trace.o\n  CC      target/mips/trace.o\n  CC      target/sparc/trace.o\n  CC      target/s390x/trace.o\n  CC      target/ppc/trace.o\n  CC      qom/trace.o\n  CC      linux-user/trace.o\n  CC      qapi/trace.o\n  CC      accel/tcg/trace.o\n  CC      accel/kvm/trace.o\n  CC      nbd/trace.o\n  CC      crypto/pbkdf-stub.o\n  CC      stubs/arch-query-cpu-def.o\n  CC      stubs/arch-query-cpu-model-expansion.o\n  CC      stubs/arch-query-cpu-model-comparison.o\n  CC      stubs/arch-query-cpu-model-baseline.o\n  CC      stubs/bdrv-next-monitor-owned.o\n  CC      stubs/blk-commit-all.o\n  CC      stubs/blockdev-close-all-bdrv-states.o\n  CC      stubs/clock-warp.o\n  CC      stubs/cpu-get-clock.o\n  CC      stubs/cpu-get-icount.o\n  CC      stubs/dump.o\n  CC      stubs/error-printf.o\n  CC      stubs/fdset.o\n  CC      stubs/gdbstub.o\n  CC      stubs/get-vm-name.o\n  CC      stubs/instrument.o\n  CC      stubs/iothread.o\n  CC      stubs/iothread-lock.o\n  CC      stubs/is-daemonized.o\n  CC      stubs/machine-init-done.o\n  CC      stubs/migr-blocker.o\n  CC      stubs/change-state-handler.o\n  CC      stubs/monitor.o\n  CC      stubs/notify-event.o\n  CC      stubs/qtest.o\n  CC      stubs/replay.o\n/tmp/qemu-test/src/stubs/instrument.c:22: error: redefinition of typedef ‘strList’\n./qapi-types.h:131: note: previous declaration of ‘strList’ was here\n  CC      stubs/runstate-check.o\nmake: *** [stubs/instrument.o] Error 1\nmake: *** Waiting for unfinished jobs....\nTraceback (most recent call last):\n  File \"./tests/docker/docker.py\", line 384, in <module>\n    sys.exit(main())\n  File \"./tests/docker/docker.py\", line 381, in main\n    return args.cmdobj.run(args, argv)\n  File \"./tests/docker/docker.py\", line 239, in run\n    return Docker().run(argv, args.keep, quiet=args.quiet)\n  File \"./tests/docker/docker.py\", line 207, in run\n    quiet=quiet)\n  File \"./tests/docker/docker.py\", line 125, in _do_check\n    return subprocess.check_call(self._command + cmd, **kwargs)\n  File \"/usr/lib64/python2.7/subprocess.py\", line 186, in check_call\n    raise CalledProcessError(retcode, cmd)\nsubprocess.CalledProcessError: Command '['docker', 'run', '--label', 'com.qemu.instance.uuid=9670c49e980a11e7950952540069c830', '-u', '0', '-t', '--rm', '--net=none', '-e', 'TARGET_LIST=', '-e', 'EXTRA_CONFIGURE_OPTS=', '-e', 'V=', '-e', 'J=8', '-e', 'DEBUG=', '-e', 'SHOW_ENV=1', '-e', 'CCACHE_DIR=/var/tmp/ccache', '-v', '/var/tmp/patchew-tester-tmp-lf5qg9x6/src/docker-src.2017-09-12-18.34.46.13887:/var/tmp/qemu:z,ro', '-v', '/root/.cache/qemu-docker-ccache:/var/tmp/ccache:z', 'qemu:centos6', '/var/tmp/qemu/run', 'test-quick']' returned non-zero exit status 2\nmake[1]: *** [tests/docker/Makefile.include:139: docker-run] Error 1\nmake[1]: Leaving directory '/var/tmp/patchew-tester-tmp-lf5qg9x6/src'\nmake: *** [tests/docker/Makefile.include:168: docker-run-test-quick@centos6] Error 2\n\nreal\t1m25.085s\nuser\t0m5.300s\nsys\t0m1.695s\n=== OUTPUT END ===\n\nTest command exited with code: 2\n\n\n---\nEmail generated automatically by Patchew [http://patchew.org/].\nPlease send your feedback to patchew-devel@freelists.org","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xsKRV0KwCz9sNw\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 13 Sep 2017 08:37:09 +1000 (AEST)","from localhost ([::1]:39055 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1drtno-00005A-2q\n\tfor incoming@patchwork.ozlabs.org; Tue, 12 Sep 2017 18:37:08 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:59155)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <no-reply@patchew.org>) id 1drtn8-0008TA-Oi\n\tfor qemu-devel@nongnu.org; Tue, 12 Sep 2017 18:36:31 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <no-reply@patchew.org>) id 1drtn5-0004CT-3w\n\tfor qemu-devel@nongnu.org; Tue, 12 Sep 2017 18:36:26 -0400","from sender-of-o52.zoho.com ([135.84.80.217]:21399)\n\tby eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from <no-reply@patchew.org>)\n\tid 1drtn4-0004BX-I5\n\tfor qemu-devel@nongnu.org; Tue, 12 Sep 2017 18:36:22 -0400","from [172.17.0.2] (23.253.156.214 [23.253.156.214]) by\n\tmx.zohomail.com with SMTPS id 1505255770859560.162572112828;\n\tTue, 12 Sep 2017 15:36:10 -0700 (PDT)"],"Resent-Date":"Tue, 12 Sep 2017 18:36:26 -0400","Resent-Message-Id":"<E1drtn5-0004CT-3w@eggs.gnu.org>","In-Reply-To":"<150525010239.15988.8172586618197849619.stgit@frigg.lan>","Message-ID":"<150525577025.321.3001704927093785609@6d89bd104fc3>","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Resent-From":"","From":"no-reply@patchew.org","To":"vilanova@ac.upc.edu","Date":"Tue, 12 Sep 2017 15:36:10 -0700 (PDT)","X-ZohoMailClient":"External","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 3.x [fuzzy]","X-Received-From":"135.84.80.217","Subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Reply-To":"qemu-devel@nongnu.org","Cc":"cota@braap.org, famz@redhat.com, qemu-devel@nongnu.org,\n\tstefanha@redhat.com, armbru@redhat.com","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1767722,"web_url":"http://patchwork.ozlabs.org/comment/1767722/","msgid":"<874ls7ue31.fsf@frigg.lan>","list_archive_url":null,"date":"2017-09-13T09:45:22","subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","submitter":{"id":9099,"url":"http://patchwork.ozlabs.org/api/people/9099/","name":"Lluís Vilanova","email":"vilanova@ac.upc.edu"},"content":"no-reply  writes:\n\n> Hi,\n> This series seems to have some coding style problems. See output below for\n> more information:\n\n> Subject: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event instrumentation\n> Message-id: 150525010239.15988.8172586618197849619.stgit@frigg.lan\n> Type: series\n\n> === TEST SCRIPT BEGIN ===\n> #!/bin/bash\n\n> BASE=base\n> n=1\n> total=$(git log --oneline $BASE.. | wc -l)\n> failed=0\n\n> git config --local diff.renamelimit 0\n> git config --local diff.renames True\n\n> commits=\"$(git log --format=%H --reverse $BASE..)\"\n> for c in $commits; do\n>     echo \"Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)...\"\n>     if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then\n>         failed=1\n>         echo\n>     fi\n>     n=$((n+1))\n> done\n\n> exit $failed\n> === TEST SCRIPT END ===\n\n> Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384\n> From https://github.com/patchew-project/qemu\n>  * [new tag]               patchew/150525010239.15988.8172586618197849619.stgit@frigg.lan -> patchew/150525010239.15988.8172586618197849619.stgit@frigg.lan\n>  t [tag update]            patchew/20170912144459.11359-1-pbonzini@redhat.com -> patchew/20170912144459.11359-1-pbonzini@redhat.com\n> Switched to a new branch 'test'\n> 1ab48ae9b7 instrument: Add API to manipulate guest memory\n> 7e0bd2cad7 instrument: Add event 'guest_user_syscall_ret'\n> 334caef899 instrument: Add event 'guest_user_syscall'\n> 09a1773791 instrument: Add event 'guest_mem_before_exec'\n> 2bd64563d3 instrument: Add event 'guest_mem_before_trans'\n> 5b344ec1c3 trace: Introduce a proper structure to describe memory accesses\n> 04e5b883b1 instrument: Add event 'guest_cpu_reset'\n> 7971d0f2a4 instrument: Add event 'guest_cpu_exit'\n> 53dbc9ad88 exec: Add function to synchronously flush TB on a stopped vCPU\n> d8b51515d2 instrument: Support synchronous modification of vCPU state\n> 08d492e35f instrument: Add event 'guest_cpu_enter'\n> 0be52b1bbd instrument: Track vCPUs\n> 7ab01f20f5 instrument: Add support for tracing events\n> 78676cff2d instrument: Add basic control interface\n> 00172972ae instrument: [hmp] Add library loader\n> 34ccf831e6 instrument: [qapi] Add library loader\n> d1ab648b00 instrument: [softmmu] Add command line library loader\n> 150ad4a651 instrument: [bsd-user] Add command line library loader\n> a064b1621a instrument: [linux-user] Add command line library loader\n> aa78ee9f5a instrument: Add generic library loader\n> f10357e313 instrument: Add configure-time flag\n> 4d324ad619 instrument: Add documentation\n\n> === OUTPUT BEGIN ===\n> Checking PATCH 1/22: instrument: Add documentation...\n> Checking PATCH 2/22: instrument: Add configure-time flag...\n> Checking PATCH 3/22: instrument: Add generic library loader...\n> Checking PATCH 4/22: instrument: [linux-user] Add command line library loader...\n> Checking PATCH 5/22: instrument: [bsd-user] Add command line library loader...\n> Checking PATCH 6/22: instrument: [softmmu] Add command line library loader...\n> Checking PATCH 7/22: instrument: [qapi] Add library loader...\n> ERROR: externs should be avoided in .c files\n> #254: FILE: stubs/instrument.c:40:\n> +void qmp_instr_unload(const char *id, Error **errp);\n\n> total: 1 errors, 0 warnings, 204 lines checked\n\n> Your patch has style problems, please review.  If any of these errors\n> are false positives report them to the maintainer, see\n> CHECKPATCH in MAINTAINERS.\n\n> Checking PATCH 8/22: instrument: [hmp] Add library loader...\n> Checking PATCH 9/22: instrument: Add basic control interface...\n> WARNING: architecture specific defines should be avoided\n> #52: FILE: include/qemu/compiler.h:119:\n> +#if defined _WIN32 || defined __CYGWIN__\n\n> WARNING: architecture specific defines should be avoided\n> #53: FILE: include/qemu/compiler.h:120:\n> +  #ifdef __GNUC__\n\n> WARNING: architecture specific defines should be avoided\n> #59: FILE: include/qemu/compiler.h:126:\n> +  #if __GNUC__ >= 4\n\n> WARNING: architecture specific defines should be avoided\n> #343: FILE: instrument/qemu-instr/control.h:13:\n> +#ifdef __cplusplus\n\n> WARNING: architecture specific defines should be avoided\n> #372: FILE: instrument/qemu-instr/control.h:42:\n> +#ifdef __cplusplus\n\n> total: 0 errors, 5 warnings, 309 lines checked\n\n> Your patch has style problems, please review.  If any of these errors\n> are false positives report them to the maintainer, see\n> CHECKPATCH in MAINTAINERS.\n> Checking PATCH 10/22: instrument: Add support for tracing events...\n> WARNING: architecture specific defines should be avoided\n> #77: FILE: instrument/qemu-instr/types.h:13:\n> +#ifdef __cplusplus\n\n> WARNING: architecture specific defines should be avoided\n> #111: FILE: instrument/qemu-instr/types.h:47:\n> +#ifdef __cplusplus\n\n> total: 0 errors, 2 warnings, 225 lines checked\n\n> Your patch has style problems, please review.  If any of these errors\n> are false positives report them to the maintainer, see\n> CHECKPATCH in MAINTAINERS.\n> Checking PATCH 11/22: instrument: Track vCPUs...\n> Checking PATCH 12/22: instrument: Add event 'guest_cpu_enter'...\n> Checking PATCH 13/22: instrument: Support synchronous modification of vCPU state...\n> WARNING: line over 80 characters\n> #73: FILE: instrument/control.c:85:\n> +        async_run_on_cpu(cpu, instr_cpu_stop_all__cb, RUN_ON_CPU_HOST_PTR(info));\n\nFixed in next series. All above are false positives.\n\n\n> total: 0 errors, 1 warnings, 127 lines checked\n\n> Your patch has style problems, please review.  If any of these errors\n> are false positives report them to the maintainer, see\n> CHECKPATCH in MAINTAINERS.\n> Checking PATCH 14/22: exec: Add function to synchronously flush TB on a stopped vCPU...\n> Checking PATCH 15/22: instrument: Add event 'guest_cpu_exit'...\n> Checking PATCH 16/22: instrument: Add event 'guest_cpu_reset'...\n> Checking PATCH 17/22: trace: Introduce a proper structure to describe memory accesses...\n> ERROR: spaces prohibited around that ':' (ctx:WxW)\n> #244: FILE: trace/mem.h:29:\n> +            uint8_t size_shift : 2;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:VxW)\n> #245: FILE: trace/mem.h:30:\n> +            bool    sign_extend: 1;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:WxW)\n> #246: FILE: trace/mem.h:31:\n> +            uint8_t endianness : 1;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:WxW)\n> #247: FILE: trace/mem.h:32:\n> +            bool    store      : 1;\n>                                 ^\n\n> total: 4 errors, 0 warnings, 227 lines checked\n\n> Your patch has style problems, please review.  If any of these errors\n> are false positives report them to the maintainer, see\n> CHECKPATCH in MAINTAINERS.\n\n> Checking PATCH 18/22: instrument: Add event 'guest_mem_before_trans'...\n> ERROR: spaces prohibited around that ':' (ctx:WxW)\n> #302: FILE: instrument/qemu-instr/types.h:64:\n> +            uint8_t size_shift : 2;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:VxW)\n> #303: FILE: instrument/qemu-instr/types.h:65:\n> +            bool    sign_extend: 1;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:WxW)\n> #304: FILE: instrument/qemu-instr/types.h:66:\n> +            uint8_t endianness : 1;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:WxW)\n> #305: FILE: instrument/qemu-instr/types.h:67:\n> +            bool    store      : 1;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:WxW)\n> #430: FILE: trace/control.h:37:\n> +            uint8_t size_shift : 2;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:VxW)\n> #431: FILE: trace/control.h:38:\n> +            bool    sign_extend: 1;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:WxW)\n> #432: FILE: trace/control.h:39:\n> +            uint8_t endianness : 1;\n>                                 ^\n\n> ERROR: spaces prohibited around that ':' (ctx:WxW)\n> #433: FILE: trace/control.h:40:\n> +            bool    store      : 1;\n>                                 ^\n\nIgnoring; these keep field lengths nicely aligned.\n\n\n> total: 8 errors, 0 warnings, 389 lines checked\n\n> Your patch has style problems, please review.  If any of these errors\n> are false positives report them to the maintainer, see\n> CHECKPATCH in MAINTAINERS.\n\n> Checking PATCH 19/22: instrument: Add event 'guest_mem_before_exec'...\n> ERROR: externs should be avoided in .c files\n> #337: FILE: stubs/instrument.c:61:\n> +void helper_instr_guest_mem_before_exec(\n\nFalse positive. Cannot include the proper helper headers on the stub file.\n\n\n> total: 1 errors, 0 warnings, 258 lines checked\n\n> Your patch has style problems, please review.  If any of these errors\n> are false positives report them to the maintainer, see\n> CHECKPATCH in MAINTAINERS.\n\n> Checking PATCH 20/22: instrument: Add event 'guest_user_syscall'...\n> Checking PATCH 21/22: instrument: Add event 'guest_user_syscall_ret'...\n> Checking PATCH 22/22: instrument: Add API to manipulate guest memory...\n> WARNING: architecture specific defines should be avoided\n> #41: FILE: instrument/qemu-instr/state.h:13:\n> +#ifdef __cplusplus\n\n> WARNING: architecture specific defines should be avoided\n> #128: FILE: instrument/qemu-instr/state.h:100:\n> +#ifdef __cplusplus\n\n> total: 0 errors, 2 warnings, 181 lines checked\n\n> Your patch has style problems, please review.  If any of these errors\n> are false positives report them to the maintainer, see\n> CHECKPATCH in MAINTAINERS.\n> === OUTPUT END ===\n\n> Test command exited with code: 1\n\nMore false positives.\n\n\nThanks,\n  Lluis","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xscHW58Jfz9s7g\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 13 Sep 2017 19:46:12 +1000 (AEST)","from localhost ([::1]:41237 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1ds4FE-0007Vt-9y\n\tfor incoming@patchwork.ozlabs.org; Wed, 13 Sep 2017 05:46:08 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:55619)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <vilanova@ac.upc.edu>) id 1ds4Er-0007VX-Px\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 05:45:47 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <vilanova@ac.upc.edu>) id 1ds4El-0005Fn-3Y\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 05:45:45 -0400","from roura.ac.upc.es ([147.83.33.10]:54357)\n\tby eggs.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <vilanova@ac.upc.edu>) id 1ds4Ek-0005FI-BK\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 05:45:38 -0400","from correu-1.ac.upc.es (correu-1.ac.upc.es [147.83.30.91])\n\tby roura.ac.upc.es (8.13.8/8.13.8) with ESMTP id v8D9jUN1007255;\n\tWed, 13 Sep 2017 11:45:30 +0200","from localhost (unknown [132.68.137.204])\n\tby correu-1.ac.upc.es (Postfix) with ESMTPSA id AFB5596A;\n\tWed, 13 Sep 2017 11:45:24 +0200 (CEST)"],"From":"=?utf-8?q?Llu=C3=ADs_Vilanova?= <vilanova@ac.upc.edu>","To":"no-reply@patchew.org","References":"<150525566622.321.14749941518071789021@6d89bd104fc3>","Mail-Followup-To":"no-reply@patchew.org, qemu-devel@nongnu.org, cota@braap.org,\n\tfamz@redhat.com, stefanha@redhat.com, armbru@redhat.com","Date":"Wed, 13 Sep 2017 12:45:22 +0300","In-Reply-To":"<150525566622.321.14749941518071789021@6d89bd104fc3>\n\t(no-reply@patchew.org's message of \"Tue, 12 Sep 2017 15:34:26 -0700\n\t(PDT)\")","Message-ID":"<874ls7ue31.fsf@frigg.lan>","User-Agent":"Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)","MIME-Version":"1.0","Content-Type":"text/plain","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 2.6.x [fuzzy]","X-Received-From":"147.83.33.10","Subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Cc":"cota@braap.org, famz@redhat.com, qemu-devel@nongnu.org,\n\tstefanha@redhat.com, armbru@redhat.com","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1767732,"web_url":"http://patchwork.ozlabs.org/comment/1767732/","msgid":"<87shfrsz9o.fsf@frigg.lan>","list_archive_url":null,"date":"2017-09-13T09:50:43","subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","submitter":{"id":9099,"url":"http://patchwork.ozlabs.org/api/people/9099/","name":"Lluís Vilanova","email":"vilanova@ac.upc.edu"},"content":"no-reply  writes:\n\n> Hi,\n> This series failed automatic build test. Please find the testing commands and\n> their output below. If you have docker installed, you can probably reproduce it\n> locally.\n\n> Subject: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event instrumentation\n> Message-id: 150525010239.15988.8172586618197849619.stgit@frigg.lan\n> Type: series\n\n> === TEST SCRIPT BEGIN ===\n> #!/bin/bash\n> set -e\n> git submodule update --init dtc\n> # Let docker tests dump environment info\n> export SHOW_ENV=1\n> export J=8\n> time make docker-test-quick@centos6\n> time make docker-test-build@min-glib\n> time make docker-test-mingw@fedora\n> === TEST SCRIPT END ===\n\n> Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384\n> Switched to a new branch 'test'\n> 1ab48ae9b7 instrument: Add API to manipulate guest memory\n> 7e0bd2cad7 instrument: Add event 'guest_user_syscall_ret'\n> 334caef899 instrument: Add event 'guest_user_syscall'\n> 09a1773791 instrument: Add event 'guest_mem_before_exec'\n> 2bd64563d3 instrument: Add event 'guest_mem_before_trans'\n> 5b344ec1c3 trace: Introduce a proper structure to describe memory accesses\n> 04e5b883b1 instrument: Add event 'guest_cpu_reset'\n> 7971d0f2a4 instrument: Add event 'guest_cpu_exit'\n> 53dbc9ad88 exec: Add function to synchronously flush TB on a stopped vCPU\n> d8b51515d2 instrument: Support synchronous modification of vCPU state\n> 08d492e35f instrument: Add event 'guest_cpu_enter'\n> 0be52b1bbd instrument: Track vCPUs\n> 7ab01f20f5 instrument: Add support for tracing events\n> 78676cff2d instrument: Add basic control interface\n> 00172972ae instrument: [hmp] Add library loader\n> 34ccf831e6 instrument: [qapi] Add library loader\n> d1ab648b00 instrument: [softmmu] Add command line library loader\n> 150ad4a651 instrument: [bsd-user] Add command line library loader\n> a064b1621a instrument: [linux-user] Add command line library loader\n> aa78ee9f5a instrument: Add generic library loader\n> f10357e313 instrument: Add configure-time flag\n> 4d324ad619 instrument: Add documentation\n\n> === OUTPUT BEGIN ===\n> Submodule 'dtc' (git://git.qemu-project.org/dtc.git) registered for path 'dtc'\n> Cloning into '/var/tmp/patchew-tester-tmp-lf5qg9x6/src/dtc'...\n> Submodule path 'dtc': checked out '558cd81bdd432769b59bff01240c44f82cfb1a9d'\n>   BUILD   centos6\n> make[1]: Entering directory '/var/tmp/patchew-tester-tmp-lf5qg9x6/src'\n>   ARCHIVE qemu.tgz\n>   ARCHIVE dtc.tgz\n>   COPY    RUNNER\n>     RUN test-quick in qemu:centos6 \n> Packages installed:\n> SDL-devel-1.2.14-7.el6_7.1.x86_64\n> bison-2.4.1-5.el6.x86_64\n> bzip2-devel-1.0.5-7.el6_0.x86_64\n> ccache-3.1.6-2.el6.x86_64\n> csnappy-devel-0-6.20150729gitd7bc683.el6.x86_64\n> flex-2.5.35-9.el6.x86_64\n> gcc-4.4.7-18.el6.x86_64\n> git-1.7.1-8.el6.x86_64\n> glib2-devel-2.28.8-9.el6.x86_64\n> libepoxy-devel-1.2-3.el6.x86_64\n> libfdt-devel-1.4.0-1.el6.x86_64\n> librdmacm-devel-1.0.21-0.el6.x86_64\n> lzo-devel-2.03-3.1.el6_5.1.x86_64\n> make-3.81-23.el6.x86_64\n> mesa-libEGL-devel-11.0.7-4.el6.x86_64\n> mesa-libgbm-devel-11.0.7-4.el6.x86_64\n> package g++ is not installed\n> pixman-devel-0.32.8-1.el6.x86_64\n> spice-glib-devel-0.26-8.el6.x86_64\n> spice-server-devel-0.12.4-16.el6.x86_64\n> tar-1.23-15.el6_8.x86_64\n> vte-devel-0.25.1-9.el6.x86_64\n> xen-devel-4.6.3-15.el6.x86_64\n> zlib-devel-1.2.3-29.el6.x86_64\n\n> Environment variables:\n> PACKAGES=bison     bzip2-devel     ccache     csnappy-devel     flex     g++     gcc     git     glib2-devel     libepoxy-devel     libfdt-devel     librdmacm-devel     lzo-devel     make     mesa-libEGL-devel     mesa-libgbm-devel     pixman-devel     SDL-devel     spice-glib-devel     spice-server-devel     tar     vte-devel     xen-devel     zlib-devel\n> HOSTNAME=fd698ce7a83b\n> TERM=xterm\n> MAKEFLAGS= -j8\n> HISTSIZE=1000\n> J=8\n> USER=root\n> LS_COLORS=rs=0:di=01;34:ln=01;36:mh=00:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:mi=01;05;37;41:su=37;41:sg=30;43:ca=30;41:tw=30;42:ow=34;42:st=37;44:ex=01;32:*.tar=01;31:*.tgz=01;31:*.arj=01;31:*.taz=01;31:*.lzh=01;31:*.lzma=01;31:*.tlz=01;31:*.txz=01;31:*.zip=01;31:*.z=01;31:*.Z=01;31:*.dz=01;31:*.gz=01;31:*.lz=01;31:*.xz=01;31:*.bz2=01;31:*.tbz=01;31:*.tbz2=01;31:*.bz=01;31:*.tz=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.rar=01;31:*.ace=01;31:*.zoo=01;31:*.cpio=01;31:*.7z=01;31:*.rz=01;31:*.jpg=01;35:*.jpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.svg=01;35:*.svgz=01;35:*.mng=01;35:*.pcx=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.m2v=01;35:*.mkv=01;35:*.ogm=01;35:*.mp4=01;35:*.m4v=01;35:*.mp4v=01;35:*.vob=01;35:*.qt=01;35:*.nuv=01;35:*.wmv=01;35:*.asf=01;35:*.rm=01;35:*.rmvb=01;35:*.flc=01;35:*.avi=01;35:*.fli=01;35:*.flv=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.yuv=01;35:*.cgm=01;35:*.emf=01;35:*.axv=01;35:*.anx=01;35:*.ogv=01;35:*.ogx=01;35:*.aac=01;36:*.au=01;36:*.flac=01;36:*.mid=01;36:*.midi=01;36:*.mka=01;36:*.mp3=01;36:*.mpc=01;36:*.ogg=01;36:*.ra=01;36:*.wav=01;36:*.axa=01;36:*.oga=01;36:*.spx=01;36:*.xspf=01;36:\n> CCACHE_DIR=/var/tmp/ccache\n> EXTRA_CONFIGURE_OPTS=\n> V=\n> SHOW_ENV=1\n> MAIL=/var/spool/mail/root\n> PATH=/usr/lib/ccache:/usr/lib64/ccache:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin\n> PWD=/\n> LANG=en_US.UTF-8\n> TARGET_LIST=\n> HISTCONTROL=ignoredups\n> SHLVL=1\n> HOME=/root\n> TEST_DIR=/tmp/qemu-test\n> LOGNAME=root\n> LESSOPEN=||/usr/bin/lesspipe.sh %s\n> FEATURES= dtc\n> DEBUG=\n> G_BROKEN_FILENAMES=1\n> CCACHE_HASHDIR=\n> _=/usr/bin/env\n\n> Configure options:\n> --enable-werror --target-list=x86_64-softmmu,aarch64-softmmu --prefix=/var/tmp/qemu-build/install\n> No C++ compiler available; disabling C++ specific optional code\n> Install prefix    /var/tmp/qemu-build/install\n> BIOS directory    /var/tmp/qemu-build/install/share/qemu\n> binary directory  /var/tmp/qemu-build/install/bin\n> library directory /var/tmp/qemu-build/install/lib\n> module directory  /var/tmp/qemu-build/install/lib/qemu\n> libexec directory /var/tmp/qemu-build/install/libexec\n> include directory /var/tmp/qemu-build/install/include\n> config directory  /var/tmp/qemu-build/install/etc\n> local state directory   /var/tmp/qemu-build/install/var\n> Manual directory  /var/tmp/qemu-build/install/share/man\n> ELF interp prefix /usr/gnemul/qemu-%M\n> Source path       /tmp/qemu-test/src\n> C compiler        cc\n> Host C compiler   cc\n> C++ compiler      \n> Objective-C compiler cc\n> ARFLAGS           rv\n> CFLAGS            -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -g \n> QEMU_CFLAGS       -I/usr/include/pixman-1   -I$(SRC_PATH)/dtc/libfdt -pthread -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include   -DNCURSES_WIDECHAR   -fPIE -DPIE -m64 -mcx16 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv  -Wendif-labels -Wno-missing-include-dirs -Wempty-body -Wnested-externs -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration -Wold-style-definition -Wtype-limits -fstack-protector-all  -I/usr/include/libpng12   -I/usr/include/libdrm     -I/usr/include/spice-server -I/usr/include/cacard -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/nss3 -I/usr/include/nspr4 -I/usr/include/spice-1   -I/usr/include/cacard -I/usr/include/nss3 -I/usr/include/nspr4  \n> LDFLAGS           -Wl,--warn-common -Wl,-z,relro -Wl,-z,now -pie -m64 -g \n> make              make\n> install           install\n> python            python -B\n> smbd              /usr/sbin/smbd\n> module support    no\n> host CPU          x86_64\n> host big endian   no\n> target list       x86_64-softmmu aarch64-softmmu\n> gprof enabled     no\n> sparse enabled    no\n> strip binaries    yes\n> profiler          no\n> static build      no\n> pixman            system\n> SDL support       yes (1.2.14)\n> GTK support       yes (2.24.23)\n> GTK GL support    no\n> VTE support       yes (0.25.1)\n> TLS priority      NORMAL\n> GNUTLS support    no\n> GNUTLS rnd        no\n> libgcrypt         no\n> libgcrypt kdf     no\n> nettle            no \n> nettle kdf        no\n> libtasn1          no\n> curses support    yes\n> virgl support     no\n> curl support      no\n> mingw32 support   no\n> Audio drivers     oss\n> Block whitelist (rw) \n> Block whitelist (ro) \n> VirtFS support    no\n> VNC support       yes\n> VNC SASL support  no\n> VNC JPEG support  yes\n> VNC PNG support   yes\n> xen support       yes\n> xen ctrl version  40600\n> pv dom build      no\n> brlapi support    no\n> bluez  support    no\n> Documentation     no\n> PIE               yes\n> vde support       no\n> netmap support    no\n> Linux AIO support no\n> ATTR/XATTR support yes\n> Install blobs     yes\n> KVM support       yes\n> HAX support       no\n> TCG support       yes\n> TCG debug enabled no\n> TCG interpreter   no\n> RDMA support      yes\n> fdt support       yes\n> preadv support    yes\n> fdatasync         yes\n> madvise           yes\n> posix_madvise     yes\n> libcap-ng support no\n> vhost-net support yes\n> vhost-scsi support yes\n> vhost-vsock support yes\n> vhost-user support yes\n> Trace backends    log\n> instrumentation   no\n> spice support     yes (0.12.6/0.12.4)\n> rbd support       no\n> xfsctl support    no\n> smartcard support yes\n> libusb            no\n> usb net redir     no\n> OpenGL support    yes\n> OpenGL dmabufs    no\n> libiscsi support  no\n> libnfs support    no\n> build guest agent yes\n> QGA VSS support   no\n> QGA w32 disk info no\n> QGA MSI support   no\n> seccomp support   no\n> coroutine backend ucontext\n> coroutine pool    yes\n> debug stack usage no\n> crypto afalg      no\n> GlusterFS support no\n> gcov              gcov\n> gcov enabled      no\n> TPM support       yes\n> libssh2 support   no\n> TPM passthrough   yes\n> QOM debugging     yes\n> Live block migration yes\n> lzo support       yes\n> snappy support    no\n> bzip2 support     yes\n> NUMA host support no\n> tcmalloc support  no\n> jemalloc support  no\n> avx2 optimization no\n> replication support yes\n> VxHS block device no\n> mkdir -p dtc/libfdt\n> mkdir -p dtc/tests\n>   GEN     aarch64-softmmu/config-devices.mak.tmp\n>   GEN     x86_64-softmmu/config-devices.mak.tmp\n>   GEN     config-host.h\n>   GEN     qemu-options.def\n>   GEN     qmp-commands.h\n>   GEN     qapi-types.h\n>   GEN     qapi-visit.h\n>   GEN     qapi-event.h\n>   GEN     x86_64-softmmu/config-devices.mak\n>   GEN     qmp-marshal.c\n>   GEN     aarch64-softmmu/config-devices.mak\n>   GEN     qapi-types.c\n>   GEN     qapi-visit.c\n>   GEN     qapi-event.c\n>   GEN     qmp-introspect.h\n>   GEN     qmp-introspect.c\n>   GEN     trace/generated-tcg-tracers.h\n>   GEN     trace/generated-helpers-wrappers.h\n>   GEN     trace/generated-helpers.h\n>   GEN     trace/generated-helpers.c\n>   GEN     module_block.h\n>   GEN     tests/test-qapi-types.h\n>   GEN     tests/test-qapi-visit.h\n>   GEN     tests/test-qmp-commands.h\n>   GEN     tests/test-qapi-event.h\n>   GEN     tests/test-qmp-introspect.h\n>   GEN     trace-root.h\n>   GEN     util/trace.h\n>   GEN     crypto/trace.h\n>   GEN     io/trace.h\n>   GEN     migration/trace.h\n>   GEN     block/trace.h\n>   GEN     chardev/trace.h\n>   GEN     hw/block/trace.h\n>   GEN     hw/block/dataplane/trace.h\n>   GEN     hw/char/trace.h\n>   GEN     hw/intc/trace.h\n>   GEN     hw/net/trace.h\n>   GEN     hw/virtio/trace.h\n>   GEN     hw/audio/trace.h\n>   GEN     hw/misc/trace.h\n>   GEN     hw/usb/trace.h\n>   GEN     hw/scsi/trace.h\n>   GEN     hw/nvram/trace.h\n>   GEN     hw/display/trace.h\n>   GEN     hw/input/trace.h\n>   GEN     hw/timer/trace.h\n>   GEN     hw/dma/trace.h\n>   GEN     hw/sparc/trace.h\n>   GEN     hw/sd/trace.h\n>   GEN     hw/isa/trace.h\n>   GEN     hw/mem/trace.h\n>   GEN     hw/i386/trace.h\n>   GEN     hw/i386/xen/trace.h\n>   GEN     hw/9pfs/trace.h\n>   GEN     hw/ppc/trace.h\n>   GEN     hw/pci/trace.h\n>   GEN     hw/s390x/trace.h\n>   GEN     hw/vfio/trace.h\n>   GEN     hw/acpi/trace.h\n>   GEN     hw/arm/trace.h\n>   GEN     hw/alpha/trace.h\n>   GEN     hw/xen/trace.h\n>   GEN     ui/trace.h\n>   GEN     audio/trace.h\n>   GEN     net/trace.h\n>   GEN     target/arm/trace.h\n>   GEN     target/i386/trace.h\n>   GEN     target/mips/trace.h\n>   GEN     target/sparc/trace.h\n>   GEN     target/s390x/trace.h\n>   GEN     target/ppc/trace.h\n>   GEN     qom/trace.h\n>   GEN     linux-user/trace.h\n>   GEN     qapi/trace.h\n>   GEN     accel/tcg/trace.h\n>   GEN     accel/kvm/trace.h\n>   GEN     nbd/trace.h\n>   GEN     trace-root.c\n>   GEN     util/trace.c\n>   GEN     crypto/trace.c\n>   GEN     io/trace.c\n>   GEN     migration/trace.c\n>   GEN     block/trace.c\n>   GEN     chardev/trace.c\n>   GEN     hw/block/trace.c\n>   GEN     hw/block/dataplane/trace.c\n>   GEN     hw/char/trace.c\n>   GEN     hw/intc/trace.c\n>   GEN     hw/net/trace.c\n>   GEN     hw/virtio/trace.c\n>   GEN     hw/audio/trace.c\n>   GEN     hw/misc/trace.c\n>   GEN     hw/usb/trace.c\n>   GEN     hw/scsi/trace.c\n>   GEN     hw/nvram/trace.c\n>   GEN     hw/display/trace.c\n>   GEN     hw/input/trace.c\n>   GEN     hw/timer/trace.c\n>   GEN     hw/dma/trace.c\n>   GEN     hw/sparc/trace.c\n>   GEN     hw/sd/trace.c\n>   GEN     hw/isa/trace.c\n>   GEN     hw/mem/trace.c\n>   GEN     hw/i386/trace.c\n>   GEN     hw/i386/xen/trace.c\n>   GEN     hw/9pfs/trace.c\n>   GEN     hw/ppc/trace.c\n>   GEN     hw/pci/trace.c\n>   GEN     hw/s390x/trace.c\n>   GEN     hw/vfio/trace.c\n>   GEN     hw/acpi/trace.c\n>   GEN     hw/arm/trace.c\n>   GEN     hw/alpha/trace.c\n>   GEN     hw/xen/trace.c\n>   GEN     ui/trace.c\n>   GEN     audio/trace.c\n>   GEN     net/trace.c\n>   GEN     target/arm/trace.c\n>   GEN     target/i386/trace.c\n>   GEN     target/mips/trace.c\n>   GEN     target/sparc/trace.c\n>   GEN     target/s390x/trace.c\n>   GEN     target/ppc/trace.c\n>   GEN     qom/trace.c\n>   GEN     linux-user/trace.c\n>   GEN     qapi/trace.c\n>   GEN     accel/tcg/trace.c\n>   GEN     accel/kvm/trace.c\n>   GEN     nbd/trace.c\n>   GEN     config-all-devices.mak\n> \t DEP /tmp/qemu-test/src/dtc/tests/dumptrees.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/trees.S\n> \t DEP /tmp/qemu-test/src/dtc/tests/testutils.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/value-labels.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/asm_tree_dump.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/truncated_property.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/check_path.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/overlay_bad_fixup.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/overlay.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/subnode_iterate.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/property_iterate.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/integer-expressions.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/utilfdt_test.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/path_offset_aliases.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/add_subnode_with_nops.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/dtbs_equal_unordered.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/dtb_reverse.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/dtbs_equal_ordered.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/extra-terminating-null.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/incbin.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/boot-cpuid.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/phandle_format.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/path-references.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/references.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/string_escapes.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/propname_escapes.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/appendprop2.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/appendprop1.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/del_node.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/del_property.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/setprop.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/set_name.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/rw_tree1.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/open_pack.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/nopulate.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/mangle-layout.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/move_and_save.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/sw_tree1.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/nop_node.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/nop_property.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/setprop_inplace.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/stringlist.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/addr_size_cells.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/notfound.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/sized_cells.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/get_alias.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/char_literal.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/node_offset_by_compatible.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/node_check_compatible.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/node_offset_by_phandle.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/node_offset_by_prop_value.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/parent_offset.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/supernode_atdepth_offset.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/get_path.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/get_phandle.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/getprop.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/get_name.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/path_offset.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/subnode_offset.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/find_property.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/root_node.c\n> \t DEP /tmp/qemu-test/src/dtc/tests/get_mem_rsv.c\n> \t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_overlay.c\n> \t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_addresses.c\n> \t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_empty_tree.c\n> \t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_strerror.c\n> \t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_rw.c\n> \t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_sw.c\n> \t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_ro.c\n> \t DEP /tmp/qemu-test/src/dtc/libfdt/fdt_wip.c\n> \t DEP /tmp/qemu-test/src/dtc/libfdt/fdt.c\n> \t DEP /tmp/qemu-test/src/dtc/util.c\n> \t DEP /tmp/qemu-test/src/dtc/fdtput.c\n> \t DEP /tmp/qemu-test/src/dtc/fdtget.c\n> \t DEP /tmp/qemu-test/src/dtc/fdtdump.c\n> \t LEX convert-dtsv0-lexer.lex.c\n> \t DEP /tmp/qemu-test/src/dtc/srcpos.c\n> \t BISON dtc-parser.tab.c\n> \t LEX dtc-lexer.lex.c\n> \t DEP /tmp/qemu-test/src/dtc/treesource.c\n> \t DEP /tmp/qemu-test/src/dtc/livetree.c\n> \t DEP /tmp/qemu-test/src/dtc/flattree.c\n> \t DEP /tmp/qemu-test/src/dtc/fstree.c\n> \t DEP /tmp/qemu-test/src/dtc/dtc.c\n> \t DEP /tmp/qemu-test/src/dtc/data.c\n> \t DEP /tmp/qemu-test/src/dtc/checks.c\n> \t DEP convert-dtsv0-lexer.lex.c\n> \t DEP dtc-parser.tab.c\n> \t DEP dtc-lexer.lex.c\n> \tCHK version_gen.h\n> \tUPD version_gen.h\n> \t DEP /tmp/qemu-test/src/dtc/util.c\n> \t CC libfdt/fdt.o\n> \t CC libfdt/fdt_ro.o\n> \t CC libfdt/fdt_wip.o\n> \t CC libfdt/fdt_sw.o\n> \t CC libfdt/fdt_rw.o\n> \t CC libfdt/fdt_empty_tree.o\n> \t CC libfdt/fdt_strerror.o\n> \t CC libfdt/fdt_addresses.o\n> \t CC libfdt/fdt_overlay.o\n> \t AR libfdt/libfdt.a\n> ar: creating libfdt/libfdt.a\n> a - libfdt/fdt.o\n> a - libfdt/fdt_ro.o\n> a - libfdt/fdt_wip.o\n> a - libfdt/fdt_sw.o\n> a - libfdt/fdt_rw.o\n> a - libfdt/fdt_strerror.o\n> a - libfdt/fdt_empty_tree.o\n> a - libfdt/fdt_addresses.o\n> a - libfdt/fdt_overlay.o\n>   CC      tests/qemu-iotests/socket_scm_helper.o\n>   GEN     qga/qapi-generated/qga-qapi-types.h\n>   GEN     qga/qapi-generated/qga-qapi-visit.h\n>   GEN     qga/qapi-generated/qga-qapi-types.c\n>   GEN     qga/qapi-generated/qga-qmp-marshal.c\n>   GEN     qga/qapi-generated/qga-qapi-visit.c\n>   GEN     qga/qapi-generated/qga-qmp-commands.h\n>   CC      qmp-introspect.o\n>   CC      qapi-types.o\n>   CC      qapi-visit.o\n>   CC      qapi-event.o\n>   CC      qapi/qapi-visit-core.o\n>   CC      qapi/qapi-dealloc-visitor.o\n>   CC      qapi/qobject-input-visitor.o\n>   CC      qapi/qobject-output-visitor.o\n>   CC      qapi/qmp-registry.o\n>   CC      qapi/qmp-dispatch.o\n>   CC      qapi/string-input-visitor.o\n>   CC      qapi/string-output-visitor.o\n>   CC      qapi/opts-visitor.o\n>   CC      qapi/qapi-clone-visitor.o\n>   CC      qapi/qmp-event.o\n>   CC      qapi/qapi-util.o\n>   CC      qobject/qnull.o\n>   CC      qobject/qnum.o\n>   CC      qobject/qstring.o\n>   CC      qobject/qdict.o\n>   CC      qobject/qlist.o\n>   CC      qobject/qbool.o\n>   CC      qobject/qlit.o\n>   CC      qobject/qjson.o\n>   CC      qobject/qobject.o\n>   CC      qobject/json-lexer.o\n>   CC      qobject/json-streamer.o\n>   CC      qobject/json-parser.o\n>   CC      trace/control.o\n>   CC      trace/qmp.o\n>   CC      util/osdep.o\n>   CC      util/cutils.o\n>   CC      util/unicode.o\n>   CC      util/qemu-timer-common.o\n>   CC      util/bufferiszero.o\n>   CC      util/lockcnt.o\n>   CC      util/aiocb.o\n>   CC      util/async.o\n>   CC      util/thread-pool.o\n>   CC      util/qemu-timer.o\n>   CC      util/main-loop.o\n>   CC      util/iohandler.o\n>   CC      util/aio-posix.o\n>   CC      util/compatfd.o\n>   CC      util/event_notifier-posix.o\n>   CC      util/mmap-alloc.o\n>   CC      util/oslib-posix.o\n>   CC      util/qemu-openpty.o\n>   CC      util/qemu-thread-posix.o\n>   CC      util/memfd.o\n>   CC      util/envlist.o\n>   CC      util/path.o\n>   CC      util/module.o\n>   CC      util/host-utils.o\n>   CC      util/bitmap.o\n>   CC      util/bitops.o\n>   CC      util/hbitmap.o\n>   CC      util/fifo8.o\n>   CC      util/acl.o\n>   CC      util/cacheinfo.o\n>   CC      util/error.o\n>   CC      util/qemu-error.o\n>   CC      util/id.o\n>   CC      util/iov.o\n>   CC      util/qemu-config.o\n>   CC      util/qemu-sockets.o\n>   CC      util/uri.o\n>   CC      util/notify.o\n>   CC      util/qemu-option.o\n>   CC      util/qemu-progress.o\n>   CC      util/keyval.o\n>   CC      util/hexdump.o\n>   CC      util/crc32c.o\n>   CC      util/uuid.o\n>   CC      util/throttle.o\n>   CC      util/getauxval.o\n>   CC      util/readline.o\n>   CC      util/rcu.o\n>   CC      util/qemu-coroutine.o\n>   CC      util/qemu-coroutine-lock.o\n>   CC      util/qemu-coroutine-io.o\n>   CC      util/qemu-coroutine-sleep.o\n>   CC      util/coroutine-ucontext.o\n>   CC      util/buffer.o\n>   CC      util/timed-average.o\n>   CC      util/base64.o\n>   CC      util/log.o\n>   CC      util/qdist.o\n>   CC      util/qht.o\n>   CC      util/range.o\n>   CC      util/stats64.o\n>   CC      util/systemd.o\n>   CC      trace-root.o\n>   CC      util/trace.o\n>   CC      crypto/trace.o\n>   CC      io/trace.o\n>   CC      migration/trace.o\n>   CC      block/trace.o\n>   CC      chardev/trace.o\n>   CC      hw/block/trace.o\n>   CC      hw/block/dataplane/trace.o\n>   CC      hw/char/trace.o\n>   CC      hw/intc/trace.o\n>   CC      hw/net/trace.o\n>   CC      hw/virtio/trace.o\n>   CC      hw/audio/trace.o\n>   CC      hw/misc/trace.o\n>   CC      hw/usb/trace.o\n>   CC      hw/scsi/trace.o\n>   CC      hw/nvram/trace.o\n>   CC      hw/display/trace.o\n>   CC      hw/input/trace.o\n>   CC      hw/timer/trace.o\n>   CC      hw/dma/trace.o\n>   CC      hw/sparc/trace.o\n>   CC      hw/sd/trace.o\n>   CC      hw/isa/trace.o\n>   CC      hw/mem/trace.o\n>   CC      hw/i386/trace.o\n>   CC      hw/i386/xen/trace.o\n>   CC      hw/9pfs/trace.o\n>   CC      hw/ppc/trace.o\n>   CC      hw/pci/trace.o\n>   CC      hw/s390x/trace.o\n>   CC      hw/vfio/trace.o\n>   CC      hw/acpi/trace.o\n>   CC      hw/arm/trace.o\n>   CC      hw/alpha/trace.o\n>   CC      hw/xen/trace.o\n>   CC      ui/trace.o\n>   CC      audio/trace.o\n>   CC      net/trace.o\n>   CC      target/arm/trace.o\n>   CC      target/i386/trace.o\n>   CC      target/mips/trace.o\n>   CC      target/sparc/trace.o\n>   CC      target/s390x/trace.o\n>   CC      target/ppc/trace.o\n>   CC      qom/trace.o\n>   CC      linux-user/trace.o\n>   CC      qapi/trace.o\n>   CC      accel/tcg/trace.o\n>   CC      accel/kvm/trace.o\n>   CC      nbd/trace.o\n>   CC      crypto/pbkdf-stub.o\n>   CC      stubs/arch-query-cpu-def.o\n>   CC      stubs/arch-query-cpu-model-expansion.o\n>   CC      stubs/arch-query-cpu-model-comparison.o\n>   CC      stubs/arch-query-cpu-model-baseline.o\n>   CC      stubs/bdrv-next-monitor-owned.o\n>   CC      stubs/blk-commit-all.o\n>   CC      stubs/blockdev-close-all-bdrv-states.o\n>   CC      stubs/clock-warp.o\n>   CC      stubs/cpu-get-clock.o\n>   CC      stubs/cpu-get-icount.o\n>   CC      stubs/dump.o\n>   CC      stubs/error-printf.o\n>   CC      stubs/fdset.o\n>   CC      stubs/gdbstub.o\n>   CC      stubs/get-vm-name.o\n>   CC      stubs/instrument.o\n>   CC      stubs/iothread.o\n>   CC      stubs/iothread-lock.o\n>   CC      stubs/is-daemonized.o\n>   CC      stubs/machine-init-done.o\n>   CC      stubs/migr-blocker.o\n>   CC      stubs/change-state-handler.o\n>   CC      stubs/monitor.o\n>   CC      stubs/notify-event.o\n>   CC      stubs/qtest.o\n>   CC      stubs/replay.o\n> /tmp/qemu-test/src/stubs/instrument.c:22: error: redefinition of typedef ‘strList’\n> ./qapi-types.h:131: note: previous declaration of ‘strList’ was here\n>   CC      stubs/runstate-check.o\n> make: *** [stubs/instrument.o] Error 1\n> make: *** Waiting for unfinished jobs....\n> Traceback (most recent call last):\n>   File \"./tests/docker/docker.py\", line 384, in <module>\n>     sys.exit(main())\n>   File \"./tests/docker/docker.py\", line 381, in main\n>     return args.cmdobj.run(args, argv)\n>   File \"./tests/docker/docker.py\", line 239, in run\n>     return Docker().run(argv, args.keep, quiet=args.quiet)\n>   File \"./tests/docker/docker.py\", line 207, in run\n>     quiet=quiet)\n>   File \"./tests/docker/docker.py\", line 125, in _do_check\n>     return subprocess.check_call(self._command + cmd, **kwargs)\n>   File \"/usr/lib64/python2.7/subprocess.py\", line 186, in check_call\n>     raise CalledProcessError(retcode, cmd)\n> subprocess.CalledProcessError: Command '['docker', 'run', '--label', 'com.qemu.instance.uuid=9670c49e980a11e7950952540069c830', '-u', '0', '-t', '--rm', '--net=none', '-e', 'TARGET_LIST=', '-e', 'EXTRA_CONFIGURE_OPTS=', '-e', 'V=', '-e', 'J=8', '-e', 'DEBUG=', '-e', 'SHOW_ENV=1', '-e', 'CCACHE_DIR=/var/tmp/ccache', '-v', '/var/tmp/patchew-tester-tmp-lf5qg9x6/src/docker-src.2017-09-12-18.34.46.13887:/var/tmp/qemu:z,ro', '-v', '/root/.cache/qemu-docker-ccache:/var/tmp/ccache:z', 'qemu:centos6', '/var/tmp/qemu/run', 'test-quick']' returned non-zero exit status 2\n> make[1]: *** [tests/docker/Makefile.include:139: docker-run] Error 1\n> make[1]: Leaving directory '/var/tmp/patchew-tester-tmp-lf5qg9x6/src'\n> make: *** [tests/docker/Makefile.include:168: docker-run-test-quick@centos6] Error 2\n\nI think I fixed it, and will send a new series (for some reason this does not\nhappen with my GCC version).\n\n\nCheers,\n  Lluis","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","Received":["from lists.gnu.org (unknown [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xscPj2xm5z9s76\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 13 Sep 2017 19:51:36 +1000 (AEST)","from localhost ([::1]:41262 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1ds4KK-0003RP-4N\n\tfor incoming@patchwork.ozlabs.org; Wed, 13 Sep 2017 05:51:24 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:58316)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <vilanova@ac.upc.edu>) id 1ds4Jw-0003QB-Hj\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 05:51:04 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <vilanova@ac.upc.edu>) id 1ds4Jr-0000XX-GB\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 05:51:00 -0400","from roura.ac.upc.es ([147.83.33.10]:36988)\n\tby eggs.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <vilanova@ac.upc.edu>) id 1ds4Jq-0000Wk-PL\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 05:50:55 -0400","from correu-1.ac.upc.es (correu-1.ac.upc.es [147.83.30.91])\n\tby roura.ac.upc.es (8.13.8/8.13.8) with ESMTP id v8D9oodx007442;\n\tWed, 13 Sep 2017 11:50:50 +0200","from localhost (unknown [132.68.137.204])\n\tby correu-1.ac.upc.es (Postfix) with ESMTPSA id 86B0B13E;\n\tWed, 13 Sep 2017 11:50:44 +0200 (CEST)"],"From":"=?utf-8?q?Llu=C3=ADs_Vilanova?= <vilanova@ac.upc.edu>","To":"qemu-devel@nongnu.org","References":"<150525577025.321.3001704927093785609@6d89bd104fc3>","Mail-Followup-To":"qemu-devel@nongnu.org, cota@braap.org, famz@redhat.com,\n\tstefanha@redhat.com, armbru@redhat.com","Date":"Wed, 13 Sep 2017 12:50:43 +0300","In-Reply-To":"<150525577025.321.3001704927093785609@6d89bd104fc3>\n\t(no-reply@patchew.org's message of \"Tue, 12 Sep 2017 15:36:10 -0700\n\t(PDT)\")","Message-ID":"<87shfrsz9o.fsf@frigg.lan>","User-Agent":"Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Transfer-Encoding":"quoted-printable","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 2.6.x [fuzzy]","X-Received-From":"147.83.33.10","Subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Cc":"cota@braap.org, famz@redhat.com, armbru@redhat.com, stefanha@redhat.com","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1768651,"web_url":"http://patchwork.ozlabs.org/comment/1768651/","msgid":"<CAFEAcA-rmX=gUdYTLK4g5TAWVTkCfkbkXXfCxmaaivv8XGdMXg@mail.gmail.com>","list_archive_url":null,"date":"2017-09-14T14:54:28","subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","submitter":{"id":5111,"url":"http://patchwork.ozlabs.org/api/people/5111/","name":"Peter Maydell","email":"peter.maydell@linaro.org"},"content":"On 12 September 2017 at 22:01, Lluís Vilanova <vilanova@ac.upc.edu> wrote:\n> This series adds an API to add instrumentation events.\n>\n> It also provides additional APIs for:\n> * Controlling tracing events.\n> * Peek/poke guest memory.\n\n> Future APIs (for later series):\n> * Peek/poke guest registers.\n> * Add breakpoints to trigger instrumentation functions.\n> * Trigger instrumentation functions from guest code (former hypertrace).\n> * Add events for guest code translation/execution (once the respective tracing\n>   events are accepted upstream).\n> * Add events for exceptions/syscalls.\n> * Add events for TB invalidation (necessary for libraries to deallocate any data\n>   they might have allocated for the TBs they instrumented).\n>\n> The instrumentation code is dynamically loaded as a library into QEMU either\n> when it starts or later using its remote control interfaces. The loaded code\n> only has access to function explicitly exported through the QI_VPUBLIC macro.\n>\n> This series is branch 'devel-instrument' in\n> https://code.gso.ac.upc.edu/git/qemu-dbi.\n\nTo parallel the comment I sent on Emilio's series: I think the\nfirst thing we should do here is work out the API we want to\npresent to the instrumentation plugin, because that's permanent\nand we want to get it right. Then we can figure out the\nimplementation details later.\n\nParticular notes:\n * putting things in the instrumentation plugin API that basically\n   parallel some existing trace points is something I'm wary of,\n   because a lot of our existing tracing is \"output something at\n   a point and in a way that's convenient for our internal\n   implementation\", rather than \"what is the right clean interface\n   to provide this sort of information\". I'd rather we started\n   with a blank piece of paper and designed an API, and then\n   implemented it.\n * I definitely don't think we should expose to the instrumentation\n   the distinction between translate time and execute time\n\nthanks\n-- PMM","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"I0DlumAo\"; dkim-atps=neutral"],"Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xtM8m4xJ3z9s0Z\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri, 15 Sep 2017 00:58:00 +1000 (AEST)","from localhost ([::1]:48264 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1dsVaY-0007wu-Pv\n\tfor incoming@patchwork.ozlabs.org; Thu, 14 Sep 2017 10:57:58 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:54068)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <peter.maydell@linaro.org>) id 1dsVXX-0004ba-U9\n\tfor qemu-devel@nongnu.org; Thu, 14 Sep 2017 10:54:53 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <peter.maydell@linaro.org>) id 1dsVXW-0002tI-W1\n\tfor qemu-devel@nongnu.org; Thu, 14 Sep 2017 10:54:52 -0400","from mail-wr0-x234.google.com ([2a00:1450:400c:c0c::234]:48085)\n\tby eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.71) (envelope-from <peter.maydell@linaro.org>)\n\tid 1dsVXW-0002sO-OE\n\tfor qemu-devel@nongnu.org; Thu, 14 Sep 2017 10:54:50 -0400","by mail-wr0-x234.google.com with SMTP id k20so6111264wre.4\n\tfor <qemu-devel@nongnu.org>; Thu, 14 Sep 2017 07:54:50 -0700 (PDT)","by 10.223.139.215 with HTTP; Thu, 14 Sep 2017 07:54:28 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc:content-transfer-encoding;\n\tbh=d1RRIOQIw2QRsdrnNs2oV6m53ad4FvnIOq+XtOmCsYA=;\n\tb=I0DlumAoqgPd+Y0pLLmcRWhfLMDrPxOzLemmnFu3Kk1Vasv+U1I2VqUpqBQewJDYvz\n\tifHqqi8QHi5txDJ05MFf0atd4cFyNDpS79XMsvxATVdXQ6vwiKPlSdapFxO8GLCIUAJf\n\tBXnUH+iMBNeKceR7Gr72y9sSTaLnZr9YMTZYs=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc:content-transfer-encoding;\n\tbh=d1RRIOQIw2QRsdrnNs2oV6m53ad4FvnIOq+XtOmCsYA=;\n\tb=R6N8TYBORSeyNm8eUEn9Lqqt5IUQtJsfXqpx23Jg+829Spz4VPcZSe876DuKxMsFMq\n\tI8RXRcx1Aqn1sz0vGmJERoLMA4ltb/gBIHvrhixRbL2cPoN6UUUQbk51VantyMVRM6ZB\n\t89LS2xoN5vPnP9N2zoKDedK0K2EzVWYHZtnfk8375/AgmmhuVz7CyKSD+HOszld1GCCP\n\twU6NRNa0gfjAv3sSFXsLnXogA16SbuUeXpSFb8zNYZkev6Lsx428c0MP8GtXxzBXQ7jK\n\t7osUrt2JYdeMX9iINlBwIPxxX+xBa8KTCqTQ88jCquHdCV61iLQNfZGiAV+c/Aiss+Vp\n\t9ZTw==","X-Gm-Message-State":"AHPjjUjLWUuqVdvyTC6P3eltXhipncTXerbyxWzUA6PFto6IcIZheLMg\n\tnD9TeZxqqHLxzx98Fs7oUekYsPp66vR6aLoVJvW85A==","X-Google-Smtp-Source":"ADKCNb5tK/M5Un4RruT3SyHz8DRp6MLVu+Fe19zW6CQ3GhQZPhbnBvbd5imzOPu7e0pQYzB1tftfEftoQm5UQDkfFDk=","X-Received":"by 10.223.147.166 with SMTP id 35mr21178971wrp.90.1505400889476; \n\tThu, 14 Sep 2017 07:54:49 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<150525010239.15988.8172586618197849619.stgit@frigg.lan>","References":"<150525010239.15988.8172586618197849619.stgit@frigg.lan>","From":"Peter Maydell <peter.maydell@linaro.org>","Date":"Thu, 14 Sep 2017 15:54:28 +0100","Message-ID":"<CAFEAcA-rmX=gUdYTLK4g5TAWVTkCfkbkXXfCxmaaivv8XGdMXg@mail.gmail.com>","To":"=?utf-8?q?Llu=C3=ADs_Vilanova?= <vilanova@ac.upc.edu>","Content-Type":"text/plain; charset=\"UTF-8\"","Content-Transfer-Encoding":"quoted-printable","X-detected-operating-system":"by eggs.gnu.org: Genre and OS details not\n\trecognized.","X-Received-From":"2a00:1450:400c:c0c::234","Subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Cc":"\"Emilio G. Cota\" <cota@braap.org>,\n\tQEMU Developers <qemu-devel@nongnu.org>, \n\tStefan Hajnoczi <stefanha@redhat.com>,\n\tMarkus Armbruster <armbru@redhat.com>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1769200,"web_url":"http://patchwork.ozlabs.org/comment/1769200/","msgid":"<877ex0gjnd.fsf@frigg.lan>","list_archive_url":null,"date":"2017-09-15T13:45:42","subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","submitter":{"id":9099,"url":"http://patchwork.ozlabs.org/api/people/9099/","name":"Lluís Vilanova","email":"vilanova@ac.upc.edu"},"content":"Peter Maydell writes:\n\n> On 12 September 2017 at 22:01, Lluís Vilanova <vilanova@ac.upc.edu> wrote:\n>> This series adds an API to add instrumentation events.\n>> \n>> It also provides additional APIs for:\n>> * Controlling tracing events.\n>> * Peek/poke guest memory.\n\n>> Future APIs (for later series):\n>> * Peek/poke guest registers.\n>> * Add breakpoints to trigger instrumentation functions.\n>> * Trigger instrumentation functions from guest code (former hypertrace).\n>> * Add events for guest code translation/execution (once the respective tracing\n>> events are accepted upstream).\n>> * Add events for exceptions/syscalls.\n>> * Add events for TB invalidation (necessary for libraries to deallocate any data\n>> they might have allocated for the TBs they instrumented).\n>> \n>> The instrumentation code is dynamically loaded as a library into QEMU either\n>> when it starts or later using its remote control interfaces. The loaded code\n>> only has access to function explicitly exported through the QI_VPUBLIC macro.\n>> \n>> This series is branch 'devel-instrument' in\n>> https://code.gso.ac.upc.edu/git/qemu-dbi.\n\n> To parallel the comment I sent on Emilio's series: I think the\n> first thing we should do here is work out the API we want to\n> present to the instrumentation plugin, because that's permanent\n> and we want to get it right. Then we can figure out the\n> implementation details later.\n\n> Particular notes:\n>  * putting things in the instrumentation plugin API that basically\n>    parallel some existing trace points is something I'm wary of,\n>    because a lot of our existing tracing is \"output something at\n>    a point and in a way that's convenient for our internal\n>    implementation\", rather than \"what is the right clean interface\n>    to provide this sort of information\". I'd rather we started\n>    with a blank piece of paper and designed an API, and then\n>    implemented it.\n\nDo you have an opinion on specific trace points, or an observation in general?\n\nI wanted to have the following events:\n\n* cpu hotplug/hotunplug\n* cpu reset\n* memory access translation/execution\n* BBL/TB translation/execution\n* instruction translation/execution\n* exceptions / syscalls\n\n\n>  * I definitely don't think we should expose to the instrumentation\n>    the distinction between translate time and execute time\n\nPlease see my response to your email in v6.\n\nAlso, I forgot an even simpler case, like instrumenting a subset of memory\naccesses (e.g., only writes).\n\n\nThanks,\n  Lluis","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xtxWh2kFbz9sRm\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri, 15 Sep 2017 23:46:21 +1000 (AEST)","from localhost ([::1]:53451 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1dsqwl-0003nS-53\n\tfor incoming@patchwork.ozlabs.org; Fri, 15 Sep 2017 09:46:19 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:50796)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <vilanova@ac.upc.edu>) id 1dsqwR-0003n9-RF\n\tfor qemu-devel@nongnu.org; Fri, 15 Sep 2017 09:46:01 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <vilanova@ac.upc.edu>) id 1dsqwO-0000M9-My\n\tfor qemu-devel@nongnu.org; Fri, 15 Sep 2017 09:45:59 -0400","from roura.ac.upc.edu ([147.83.33.10]:58397 helo=roura.ac.upc.es)\n\tby eggs.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <vilanova@ac.upc.edu>) id 1dsqwO-0000Lt-Ap\n\tfor qemu-devel@nongnu.org; Fri, 15 Sep 2017 09:45:56 -0400","from correu-1.ac.upc.es (correu-1.ac.upc.es [147.83.30.91])\n\tby roura.ac.upc.es (8.13.8/8.13.8) with ESMTP id v8FDjmro018305;\n\tFri, 15 Sep 2017 15:45:48 +0200","from localhost (unknown [132.68.50.201])\n\tby correu-1.ac.upc.es (Postfix) with ESMTPSA id 3036E14B0;\n\tFri, 15 Sep 2017 15:45:43 +0200 (CEST)"],"From":"=?utf-8?q?Llu=C3=ADs_Vilanova?= <vilanova@ac.upc.edu>","To":"Peter Maydell <peter.maydell@linaro.org>","References":"<150525010239.15988.8172586618197849619.stgit@frigg.lan>\n\t<CAFEAcA-rmX=gUdYTLK4g5TAWVTkCfkbkXXfCxmaaivv8XGdMXg@mail.gmail.com>","Mail-Followup-To":"Peter Maydell <peter.maydell@linaro.org>, QEMU Developers\n\t<qemu-devel@nongnu.org>, \"Emilio G. Cota\" <cota@braap.org>,\n\tMarkus\n\tArmbruster <armbru@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>","Date":"Fri, 15 Sep 2017 16:45:42 +0300","In-Reply-To":"<CAFEAcA-rmX=gUdYTLK4g5TAWVTkCfkbkXXfCxmaaivv8XGdMXg@mail.gmail.com>\n\t(Peter Maydell's message of \"Thu, 14 Sep 2017 15:54:28 +0100\")","Message-ID":"<877ex0gjnd.fsf@frigg.lan>","User-Agent":"Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Transfer-Encoding":"quoted-printable","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 2.6.x [fuzzy]","X-Received-From":"147.83.33.10","Subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Cc":"\"Emilio G. Cota\" <cota@braap.org>,\n\tQEMU Developers <qemu-devel@nongnu.org>, \n\tStefan Hajnoczi <stefanha@redhat.com>,\n\tMarkus Armbruster <armbru@redhat.com>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1769202,"web_url":"http://patchwork.ozlabs.org/comment/1769202/","msgid":"<CAFEAcA9Ux91tSfYpE_giuSHBEuZEuHeNsp=79h=aHwO2N85cgg@mail.gmail.com>","list_archive_url":null,"date":"2017-09-15T13:49:42","subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","submitter":{"id":5111,"url":"http://patchwork.ozlabs.org/api/people/5111/","name":"Peter Maydell","email":"peter.maydell@linaro.org"},"content":"On 15 September 2017 at 14:45, Lluís Vilanova <vilanova@ac.upc.edu> wrote:\n> Do you have an opinion on specific trace points, or an observation in general?\n>\n> I wanted to have the following events:\n>\n> * cpu hotplug/hotunplug\n> * cpu reset\n> * memory access translation/execution\n> * BBL/TB translation/execution\n> * instruction translation/execution\n> * exceptions / syscalls\n\nMy initial stab at a \"basic\" set of events would be:\n * instruction executes\n * memory accesses\n * exceptions\n\nBut the better approach would probably be to survey the\nexisting trace APIs in other models.\n\nthanks\n-- PMM","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"giu57sbX\"; dkim-atps=neutral"],"Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xtxcT0hcKz9sxR\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri, 15 Sep 2017 23:50:33 +1000 (AEST)","from localhost ([::1]:53459 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1dsr0p-0005ig-5m\n\tfor incoming@patchwork.ozlabs.org; Fri, 15 Sep 2017 09:50:31 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:51982)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <peter.maydell@linaro.org>) id 1dsr0P-0005hM-EE\n\tfor qemu-devel@nongnu.org; Fri, 15 Sep 2017 09:50:06 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <peter.maydell@linaro.org>) id 1dsr0O-0003x6-Gw\n\tfor qemu-devel@nongnu.org; Fri, 15 Sep 2017 09:50:05 -0400","from mail-wm0-x22f.google.com ([2a00:1450:400c:c09::22f]:43735)\n\tby eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.71) (envelope-from <peter.maydell@linaro.org>)\n\tid 1dsr0O-0003wa-8f\n\tfor qemu-devel@nongnu.org; Fri, 15 Sep 2017 09:50:04 -0400","by mail-wm0-x22f.google.com with SMTP id a137so2923845wma.0\n\tfor <qemu-devel@nongnu.org>; Fri, 15 Sep 2017 06:50:04 -0700 (PDT)","by 10.223.139.215 with HTTP; Fri, 15 Sep 2017 06:49:42 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:content-transfer-encoding;\n\tbh=ykr4hJrV3Jub4IEJ1HiKELoTwqBH50DDsVPc8pS8xAo=;\n\tb=giu57sbXH3wiN0SPzzAFY3W4ng3O1bFIV3Kh8IHLA3voObx6yX0T3fvB22pa6Rj1Cv\n\tIgEFCNIMAuTyu2RSpaFrSzZowVe5QRd/uTtN56aWixR7PUvblab7dMQdxTB4107vq4k3\n\thW5H/Ck1uMQLkFDJaHmrs7PDUuC6Ng8OnBzjw=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:content-transfer-encoding;\n\tbh=ykr4hJrV3Jub4IEJ1HiKELoTwqBH50DDsVPc8pS8xAo=;\n\tb=PvJzHlfmOXXNaCRPnyujxdFU6lHt1TtggPMjWlcFDIytGhrJPUs12ZP34w7Q9+hxIi\n\tIVqBQ/oA4FoqGkn7lK0WQKOAa5CknACVbed0tTc/XYH4JJXM/7Ti+h65AgowtzcFrGN8\n\tqwidi1s5wOsR1F3ar9cqAQHS9v8Ci/kingcnZpIpTYH24tFkar/IWe0VRpS2chIZUhwt\n\tmJ43zTNaCRl/YjhZbrO1Dg3i8t1qL6GNRLiZt1anqndFiOSq2YoTOyDnTDsEBT0TqhME\n\tAr3xLqR/9bdfsmgrrYVnZYiblxAPIc/89m1nncEZd47V0VoQMMc8fp/m3aasr6e5u+EF\n\tDIaw==","X-Gm-Message-State":"AHPjjUgSi8XGg1QlG/TmYVZs/epyDk/zjbbdesYBJNVdlkqVho835Wrl\n\tU/LrZGs9mD4nAqc9XHcpx9qE6kJK5/6ILVzAqzo3qdLK","X-Google-Smtp-Source":"AOwi7QB2rrhRg6Gp4vq6avG/qkEfzoU1R9PsNnF0U4m7qgbCdXWQmPWNhJwIeDpzoCKMRrq04ZDajmcKCt5HECYpL6c=","X-Received":"by 10.28.166.135 with SMTP id p129mr2992477wme.147.1505483403017;\n\tFri, 15 Sep 2017 06:50:03 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<877ex0gjnd.fsf@frigg.lan>","References":"<150525010239.15988.8172586618197849619.stgit@frigg.lan>\n\t<CAFEAcA-rmX=gUdYTLK4g5TAWVTkCfkbkXXfCxmaaivv8XGdMXg@mail.gmail.com>\n\t<877ex0gjnd.fsf@frigg.lan>","From":"Peter Maydell <peter.maydell@linaro.org>","Date":"Fri, 15 Sep 2017 14:49:42 +0100","Message-ID":"<CAFEAcA9Ux91tSfYpE_giuSHBEuZEuHeNsp=79h=aHwO2N85cgg@mail.gmail.com>","To":"Peter Maydell <peter.maydell@linaro.org>,\n\tQEMU Developers <qemu-devel@nongnu.org>, \n\t\"Emilio G. Cota\" <cota@braap.org>,\n\tMarkus Armbruster <armbru@redhat.com>, \n\tStefan Hajnoczi <stefanha@redhat.com>","Content-Type":"text/plain; charset=\"UTF-8\"","Content-Transfer-Encoding":"quoted-printable","X-detected-operating-system":"by eggs.gnu.org: Genre and OS details not\n\trecognized.","X-Received-From":"2a00:1450:400c:c09::22f","Subject":"Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event\n\tinstrumentation","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}}]