[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [Qemu-devel] [PATCH v1 4/5] s390-ccw: interactive boot
From: |
Cornelia Huck |
Subject: |
Re: [qemu-s390x] [Qemu-devel] [PATCH v1 4/5] s390-ccw: interactive boot menu for eckd dasd |
Date: |
Tue, 28 Nov 2017 13:45:28 +0100 |
On Tue, 28 Nov 2017 13:36:38 +0100
Thomas Huth <address@hidden> wrote:
> On 27.11.2017 21:55, Collin L. Walling wrote:
> > +static int zipl_boot_menu(block_number_t s1b_block_nr)
> > +{
> > + void *stage2_data, *menu_offset;
> > +
> > + read_stage2(s1b_block_nr, &stage2_data);
> > + menu_offset = stage2_data;
> > +
> > + /* Menu banner starts with "zIPL" */
> > + while (menu_offset < stage2_data + STAGE2_MAX_SIZE - 4) {
> > + if (magic_match(menu_offset, ZIPL_MAGIC_EBCDIC)) {
> > + return menu_get_zipl_boot_index(menu_offset);
> > + }
> > + menu_offset++;
> > + }
> > +
> > + panic("\n! No menu data found !\n");
>
> Maybe remove the initial exclamation mark?
It does match the other panics in the code, though.
>
> > + /* should not reach here */
> > + return 0;
> > +}
- [qemu-s390x] [PATCH v1 0/5] (FIXED) Interactive Boot Menu for DASD and SCSI Guests on s390x, Collin L. Walling, 2017/11/27
- [qemu-s390x] [PATCH v1 3/5] s390-ccw: parse and set boot menu options, Collin L. Walling, 2017/11/27
- [qemu-s390x] [PATCH v1 5/5] s390-ccw: interactive boot menu for scsi, Collin L. Walling, 2017/11/27
- [qemu-s390x] [PATCH v1 4/5] s390-ccw: interactive boot menu for eckd dasd, Collin L. Walling, 2017/11/27
- [qemu-s390x] [PATCH v1 1/5] s390-ccw: update libc.h, Collin L. Walling, 2017/11/27
- [qemu-s390x] [PATCH v1 2/5] s390-ccw: ipl structs for eckd cdl/ldl, Collin L. Walling, 2017/11/27
- Re: [qemu-s390x] [Qemu-devel] [PATCH v1 0/5] (FIXED) Interactive Boot Menu for DASD and SCSI Guests on s390x, Cornelia Huck, 2017/11/28