[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH-for-5.0 v2 00/11] misc: Trivial static code analyzer fixes
From: |
Chenqun (kuhn) |
Subject: |
RE: [PATCH-for-5.0 v2 00/11] misc: Trivial static code analyzer fixes |
Date: |
Tue, 24 Mar 2020 07:33:46 +0000 |
>-----Original Message-----
>From: Qemu-devel [mailto:qemu-devel-
>bounces+kuhn.chenqun=address@hidden] On Behalf Of Laurent Vivier
>Sent: Monday, March 23, 2020 10:56 PM
>To: Philippe Mathieu-Daudé <address@hidden>; address@hidden
>Cc: Fam Zheng <address@hidden>; Peter Maydell <address@hidden>;
>Michael S. Tsirkin <address@hidden>; Mark Cave-Ayland <mark.cave-
>address@hidden>; address@hidden; address@hidden;
>Markus Armbruster <address@hidden>; Hervé Poussineau
><address@hidden>; Joel Stanley <address@hidden>; Michael Tokarev
><address@hidden>; Alistair Francis <address@hidden>; qemu-
>address@hidden; Cédric Le Goater <address@hidden>; John Snow
><address@hidden>; David Gibson <address@hidden>; Kevin Wolf
><address@hidden>; Andrew Jeffery <address@hidden>; Max Reitz
><address@hidden>; Igor Mitsyanko <address@hidden>; qemu-
>address@hidden; Paolo Bonzini <address@hidden>
>Subject: Re: [PATCH-for-5.0 v2 00/11] misc: Trivial static code analyzer fixes
>
>Le 23/03/2020 à 15:45, Philippe Mathieu-Daudé a écrit :
>> On 3/23/20 3:32 PM, Laurent Vivier wrote:
>>> Le 21/03/2020 à 15:40, Philippe Mathieu-Daudé a écrit :
>>>> Fix trivial warnings reported by the Clang static code analyzer.
>>>>
>>>> Since v1:
>>>> - Addressed Markus/Zoltan/Aleksandar review comments
>>>>
>>>> Philippe Mathieu-Daudé (11):
>>>> block: Avoid dead assignment
>>>> blockdev: Remove dead assignment
>>>> hw/i2c/pm_smbus: Remove dead assignment
>>>> hw/input/adb-kbd: Remove dead assignment
>>>> hw/ide/sii3112: Remove dead assignment
>>>> hw/isa/i82378: Remove dead assignment
>>>> hw/gpio/aspeed_gpio: Remove dead assignment
>>>> hw/timer/exynos4210_mct: Remove dead assignments
Same as this:
https://patchwork.kernel.org/patch/11415527/
>>>> hw/timer/stm32f2xx_timer: Remove dead assignment
>>>> hw/timer/pxa2xx_timer: Add assertion to silent static analyzer
>>>> warning
>>>> hw/scsi/esp-pci: Remove dead assignment
Same as this:
https://patchwork.kernel.org/patch/11415529/
>>>>
>>>> block.c | 2 +-
>>>> blockdev.c | 2 +-
>>>> hw/gpio/aspeed_gpio.c | 2 +-
>>>> hw/i2c/pm_smbus.c | 1 -
>>>> hw/ide/sii3112.c | 5 +++--
>>>> hw/input/adb-kbd.c | 6 +-----
>>>> hw/isa/i82378.c | 8 ++++----
>>>> hw/scsi/esp-pci.c | 1 -
>>>> hw/timer/exynos4210_mct.c | 3 ---
>>>> hw/timer/pxa2xx_timer.c | 1 +
>>>> hw/timer/stm32f2xx_timer.c | 1 -
>>>> 11 files changed, 12 insertions(+), 20 deletions(-)
>>>>
>>>
>>> I think your series covers cases already covered by:
>>>
>>> [PATCH v3 00/12] redundant code: Fix warnings reported by Clang
>>> static code analyzer
>>> https://patchew.org/QEMU/20200302130715.29440-1-kuhn.ch
>>
>> Unfortunately [for me...] I don't have v3 in my INBOX... *sigh* This
>> was 3 weeks ago. *sigh*.
>>
>> I can see the series in the archives:
>> https://lists.gnu.org/archive/html/qemu-devel/2020-03/msg00219.html
>> But I can't find the outcome, was it queued in the trivial tree?
>> Any idea when this will be merged in the master tree?
>
>Some patches are already merged via trivial (1, 2 (should go by SCSI
>queue) 3, 5, 6, 7, 9, 11 (by USB queue), 12).
>
>But others needed R-b tags or new version. I didn't check which of your patches
>are already covered by this series.
>
Hi, Laurent and Philippe
Yes, there are currently three patches( 4、8 、10) that are not merged.
And, only patch4 and patch10 have the same points as this series.
I should update it again earlier.
Thanks.
- Re: [PATCH-for-5.0 v2 09/11] hw/timer/stm32f2xx_timer: Remove dead assignment, (continued)