* [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci
@ 2017-02-14 13:33 Laurent Vivier
2017-02-14 14:07 ` Thomas Huth
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Laurent Vivier @ 2017-02-14 13:33 UTC (permalink / raw)
To: qemu-devel
Cc: Laurent Vivier, David Gibson, Dr . David Alan Gilbert,
Thomas Huth, Greg Kurz, Alexey Kardashevskiy
To fix migration between 2.7 and 2.8, some fields have
been renamed and managed with the help of a PHB property
(pre_2_8_migration):
5c4537b spapr: Fix 2.7<->2.8 migration of PCI host bridge
So we need to add them to the white list:
dma_liobn[0],
mem_win_addr, mem_win_size,
io_win_addr, io_win_size
become
mig_liobn,
mig_mem_win_addr, mig_mem_win_size,
mig_io_win_addr, mig_io_win_size
CC: David Gibson <david@gibson.dropbear.id.au>
CC: Dr. David Alan Gilbert <dgilbert@redhat.com>
CC: Thomas Huth <thuth@redhat.com>
CC: Greg Kurz <groug@kaod.org>
CC: Alexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
---
scripts/vmstate-static-checker.py | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/scripts/vmstate-static-checker.py b/scripts/vmstate-static-checker.py
index 14a27e7..bcef7ee 100755
--- a/scripts/vmstate-static-checker.py
+++ b/scripts/vmstate-static-checker.py
@@ -85,6 +85,11 @@ def check_fields_match(name, s_field, d_field):
'xio3130-express-upstream-port': ['br.dev', 'parent_obj.parent_obj',
'br.dev.exp.aer_log',
'parent_obj.parent_obj.exp.aer_log'],
+ 'spapr_pci': ['dma_liobn[0]', 'mig_liobn',
+ 'mem_win_addr', 'mig_mem_win_addr',
+ 'mem_win_size', 'mig_mem_win_size',
+ 'io_win_addr', 'mig_io_win_addr',
+ 'io_win_size', 'mig_io_win_size'],
}
if not name in changed_names:
--
2.9.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci
2017-02-14 13:33 [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci Laurent Vivier
@ 2017-02-14 14:07 ` Thomas Huth
2017-02-14 22:55 ` David Gibson
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Thomas Huth @ 2017-02-14 14:07 UTC (permalink / raw)
To: Laurent Vivier, qemu-devel
Cc: David Gibson, Dr . David Alan Gilbert, Greg Kurz,
Alexey Kardashevskiy
On 14.02.2017 14:33, Laurent Vivier wrote:
> To fix migration between 2.7 and 2.8, some fields have
> been renamed and managed with the help of a PHB property
> (pre_2_8_migration):
>
> 5c4537b spapr: Fix 2.7<->2.8 migration of PCI host bridge
>
> So we need to add them to the white list:
>
> dma_liobn[0],
> mem_win_addr, mem_win_size,
> io_win_addr, io_win_size
>
> become
>
> mig_liobn,
> mig_mem_win_addr, mig_mem_win_size,
> mig_io_win_addr, mig_io_win_size
>
> CC: David Gibson <david@gibson.dropbear.id.au>
> CC: Dr. David Alan Gilbert <dgilbert@redhat.com>
> CC: Thomas Huth <thuth@redhat.com>
> CC: Greg Kurz <groug@kaod.org>
> CC: Alexey Kardashevskiy <aik@ozlabs.ru>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
> scripts/vmstate-static-checker.py | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/scripts/vmstate-static-checker.py b/scripts/vmstate-static-checker.py
> index 14a27e7..bcef7ee 100755
> --- a/scripts/vmstate-static-checker.py
> +++ b/scripts/vmstate-static-checker.py
> @@ -85,6 +85,11 @@ def check_fields_match(name, s_field, d_field):
> 'xio3130-express-upstream-port': ['br.dev', 'parent_obj.parent_obj',
> 'br.dev.exp.aer_log',
> 'parent_obj.parent_obj.exp.aer_log'],
> + 'spapr_pci': ['dma_liobn[0]', 'mig_liobn',
> + 'mem_win_addr', 'mig_mem_win_addr',
> + 'mem_win_size', 'mig_mem_win_size',
> + 'io_win_addr', 'mig_io_win_addr',
> + 'io_win_size', 'mig_io_win_size'],
> }
>
> if not name in changed_names:
>
Reviewed-by: Thomas Huth <thuth@redhat.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci
2017-02-14 13:33 [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci Laurent Vivier
2017-02-14 14:07 ` Thomas Huth
@ 2017-02-14 22:55 ` David Gibson
2017-02-24 8:26 ` Laurent Vivier
2017-02-28 9:35 ` Dr. David Alan Gilbert
3 siblings, 0 replies; 5+ messages in thread
From: David Gibson @ 2017-02-14 22:55 UTC (permalink / raw)
To: Laurent Vivier
Cc: qemu-devel, Dr . David Alan Gilbert, Thomas Huth, Greg Kurz,
Alexey Kardashevskiy
[-- Attachment #1: Type: text/plain, Size: 2023 bytes --]
On Tue, Feb 14, 2017 at 02:33:31PM +0100, Laurent Vivier wrote:
> To fix migration between 2.7 and 2.8, some fields have
> been renamed and managed with the help of a PHB property
> (pre_2_8_migration):
>
> 5c4537b spapr: Fix 2.7<->2.8 migration of PCI host bridge
>
> So we need to add them to the white list:
>
> dma_liobn[0],
> mem_win_addr, mem_win_size,
> io_win_addr, io_win_size
>
> become
>
> mig_liobn,
> mig_mem_win_addr, mig_mem_win_size,
> mig_io_win_addr, mig_io_win_size
>
> CC: David Gibson <david@gibson.dropbear.id.au>
> CC: Dr. David Alan Gilbert <dgilbert@redhat.com>
> CC: Thomas Huth <thuth@redhat.com>
> CC: Greg Kurz <groug@kaod.org>
> CC: Alexey Kardashevskiy <aik@ozlabs.ru>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
> scripts/vmstate-static-checker.py | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/scripts/vmstate-static-checker.py b/scripts/vmstate-static-checker.py
> index 14a27e7..bcef7ee 100755
> --- a/scripts/vmstate-static-checker.py
> +++ b/scripts/vmstate-static-checker.py
> @@ -85,6 +85,11 @@ def check_fields_match(name, s_field, d_field):
> 'xio3130-express-upstream-port': ['br.dev', 'parent_obj.parent_obj',
> 'br.dev.exp.aer_log',
> 'parent_obj.parent_obj.exp.aer_log'],
> + 'spapr_pci': ['dma_liobn[0]', 'mig_liobn',
> + 'mem_win_addr', 'mig_mem_win_addr',
> + 'mem_win_size', 'mig_mem_win_size',
> + 'io_win_addr', 'mig_io_win_addr',
> + 'io_win_size', 'mig_io_win_size'],
> }
>
> if not name in changed_names:
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
--
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
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci
2017-02-14 13:33 [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci Laurent Vivier
2017-02-14 14:07 ` Thomas Huth
2017-02-14 22:55 ` David Gibson
@ 2017-02-24 8:26 ` Laurent Vivier
2017-02-28 9:35 ` Dr. David Alan Gilbert
3 siblings, 0 replies; 5+ messages in thread
From: Laurent Vivier @ 2017-02-24 8:26 UTC (permalink / raw)
To: qemu-devel
Cc: David Gibson, Dr . David Alan Gilbert, Thomas Huth, Greg Kurz,
Alexey Kardashevskiy
Ping?
Laurent
On 14/02/2017 14:33, Laurent Vivier wrote:
> To fix migration between 2.7 and 2.8, some fields have
> been renamed and managed with the help of a PHB property
> (pre_2_8_migration):
>
> 5c4537b spapr: Fix 2.7<->2.8 migration of PCI host bridge
>
> So we need to add them to the white list:
>
> dma_liobn[0],
> mem_win_addr, mem_win_size,
> io_win_addr, io_win_size
>
> become
>
> mig_liobn,
> mig_mem_win_addr, mig_mem_win_size,
> mig_io_win_addr, mig_io_win_size
>
> CC: David Gibson <david@gibson.dropbear.id.au>
> CC: Dr. David Alan Gilbert <dgilbert@redhat.com>
> CC: Thomas Huth <thuth@redhat.com>
> CC: Greg Kurz <groug@kaod.org>
> CC: Alexey Kardashevskiy <aik@ozlabs.ru>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
> scripts/vmstate-static-checker.py | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/scripts/vmstate-static-checker.py b/scripts/vmstate-static-checker.py
> index 14a27e7..bcef7ee 100755
> --- a/scripts/vmstate-static-checker.py
> +++ b/scripts/vmstate-static-checker.py
> @@ -85,6 +85,11 @@ def check_fields_match(name, s_field, d_field):
> 'xio3130-express-upstream-port': ['br.dev', 'parent_obj.parent_obj',
> 'br.dev.exp.aer_log',
> 'parent_obj.parent_obj.exp.aer_log'],
> + 'spapr_pci': ['dma_liobn[0]', 'mig_liobn',
> + 'mem_win_addr', 'mig_mem_win_addr',
> + 'mem_win_size', 'mig_mem_win_size',
> + 'io_win_addr', 'mig_io_win_addr',
> + 'io_win_size', 'mig_io_win_size'],
> }
>
> if not name in changed_names:
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci
2017-02-14 13:33 [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci Laurent Vivier
` (2 preceding siblings ...)
2017-02-24 8:26 ` Laurent Vivier
@ 2017-02-28 9:35 ` Dr. David Alan Gilbert
3 siblings, 0 replies; 5+ messages in thread
From: Dr. David Alan Gilbert @ 2017-02-28 9:35 UTC (permalink / raw)
To: Laurent Vivier
Cc: qemu-devel, David Gibson, Thomas Huth, Greg Kurz,
Alexey Kardashevskiy
* Laurent Vivier (lvivier@redhat.com) wrote:
> To fix migration between 2.7 and 2.8, some fields have
> been renamed and managed with the help of a PHB property
> (pre_2_8_migration):
>
> 5c4537b spapr: Fix 2.7<->2.8 migration of PCI host bridge
>
> So we need to add them to the white list:
>
> dma_liobn[0],
> mem_win_addr, mem_win_size,
> io_win_addr, io_win_size
>
> become
>
> mig_liobn,
> mig_mem_win_addr, mig_mem_win_size,
> mig_io_win_addr, mig_io_win_size
>
> CC: David Gibson <david@gibson.dropbear.id.au>
> CC: Dr. David Alan Gilbert <dgilbert@redhat.com>
> CC: Thomas Huth <thuth@redhat.com>
> CC: Greg Kurz <groug@kaod.org>
> CC: Alexey Kardashevskiy <aik@ozlabs.ru>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Queued.
> ---
> scripts/vmstate-static-checker.py | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/scripts/vmstate-static-checker.py b/scripts/vmstate-static-checker.py
> index 14a27e7..bcef7ee 100755
> --- a/scripts/vmstate-static-checker.py
> +++ b/scripts/vmstate-static-checker.py
> @@ -85,6 +85,11 @@ def check_fields_match(name, s_field, d_field):
> 'xio3130-express-upstream-port': ['br.dev', 'parent_obj.parent_obj',
> 'br.dev.exp.aer_log',
> 'parent_obj.parent_obj.exp.aer_log'],
> + 'spapr_pci': ['dma_liobn[0]', 'mig_liobn',
> + 'mem_win_addr', 'mig_mem_win_addr',
> + 'mem_win_size', 'mig_mem_win_size',
> + 'io_win_addr', 'mig_io_win_addr',
> + 'io_win_size', 'mig_io_win_size'],
> }
>
> if not name in changed_names:
> --
> 2.9.3
>
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-02-28 9:36 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-02-14 13:33 [Qemu-devel] [PATCH] vmstate-static-checker: update white list with spapr_pci Laurent Vivier
2017-02-14 14:07 ` Thomas Huth
2017-02-14 22:55 ` David Gibson
2017-02-24 8:26 ` Laurent Vivier
2017-02-28 9:35 ` Dr. David Alan Gilbert
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).