[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 0/8] Net patches
From: |
Cédric Le Goater |
Subject: |
Re: [Qemu-devel] [PULL 0/8] Net patches |
Date: |
Tue, 25 Apr 2017 08:18:45 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 04/25/2017 05:57 AM, Jason Wang wrote:
>
>
> On 2017年04月25日 00:02, Cédric Le Goater wrote:
>> On 04/24/2017 03:49 PM, Peter Maydell wrote:
>>> On 24 April 2017 at 06:15, Jason Wang <address@hidden> wrote:
>>>> The following changes since commit
>>>> 32c7e0ab755745e961f1772e95cac381cc68769d:
>>>>
>>>> Merge remote-tracking branch
>>>> 'remotes/juanquintela/tags/migration/20170421' into staging (2017-04-21
>>>> 15:59:27 +0100)
>>>>
>>>> are available in the git repository at:
>>>>
>>>> https://github.com/jasowang/qemu.git tags/net-pull-request
>>>>
>>>> for you to fetch changes up to 049f6d8237dd0b14dee02e4c22b20114c43cecff:
>>>>
>>>> COLO-compare: Optimize tcp compare trace event (2017-04-24 11:30:36
>>>> +0800)
>>>>
>>>> ----------------------------------------------------------------
>>>>
>>>> ----------------------------------------------------------------
>>> Hi. Clang picks up what looks like a typo:
>>>
>>> /Users/pm215/src/qemu-for-merges/hw/net/ftgmac100.c:809:33: error: use
>>> of logical '&&' with constant operand
>>> [-Werror,-Wconstant-logical-operand]
>>> if (size < 64 && !(s->maccr && FTGMAC100_MACCR_RX_RUNT)) {
>>> ^ ~~~~~~~~~~~~~~~~~~~~~~~
>>> /Users/pm215/src/qemu-for-merges/hw/net/ftgmac100.c:809:33: note: use
>>> '&' for a bitwise operation
>>> if (size < 64 && !(s->maccr && FTGMAC100_MACCR_RX_RUNT)) {
>>> ^~
>>> &
>>> /Users/pm215/src/qemu-for-merges/hw/net/ftgmac100.c:809:33: note:
>>> remove constant to silence this warning
>>> if (size < 64 && !(s->maccr && FTGMAC100_MACCR_RX_RUNT)) {
>>> ~^~~~~~~~~~~~~~~~~~~~~~~~~~
>> Jason,
>>
>> How do you want to handle that ? A resend of the patch or a fix ?
>>
>> Thanks,
>>
>> C.
>>
>
> The fix looks trivial, let me fix it.
OK. Thanks,
C.
- [Qemu-devel] [PULL 2/8] hw/net: add MII definitions, (continued)
- [Qemu-devel] [PULL 2/8] hw/net: add MII definitions, Jason Wang, 2017/04/24
- [Qemu-devel] [PULL 4/8] net/ftgmac100: add a 'aspeed' property, Jason Wang, 2017/04/24
- [Qemu-devel] [PULL 5/8] aspeed: add a FTGMAC100 nic, Jason Wang, 2017/04/24
- [Qemu-devel] [PULL 3/8] net: add FTGMAC100 support, Jason Wang, 2017/04/24
- [Qemu-devel] [PULL 6/8] slirp: add a fake NC-SI backend, Jason Wang, 2017/04/24
- [Qemu-devel] [PULL 7/8] COLO-compare: Optimize tcp compare for option field, Jason Wang, 2017/04/24
- [Qemu-devel] [PULL 8/8] COLO-compare: Optimize tcp compare trace event, Jason Wang, 2017/04/24
- Re: [Qemu-devel] [PULL 0/8] Net patches, Peter Maydell, 2017/04/24