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

hw/ppc/spapr_rtas.c: use g_autofree in rtas_ibm_get_system_parameter()

Browse Source 
Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220228175004.8862-14-danielhb413@gmail.com>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
This commit is contained in:
Daniel Henrique Barboza 2022-03-02 06:51:40 +01:00 committed by Cédric Le Goater
parent ecccc4bed7
commit 3a1229148d
1 changed files with 12 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
hw/ppc/spapr_rtas.c
View File

@ -279,30 +279,29 @@ static void rtas_ibm_get_system_parameter(PowerPCCPU *cpu,
switch (parameter) { switch (parameter) {
case RTAS_SYSPARM_SPLPAR_CHARACTERISTICS: { case RTAS_SYSPARM_SPLPAR_CHARACTERISTICS: {
char *param_val = g_strdup_printf("MaxEntCap=%d," g_autofree char *param_val = g_strdup_printf("MaxEntCap=%d,"
"DesMem=%" PRIu64 "," "DesMem=%" PRIu64 ","
"DesProcs=%d," "DesProcs=%d,"
"MaxPlatProcs=%d", "MaxPlatProcs=%d",
ms->smp.max_cpus, ms->smp.max_cpus,
ms->ram_size / MiB, ms->ram_size / MiB,
ms->smp.cpus, ms->smp.cpus,
ms->smp.max_cpus); ms->smp.max_cpus);
if (pcc->n_host_threads > 0) { if (pcc->n_host_threads > 0) {
char *hostthr_val, *old = param_val;
/* /*
* Add HostThrs property. This property is not present in PAPR but * Add HostThrs property. This property is not present in PAPR but
* is expected by some guests to communicate the number of physical * is expected by some guests to communicate the number of physical
* host threads per core on the system so that they can scale * host threads per core on the system so that they can scale
* information which varies based on the thread configuration. * information which varies based on the thread configuration.
*/ */
hostthr_val = g_strdup_printf(",HostThrs=%d", pcc->n_host_threads); g_autofree char *hostthr_val = g_strdup_printf(",HostThrs=%d",
pcc->n_host_threads);
char *old = param_val;
param_val = g_strconcat(param_val, hostthr_val, NULL); param_val = g_strconcat(param_val, hostthr_val, NULL);
g_free(hostthr_val);
g_free(old); g_free(old);
} }
ret = sysparm_st(buffer, length, param_val, strlen(param_val) + 1); ret = sysparm_st(buffer, length, param_val, strlen(param_val) + 1);
g_free(param_val);
break; break;
} }
case RTAS_SYSPARM_DIAGNOSTICS_RUN_MODE: { case RTAS_SYSPARM_DIAGNOSTICS_RUN_MODE: {