From 4507bff773fd4cc540c91c6c3696ac3cf9a21b3c Mon Sep 17 00:00:00 2001 From: Josh Rahm Date: Tue, 26 Oct 2021 00:38:58 -0600 Subject: Removed the requirement for the init data to be aligned. I don't yet know how to configure the the initscript to align the INIT_DATA_VALUES. Nothing seems to work. But whatever, it seems like things work and it's a very minor performance hit. --- src/kern/init.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) (limited to 'src/kern/init.c') diff --git a/src/kern/init.c b/src/kern/init.c index 4c2bfcd..b156dd7 100644 --- a/src/kern/init.c +++ b/src/kern/init.c @@ -44,17 +44,17 @@ init2() volatile uint32_t data_segment_start_ptr = (uint32_t) &DATA_SEGMENT_START; volatile uint32_t data_segment_stop_ptr = (uint32_t) &DATA_SEGMENT_STOP; + klogf(" .data ...\n"); + klogf(" set (%p - %p)\n", &DATA_SEGMENT_START, &DATA_SEGMENT_STOP); + klogf(" from (%p)\n", &INIT_DATA_VALUES); + if ((data_segment_start_ptr | data_segment_start_ptr) & 3) { panic(".data segment not aligned with sizeof(uint32_t)!\n"); } - if (init_data_values_ptr & 3) { - panic("init data values pointer not aligned with sizeof(uint32_t)!\n"); - } - - if ((bss_start_ptr | bss_end_ptr) & 3) { - panic(".bss data segment not aligned with sizeof(uint32_t)!\n"); - } + // if (init_data_values_ptr & 3) { + // panic("init data values pointer not aligned with sizeof(uint32_t)!\n"); + // } klogf("Copy data segments from flash ... \n"); @@ -62,10 +62,6 @@ init2() uint32_t* src; uint32_t* dest; - klogf(" .data ...\n"); - klogf(" set (%p - %p)\n", &DATA_SEGMENT_START, &DATA_SEGMENT_STOP); - klogf(" from (%p)\n", &INIT_DATA_VALUES); - src = &INIT_DATA_VALUES; dest = &DATA_SEGMENT_START; @@ -77,6 +73,10 @@ init2() klogf(" .bss ...\n"); klogf(" clear (%p - %p)\n", &BSS_START, &BSS_END); + if ((bss_start_ptr | bss_end_ptr) & 3) { + panic(".bss data segment not aligned with sizeof(uint32_t)!\n"); + } + /* Everything in the BSS segment is set to zero. */ dest = &BSS_START; @@ -118,7 +118,7 @@ void run_init_routines() klogf("Init Level 0 ...\n"); for (initfn = &INIT_ROUTINES_FLASH_START; initfn < &INIT_ROUTINES_FLASH_STOP; ++initfn) { - while (initfn == init_boundaries[initlevel] && initlevel < INIT_LEVEL_7) { + while (initfn >= init_boundaries[initlevel] && initlevel < INIT_LEVEL_7) { ++initlevel; klogf("Init Level %d ...\n", initlevel); } -- cgit