On Thu, Jan 13, 2022 at 06:51:56PM +0000, Dr. David Alan Gilbert wrote: > Hi, > Is there any easy way of getting a machine where the pbr403 vmstate > would be generated?
The condition in pbr403_needed is...
return (pvr & 0xffff0000) == 0x00200000;
.. which looks to be the PVR for ppc403 models. That makes sense with
the section name... but not so much with the fact that it's under
cpu/tlb6xx. The 6xx MMU is basically unrelated to the 40x MMU. But
it looks like the vmstate_tlbemb might be shared between then, because
of bad ideas of the past.
But in any case, we already dropped what little 403 support we ever
had - there's nothing with that PVR even listed in
target/ppc/cpu-models.h.
So I think we should just drop it.
> Given my vague understanding of vmstate subsection naming, I think
> we need:
>
> diff --git a/target/ppc/machine.c b/target/ppc/machine.c
> index 756d8de5d8..e535edb7c4 100644
> --- a/target/ppc/machine.c
> +++ b/target/ppc/machine.c
> @@ -718,7 +718,7 @@ static bool pbr403_needed(void *opaque)
> }
>
> static const VMStateDescription vmstate_pbr403 = {
> - .name = "cpu/pbr403",
> + .name = "cpu/tlb6xx/pbr403",
> .version_id = 1,
> .minimum_version_id = 1,
> .needed = pbr403_needed,
>
> to fit the rule where the name of a subsection is prefixed
> by the parent name. (Something a new check I added just triggered).
>
> Dave
>
--
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
signature.asc
Description: PGP signature
