Home
last modified time | relevance | path

Searched hist:c27f5d3a83ed2959a6a1947708b588a6774a2aec (Results 1 – 2 of 2) sorted by relevance

/openbmc/qemu/target/arm/
H A Dcpu.cdiff c27f5d3a83ed2959a6a1947708b588a6774a2aec Sun May 01 00:49:54 CDT 2022 Richard Henderson <richard.henderson@linaro.org> target/arm: Merge allocation of the cpreg and its name

Simplify freeing cp_regs hash table entries by using a single
allocation for the entire value.

This fixes a theoretical bug if we were to ever free the entire
hash table, because we've been installing string literal constants
into the cpreg structure in define_arm_vh_e2h_redirects_aliases.
However, at present we only free entries created for AArch32
wildcard cpregs which get overwritten by more specific cpregs,
so this bug is never exposed.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-id: 20220501055028.646596-13-richard.henderson@linaro.org
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
H A Dhelper.cdiff c27f5d3a83ed2959a6a1947708b588a6774a2aec Sun May 01 00:49:54 CDT 2022 Richard Henderson <richard.henderson@linaro.org> target/arm: Merge allocation of the cpreg and its name

Simplify freeing cp_regs hash table entries by using a single
allocation for the entire value.

This fixes a theoretical bug if we were to ever free the entire
hash table, because we've been installing string literal constants
into the cpreg structure in define_arm_vh_e2h_redirects_aliases.
However, at present we only free entries created for AArch32
wildcard cpregs which get overwritten by more specific cpregs,
so this bug is never exposed.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-id: 20220501055028.646596-13-richard.henderson@linaro.org
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>