On 21.02.2018 20:35, Collin L. Walling wrote: > Interactive boot menu for scsi. This follows a similar procedure > as the interactive menu for eckd dasd. An example follows: > > s390x Enumerated Boot Menu. > > 3 entries detected. Select from index 0 to 2. > > Signed-off-by: Collin L. Walling <wall...@linux.vnet.ibm.com> > Reviewed-by: Thomas Huth <th...@redhat.com> > --- > hw/s390x/ipl.c | 1 + > pc-bios/s390-ccw/bootmap.c | 4 ++++ > pc-bios/s390-ccw/main.c | 1 + > pc-bios/s390-ccw/menu.c | 14 ++++++++++++++ > pc-bios/s390-ccw/s390-ccw.h | 1 + > 5 files changed, 21 insertions(+) > > diff --git a/hw/s390x/ipl.c b/hw/s390x/ipl.c > index a0f4f40..566248e 100644 > --- a/hw/s390x/ipl.c > +++ b/hw/s390x/ipl.c > @@ -238,6 +238,7 @@ static void s390_ipl_set_boot_menu(S390IPLState *ipl) > *flags |= QIPL_FLAG_BM_OPTS_ZIPL; > return; > } > + case S390_IPL_TYPE_QEMU_SCSI: > break;
It's not a real bug, but I'm pretty sure this will cause Coverity to report an issue of unintended switch-case fall through. Could you please add a break before the new case label? (Since all other patches look fine to me, this could maybe also be fixed when the patches are picked up, so no need to respin the whole patch series just because of this) Thomas