The size of the percpu stack of Xen on x86_64 is 8 pages, not 2. While not really critical in its current usage, it should be corrected nevertheless. Signed-off-by: Juergen Gross <jgross(a)suse.com&gt; --- Makefile | 14 +++++++------- x86_64.c | 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/Makefile b/Makefile index a94a243..c4fdbe7 100644 --- a/Makefile +++ b/Makefile @@ -24,7 +24,7 @@ PROGRAM=crash # Supported targets: X86 ALPHA PPC IA64 PPC64 SPARC64 # TARGET and GDB_CONF_FLAGS will be configured automatically by configure # -TARGET= +TARGET=X86_64 GDB_CONF_FLAGS= ARCH := $(shell uname -m | sed -e s/i.86/i386/ -e s/sun4u/sparc64/ -e s/arm.*/arm/ -e s/sa110/arm/) @@ -35,10 +35,10 @@ endif # # GDB, GDB_FILES, GDB_OFILES and GDB_PATCH_FILES will be configured automatically by configure # -GDB= -GDB_FILES= -GDB_OFILES= -GDB_PATCH_FILES= +GDB=gdb-10.2 +GDB_FILES=${GDB_10.2_FILES} +GDB_OFILES=${GDB_10.2_OFILES} +GDB_PATCH_FILES=gdb-10.2.patch # # Default installation directory @@ -190,7 +190,7 @@ GDB_10.2_OFILES=${GDB}/gdb/symtab.o crash_target.o # # GDB_FLAGS is passed up from the gdb Makefile. # -GDB_FLAGS= +GDB_FLAGS=-DGDB_10_2 # # WARNING_OPTIONS and WARNING_ERROR are both applied on a per-file basis. @@ -207,7 +207,7 @@ TARGET_CFLAGS= CRASH_CFLAGS=-g -D${TARGET} ${TARGET_CFLAGS} ${GDB_FLAGS} ${CFLAGS} -GPL_FILES= +GPL_FILES=COPYING3 TAR_FILES=${SOURCE_FILES} Makefile ${GPL_FILES} README .rh_rpm_package crash.8 \ ${EXTENSION_SOURCE_FILES} ${MEMORY_DRIVER_FILES} CSCOPE_FILES=${SOURCE_FILES}

