[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 000/100] Block layer patches
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PULL 000/100] Block layer patches |
Date: |
Mon, 10 Jul 2017 13:16:49 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 10.07.2017 um 11:28 hat Peter Maydell geschrieben:
> On 7 July 2017 at 18:07, Kevin Wolf <address@hidden> wrote:
> > The following changes since commit b11365867568ba954de667a0bfe0945b8f78d6bd:
> >
> > Merge remote-tracking branch 'remotes/borntraeger/tags/s390x-20170706'
> > into staging (2017-07-06 11:42:59 +0100)
> >
> > are available in the git repository at:
> >
> >
> > git://repo.or.cz/qemu/kevin.git tags/for-upstream
> >
> > for you to fetch changes up to 7d982302db173616d011e07ee43be3b0aae872b1:
> >
> > Merge remote-tracking branch 'mreitz/tags/pull-block-2017-07-07' into
> > queue-block (2017-07-07 18:11:41 +0200)
> >
> > ----------------------------------------------------------------
> >
> > Block layer patches
>
> Hi -- I'm afraid this doesn't build with clang:
My clang didn't complain, unfortunately.
> CC block/qcow2-bitmap.o
> /home/petmay01/linaro/qemu-for-merges/block/qcow2-bitmap.c:1183:9:
> error: variable 'ret' is used uninitialized whenever 'if' condition is
> true [-Werror,-Wsometimes-uninitialized]
> if (tb_offset < 0) {
> ^~~~~~~~~~~~~
> /home/petmay01/linaro/qemu-for-merges/block/qcow2-bitmap.c:1222:12:
> note: uninitialized use occurs here
> return ret;
> ^~~
> /home/petmay01/linaro/qemu-for-merges/block/qcow2-bitmap.c:1183:5:
> note: remove the 'if' if its condition is always false
> if (tb_offset < 0) {
> ^~~~~~~~~~~~~~~~~~~~
> /home/petmay01/linaro/qemu-for-merges/block/qcow2-bitmap.c:1165:12:
> note: initialize the variable 'ret' to silence this warning
> int ret;
> ^
> = 0
> 1 error generated.
>
> Looking at the code, the compiler appears to be correct
> that that error-exit path has forgotten to set ret.
Indeed. This is from commit 519d4789 ('qcow2: add persistent dirty
bitmaps support') and came through Max' tree.
Max, I'll split the pull request and send only my tree in v2. Please
submit a separate pull request for the patches in your tree once the
build is fixed.
Kevin
- [Qemu-devel] [PULL 089/100] block/dirty-bitmap: add bdrv_remove_persistent_dirty_bitmap, (continued)
- [Qemu-devel] [PULL 089/100] block/dirty-bitmap: add bdrv_remove_persistent_dirty_bitmap, Kevin Wolf, 2017/07/07
- [Qemu-devel] [PULL 090/100] qcow2: add .bdrv_remove_persistent_dirty_bitmap, Kevin Wolf, 2017/07/07
- [Qemu-devel] [PULL 094/100] iotests: fix remainining tests to work with LUKS, Kevin Wolf, 2017/07/07
- [Qemu-devel] [PULL 098/100] iotests: Use absolute paths for executables, Kevin Wolf, 2017/07/07
- [Qemu-devel] [PULL 099/100] iotests: Add test for colon handling, Kevin Wolf, 2017/07/07
- [Qemu-devel] [PULL 096/100] iotests: add more LUKS hash combination tests, Kevin Wolf, 2017/07/07
- [Qemu-devel] [PULL 095/100] iotests: reduce PBKDF iterations when testing LUKS, Kevin Wolf, 2017/07/07
- [Qemu-devel] [PULL 100/100] tests: Avoid non-portable 'echo -ARG', Kevin Wolf, 2017/07/07
- [Qemu-devel] [PULL 097/100] iotests: chown LUKS device before qemu-io launches, Kevin Wolf, 2017/07/07
- Re: [Qemu-devel] [PULL 000/100] Block layer patches, Peter Maydell, 2017/07/10
- Re: [Qemu-devel] [PULL 000/100] Block layer patches,
Kevin Wolf <=