* [PATCH 01/13] block/stream: Remove redundant statement in stream_run()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 21:49 ` John Snow
2020-02-25 2:09 ` [PATCH 02/13] block/iscsi:Remove redundant statement in iscsi_open() kuhn.chenqun
` (11 subsequent siblings)
12 siblings, 1 reply; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: Kevin Wolf, peter.maydell, zhang.zhanghailiang, qemu-block,
Max Reitz, Chen Qun, John Snow
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
block/stream.c:186:9: warning: Value stored to 'ret' is never read
ret = 0;
^ ~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: John Snow <jsnow@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Max Reitz <mreitz@redhat.com>
Cc: qemu-block@nongnu.org
---
block/stream.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/block/stream.c b/block/stream.c
index 5562ccbf57..d78074ac80 100644
--- a/block/stream.c
+++ b/block/stream.c
@@ -183,7 +183,6 @@ static int coroutine_fn stream_run(Job *job, Error **errp)
break;
}
}
- ret = 0;
/* Publish progress */
job_progress_update(&s->common.job, n);
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH 01/13] block/stream: Remove redundant statement in stream_run()
2020-02-25 2:09 ` [PATCH 01/13] block/stream: Remove redundant statement in stream_run() kuhn.chenqun
@ 2020-02-25 21:49 ` John Snow
0 siblings, 0 replies; 22+ messages in thread
From: John Snow @ 2020-02-25 21:49 UTC (permalink / raw)
To: kuhn.chenqun, qemu-devel, qemu-trivial
Cc: Kevin Wolf, peter.maydell, zhang.zhanghailiang, qemu-block, Max Reitz
On 2/24/20 9:09 PM, kuhn.chenqun@huawei.com wrote:
> From: Chen Qun <kuhn.chenqun@huawei.com>
>
> Clang static code analyzer show warning:
> block/stream.c:186:9: warning: Value stored to 'ret' is never read
> ret = 0;
> ^ ~
> Reported-by: Euler Robot <euler.robot@huawei.com>
> Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
> ---
> Cc: John Snow <jsnow@redhat.com>
> Cc: Kevin Wolf <kwolf@redhat.com>
> Cc: Max Reitz <mreitz@redhat.com>
> Cc: qemu-block@nongnu.org
> ---
> block/stream.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/block/stream.c b/block/stream.c
> index 5562ccbf57..d78074ac80 100644
> --- a/block/stream.c
> +++ b/block/stream.c
> @@ -183,7 +183,6 @@ static int coroutine_fn stream_run(Job *job, Error **errp)
> break;
> }
> }
> - ret = 0;
>
> /* Publish progress */
> job_progress_update(&s->common.job, n);
>
Reviewed-by: John Snow <jsnow@redhat.com>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 02/13] block/iscsi:Remove redundant statement in iscsi_open()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
2020-02-25 2:09 ` [PATCH 01/13] block/stream: Remove redundant statement in stream_run() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 2:09 ` [PATCH 03/13] block/file-posix: Remove redundant statement in raw_handle_perm_lock() kuhn.chenqun
` (10 subsequent siblings)
12 siblings, 0 replies; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: Kevin Wolf, peter.maydell, zhang.zhanghailiang, qemu-block,
Peter Lieven, Max Reitz, Ronnie Sahlberg, Chen Qun,
Paolo Bonzini
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
block/iscsi.c:1920:9: warning: Value stored to 'flags' is never read
flags &= ~BDRV_O_RDWR;
^ ~~~~~~~~~~~~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Peter Lieven <pl@kamp.de>
Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Max Reitz <mreitz@redhat.com>
Cc: qemu-block@nongnu.org
---
block/iscsi.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/block/iscsi.c b/block/iscsi.c
index 682abd8e09..ed88479ede 100644
--- a/block/iscsi.c
+++ b/block/iscsi.c
@@ -1917,7 +1917,6 @@ static int iscsi_open(BlockDriverState *bs, QDict *options, int flags,
if (ret < 0) {
goto out;
}
- flags &= ~BDRV_O_RDWR;
}
iscsi_readcapacity_sync(iscsilun, &local_err);
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 03/13] block/file-posix: Remove redundant statement in raw_handle_perm_lock()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
2020-02-25 2:09 ` [PATCH 01/13] block/stream: Remove redundant statement in stream_run() kuhn.chenqun
2020-02-25 2:09 ` [PATCH 02/13] block/iscsi:Remove redundant statement in iscsi_open() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 2:09 ` [PATCH 04/13] scsi/esp-pci: Remove redundant statement in esp_pci_io_write() kuhn.chenqun
` (9 subsequent siblings)
12 siblings, 0 replies; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: Kevin Wolf, peter.maydell, zhang.zhanghailiang, qemu-block,
Max Reitz, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
block/file-posix.c:891:9: warning: Value stored to 'op' is never read
op = RAW_PL_ABORT;
^ ~~~~~~~~~~~~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Max Reitz <mreitz@redhat.com>
Cc: qemu-block@nongnu.org
---
block/file-posix.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/block/file-posix.c b/block/file-posix.c
index 6345477112..0f77447a25 100644
--- a/block/file-posix.c
+++ b/block/file-posix.c
@@ -888,7 +888,6 @@ static int raw_handle_perm_lock(BlockDriverState *bs,
"Is another process using the image [%s]?\n",
bs->filename);
}
- op = RAW_PL_ABORT;
/* fall through to unlock bytes. */
case RAW_PL_ABORT:
raw_apply_lock_bytes(s, s->fd, s->perm, ~s->shared_perm,
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 04/13] scsi/esp-pci: Remove redundant statement in esp_pci_io_write()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (2 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 03/13] block/file-posix: Remove redundant statement in raw_handle_perm_lock() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 2:09 ` [PATCH 05/13] scsi/scsi-disk: Remove redundant statement in scsi_disk_emulate_command() kuhn.chenqun
` (8 subsequent siblings)
12 siblings, 0 replies; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: Fam Zheng, peter.maydell, Paolo Bonzini, zhang.zhanghailiang, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
hw/scsi/esp-pci.c:198:9: warning: Value stored to 'size' is never read
size = 4;
^ ~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Fam Zheng <fam@euphon.net>
---
hw/scsi/esp-pci.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/scsi/esp-pci.c b/hw/scsi/esp-pci.c
index d5a1f9e017..2e6cc07d4e 100644
--- a/hw/scsi/esp-pci.c
+++ b/hw/scsi/esp-pci.c
@@ -195,7 +195,6 @@ static void esp_pci_io_write(void *opaque, hwaddr addr,
val <<= shift;
val |= current & ~(mask << shift);
addr &= ~3;
- size = 4;
}
if (addr < 0x40) {
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 05/13] scsi/scsi-disk: Remove redundant statement in scsi_disk_emulate_command()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (3 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 04/13] scsi/esp-pci: Remove redundant statement in esp_pci_io_write() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 2:09 ` [PATCH 06/13] display/pxa2xx_lcd: Remove redundant statement in pxa2xx_palette_parse() kuhn.chenqun
` (7 subsequent siblings)
12 siblings, 0 replies; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: Fam Zheng, peter.maydell, Paolo Bonzini, zhang.zhanghailiang, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
scsi/scsi-disk.c:1918:5: warning: Value stored to 'buflen' is never read
buflen = req->cmd.xfer;
^ ~~~~~~~~~~~~~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Fam Zheng <fam@euphon.net>
---
hw/scsi/scsi-disk.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
index 10d0794d60..1c0cb63a6f 100644
--- a/hw/scsi/scsi-disk.c
+++ b/hw/scsi/scsi-disk.c
@@ -1915,7 +1915,6 @@ static int32_t scsi_disk_emulate_command(SCSIRequest *req, uint8_t *buf)
r->iov.iov_base = blk_blockalign(s->qdev.conf.blk, r->buflen);
}
- buflen = req->cmd.xfer;
outbuf = r->iov.iov_base;
memset(outbuf, 0, r->buflen);
switch (req->cmd.buf[0]) {
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 06/13] display/pxa2xx_lcd: Remove redundant statement in pxa2xx_palette_parse()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (4 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 05/13] scsi/scsi-disk: Remove redundant statement in scsi_disk_emulate_command() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 2:09 ` [PATCH 07/13] display/exynos4210_fimd: Remove redundant statement in exynos4210_fimd_update() kuhn.chenqun
` (6 subsequent siblings)
12 siblings, 0 replies; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: peter.maydell, qemu-arm, zhang.zhanghailiang, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
hw/display/pxa2xx_lcd.c:596:9: warning: Value stored to 'format' is never read
format = 0;
^ ~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Andrzej Zaborowski <balrogg@gmail.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-arm@nongnu.org
---
hw/display/pxa2xx_lcd.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/display/pxa2xx_lcd.c b/hw/display/pxa2xx_lcd.c
index 05f5f84671..464e93161a 100644
--- a/hw/display/pxa2xx_lcd.c
+++ b/hw/display/pxa2xx_lcd.c
@@ -593,7 +593,6 @@ static void pxa2xx_palette_parse(PXA2xxLCDState *s, int ch, int bpp)
n = 256;
break;
default:
- format = 0;
return;
}
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 07/13] display/exynos4210_fimd: Remove redundant statement in exynos4210_fimd_update()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (5 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 06/13] display/pxa2xx_lcd: Remove redundant statement in pxa2xx_palette_parse() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 2:09 ` [PATCH 08/13] display/blizzard: Remove redundant statement in blizzard_draw_line16_32() kuhn.chenqun
` (5 subsequent siblings)
12 siblings, 0 replies; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: Igor Mitsyanko, peter.maydell, qemu-arm, zhang.zhanghailiang, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
hw/display/exynos4210_fimd.c:1313:17: warning: Value stored to 'is_dirty' is never read
is_dirty = false;
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Igor Mitsyanko <i.mitsyanko@gmail.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-arm@nongnu.org
---
hw/display/exynos4210_fimd.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/display/exynos4210_fimd.c b/hw/display/exynos4210_fimd.c
index c1071ecd46..05d3265b76 100644
--- a/hw/display/exynos4210_fimd.c
+++ b/hw/display/exynos4210_fimd.c
@@ -1310,7 +1310,6 @@ static void exynos4210_fimd_update(void *opaque)
}
host_fb_addr += inc_size;
fb_line_addr += inc_size;
- is_dirty = false;
}
g_free(snap);
blend = true;
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 08/13] display/blizzard: Remove redundant statement in blizzard_draw_line16_32()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (6 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 07/13] display/exynos4210_fimd: Remove redundant statement in exynos4210_fimd_update() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 2:09 ` [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in zdma_write_dst() kuhn.chenqun
` (4 subsequent siblings)
12 siblings, 0 replies; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: peter.maydell, qemu-arm, zhang.zhanghailiang, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
hw/display/blizzard.c:940:9: warning: Value stored to 'data' is never read
data >>= 5;
^ ~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Andrzej Zaborowski <balrogg@gmail.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-arm@nongnu.org
---
hw/display/blizzard.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/display/blizzard.c b/hw/display/blizzard.c
index 359e399c2a..62517bdf75 100644
--- a/hw/display/blizzard.c
+++ b/hw/display/blizzard.c
@@ -937,7 +937,6 @@ static void blizzard_draw_line16_32(uint32_t *dest,
g = (data & 0x3f) << 2;
data >>= 6;
r = (data & 0x1f) << 3;
- data >>= 5;
*dest++ = rgb_to_pixel32(r, g, b);
}
}
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in zdma_write_dst()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (7 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 08/13] display/blizzard: Remove redundant statement in blizzard_draw_line16_32() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 9:36 ` Philippe Mathieu-Daudé
2020-02-25 2:09 ` [PATCH 10/13] migration/vmstate: Remove redundant statement in vmstate_save_state_v() kuhn.chenqun
` (3 subsequent siblings)
12 siblings, 1 reply; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: peter.maydell, zhang.zhanghailiang, Alistair Francis, qemu-arm,
Chen Qun, Edgar E. Iglesias
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
hw/dma/xlnx-zdma.c:399:13: warning: Value stored to 'dst_type' is never read
dst_type = FIELD_EX32(s->dsc_dst.words[3], ZDMA_CH_DST_DSCR_WORD3,
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Alistair Francis <alistair@alistair23.me>
Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-arm@nongnu.org
---
hw/dma/xlnx-zdma.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/hw/dma/xlnx-zdma.c b/hw/dma/xlnx-zdma.c
index 8fb83f5b07..45355c5d59 100644
--- a/hw/dma/xlnx-zdma.c
+++ b/hw/dma/xlnx-zdma.c
@@ -396,8 +396,6 @@ static void zdma_write_dst(XlnxZDMA *s, uint8_t *buf, uint32_t len)
zdma_load_descriptor(s, next, &s->dsc_dst);
dst_size = FIELD_EX32(s->dsc_dst.words[2], ZDMA_CH_DST_DSCR_WORD2,
SIZE);
- dst_type = FIELD_EX32(s->dsc_dst.words[3], ZDMA_CH_DST_DSCR_WORD3,
- TYPE);
}
/* Match what hardware does by ignoring the dst_size and only using
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in zdma_write_dst()
2020-02-25 2:09 ` [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in zdma_write_dst() kuhn.chenqun
@ 2020-02-25 9:36 ` Philippe Mathieu-Daudé
2020-02-25 10:01 ` Chenqun (kuhn)
0 siblings, 1 reply; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-02-25 9:36 UTC (permalink / raw)
To: kuhn.chenqun, qemu-devel, qemu-trivial
Cc: peter.maydell, Alistair Francis, zhang.zhanghailiang, qemu-arm
On 2/25/20 3:09 AM, kuhn.chenqun@huawei.com wrote:
> From: Chen Qun <kuhn.chenqun@huawei.com>
>
> Clang static code analyzer show warning:
> hw/dma/xlnx-zdma.c:399:13: warning: Value stored to 'dst_type' is never read
> dst_type = FIELD_EX32(s->dsc_dst.words[3], ZDMA_CH_DST_DSCR_WORD3,
> ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> Reported-by: Euler Robot <euler.robot@huawei.com>
> Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
> ---
> Cc: Alistair Francis <alistair@alistair23.me>
> Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
> Cc: Peter Maydell <peter.maydell@linaro.org>
> Cc: qemu-arm@nongnu.org
> ---
> hw/dma/xlnx-zdma.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/hw/dma/xlnx-zdma.c b/hw/dma/xlnx-zdma.c
> index 8fb83f5b07..45355c5d59 100644
> --- a/hw/dma/xlnx-zdma.c
> +++ b/hw/dma/xlnx-zdma.c
> @@ -396,8 +396,6 @@ static void zdma_write_dst(XlnxZDMA *s, uint8_t *buf, uint32_t len)
> zdma_load_descriptor(s, next, &s->dsc_dst);
> dst_size = FIELD_EX32(s->dsc_dst.words[2], ZDMA_CH_DST_DSCR_WORD2,
> SIZE);
> - dst_type = FIELD_EX32(s->dsc_dst.words[3], ZDMA_CH_DST_DSCR_WORD3,
> - TYPE);
Maybe move dst_type to this if() statement now?
> }
>
> /* Match what hardware does by ignoring the dst_size and only using
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* RE: [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in zdma_write_dst()
2020-02-25 9:36 ` Philippe Mathieu-Daudé
@ 2020-02-25 10:01 ` Chenqun (kuhn)
2020-02-25 10:11 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 22+ messages in thread
From: Chenqun (kuhn) @ 2020-02-25 10:01 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel, qemu-trivial
Cc: peter.maydell, Alistair Francis, Zhanghailiang, qemu-arm
>-----Original Message-----
>From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com]
>Sent: Tuesday, February 25, 2020 5:36 PM
>To: Chenqun (kuhn) <kuhn.chenqun@huawei.com>; qemu-
>devel@nongnu.org; qemu-trivial@nongnu.org
>Cc: peter.maydell@linaro.org; Zhanghailiang
><zhang.zhanghailiang@huawei.com>; Alistair Francis <alistair@alistair23.me>;
>qemu-arm@nongnu.org
>Subject: Re: [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in
>zdma_write_dst()
>
>On 2/25/20 3:09 AM, kuhn.chenqun@huawei.com wrote:
>> From: Chen Qun <kuhn.chenqun@huawei.com>
>>
>> Clang static code analyzer show warning:
>> hw/dma/xlnx-zdma.c:399:13: warning: Value stored to 'dst_type' is never
>read
>> dst_type = FIELD_EX32(s->dsc_dst.words[3],
>ZDMA_CH_DST_DSCR_WORD3,
>> ^
>~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>
>> Reported-by: Euler Robot <euler.robot@huawei.com>
>> Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
>> ---
>> Cc: Alistair Francis <alistair@alistair23.me>
>> Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
>> Cc: Peter Maydell <peter.maydell@linaro.org>
>> Cc: qemu-arm@nongnu.org
>> ---
>> hw/dma/xlnx-zdma.c | 2 --
>> 1 file changed, 2 deletions(-)
>>
>> diff --git a/hw/dma/xlnx-zdma.c b/hw/dma/xlnx-zdma.c index
>> 8fb83f5b07..45355c5d59 100644
>> --- a/hw/dma/xlnx-zdma.c
>> +++ b/hw/dma/xlnx-zdma.c
>> @@ -396,8 +396,6 @@ static void zdma_write_dst(XlnxZDMA *s, uint8_t
>*buf, uint32_t len)
>> zdma_load_descriptor(s, next, &s->dsc_dst);
>> dst_size = FIELD_EX32(s->dsc_dst.words[2],
>ZDMA_CH_DST_DSCR_WORD2,
>> SIZE);
>> - dst_type = FIELD_EX32(s->dsc_dst.words[3],
>ZDMA_CH_DST_DSCR_WORD3,
>> - TYPE);
>
>Maybe move dst_type to this if() statement now?
>
Sorry, I don't follow you. I didn't find where I could move dst_type.
Do you mean to move the first dst_type to the if().
Modify it like this:
while (len) {
dst_size = FIELD_EX32(s->dsc_dst.words[2], ZDMA_CH_DST_DSCR_WORD2,
SIZE);
if (dst_size == 0 && ptype == PT_MEM) {
uint64_t next;
dst_type = FIELD_EX32(s->dsc_dst.words[3], ZDMA_CH_DST_DSCR_WORD3,
TYPE);
next = zdma_update_descr_addr(s, dst_type,
R_ZDMA_CH_DST_CUR_DSCR_LSB);
zdma_load_descriptor(s, next, &s->dsc_dst);
dst_size = FIELD_EX32(s->dsc_dst.words[2], ZDMA_CH_DST_DSCR_WORD2,
SIZE);
}
...
}
Thanks.
>> }
>>
>> /* Match what hardware does by ignoring the dst_size and
>> only using
>>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in zdma_write_dst()
2020-02-25 10:01 ` Chenqun (kuhn)
@ 2020-02-25 10:11 ` Philippe Mathieu-Daudé
2020-02-25 10:25 ` Chenqun (kuhn)
0 siblings, 1 reply; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-02-25 10:11 UTC (permalink / raw)
To: Chenqun (kuhn), qemu-devel, qemu-trivial
Cc: peter.maydell, Alistair Francis, Zhanghailiang, qemu-arm
On 2/25/20 11:01 AM, Chenqun (kuhn) wrote:
>> -----Original Message-----
>> From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com]
>> Sent: Tuesday, February 25, 2020 5:36 PM
>> To: Chenqun (kuhn) <kuhn.chenqun@huawei.com>; qemu-
>> devel@nongnu.org; qemu-trivial@nongnu.org
>> Cc: peter.maydell@linaro.org; Zhanghailiang
>> <zhang.zhanghailiang@huawei.com>; Alistair Francis <alistair@alistair23.me>;
>> qemu-arm@nongnu.org
>> Subject: Re: [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in
>> zdma_write_dst()
>>
>> On 2/25/20 3:09 AM, kuhn.chenqun@huawei.com wrote:
>>> From: Chen Qun <kuhn.chenqun@huawei.com>
>>>
>>> Clang static code analyzer show warning:
>>> hw/dma/xlnx-zdma.c:399:13: warning: Value stored to 'dst_type' is never
>> read
>>> dst_type = FIELD_EX32(s->dsc_dst.words[3],
>> ZDMA_CH_DST_DSCR_WORD3,
>>> ^
>> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>>
>>> Reported-by: Euler Robot <euler.robot@huawei.com>
>>> Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
>>> ---
>>> Cc: Alistair Francis <alistair@alistair23.me>
>>> Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
>>> Cc: Peter Maydell <peter.maydell@linaro.org>
>>> Cc: qemu-arm@nongnu.org
>>> ---
>>> hw/dma/xlnx-zdma.c | 2 --
>>> 1 file changed, 2 deletions(-)
>>>
>>> diff --git a/hw/dma/xlnx-zdma.c b/hw/dma/xlnx-zdma.c index
>>> 8fb83f5b07..45355c5d59 100644
>>> --- a/hw/dma/xlnx-zdma.c
>>> +++ b/hw/dma/xlnx-zdma.c
>>> @@ -396,8 +396,6 @@ static void zdma_write_dst(XlnxZDMA *s, uint8_t
>> *buf, uint32_t len)
>>> zdma_load_descriptor(s, next, &s->dsc_dst);
>>> dst_size = FIELD_EX32(s->dsc_dst.words[2],
>> ZDMA_CH_DST_DSCR_WORD2,
>>> SIZE);
>>> - dst_type = FIELD_EX32(s->dsc_dst.words[3],
>> ZDMA_CH_DST_DSCR_WORD3,
>>> - TYPE);
>>
>> Maybe move dst_type to this if() statement now?
>>
> Sorry, I don't follow you. I didn't find where I could move dst_type.
> Do you mean to move the first dst_type to the if().
> Modify it like this:
> while (len) {
> dst_size = FIELD_EX32(s->dsc_dst.words[2], ZDMA_CH_DST_DSCR_WORD2,
> SIZE);
> if (dst_size == 0 && ptype == PT_MEM) {
> uint64_t next;
> dst_type = FIELD_EX32(s->dsc_dst.words[3], ZDMA_CH_DST_DSCR_WORD3,
> TYPE);
> next = zdma_update_descr_addr(s, dst_type,
> R_ZDMA_CH_DST_CUR_DSCR_LSB);
> zdma_load_descriptor(s, next, &s->dsc_dst);
> dst_size = FIELD_EX32(s->dsc_dst.words[2], ZDMA_CH_DST_DSCR_WORD2,
> SIZE);
> }
> ...
> }
No, like this:
-- >8 --
@@ -373,7 +373,7 @@ static uint64_t zdma_update_descr_addr(XlnxZDMA *s,
bool type,
static void zdma_write_dst(XlnxZDMA *s, uint8_t *buf, uint32_t len)
{
uint32_t dst_size, dlen;
- bool dst_intr, dst_type;
+ bool dst_intr;
unsigned int ptype = ARRAY_FIELD_EX32(s->regs, ZDMA_CH_CTRL0,
POINT_TYPE);
unsigned int rw_mode = ARRAY_FIELD_EX32(s->regs, ZDMA_CH_CTRL0, MODE);
unsigned int burst_type = ARRAY_FIELD_EX32(s->regs, ZDMA_CH_DATA_ATTR,
@@ -387,17 +387,17 @@ static void zdma_write_dst(XlnxZDMA *s, uint8_t
*buf, uint32_t len)
while (len) {
dst_size = FIELD_EX32(s->dsc_dst.words[2], ZDMA_CH_DST_DSCR_WORD2,
SIZE);
- dst_type = FIELD_EX32(s->dsc_dst.words[3], ZDMA_CH_DST_DSCR_WORD3,
- TYPE);
if (dst_size == 0 && ptype == PT_MEM) {
uint64_t next;
+ bool dst_type;
+
+ dst_type = FIELD_EX32(s->dsc_dst.words[3],
ZDMA_CH_DST_DSCR_WORD3,
+ TYPE);
next = zdma_update_descr_addr(s, dst_type,
R_ZDMA_CH_DST_CUR_DSCR_LSB);
zdma_load_descriptor(s, next, &s->dsc_dst);
dst_size = FIELD_EX32(s->dsc_dst.words[2],
ZDMA_CH_DST_DSCR_WORD2,
SIZE);
- dst_type = FIELD_EX32(s->dsc_dst.words[3],
ZDMA_CH_DST_DSCR_WORD3,
- TYPE);
}
---
^ permalink raw reply [flat|nested] 22+ messages in thread
* RE: [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in zdma_write_dst()
2020-02-25 10:11 ` Philippe Mathieu-Daudé
@ 2020-02-25 10:25 ` Chenqun (kuhn)
0 siblings, 0 replies; 22+ messages in thread
From: Chenqun (kuhn) @ 2020-02-25 10:25 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel, qemu-trivial
Cc: peter.maydell, Alistair Francis, Zhanghailiang, qemu-arm
>-----Original Message-----
>From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com]
>Sent: Tuesday, February 25, 2020 6:12 PM
>To: Chenqun (kuhn) <kuhn.chenqun@huawei.com>; qemu-
>devel@nongnu.org; qemu-trivial@nongnu.org
>Cc: peter.maydell@linaro.org; Zhanghailiang
><zhang.zhanghailiang@huawei.com>; Alistair Francis <alistair@alistair23.me>;
>qemu-arm@nongnu.org
>Subject: Re: [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in
>zdma_write_dst()
>
>On 2/25/20 11:01 AM, Chenqun (kuhn) wrote:
>>> -----Original Message-----
>>> From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com]
>>> Sent: Tuesday, February 25, 2020 5:36 PM
>>> To: Chenqun (kuhn) <kuhn.chenqun@huawei.com>; qemu-
>devel@nongnu.org;
>>> qemu-trivial@nongnu.org
>>> Cc: peter.maydell@linaro.org; Zhanghailiang
>>> <zhang.zhanghailiang@huawei.com>; Alistair Francis
>>> <alistair@alistair23.me>; qemu-arm@nongnu.org
>>> Subject: Re: [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement
>>> in
>>> zdma_write_dst()
>>>
>>> On 2/25/20 3:09 AM, kuhn.chenqun@huawei.com wrote:
>>>> From: Chen Qun <kuhn.chenqun@huawei.com>
>>>>
>>>> Clang static code analyzer show warning:
>>>> hw/dma/xlnx-zdma.c:399:13: warning: Value stored to 'dst_type' is
>>>> never
>>> read
>>>> dst_type = FIELD_EX32(s->dsc_dst.words[3],
>>> ZDMA_CH_DST_DSCR_WORD3,
>>>> ^
>>> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>>>
>>>> Reported-by: Euler Robot <euler.robot@huawei.com>
>>>> Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
>>>> ---
>>>> Cc: Alistair Francis <alistair@alistair23.me>
>>>> Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
>>>> Cc: Peter Maydell <peter.maydell@linaro.org>
>>>> Cc: qemu-arm@nongnu.org
>>>> ---
>>>> hw/dma/xlnx-zdma.c | 2 --
>>>> 1 file changed, 2 deletions(-)
>>>>
>>>> diff --git a/hw/dma/xlnx-zdma.c b/hw/dma/xlnx-zdma.c index
>>>> 8fb83f5b07..45355c5d59 100644
>>>> --- a/hw/dma/xlnx-zdma.c
>>>> +++ b/hw/dma/xlnx-zdma.c
>>>> @@ -396,8 +396,6 @@ static void zdma_write_dst(XlnxZDMA *s, uint8_t
>>> *buf, uint32_t len)
>>>> zdma_load_descriptor(s, next, &s->dsc_dst);
>>>> dst_size = FIELD_EX32(s->dsc_dst.words[2],
>>> ZDMA_CH_DST_DSCR_WORD2,
>>>> SIZE);
>>>> - dst_type = FIELD_EX32(s->dsc_dst.words[3],
>>> ZDMA_CH_DST_DSCR_WORD3,
>>>> - TYPE);
>>>
>>> Maybe move dst_type to this if() statement now?
>>>
>> Sorry, I don't follow you. I didn't find where I could move dst_type.
>> Do you mean to move the first dst_type to the if().
>> Modify it like this:
>> while (len) {
>> dst_size = FIELD_EX32(s->dsc_dst.words[2],
>ZDMA_CH_DST_DSCR_WORD2,
>> SIZE);
>> if (dst_size == 0 && ptype == PT_MEM) {
>> uint64_t next;
>> dst_type = FIELD_EX32(s->dsc_dst.words[3],
>ZDMA_CH_DST_DSCR_WORD3,
>> TYPE);
>> next = zdma_update_descr_addr(s, dst_type,
>> R_ZDMA_CH_DST_CUR_DSCR_LSB);
>> zdma_load_descriptor(s, next, &s->dsc_dst);
>> dst_size = FIELD_EX32(s->dsc_dst.words[2],
>ZDMA_CH_DST_DSCR_WORD2,
>> SIZE);
>> }
>> ...
>> }
>
>No, like this:
>
>-- >8 --
>@@ -373,7 +373,7 @@ static uint64_t zdma_update_descr_addr(XlnxZDMA
>*s, bool type,
> static void zdma_write_dst(XlnxZDMA *s, uint8_t *buf, uint32_t len)
> {
> uint32_t dst_size, dlen;
>- bool dst_intr, dst_type;
>+ bool dst_intr;
> unsigned int ptype = ARRAY_FIELD_EX32(s->regs, ZDMA_CH_CTRL0,
>POINT_TYPE);
> unsigned int rw_mode = ARRAY_FIELD_EX32(s->regs, ZDMA_CH_CTRL0,
>MODE);
> unsigned int burst_type = ARRAY_FIELD_EX32(s->regs,
>ZDMA_CH_DATA_ATTR, @@ -387,17 +387,17 @@ static void
>zdma_write_dst(XlnxZDMA *s, uint8_t *buf, uint32_t len)
> while (len) {
> dst_size = FIELD_EX32(s->dsc_dst.words[2],
>ZDMA_CH_DST_DSCR_WORD2,
> SIZE);
>- dst_type = FIELD_EX32(s->dsc_dst.words[3],
>ZDMA_CH_DST_DSCR_WORD3,
>- TYPE);
> if (dst_size == 0 && ptype == PT_MEM) {
> uint64_t next;
>+ bool dst_type;
>+
>+ dst_type = FIELD_EX32(s->dsc_dst.words[3],
>ZDMA_CH_DST_DSCR_WORD3,
>+ TYPE);
> next = zdma_update_descr_addr(s, dst_type,
> R_ZDMA_CH_DST_CUR_DSCR_LSB);
> zdma_load_descriptor(s, next, &s->dsc_dst);
> dst_size = FIELD_EX32(s->dsc_dst.words[2],
>ZDMA_CH_DST_DSCR_WORD2,
> SIZE);
>- dst_type = FIELD_EX32(s->dsc_dst.words[3],
>ZDMA_CH_DST_DSCR_WORD3,
>- TYPE);
> }
Hmm, this is better.
I will modify it later in V2.
Thanks.
>---
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 10/13] migration/vmstate: Remove redundant statement in vmstate_save_state_v()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (8 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 09/13] dma/xlnx-zdma: Remove redundant statement in zdma_write_dst() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 2:09 ` [PATCH 11/13] timer/exynos4210_mct: Remove redundant statement in exynos4210_mct_write() kuhn.chenqun
` (2 subsequent siblings)
12 siblings, 0 replies; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: peter.maydell, Juan Quintela, zhang.zhanghailiang,
Dr. David Alan Gilbert, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
The "ret" has been assigned in all branches. It didn't need to be
assigned separately.
Clang static code analyzer show warning:
migration/vmstate.c:365:17: warning: Value stored to 'ret' is never read
ret = 0;
^ ~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Juan Quintela <quintela@redhat.com>
Cc: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
---
migration/vmstate.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/migration/vmstate.c b/migration/vmstate.c
index 7dd8ef66c6..bafa890384 100644
--- a/migration/vmstate.c
+++ b/migration/vmstate.c
@@ -362,7 +362,6 @@ int vmstate_save_state_v(QEMUFile *f, const VMStateDescription *vmsd,
}
for (i = 0; i < n_elems; i++) {
void *curr_elem = first_elem + size * i;
- ret = 0;
vmsd_desc_field_start(vmsd, vmdesc_loop, field, i, n_elems);
old_offset = qemu_ftell_fast(f);
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 11/13] timer/exynos4210_mct: Remove redundant statement in exynos4210_mct_write()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (9 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 10/13] migration/vmstate: Remove redundant statement in vmstate_save_state_v() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 2:09 ` [PATCH 12/13] usb/hcd-ehci: Remove redundant statements kuhn.chenqun
2020-02-25 2:09 ` [PATCH 13/13] monitor/hmp-cmds: Remove redundant statement in hmp_rocker_of_dpa_groups() kuhn.chenqun
12 siblings, 0 replies; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: Igor Mitsyanko, peter.maydell, qemu-arm, zhang.zhanghailiang, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
hw/timer/exynos4210_mct.c:1370:9: warning: Value stored to 'index' is never read
index = GET_L_TIMER_CNT_REG_IDX(offset, lt_i);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
hw/timer/exynos4210_mct.c:1399:9: warning: Value stored to 'index' is never read
index = GET_L_TIMER_CNT_REG_IDX(offset, lt_i);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
hw/timer/exynos4210_mct.c:1441:9: warning: Value stored to 'index' is never read
index = GET_L_TIMER_CNT_REG_IDX(offset, lt_i);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Igor Mitsyanko <i.mitsyanko@gmail.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-arm@nongnu.org
---
hw/timer/exynos4210_mct.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/hw/timer/exynos4210_mct.c b/hw/timer/exynos4210_mct.c
index 944120aea5..570cf7075b 100644
--- a/hw/timer/exynos4210_mct.c
+++ b/hw/timer/exynos4210_mct.c
@@ -1367,7 +1367,6 @@ static void exynos4210_mct_write(void *opaque, hwaddr offset,
case L0_TCNTB: case L1_TCNTB:
lt_i = GET_L_TIMER_IDX(offset);
- index = GET_L_TIMER_CNT_REG_IDX(offset, lt_i);
/*
* TCNTB is updated to internal register only after CNT expired.
@@ -1396,7 +1395,6 @@ static void exynos4210_mct_write(void *opaque, hwaddr offset,
case L0_ICNTB: case L1_ICNTB:
lt_i = GET_L_TIMER_IDX(offset);
- index = GET_L_TIMER_CNT_REG_IDX(offset, lt_i);
s->l_timer[lt_i].reg.wstat |= L_WSTAT_ICNTB_WRITE;
s->l_timer[lt_i].reg.cnt[L_REG_CNT_ICNTB] = value &
@@ -1438,8 +1436,6 @@ static void exynos4210_mct_write(void *opaque, hwaddr offset,
case L0_FRCNTB: case L1_FRCNTB:
lt_i = GET_L_TIMER_IDX(offset);
- index = GET_L_TIMER_CNT_REG_IDX(offset, lt_i);
-
DPRINTF("local timer[%d] FRCNTB write %llx\n", lt_i, value);
s->l_timer[lt_i].reg.wstat |= L_WSTAT_FRCCNTB_WRITE;
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 12/13] usb/hcd-ehci: Remove redundant statements
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (10 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 11/13] timer/exynos4210_mct: Remove redundant statement in exynos4210_mct_write() kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 9:38 ` Philippe Mathieu-Daudé
2020-02-25 2:09 ` [PATCH 13/13] monitor/hmp-cmds: Remove redundant statement in hmp_rocker_of_dpa_groups() kuhn.chenqun
12 siblings, 1 reply; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: peter.maydell, Gerd Hoffmann, zhang.zhanghailiang, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
The "again" assignment is meaningless before g_assert_not_reached.
In addition, the break statements no longer needs to be after
g_assert_not_reached.
Clang static code analyzer show warning:
hw/usb/hcd-ehci.c:2108:13: warning: Value stored to 'again' is never read
again = -1;
^ ~~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: Gerd Hoffmann <kraxel@redhat.com>
---
hw/usb/hcd-ehci.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c
index 56ab2f457f..29d49c2d7e 100644
--- a/hw/usb/hcd-ehci.c
+++ b/hw/usb/hcd-ehci.c
@@ -1301,7 +1301,6 @@ static void ehci_execute_complete(EHCIQueue *q)
/* should not be triggerable */
fprintf(stderr, "USB invalid response %d\n", p->packet.status);
g_assert_not_reached();
- break;
}
/* TODO check 4.12 for splits */
@@ -2105,9 +2104,7 @@ static void ehci_advance_state(EHCIState *ehci, int async)
default:
fprintf(stderr, "Bad state!\n");
- again = -1;
g_assert_not_reached();
- break;
}
if (again < 0 || itd_count > 16) {
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH 12/13] usb/hcd-ehci: Remove redundant statements
2020-02-25 2:09 ` [PATCH 12/13] usb/hcd-ehci: Remove redundant statements kuhn.chenqun
@ 2020-02-25 9:38 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-02-25 9:38 UTC (permalink / raw)
To: kuhn.chenqun, qemu-devel, qemu-trivial
Cc: peter.maydell, Gerd Hoffmann, zhang.zhanghailiang
On 2/25/20 3:09 AM, kuhn.chenqun@huawei.com wrote:
> From: Chen Qun <kuhn.chenqun@huawei.com>
>
> The "again" assignment is meaningless before g_assert_not_reached.
> In addition, the break statements no longer needs to be after
> g_assert_not_reached.
>
> Clang static code analyzer show warning:
> hw/usb/hcd-ehci.c:2108:13: warning: Value stored to 'again' is never read
> again = -1;
> ^ ~~
>
> Reported-by: Euler Robot <euler.robot@huawei.com>
> Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
> ---
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> ---
> hw/usb/hcd-ehci.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c
> index 56ab2f457f..29d49c2d7e 100644
> --- a/hw/usb/hcd-ehci.c
> +++ b/hw/usb/hcd-ehci.c
> @@ -1301,7 +1301,6 @@ static void ehci_execute_complete(EHCIQueue *q)
> /* should not be triggerable */
> fprintf(stderr, "USB invalid response %d\n", p->packet.status);
> g_assert_not_reached();
> - break;
> }
>
> /* TODO check 4.12 for splits */
> @@ -2105,9 +2104,7 @@ static void ehci_advance_state(EHCIState *ehci, int async)
>
> default:
> fprintf(stderr, "Bad state!\n");
> - again = -1;
> g_assert_not_reached();
> - break;
> }
>
> if (again < 0 || itd_count > 16) {
>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 13/13] monitor/hmp-cmds: Remove redundant statement in hmp_rocker_of_dpa_groups()
2020-02-25 2:09 [PATCH 00/13]redundant code: Fix warnings reported by Clang static code analyzer kuhn.chenqun
` (11 preceding siblings ...)
2020-02-25 2:09 ` [PATCH 12/13] usb/hcd-ehci: Remove redundant statements kuhn.chenqun
@ 2020-02-25 2:09 ` kuhn.chenqun
2020-02-25 9:44 ` Philippe Mathieu-Daudé
12 siblings, 1 reply; 22+ messages in thread
From: kuhn.chenqun @ 2020-02-25 2:09 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: peter.maydell, zhang.zhanghailiang, Dr. David Alan Gilbert, Chen Qun
From: Chen Qun <kuhn.chenqun@huawei.com>
Clang static code analyzer show warning:
monitor/hmp-cmds.c:2867:17: warning: Value stored to 'set' is never read
set = true;
^ ~~~~
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
Cc: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
---
monitor/hmp-cmds.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/monitor/hmp-cmds.c b/monitor/hmp-cmds.c
index 53bc3f76c4..84f94647cd 100644
--- a/monitor/hmp-cmds.c
+++ b/monitor/hmp-cmds.c
@@ -2864,7 +2864,6 @@ void hmp_rocker_of_dpa_groups(Monitor *mon, const QDict *qdict)
if (group->has_set_eth_dst) {
if (!set) {
- set = true;
monitor_printf(mon, " set");
}
monitor_printf(mon, " dst %s", group->set_eth_dst);
--
2.23.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH 13/13] monitor/hmp-cmds: Remove redundant statement in hmp_rocker_of_dpa_groups()
2020-02-25 2:09 ` [PATCH 13/13] monitor/hmp-cmds: Remove redundant statement in hmp_rocker_of_dpa_groups() kuhn.chenqun
@ 2020-02-25 9:44 ` Philippe Mathieu-Daudé
2020-02-25 10:12 ` Chenqun (kuhn)
0 siblings, 1 reply; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-02-25 9:44 UTC (permalink / raw)
To: kuhn.chenqun, qemu-devel, qemu-trivial
Cc: peter.maydell, zhang.zhanghailiang, Dr. David Alan Gilbert
On 2/25/20 3:09 AM, kuhn.chenqun@huawei.com wrote:
> From: Chen Qun <kuhn.chenqun@huawei.com>
>
> Clang static code analyzer show warning:
> monitor/hmp-cmds.c:2867:17: warning: Value stored to 'set' is never read
> set = true;
> ^ ~~~~
>
> Reported-by: Euler Robot <euler.robot@huawei.com>
> Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
> ---
> Cc: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> ---
> monitor/hmp-cmds.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/monitor/hmp-cmds.c b/monitor/hmp-cmds.c
> index 53bc3f76c4..84f94647cd 100644
> --- a/monitor/hmp-cmds.c
> +++ b/monitor/hmp-cmds.c
> @@ -2864,7 +2864,6 @@ void hmp_rocker_of_dpa_groups(Monitor *mon, const QDict *qdict)
>
> if (group->has_set_eth_dst) {
> if (!set) {
> - set = true;
> monitor_printf(mon, " set");
> }
> monitor_printf(mon, " dst %s", group->set_eth_dst);
>
Can you move the 'set' declaration to the for() statement and also
remove the last "set = false;"?
^ permalink raw reply [flat|nested] 22+ messages in thread
* RE: [PATCH 13/13] monitor/hmp-cmds: Remove redundant statement in hmp_rocker_of_dpa_groups()
2020-02-25 9:44 ` Philippe Mathieu-Daudé
@ 2020-02-25 10:12 ` Chenqun (kuhn)
0 siblings, 0 replies; 22+ messages in thread
From: Chenqun (kuhn) @ 2020-02-25 10:12 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel, qemu-trivial
Cc: peter.maydell, Zhanghailiang, Dr. David Alan Gilbert
>-----Original Message-----
>From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com]
>Sent: Tuesday, February 25, 2020 5:45 PM
>To: Chenqun (kuhn) <kuhn.chenqun@huawei.com>; qemu-
>devel@nongnu.org; qemu-trivial@nongnu.org
>Cc: peter.maydell@linaro.org; Zhanghailiang
><zhang.zhanghailiang@huawei.com>; Dr. David Alan Gilbert
><dgilbert@redhat.com>
>Subject: Re: [PATCH 13/13] monitor/hmp-cmds: Remove redundant
>statement in hmp_rocker_of_dpa_groups()
>
>On 2/25/20 3:09 AM, kuhn.chenqun@huawei.com wrote:
>> From: Chen Qun <kuhn.chenqun@huawei.com>
>>
>> Clang static code analyzer show warning:
>> monitor/hmp-cmds.c:2867:17: warning: Value stored to 'set' is never read
>> set = true;
>> ^ ~~~~
>>
>> Reported-by: Euler Robot <euler.robot@huawei.com>
>> Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
>> ---
>> Cc: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>> ---
>> monitor/hmp-cmds.c | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/monitor/hmp-cmds.c b/monitor/hmp-cmds.c index
>> 53bc3f76c4..84f94647cd 100644
>> --- a/monitor/hmp-cmds.c
>> +++ b/monitor/hmp-cmds.c
>> @@ -2864,7 +2864,6 @@ void hmp_rocker_of_dpa_groups(Monitor *mon,
>> const QDict *qdict)
>>
>> if (group->has_set_eth_dst) {
>> if (!set) {
>> - set = true;
>> monitor_printf(mon, " set");
>> }
>> monitor_printf(mon, " dst %s", group->set_eth_dst);
>>
>
>Can you move the 'set' declaration to the for() statement and also remove the
>last "set = false;"?
Yes, you are right! It will be better! I will modify it later in V2.
Thanks.
^ permalink raw reply [flat|nested] 22+ messages in thread