Skip to content

Commit

Permalink
pc-bios/s390-ccw: rename MAX_TABLE_ENTRIES to MAX_BOOT_ENTRIES
Browse files Browse the repository at this point in the history
The MAX_TABLE_ENTRIES constant has a name that is too generic. As we
want to declare a limit for boot menu entries, let's rename it to a more
fitting MAX_BOOT_ENTRIES and set its value to 31 (30 boot entries and
1 default entry). Also we move it from bootmap.h to s390-ccw.h to make
it available for menu.c in a later patch.

Signed-off-by: Collin Walling <walling@linux.ibm.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Janosch Frank <frankja@linux.ibm.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
  • Loading branch information
collinwalling authored and huth committed May 2, 2018
1 parent e4f8696 commit 6df2a82
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions pc-bios/s390-ccw/bootmap.c
Expand Up @@ -297,7 +297,7 @@ static void run_eckd_boot_script(block_number_t bmt_block_nr,
}

debug_print_int("loadparm", loadparm);
IPL_assert(loadparm <= MAX_TABLE_ENTRIES, "loadparm value greater than"
IPL_assert(loadparm < MAX_BOOT_ENTRIES, "loadparm value greater than"
" maximum number of boot entries allowed");

memset(sec, FREE_SPACE_FILLER, sizeof(sec));
Expand Down Expand Up @@ -585,7 +585,7 @@ static void ipl_scsi(void)
read_block(mbr->pt.blockno, sec, "Error reading Program Table");
IPL_assert(magic_match(sec, ZIPL_MAGIC), "No zIPL magic in PT");

while (program_table_entries <= MAX_TABLE_ENTRIES) {
while (program_table_entries < MAX_BOOT_ENTRIES) {
if (!prog_table->entry[program_table_entries].scsi.blockno) {
break;
}
Expand All @@ -600,7 +600,7 @@ static void ipl_scsi(void)
}

debug_print_int("loadparm", loadparm);
IPL_assert(loadparm <= MAX_TABLE_ENTRIES, "loadparm value greater than"
IPL_assert(loadparm < MAX_BOOT_ENTRIES, "loadparm value greater than"
" maximum number of boot entries allowed");

zipl_run(&prog_table->entry[loadparm].scsi); /* no return */
Expand Down
2 changes: 0 additions & 2 deletions pc-bios/s390-ccw/bootmap.h
Expand Up @@ -57,8 +57,6 @@ typedef union BootMapPointer {
ExtEckdBlockPtr xeckd;
} __attribute__ ((packed)) BootMapPointer;

#define MAX_TABLE_ENTRIES 30

/* aka Program Table */
typedef struct BootMapTable {
uint8_t magic[4];
Expand Down
2 changes: 2 additions & 0 deletions pc-bios/s390-ccw/s390-ccw.h
Expand Up @@ -94,6 +94,8 @@ bool menu_is_enabled_zipl(void);
int menu_get_enum_boot_index(int entries);
bool menu_is_enabled_enum(void);

#define MAX_BOOT_ENTRIES 31

static inline void fill_hex(char *out, unsigned char val)
{
const char hex[] = "0123456789abcdef";
Expand Down

0 comments on commit 6df2a82

Please sign in to comment.