Re: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics

2016-09-15 Thread Alex Bennée
Richard Henderson writes: > Changes since v2: > * Fix build for 32-bit host without 64-bit atomics. > Tested with --extra-cflags='-march=-i486'. > This is patch 15, which might ought to get folded back into > patch 13 for bisection, but is ugly for review. > > * Move a lot of stu

Re: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics

2016-09-09 Thread Richard Henderson
On 09/09/2016 12:29 PM, Alex Bennée wrote: config-temp/qemu-conf.c:12:8: error: unused variable 'is_host64' [-Werror=unused-variable] cc1: all warnings being treated as errors Try this. r~ diff --git a/configure b/configure index 519de5d..bd59cdd 100755 --- a/configure +++ b/configure @@ -

Re: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics

2016-09-09 Thread Richard Henderson
On 09/09/2016 12:29 PM, Alex Bennée wrote: config-temp/qemu-conf.c: In function 'main': config-temp/qemu-conf.c:12:8: error: unused variable 'is_host64' [-Werror=unused-variable] cc1: all warnings being treated as errors Ah hah. All I needed was a compiler without __atomic support. I thought

Re: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics

2016-09-09 Thread Alex Bennée
Richard Henderson writes: > On 09/09/2016 11:33 AM, Alex Bennée wrote: >> >> Richard Henderson writes: >> >>> Changes since v2: >>> * Fix build for 32-bit host without 64-bit atomics. >>> Tested with --extra-cflags='-march=-i486'. >>> This is patch 15, which might ought to get folded

Re: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics

2016-09-09 Thread Richard Henderson
On 09/09/2016 11:33 AM, Alex Bennée wrote: Richard Henderson writes: Changes since v2: * Fix build for 32-bit host without 64-bit atomics. Tested with --extra-cflags='-march=-i486'. This is patch 15, which might ought to get folded back into patch 13 for bisection, but is ugly f

Re: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics

2016-09-09 Thread Alex Bennée
Richard Henderson writes: > Changes since v2: > * Fix build for 32-bit host without 64-bit atomics. > Tested with --extra-cflags='-march=-i486'. > This is patch 15, which might ought to get folded back into > patch 13 for bisection, but is ugly for review. Hmm I wonder what's brea

Re: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics

2016-09-03 Thread no-reply
Hi, Your series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics Type: series Message-id: 1472935202-3342-1-git-send

Re: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics

2016-09-03 Thread no-reply
Hi, Your series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics Type: series Message-id: 1472935202-3342-1-git-send-email-...@twiddle.net === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n

[Qemu-devel] [PATCH v3 00/34] cmpxchg-based emulation of atomics

2016-09-03 Thread Richard Henderson
Changes since v2: * Fix build for 32-bit host without 64-bit atomics. Tested with --extra-cflags='-march=-i486'. This is patch 15, which might ought to get folded back into patch 13 for bisection, but is ugly for review. * Move a lot of stuff out of softmmu_template.h, and into cpu