On 11/04/2024 7:20 am, Michal Orzel wrote: > Hi Andrew, > > On 10/04/2024 20:42, Andrew Cooper wrote: >> >> The start/stop1/etc linkage scheme predates struct virtual_region, and as >> setup_virtual_regions() shows, it's awkward to express in the new scheme. >> >> Change the linker to provide explicit start/stop symbols for each bugframe >> type, and change virtual_region to have a stop pointer rather than a count. >> >> This marginly simplifies both do_bug_frame()s and prepare_payload(), but it > NIT: s/marginly/marginally
Fixed. > >> massively simplifies setup_virtual_regions() by allowing the compiler to >> initialise the .frame[] array at build time. >> >> virtual_region.c is the only user of the linker symbols, and this is unlikely >> to change given the purpose of struct virtual_region, so move their externs >> out of bug.h >> >> No functional change. >> >> Signed-off-by: Andrew Cooper <[email protected]> >> Reviewed-by: Ross Lagerwall <[email protected]> >> Reviewed-by: Jan Beulich <[email protected]> > For Arm: > Acked-by: Michal Orzel <[email protected]> Thanks. ~Andrew
