[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] aspeed/timer: Fix slowdowns in recent Linux
From: |
Cédric Le Goater |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] aspeed/timer: Fix slowdowns in recent Linux |
Date: |
Thu, 14 Mar 2019 10:06:05 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
Hello,
For the time being, we can drop patch 5 from this series.
Thanks,
C.
On 3/14/19 9:59 AM, address@hidden wrote:
> Patchew URL: https://patchew.org/QEMU/address@hidden/
>
>
>
> Hi,
>
> This series failed the address@hidden build test. Please find the testing
> commands and
> their output below. If you have Docker installed, you can probably reproduce
> it
> locally.
>
> === TEST SCRIPT BEGIN ===
> #!/bin/bash
> time make address@hidden SHOW_ENV=1 J=14 NETWORK=1
> === TEST SCRIPT END ===
>
> CC migration/xbzrle.o
> CC migration/postcopy-ram.o
> /tmp/qemu-test/src/hw/timer/aspeed_timer.c: In function
> 'aspeed_timer_set_value':
> /tmp/qemu-test/src/hw/timer/aspeed_timer.c:279:19: error: '__int128_t'
> undeclared (first use in this function); did you mean '__int128'?
> ((__int128_t)delta * NANOSECONDS_PER_SECOND / rate);
> ^~~~~~~~~~
> __int128
> /tmp/qemu-test/src/hw/timer/aspeed_timer.c:279:19: note: each undeclared
> identifier is reported only once for each function it appears in
> /tmp/qemu-test/src/hw/timer/aspeed_timer.c:279:30: error: expected ')' before
> 'delta'
> ((__int128_t)delta * NANOSECONDS_PER_SECOND / rate);
> ~ ^~~~~
> )
> /tmp/qemu-test/src/hw/timer/aspeed_timer.c:276:22: error: unused variable
> 'rate' [-Werror=unused-variable]
> uint32_t rate = calculate_rate(t);
> ^~~~
> /tmp/qemu-test/src/hw/timer/aspeed_timer.c:275:21: error: unused variable
> 'delta' [-Werror=unused-variable]
> int64_t delta = (int64_t) value - (int64_t) calculate_ticks(t,
> now);
> ^~~~~
> cc1: all warnings being treated as errors
>
>
> The full log is available at
> http://patchew.org/logs/address@hidden/address@hidden/?type=message.
> ---
> Email generated automatically by Patchew [http://patchew.org/].
> Please send your feedback to address@hidden
>
- [Qemu-devel] [PATCH 2/5] aspeed/timer: Status register contains reload for stopped timer, (continued)
- [Qemu-devel] [PATCH 2/5] aspeed/timer: Status register contains reload for stopped timer, Cédric Le Goater, 2019/03/14
- [Qemu-devel] [PATCH 3/5] aspeed/timer: Fix match calculations, Cédric Le Goater, 2019/03/14
- [Qemu-devel] [PATCH 4/5] aspeed/timer: Provide back-pressure information for short periods, Cédric Le Goater, 2019/03/14
- [Qemu-devel] [PATCH 5/5] aspeed/timer: Use signed muldiv for timer resets, Cédric Le Goater, 2019/03/14
- Re: [Qemu-devel] [PATCH 5/5] aspeed/timer: Use signed muldiv for timer resets, Peter Maydell, 2019/03/14
- Re: [Qemu-devel] [PATCH 0/5] aspeed/timer: Fix slowdowns in recent Linux, no-reply, 2019/03/14
- Re: [Qemu-devel] [PATCH 0/5] aspeed/timer: Fix slowdowns in recent Linux,
Cédric Le Goater <=