Message ID | 20220113194452.254011-2-dgilbert@redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | vmsd checks | expand |
On Thu, 13 Jan 2022 at 19:45, Dr. David Alan Gilbert (git) <dgilbert@redhat.com> wrote: > > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com> > > The pbr403 subsection is part of the tlb6xx state, so I believe it's > name needs to be: > > .name = "cpu/tlb6xx/pbr403", > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com> > --- > target/ppc/machine.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > 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, > -- > 2.34.1 Reviewed-by: Peter Maydell <peter.maydell@linaro.org> thanks -- PMM
* Peter Maydell (peter.maydell@linaro.org) wrote: > On Thu, 13 Jan 2022 at 19:45, Dr. David Alan Gilbert (git) > <dgilbert@redhat.com> wrote: > > > > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com> > > > > The pbr403 subsection is part of the tlb6xx state, so I believe it's > > name needs to be: > > > > .name = "cpu/tlb6xx/pbr403", > > > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com> > > --- > > target/ppc/machine.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > 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, > > -- > > 2.34.1 > > Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Actually, we'll drop this one; Cédric Le Goater has just published 'target/ppc: Finish removal of 401/403 CPUs' that kills this code off. Dave > thanks > -- PMM >
"Dr. David Alan Gilbert (git)" <dgilbert@redhat.com> wrote: > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com> > > The pbr403 subsection is part of the tlb6xx state, so I believe it's > name needs to be: > > .name = "cpu/tlb6xx/pbr403", > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com> Dropped as code don't exist anymore. > --- > target/ppc/machine.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > 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,
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,