xen/arm: Rename init_secondary_pagetables() to prepare_secondary_mm()

init_secondary_pagetables() is a function in the common code path
of both MMU and future MPU support. Since "page table" is a MMU
specific concept, rename init_secondary_pagetables() to a generic
name prepare_secondary_mm() as the preparation for MPU support.

Reword the in-code comment on top of prepare_secondary_mm() because
this function is now supposed to be MMU/MPU agnostic.

Take the opportunity to fix the incorrect coding style of the in-code
comments.

Signed-off-by: Penny Zheng <penny.zheng@arm.com>
Signed-off-by: Henry Wang <Henry.Wang@arm.com>
Reviewed-by: Julien Grall <jgrall@amazon.com>
This commit is contained in:
Penny Zheng authored and Julien Grall committed 2023-11-20 19:07:45 +00:00
1 parent e6bef277d4
commit 1bf5c104fe
4 files changed
+6 -7

No files matched your search

+1 -1
View File
@@ -257,7 +257,7 @@ GLOBAL(init_secondary)
secondary_switched:
/*
* Non-boot CPUs need to move on to the proper pagetables, which were
* setup in init_secondary_pagetables.
* setup in prepare_secondary_mm.
*
* XXX: This is not compliant with the Arm Arm.
*/
+2 -3
View File
@@ -204,9 +204,8 @@ extern void setup_pagetables(unsigned long boot_phys_offset);
extern void *early_fdt_map(paddr_t fdt_paddr);
/* Remove early mappings */
extern void remove_early_mappings(void);
/* Allocate and initialise pagetables for a secondary CPU. Sets init_ttbr to the
* new page table */
extern int init_secondary_pagetables(int cpu);
/* Prepare the memory subystem to bring-up the given secondary CPU */
extern int prepare_secondary_mm(int cpu);
/* Map a frame table to cover physical addresses ps through pe */
extern void setup_frametable_mappings(paddr_t ps, paddr_t pe);
/* map a physical range in virtual memory */
+2 -2
View File
@@ -67,7 +67,7 @@ static void clear_boot_pagetables(void)
}
#ifdef CONFIG_ARM_64
int init_secondary_pagetables(int cpu)
int prepare_secondary_mm(int cpu)
{
clear_boot_pagetables();
@@ -80,7 +80,7 @@ int init_secondary_pagetables(int cpu)
return 0;
}
#else
int init_secondary_pagetables(int cpu)
int prepare_secondary_mm(int cpu)
{
lpae_t *first;
+1 -1
View File
@@ -448,7 +448,7 @@ int __cpu_up(unsigned int cpu)
printk("Bringing up CPU%d\n", cpu);
rc = init_secondary_pagetables(cpu);
rc = prepare_secondary_mm(cpu);
if ( rc < 0 )
return rc;
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.