qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Richard Henderson <richard.henderson@linaro.org>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PULL 00/21] tcg patch queue
Date: Thu, 18 Oct 2018 23:10:53 -0700	[thread overview]
Message-ID: <12592ba3-ea47-e7c2-d67b-c58012c64287@linaro.org> (raw)
In-Reply-To: <CAFEAcA-NEN7MymyzmeOwz7bX99P_o6D9dMraHYA8LcFS7w9=_A@mail.gmail.com>

On 10/18/18 3:34 AM, Peter Maydell wrote:
> Hi; I get compile failures, I'm afraid:
> 
> FreeBSD's compiler gives warnings:
> In file included from /var/tmp/qemu-test.bVaUEj/accel/tcg/user-exec.c:28:
> /var/tmp/qemu-test.bVaUEj/include/qemu/atomic128.h:148:23: warning:
> unknown attribute 'error' ignored [-Wunknown-attributes]
> Int128 __attribute__((error("unsupported atomic")))
>                       ^
> /var/tmp/qemu-test.bVaUEj/include/qemu/atomic128.h:150:21: warning:
> unknown attribute 'error' ignored [-Wunknown-attributes]
> void __attribute__((error("unsupported atomic")))
>                     ^
> 
> x86-64 Linux clang fails with the same issue:
> In file included from
> /home/petmay01/linaro/qemu-for-merges/accel/tcg/user-exec.c:28:
> /home/petmay01/linaro/qemu-for-merges/include/qemu/atomic128.h:148:23:
> error: unknown attribute 'error' ignored
> [-Werror,-Wunknown-attributes]
> Int128 __attribute__((error("unsupported atomic")))
>                       ^
> /home/petmay01/linaro/qemu-for-merges/include/qemu/atomic128.h:150:21:
> error: unknown attribute 'error' ignored
> [-Werror,-Wunknown-attributes]
> void __attribute__((error("unsupported atomic")))

I used __has_attribute for this one.  Tested with clang 7.


> The win32 build fails with:
> 
> target/arm/helper-a64.o: In function `helper_paired_cmpxchg64_le_parallel':
> /home/petmay01/qemu-for-merges/target/arm/helper-a64.c:573: undefined
> reference to `helper_atomic_cmpxchgo_le_mmu'
> target/arm/helper-a64.o: In function `helper_paired_cmpxchg64_be_parallel':
> /home/petmay01/qemu-for-merges/target/arm/helper-a64.c:647: undefined
> reference to `helper_atomic_cmpxchgo_be_mmu'
> target/arm/helper-a64.o: In function `helper_casp_le_parallel':
> /home/petmay01/qemu-for-merges/target/arm/helper-a64.c:669: undefined
> reference to `helper_atomic_cmpxchgo_le_mmu'
> target/arm/helper-a64.o: In function `helper_casp_be_parallel':
> /home/petmay01/qemu-for-merges/target/arm/helper-a64.c:690: undefined
> reference to `helper_atomic_cmpxchgo_be_mmu'

This one I could not reproduce.  Tested with i686-w64-mingw32-gcc
and x86_64-w64-mingw32-gcc, both version 6.2.

I suspect configure misfired.  If you can, start this one over with a blank
build directory.

I've sent you a v2 pull.


r~

  reply	other threads:[~2018-10-19  6:11 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-16 17:48 [Qemu-devel] [PULL 00/21] tcg patch queue Richard Henderson
2018-10-16 17:48 ` [Qemu-devel] [PULL 01/21] tcg: Implement CPU_LOG_TB_NOCHAIN during expansion Richard Henderson
2018-10-16 17:48 ` [Qemu-devel] [PULL 02/21] tcg: access cpu->icount_decr.u16.high with atomics Richard Henderson
2018-10-16 17:48 ` [Qemu-devel] [PULL 03/21] tcg: fix use of uninitialized variable under CONFIG_PROFILER Richard Henderson
2018-10-16 17:48 ` [Qemu-devel] [PULL 04/21] tcg: plug holes in struct TCGProfile Richard Henderson
2018-10-16 17:48 ` [Qemu-devel] [PULL 05/21] tcg: distribute tcg_time into TCG contexts Richard Henderson
2018-10-16 17:48 ` [Qemu-devel] [PULL 06/21] target/alpha: remove tlb_flush from alpha_cpu_initfn Richard Henderson
2018-10-16 17:48 ` [Qemu-devel] [PULL 07/21] target/unicore32: remove tlb_flush from uc32_init_fn Richard Henderson
2018-10-16 17:48 ` [Qemu-devel] [PULL 08/21] exec: introduce tlb_init Richard Henderson
2018-10-16 17:48 ` [Qemu-devel] [PULL 09/21] cputlb: fix assert_cpu_is_self macro Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 10/21] cputlb: serialize tlb updates with env->tlb_lock Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 11/21] tcg: Add tlb_index and tlb_entry helpers Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 12/21] tcg: Split CONFIG_ATOMIC128 Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 13/21] target/i386: Convert to HAVE_CMPXCHG128 Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 14/21] target/arm: " Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 15/21] target/arm: Check HAVE_CMPXCHG128 at translate time Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 16/21] target/ppc: Convert to HAVE_CMPXCHG128 and HAVE_ATOMIC128 Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 17/21] target/s390x: " Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 18/21] target/s390x: Split do_cdsg, do_lpq, do_stpq Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 19/21] target/s390x: Skip wout, cout helpers if op helper does not return Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 20/21] target/s390x: Check HAVE_ATOMIC128 and HAVE_CMPXCHG128 at translate Richard Henderson
2018-10-16 17:49 ` [Qemu-devel] [PULL 21/21] cputlb: read CPUTLBEntry.addr_write atomically Richard Henderson
2018-10-18 10:34 ` [Qemu-devel] [PULL 00/21] tcg patch queue Peter Maydell
2018-10-19  6:10   ` Richard Henderson [this message]
2018-10-21 15:01     ` Peter Maydell
2018-10-21 15:21       ` Peter Maydell
2018-10-22 17:47         ` Richard Henderson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=12592ba3-ea47-e7c2-d67b-c58012c64287@linaro.org \
    --to=richard.henderson@linaro.org \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).