[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough
From: |
Thomas Huth |
Subject: |
Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough |
Date: |
Thu, 17 Dec 2020 14:03:47 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0 |
On 17/12/2020 13.51, Peter Maydell wrote:
> On Wed, 16 Dec 2020 at 17:29, Thomas Huth <thuth@redhat.com> wrote:
>>
>> Hi!
>>
>> The following changes since commit af3f37319cb1e1ca0c42842ecdbd1bcfc64a4b6f:
>>
>> Merge remote-tracking branch 'remotes/bonzini-gitlab/tags/for-upstream'
>> into staging (2020-12-15 21:24:31 +0000)
>>
>> are available in the Git repository at:
>>
>> https://gitlab.com/huth/qemu.git tags/pull-request-2020-12-16
>>
>> for you to fetch changes up to cbbedfeeb77e25b065f8a2b0c33e81403edaf728:
>>
>> configure: Compile with -Wimplicit-fallthrough=2 (2020-12-16 12:52:20
>> +0100)
>>
>> ----------------------------------------------------------------
>> * Compile QEMU with -Wimplicit-fallthrough=2 to avoid bugs in
>> switch-case statements
>> ----------------------------------------------------------------
>
> Hi; this generates a new warning on the NetBSD build:
>
> ../src/bsd-user/main.c: In function 'cpu_loop':
> ../src/bsd-user/main.c:513:16: warning: this statement may fall
> through [-Wimplicit-fallthrough=]
> if (bsd_type != target_freebsd)
> ^
> ../src/bsd-user/main.c:515:9: note: here
> case 0x100:
> ^~~~
Oh man, can't we just ditch the bsd-user folder now? It's known to be broken
since many releases, so it's currently only causing additional effort to
keep this code compilable (also with regards to the automatic code scan tool
reports that we've seen during the past months), without real benefit. Even
if the BSD folks finally upstream their fixed version again, it's more
likely that they will start from scratch again instead of fixing the old
folder, I guess?
Thomas
- [PULL 06/12] hw/intc/arm_gicv3_kvm: silence the compiler warnings, (continued)
- [PULL 06/12] hw/intc/arm_gicv3_kvm: silence the compiler warnings, Thomas Huth, 2020/12/16
- [PULL 09/12] target/sparc/win_helper: silence the compiler warnings, Thomas Huth, 2020/12/16
- [PULL 07/12] accel/tcg/user-exec: silence the compiler warnings, Thomas Huth, 2020/12/16
- [PULL 11/12] tests/fp: Do not emit implicit-fallthrough warnings in the softfloat tests, Thomas Huth, 2020/12/16
- [PULL 12/12] configure: Compile with -Wimplicit-fallthrough=2, Thomas Huth, 2020/12/16
- [PULL 10/12] tcg/optimize: Add fallthrough annotations, Thomas Huth, 2020/12/16
- [PULL 08/12] target/sparc/translate: silence the compiler warnings, Thomas Huth, 2020/12/16
- Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough, no-reply, 2020/12/16
- Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough, Peter Maydell, 2020/12/17
- Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough,
Thomas Huth <=
- Status/future of QEMU bsd-user impl ? (Wea Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough), Daniel P . Berrangé, 2020/12/17
- Re: Status/future of QEMU bsd-user impl ? (Wea Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough), Warner Losh, 2020/12/17
- Re: Status/future of QEMU bsd-user impl ? (Wea Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough), Peter Maydell, 2020/12/17
- Re: Status/future of QEMU bsd-user impl ? (Wea Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough), Warner Losh, 2020/12/17
- Re: Status/future of QEMU bsd-user impl ? (Wea Re: [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough), Warner Losh, 2020/12/17