cpu: Remove page_size_init
Move qemu_host_page_{size,mask} and HOST_PAGE_ALIGN into bsd-user. It should be removed from bsd-user as well, but defer that cleanup. Reviewed-by: Warner Losh <imp@bsdimp.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Tested-by: Ilya Leoshkevich <iii@linux.ibm.com> Acked-by: Helge Deller <deller@gmx.de> Message-Id: <20240102015808.132373-28-richard.henderson@linaro.org>
This commit is contained in:
parent
01e449809b
commit
8c45039f9e
@ -256,7 +256,6 @@ bool cpu_unwind_state_data(CPUState *cpu, uintptr_t host_pc, uint64_t *data)
|
||||
|
||||
void page_init(void)
|
||||
{
|
||||
page_size_init();
|
||||
page_table_config_init();
|
||||
}
|
||||
|
||||
|
@ -49,6 +49,13 @@
|
||||
#include "host-os.h"
|
||||
#include "target_arch_cpu.h"
|
||||
|
||||
|
||||
/*
|
||||
* TODO: Remove these and rely only on qemu_real_host_page_size().
|
||||
*/
|
||||
uintptr_t qemu_host_page_size;
|
||||
intptr_t qemu_host_page_mask;
|
||||
|
||||
static bool opt_one_insn_per_tb;
|
||||
uintptr_t guest_base;
|
||||
bool have_guest_base;
|
||||
@ -307,6 +314,9 @@ int main(int argc, char **argv)
|
||||
(void) envlist_setenv(envlist, *wrk);
|
||||
}
|
||||
|
||||
qemu_host_page_size = getpagesize();
|
||||
qemu_host_page_size = MAX(qemu_host_page_size, TARGET_PAGE_SIZE);
|
||||
|
||||
cpu_model = NULL;
|
||||
|
||||
qemu_add_opts(&qemu_trace_opts);
|
||||
@ -404,6 +414,8 @@ int main(int argc, char **argv)
|
||||
}
|
||||
}
|
||||
|
||||
qemu_host_page_mask = -qemu_host_page_size;
|
||||
|
||||
/* init debug */
|
||||
{
|
||||
int mask = 0;
|
||||
|
@ -39,6 +39,13 @@ extern char **environ;
|
||||
#include "qemu/clang-tsa.h"
|
||||
|
||||
#include "qemu-os.h"
|
||||
/*
|
||||
* TODO: Remove these and rely only on qemu_real_host_page_size().
|
||||
*/
|
||||
extern uintptr_t qemu_host_page_size;
|
||||
extern intptr_t qemu_host_page_mask;
|
||||
#define HOST_PAGE_ALIGN(addr) ROUND_UP((addr), qemu_host_page_size)
|
||||
|
||||
/*
|
||||
* This struct is used to hold certain information about the image. Basically,
|
||||
* it replicates in user space what would be certain task_struct fields in the
|
||||
|
16
cpu-target.c
16
cpu-target.c
@ -45,9 +45,6 @@
|
||||
#include "trace/trace-root.h"
|
||||
#include "qemu/accel.h"
|
||||
|
||||
uintptr_t qemu_host_page_size;
|
||||
intptr_t qemu_host_page_mask;
|
||||
|
||||
#ifndef CONFIG_USER_ONLY
|
||||
static int cpu_common_post_load(void *opaque, int version_id)
|
||||
{
|
||||
@ -474,16 +471,3 @@ const char *target_name(void)
|
||||
{
|
||||
return TARGET_NAME;
|
||||
}
|
||||
|
||||
void page_size_init(void)
|
||||
{
|
||||
/* NOTE: we can always suppose that qemu_host_page_size >=
|
||||
TARGET_PAGE_SIZE */
|
||||
if (qemu_host_page_size == 0) {
|
||||
qemu_host_page_size = qemu_real_host_page_size();
|
||||
}
|
||||
if (qemu_host_page_size < TARGET_PAGE_SIZE) {
|
||||
qemu_host_page_size = TARGET_PAGE_SIZE;
|
||||
}
|
||||
qemu_host_page_mask = -(intptr_t)qemu_host_page_size;
|
||||
}
|
||||
|
@ -20,13 +20,6 @@
|
||||
void cpu_exec_init_all(void);
|
||||
void cpu_exec_step_atomic(CPUState *cpu);
|
||||
|
||||
/* Using intptr_t ensures that qemu_*_page_mask is sign-extended even
|
||||
* when intptr_t is 32-bit and we are aligning a long long.
|
||||
*/
|
||||
extern uintptr_t qemu_host_page_size;
|
||||
extern intptr_t qemu_host_page_mask;
|
||||
|
||||
#define HOST_PAGE_ALIGN(addr) ROUND_UP((addr), qemu_host_page_size)
|
||||
#define REAL_HOST_PAGE_ALIGN(addr) ROUND_UP((addr), qemu_real_host_page_size())
|
||||
|
||||
/* The CPU list lock nests outside page_(un)lock or mmap_(un)lock */
|
||||
|
@ -1179,8 +1179,6 @@ bool target_words_bigendian(void);
|
||||
|
||||
const char *target_name(void);
|
||||
|
||||
void page_size_init(void);
|
||||
|
||||
#ifdef NEED_CPU_H
|
||||
|
||||
#ifndef CONFIG_USER_ONLY
|
||||
|
@ -2118,7 +2118,6 @@ static void qemu_create_machine(QDict *qdict)
|
||||
}
|
||||
|
||||
cpu_exec_init_all();
|
||||
page_size_init();
|
||||
|
||||
if (machine_class->hw_version) {
|
||||
qemu_set_hw_version(machine_class->hw_version);
|
||||
|
Loading…
Reference in New Issue
Block a user