Re: [qemu-s390x] [Qemu-devel] [PATCH v3 0/8] Interactive Boot Menu for DASD and SCSI Guests on s390x

[Collin L. Walling]

These warnings did not show up on my end.  I'll initialize the variables 
next
push to make the test scripts happy.

On 01/15/2018 11:58 AM, address@hidden wrote:

[...]
>    CC      s390-ccw/bootmap.o
> In file included from 
> /var/tmp/patchew-tester-tmp-dep3o2js/src/pc-bios/s390-ccw/bootmap.c:13:0:
> /var/tmp/patchew-tester-tmp-dep3o2js/src/pc-bios/s390-ccw/bootmap.c: In 
> function ‘run_eckd_boot_script’:
> /var/tmp/patchew-tester-tmp-dep3o2js/src/pc-bios/s390-ccw/bootmap.h:326:16: 
> error: ‘next_block_nr’ may be used uninitialized in this function 
> [-Werror=maybe-uninitialized]
>       IPL_assert(virtio_read(blockno, buffer) == 0, errmsg);
>                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
> /var/tmp/patchew-tester-tmp-dep3o2js/src/pc-bios/s390-ccw/bootmap.c:218:49: 
> note: ‘next_block_nr’ was declared here
>       block_number_t cur_block_nr, prev_block_nr, next_block_nr;
>                                                   ^~~~~~~~~~~~~
> In file included from 
> /var/tmp/patchew-tester-tmp-dep3o2js/src/pc-bios/s390-ccw/bootmap.c:13:0:
> /var/tmp/patchew-tester-tmp-dep3o2js/src/pc-bios/s390-ccw/bootmap.h:326:16: 
> error: ‘prev_block_nr’ may be used uninitialized in this function 
> [-Werror=maybe-uninitialized]
>       IPL_assert(virtio_read(blockno, buffer) == 0, errmsg);
>                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
> /var/tmp/patchew-tester-tmp-dep3o2js/src/pc-bios/s390-ccw/bootmap.c:218:34: 
> note: ‘prev_block_nr’ was declared here
>       block_number_t cur_block_nr, prev_block_nr, next_block_nr;
>                                    ^~~~~~~~~~~~~
> cc1: all warnings being treated as errors
> make[1]: *** [/var/tmp/patchew-tester-tmp-dep3o2js/src/rules.mak:66: bootmap.o] 
> Error 1
> make: *** [Makefile:420: romsubdir-s390-ccw] Error 2
> make: *** Waiting for unfinished jobs....
> === OUTPUT END ===


--
- Collin L Walling