[Qemu-devel] [PATCH] hw/ppc/spapr.c: consecutive 'spapr->patb_entry = 0' statements

Daniel Henrique Barboza posted 1 patch 6 years, 9 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170616172230.22755-1-danielhb@linux.vnet.ibm.com
Test FreeBSD passed
Test checkpatch passed
Test docker passed
Test s390x passed
hw/ppc/spapr.c | 1 -
1 file changed, 1 deletion(-)
[Qemu-devel] [PATCH] hw/ppc/spapr.c: consecutive 'spapr->patb_entry = 0' statements
Posted by Daniel Henrique Barboza 6 years, 9 months ago
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>
---
 hw/ppc/spapr.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
index e877d45..bd31972 100644
--- a/hw/ppc/spapr.c
+++ b/hw/ppc/spapr.c
@@ -1326,7 +1326,6 @@ static void ppc_spapr_reset(void)
          * Set the GR bit in PATB so that we know there is no HPT. */
         spapr->patb_entry = PATBE1_GR;
     } else {
-        spapr->patb_entry = 0;
         spapr_setup_hpt_and_vrma(spapr);
     }
 
-- 
2.9.4


Re: [Qemu-devel] [PATCH] hw/ppc/spapr.c: consecutive 'spapr->patb_entry = 0' statements
Posted by David Gibson 6 years, 9 months ago
On Fri, Jun 16, 2017 at 02:22:30PM -0300, Daniel Henrique Barboza wrote:
> 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>

Applied to ppc-for-2.10.

> ---
>  hw/ppc/spapr.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index e877d45..bd31972 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -1326,7 +1326,6 @@ static void ppc_spapr_reset(void)
>           * Set the GR bit in PATB so that we know there is no HPT. */
>          spapr->patb_entry = PATBE1_GR;
>      } else {
> -        spapr->patb_entry = 0;
>          spapr_setup_hpt_and_vrma(spapr);
>      }
>  

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson