summaryrefslogtreecommitdiffstats
path: root/scripts/switch-timer-api
diff options
context:
space:
mode:
authorDaniel Henrique Barboza2017-06-16 19:22:30 +0200
committerDavid Gibson2017-06-30 06:03:31 +0200
commitaca8bf9f1cbc90da82554c43ef843887ed0ab1ff (patch)
tree5804b4944cf564a477d07e62d65523ae4e9af61b /scripts/switch-timer-api
parentspapr: prevent QEMU crash when CPU realization fails (diff)
downloadqemu-aca8bf9f1cbc90da82554c43ef843887ed0ab1ff.tar.gz
qemu-aca8bf9f1cbc90da82554c43ef843887ed0ab1ff.tar.xz
qemu-aca8bf9f1cbc90da82554c43ef843887ed0ab1ff.zip
hw/ppc/spapr.c: consecutive 'spapr->patb_entry = 0' statements
In ppc_spapr_reset(), if the guest is using HPT, the code was executing: } else { spapr->patb_entry = 0; spapr_setup_hpt_and_vrma(spapr); } And, at the end of spapr_setup_hpt_and_vrma: /* We're setting up a hash table, so that means we're not radix */ spapr->patb_entry = 0; Resulting in spapr->patb_entry being assigned to 0 twice in a row. Given that 'spapr_setup_hpt_and_vrma' is also called inside 'spapr_check_setup_free_hpt' of spapr_hcall.c, this trivial patch removes the 'patb_entry = 0' assignment from the 'else' clause inside ppc_spapr_reset to avoid this behavior. Signed-off-by: Daniel Henrique Barboza <danielhb@linux.vnet.ibm.com> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'scripts/switch-timer-api')
0 files changed, 0 insertions, 0 deletions