Since many years now the stack address of each percpu stack is available via the stack_base[] array now. Use that instead of the indirect method via the percpu variables tss_init or tss_page, especially as the layout of tss_page has changed in Xen 4.16, resulting in the stack no longer to be found. Signed-off-by: Juergen Gross <jgross(a)suse.com&gt; --- xen_hyper.c | 50 ++++++++++++++++++++++++++++++-------------------- 1 file changed, 30 insertions(+), 20 deletions(-) diff --git a/xen_hyper.c b/xen_hyper.c index 1030c0a..72720e2 100644 --- a/xen_hyper.c +++ b/xen_hyper.c @@ -324,7 +324,7 @@ void xen_hyper_x86_pcpu_init(void) { ulong cpu_info; - ulong init_tss_base, init_tss; + ulong init_tss_base, init_tss, stack_base; ulong sp; struct xen_hyper_pcpu_context *pcc; char *buf, *bp; @@ -340,34 +340,44 @@ xen_hyper_x86_pcpu_init(void) } /* get physical cpu context */ xen_hyper_alloc_pcpu_context_space(XEN_HYPER_MAX_CPUS()); - if (symbol_exists("per_cpu__init_tss")) { + if (symbol_exists("stack_base")) { + stack_base = symbol_value("stack_base"); + flag = 0; + } else if (symbol_exists("per_cpu__init_tss")) { init_tss_base = symbol_value("per_cpu__init_tss"); - flag = TRUE; + flag = 1; } else if (symbol_exists("per_cpu__tss_page")) { init_tss_base = symbol_value("per_cpu__tss_page"); - flag = TRUE; + flag = 1; } else { init_tss_base = symbol_value("init_tss"); - flag = FALSE; + flag = 2; } buf = GETBUF(XEN_HYPER_SIZE(tss)); for_cpu_indexes(i, cpuid) { - if (flag) - init_tss = xen_hyper_per_cpu(init_tss_base, cpuid); - else - init_tss = init_tss_base + - XEN_HYPER_SIZE(tss) * cpuid; - if (!readmem(init_tss, KVADDR, buf, - XEN_HYPER_SIZE(tss), "init_tss", RETURN_ON_ERROR)) { - error(FATAL, "cannot read init_tss.\n"); - } - if (machine_type("X86")) { - sp = ULONG(buf + XEN_HYPER_OFFSET(tss_esp0)); - } else if (machine_type("X86_64")) { - sp = ULONG(buf + XEN_HYPER_OFFSET(tss_rsp0)); - } else - sp = 0; + if (flag) { + if (flag == 1) + init_tss = xen_hyper_per_cpu(init_tss_base, cpuid); + else + init_tss = init_tss_base + + XEN_HYPER_SIZE(tss) * cpuid; + if (!readmem(init_tss, KVADDR, buf, + XEN_HYPER_SIZE(tss), "init_tss", RETURN_ON_ERROR)) { + error(FATAL, "cannot read init_tss.\n"); + } + if (machine_type("X86")) { + sp = ULONG(buf + XEN_HYPER_OFFSET(tss_esp0)); + } else if (machine_type("X86_64")) { + sp = ULONG(buf + XEN_HYPER_OFFSET(tss_rsp0)); + } else + sp = 0; + } else { + if (!readmem(stack_base + sizeof(ulong) * cpuid, KVADDR, &sp, + sizeof(ulong), "stack_base", RETURN_ON_ERROR)) { + error(FATAL, "cannot read stack_base.\n"); + } + } cpu_info = XEN_HYPER_GET_CPU_INFO(sp); if (CRASHDEBUG(1)) { fprintf(fp, "sp=%lx, cpu_info=%lx\n", sp, cpu_info);

On 2023/03/13 22:01, Juergen Gross wrote: > There has been a significant modification regarding scheduler data in > the Xen hypervisor. Adapt to new structures and removed fields. > > Note that this is only the bare minimum to not let crash error out when > opening a vmcore in Xen mode with a recent Xen version. > > Signed-off-by: Juergen Gross <jgross(a)suse.com&gt; > --- > xen_hyper.c | 67 +++++++++++++++++++++++++++++++++--------------- > xen_hyper_defs.h | 4 ++- > 2 files changed, 49 insertions(+), 22 deletions(-) > > diff --git a/xen_hyper.c b/xen_hyper.c > index 72720e2..4c884dd 100644 > --- a/xen_hyper.c > +++ b/xen_hyper.c > @@ -417,13 +417,21 @@ void > xen_hyper_misc_init(void) > { > XEN_HYPER_STRUCT_SIZE_INIT(schedule_data, "schedule_data"); > - XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_schedule_lock, "schedule_data", "schedule_lock"); > - XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_curr, "schedule_data", "curr"); > - if (MEMBER_EXISTS("schedule_data", "idle")) > - XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_idle, "schedule_data", "idle"); > - XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_sched_priv, "schedule_data", "sched_priv"); > - XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_s_timer, "schedule_data", "s_timer"); > - XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_tick, "schedule_data", "tick"); > + XEN_HYPER_STRUCT_SIZE_INIT(sched_resource, "sched_resource"); > + if (XEN_HYPER_VALID_SIZE(schedule_data)) { > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_schedule_lock, "schedule_data", "schedule_lock"); > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_curr, "schedule_data", "curr"); > + if (MEMBER_EXISTS("schedule_data", "idle")) > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_idle, "schedule_data", "idle"); > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_sched_priv, "schedule_data", "sched_priv"); > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_s_timer, "schedule_data", "s_timer"); > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_tick, "schedule_data", "tick"); > + } else if (XEN_HYPER_VALID_SIZE(sched_resource)) { > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_schedule_lock, "sched_resource", "schedule_lock"); > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_curr, "sched_resource", "curr"); > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_sched_priv, "sched_resource", "sched_priv"); > + XEN_HYPER_MEMBER_OFFSET_INIT(schedule_data_s_timer, "sched_resource", "s_timer"); > + } > > XEN_HYPER_STRUCT_SIZE_INIT(scheduler, "scheduler"); > XEN_HYPER_MEMBER_OFFSET_INIT(scheduler_name, "scheduler", "name"); > @@ -467,6 +475,7 @@ xen_hyper_schedule_init(void) > long *schedulers_buf; > int nr_schedulers; > struct xen_hyper_sched_context *schc; > + long buf_size; > char *buf; > char opt_name_buf[XEN_HYPER_OPT_SCHED_SIZE]; > int i, cpuid, flag; > @@ -561,28 +570,43 @@ xen_hyper_schedule_init(void) > } > BZERO(xhscht->sched_context_array, > sizeof(struct xen_hyper_sched_context) * XEN_HYPER_MAX_CPUS()); > - buf = GETBUF(XEN_HYPER_SIZE(schedule_data)); > - if (symbol_exists("per_cpu__schedule_data")) { > + if (symbol_exists("per_cpu__sched_res")) { > + addr = symbol_value("per_cpu__sched_res"); > + buf_size = XEN_HYPER_SIZE(sched_resource); > + flag = 0; > + } else if (symbol_exists("per_cpu__schedule_data")) { > addr = symbol_value("per_cpu__schedule_data"); > - flag = TRUE; > + buf_size = XEN_HYPER_SIZE(schedule_data); > + flag = 1; > } else { > addr = symbol_value("schedule_data"); > - flag = FALSE; > + buf_size = XEN_HYPER_SIZE(schedule_data); > + flag = 2; > } > + buf = GETBUF(buf_size); > for_cpu_indexes(i, cpuid) > { > schc = &xhscht->sched_context_array[cpuid]; > if (flag) { > - schc->schedule_data = > - xen_hyper_per_cpu(addr, i); > + if (flag == 1) { > + schc->schedule_data = > + xen_hyper_per_cpu(addr, i); > + } else { > + schc->schedule_data = addr + > + XEN_HYPER_SIZE(schedule_data) * i; > + } > + if (!readmem(schc->schedule_data, > + KVADDR, buf, XEN_HYPER_SIZE(schedule_data), > + "schedule_data", RETURN_ON_ERROR)) { > + error(FATAL, "cannot read schedule_data.\n"); > + } > } else { > - schc->schedule_data = addr + > - XEN_HYPER_SIZE(schedule_data) * i; > - } > - if (!readmem(schc->schedule_data, > - KVADDR, buf, XEN_HYPER_SIZE(schedule_data), > - "schedule_data", RETURN_ON_ERROR)) { > - error(FATAL, "cannot read schedule_data.\n"); > + schc->sched_resource = xen_hyper_per_cpu(addr, i); > + if (!readmem(schc->sched_resource, > + KVADDR, buf, XEN_HYPER_SIZE(sched_resource), > + "sched_resource", RETURN_ON_ERROR)) { > + error(FATAL, "cannot read sched_resource.\n"); > + } > } > schc->cpu_id = cpuid; > schc->curr = ULONG(buf + XEN_HYPER_OFFSET(schedule_data_curr)); > @@ -1599,7 +1623,8 @@ xen_hyper_store_vcpu_context(struct xen_hyper_vcpu_context *vcc, > vcc->next_in_list = ULONG(vcp + XEN_HYPER_OFFSET(vcpu_next_in_list)); > if (XEN_HYPER_VALID_MEMBER(vcpu_sleep_tick)) > vcc->sleep_tick = ULONG(vcp + XEN_HYPER_OFFSET(vcpu_sleep_tick)); > - vcc->sched_priv = ULONG(vcp + XEN_HYPER_OFFSET(vcpu_sched_priv)); > + if (XEN_HYPER_VALID_MEMBER(vcpu_sched_priv)) > + vcc->sched_priv = ULONG(vcp + XEN_HYPER_OFFSET(vcpu_sched_priv)); > vcc->state = INT(vcp + XEN_HYPER_OFFSET(vcpu_runstate) + > XEN_HYPER_OFFSET(vcpu_runstate_info_state)); > vcc->state_entry_time = ULONGLONG(vcp + > diff --git a/xen_hyper_defs.h b/xen_hyper_defs.h > index acf910a..908b470 100644 > --- a/xen_hyper_defs.h > +++ b/xen_hyper_defs.h > @@ -547,6 +547,7 @@ struct xen_hyper_pcpu_table { > struct xen_hyper_sched_context { > uint cpu_id; > ulong schedule_data; > + ulong sched_resource; Please add this to xen_hyper_dump_xen_hyper_sched_table(). > ulong curr; > ulong idle; > ulong sched_priv; > @@ -595,6 +596,7 @@ struct xen_hyper_size_table { > #endif > long note_buf_t; /* elf note v1 */ > long schedule_data; > + long sched_resource; Please add this to xen_hyper_dump_xen_hyper_size_table().

Also, could you move these two above to the last member of each table? It's to avoid breaking extension modules previously compiled with old definitions, for example, like commit c477b04aee34.