OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

hmp: change hmp_info_cpus to use query-cpus-fast

Browse Source 
Changing the implementation of hmp_info_cpus() to call
qmp_query_cpus_fast() instead of qmp_query_cpus. This has the
following consequences:

  o No further code change required for qmp_query_cpus deprecation

  o HMP profits from the less disruptive cpu information retrieval

  o HMP 'info cpus' won't display architecture specific data anymore,
    which should be tolerable in the light of the deprecation of
    query-cpus.

In order to allow 'info cpus' to be executed completely on the
fast path, monitor_get_cpu_index() has been adapted to not synchronize
the cpu state.

Signed-off-by: Viktor Mihajlovski <mihajlov@linux.vnet.ibm.com>
Message-Id: <1518797321-28356-6-git-send-email-mihajlov@linux.vnet.ibm.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Cornelia Huck <cohuck@redhat.com>
This commit is contained in:
Viktor Mihajlovski 2018-02-16 17:08:41 +01:00 committed by Cornelia Huck
parent ff9a915655
commit 137b5cb6ab
2 changed files with 17 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
hmp.c
View File

@ -360,50 +360,23 @@ void hmp_info_migrate_cache_size(Monitor *mon, const QDict *qdict)
void hmp_info_cpus(Monitor *mon, const QDict *qdict) void hmp_info_cpus(Monitor *mon, const QDict *qdict)
{ {
CpuInfoList *cpu_list, *cpu; CpuInfoFastList *cpu_list, *cpu;
cpu_list = qmp_query_cpus(NULL); cpu_list = qmp_query_cpus_fast(NULL);
for (cpu = cpu_list; cpu; cpu = cpu->next) { for (cpu = cpu_list; cpu; cpu = cpu->next) {
int active = ' '; int active = ' ';
if (cpu->value->CPU == monitor_get_cpu_index()) { if (cpu->value->cpu_index == monitor_get_cpu_index()) {
active = '*'; active = '*';
} }
monitor_printf(mon, "%c CPU #%" PRId64 ":", active, cpu->value->CPU); monitor_printf(mon, "%c CPU #%" PRId64 ":", active,
cpu->value->cpu_index);
switch (cpu->value->arch) { monitor_printf(mon, " thread-id=%" PRId64 "\n", cpu->value->thread_id);
case CPU_INFO_ARCH_X86:
monitor_printf(mon, " pc=0x%016" PRIx64, cpu->value->u.x86.pc);
break;
case CPU_INFO_ARCH_PPC:
monitor_printf(mon, " nip=0x%016" PRIx64, cpu->value->u.ppc.nip);
break;
case CPU_INFO_ARCH_SPARC:
monitor_printf(mon, " pc=0x%016" PRIx64,
cpu->value->u.q_sparc.pc);
monitor_printf(mon, " npc=0x%016" PRIx64,
cpu->value->u.q_sparc.npc);
break;
case CPU_INFO_ARCH_MIPS:
monitor_printf(mon, " PC=0x%016" PRIx64, cpu->value->u.q_mips.PC);
break;
case CPU_INFO_ARCH_TRICORE:
monitor_printf(mon, " PC=0x%016" PRIx64, cpu->value->u.tricore.PC);
break;
default:
break;
}
if (cpu->value->halted) {
monitor_printf(mon, " (halted)");
}
monitor_printf(mon, " thread_id=%" PRId64 "\n", cpu->value->thread_id);
} }
qapi_free_CpuInfoList(cpu_list); qapi_free_CpuInfoFastList(cpu_list);
} }
static void print_block_info(Monitor *mon, BlockInfo *info, static void print_block_info(Monitor *mon, BlockInfo *info,

13
monitor.c
View File

@ -1055,7 +1055,7 @@ int monitor_set_cpu(int cpu_index)
return 0; return 0;
} }
CPUState *mon_get_cpu(void) static CPUState *mon_get_cpu_sync(bool synchronize)
{ {
CPUState *cpu; CPUState *cpu;
@ -1074,10 +1074,17 @@ CPUState *mon_get_cpu(void)
monitor_set_cpu(first_cpu->cpu_index); monitor_set_cpu(first_cpu->cpu_index);
cpu = first_cpu; cpu = first_cpu;
} }
cpu_synchronize_state(cpu); if (synchronize) {
cpu_synchronize_state(cpu);
}
return cpu; return cpu;
} }
CPUState *mon_get_cpu(void)
{
return mon_get_cpu_sync(true);
}
CPUArchState *mon_get_cpu_env(void) CPUArchState *mon_get_cpu_env(void)
{ {
CPUState *cs = mon_get_cpu(); CPUState *cs = mon_get_cpu();
@ -1087,7 +1094,7 @@ CPUArchState *mon_get_cpu_env(void)
int monitor_get_cpu_index(void) int monitor_get_cpu_index(void)
{ {
CPUState *cs = mon_get_cpu(); CPUState *cs = mon_get_cpu_sync(false);
return cs ? cs->cpu_index : UNASSIGNED_CPU_INDEX; return cs ? cs->cpu_index : UNASSIGNED_CPU_INDEX;
} }