From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42277) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZnTXX-0008Uj-Jt for qemu-devel@nongnu.org; Sat, 17 Oct 2015 11:37:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZnTXU-0005VJ-BH for qemu-devel@nongnu.org; Sat, 17 Oct 2015 11:36:59 -0400 Received: from mail-ob0-x230.google.com ([2607:f8b0:4003:c01::230]:33262) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZnTXU-0005V1-5H for qemu-devel@nongnu.org; Sat, 17 Oct 2015 11:36:56 -0400 Received: by obbwb3 with SMTP id wb3so84927025obb.0 for ; Sat, 17 Oct 2015 08:36:55 -0700 (PDT) Sender: fluxion Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Michael Roth In-Reply-To: References: <1444925118-10629-1-git-send-email-mdroth@linux.vnet.ibm.com> Message-ID: <20151017153642.15370.71573@loki> Date: Sat, 17 Oct 2015 10:36:42 -0500 Subject: Re: [Qemu-devel] [PULL v2 00/12] qemu-ga patch queue List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: QEMU Developers Quoting Peter Maydell (2015-10-16 13:05:28) > On 15 October 2015 at 17:05, Michael Roth wro= te: > > Hi Peter, > > > > Please note that 'glib-compat: add 2.38/2.40/2.46 asserts' is also in > > Marc-Andr=C3=A9's recent ivshmem PULL. The 2 versions of the patches ar= e identical, > > but let me know if you'd prefer a re-send/re-base later. > > > > The following changes since commit c49d3411faae8ffaab8f7e5db47405a00841= 1c10: > > > > Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2015-10-1= 2' into staging (2015-10-13 10:42:06 +0100) > > > > are available in the git repository at: > > > > > > git://github.com/mdroth/qemu.git tags/qga-pull-2015-10-14-v2-tag > > > > for you to fetch changes up to 32bcf9327f9ad614712999f514707415a53ec620: > > > > qga: guest-exec simple stdin/stdout/stderr redirection (2015-10-14 22= :39:09 -0500) > > > > ---------------------------------------------------------------- > > qemu-ga patch queue > > > > * add unit tests for qemu-ga > > * add guest-exec support for posix/w32 guests > > * added 'qemu-ga' target for w32. this allows us to do full MSI build, > > without overloading 'qemu-ga.exe' target with uneeded dependencies. > > * number of s/g_new/g_malloc/ conversions for qga > > > > v2: > > * commit message and qapi documentation spelling fixes > > * rename 'inp-data' guest-exec param to 'input-data' > > > = > Hi; I'm afraid this fails my build tests. > = > Compile failures on OSX: > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:366:51: warning: > format specifies type 'long' but the argument has type 'int64_t' (aka > 'long long') [-Wformat] > " 'buf-b64': '%s' } }", id, enc); > ^~ > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:382:27: warning: > format specifies type 'long' but the argument has type 'int64_t' (aka > 'long long') [-Wformat] > id); > ^~ > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:390:27: warning: > format specifies type 'long' but the argument has type 'int64_t' (aka > 'long long') [-Wformat] > id); > ^~ > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:416:27: warning: > format specifies type 'long' but the argument has type 'int64_t' (aka > 'long long') [-Wformat] > id); > ^~ > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:433:27: warning: > format specifies type 'long' but the argument has type 'int64_t' (aka > 'long long') [-Wformat] > id); > ^~ > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:449:27: warning: > format specifies type 'long' but the argument has type 'int64_t' (aka > 'long long') [-Wformat] > id, 6, SEEK_SET); > ^~ > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:463:27: warning: > format specifies type 'long' but the argument has type 'int64_t' (aka > 'long long') [-Wformat] > id); > ^~ > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:482:27: warning: > format specifies type 'long' but the argument has type 'int64_t' (aka > 'long long') [-Wformat] > id); > ^~ > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:537:27: warning: > format specifies type 'long' but the argument has type 'long long' > [-Wformat] > current + time * 1000); > ^~~~~~~~~~~~~~~~~~~~~ > /Users/pm215/src/qemu-for-merges/tests/test-qga.c:632:35: error: use > of undeclared identifier 'G_SPAWN_DEFAULT' > g_spawn_sync(NULL, argv, env, G_SPAWN_DEFAULT, > ^ > 9 warnings and 1 error generated. > = > Most of these look like trying to use G_GINT64_FORMAT > for int64_t rather than PRId64. > = > The last one is using a value that wasn't implemented until > after our current minimum glib version (I think it came in > in 2.37, though the glib docs fail to mention this). Needs > to use "0" instead. > = > Win32 also fails to build: > = > /home/petmay01/linaro/qemu-for-merges/qga/commands.c: In function > =E2=80=98qmp_guest_exec_status=E2=80=99: > /home/petmay01/linaro/qemu-for-merges/qga/commands.c:138: warning: > cast to pointer from integer of different size > /home/petmay01/linaro/qemu-for-merges/qga/commands.c: In function > =E2=80=98qmp_guest_exec=E2=80=99: > /home/petmay01/linaro/qemu-for-merges/qga/commands.c:407: warning: > cast from pointer to integer of different size > = > which I think is because GPid is a pointer type on Windows > and the code is assuming it's an integer. Thanks for the helpful hints. I should have everything squared away now, but just for reference I've pushed the unsquashed changes here: https://github.com/mdroth/qemu/commits/qga-pull-2015-10-14-v3-unsquashed Will send a v3 shortly. I noticed an uninitialized value warning was still present for win32 so I've added a 1-line trivial fix to the series to address it. > = > thanks > -- PMM >=20