[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] VDI patches
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] VDI patches |
Date: |
Mon, 19 Mar 2012 12:46:39 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120209 Thunderbird/10.0.1 |
Am 17.03.2012 08:10, schrieb Stefan Weil:
> Am 17.03.2012 04:11, schrieb address@hidden:
>> The Buildbot has detected a new failure on builder
>> block_i386_debian_6_0 while building qemu.
>> Full details are available at:
>> http://buildbot.b1-systems.de/qemu/builders/block_i386_debian_6_0/builds/215
>>
>> Buildbot URL: http://buildbot.b1-systems.de/qemu/
>>
>> Buildslave for this Build: yuzuki
>>
>> Build Reason: The Nightly scheduler named 'nightly_block' triggered
>> this build
>> Build Source Stamp: [branch block] HEAD
>> Blamelist:
>>
>> BUILD FAILED: failed compile
>>
>> sincerely,
>> -The Buildbot
>
> Hi Kevin, hi Paolo,
>
> the build failure is caused by the vdi changes: some versions of gcc
> detect uninitialized local variables (my gcc 4.4.5 does not).
>
> I don't think that it's really an error because the code is guarded
> by the local variable 'block', but not all compilers are clever enough,
> so it should be fixed.
>
> It can be fixed by modifying commit af93911fd45f4de34ca2d5813905e2cf43675c98
> (vdi: move aiocb fields to locals) like that:
>
> --- a/block/vdi.c
> +++ b/block/vdi.c
> @@ -519,8 +519,8 @@ static int vdi_co_write(BlockDriverState *bs,
> uint32_t block_index;
> uint32_t sector_in_block;
> uint32_t n_sectors;
> - uint32_t bmap_first;
> - uint32_t bmap_last;
> + uint32_t bmap_first = VDI_UNALLOCATED;
> + uint32_t bmap_last = VDI_UNALLOCATED;
> uint8_t *block = NULL;
> int ret = 0;
Paolo, can you send a v2?
Kevin