scripts/vmstate-static-checker.py: Add new hpet entry for num_timers

[PATCH] scripts/vmstate-static-checker.py: Add new hpet entry for num_timers

Peter Xu posted 1 patch 6 months, 2 weeks ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20250501151235.636709-1-peterx@redhat.com

Maintainers: John Snow <jsnow@redhat.com>, Cleber Rosa <crosa@redhat.com>, Peter Xu <peterx@redhat.com>, Fabiano Rosas <farosas@suse.de>

scripts/vmstate-static-checker.py | 1 +
1 file changed, 1 insertion(+)

Expand all Fold all

[PATCH] scripts/vmstate-static-checker.py: Add new hpet entry for num_timers

Posted by Peter Xu 6 months, 2 weeks ago

The old "num_timers" got a rename.  See commit 1433e38cc8 ("hpet: do not
overwrite properties on post_load") for more details.  Teach the script to
accept the new name.

Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Thomas Huth <thuth@redhat.com>
Signed-off-by: Peter Xu <peterx@redhat.com>
---
 scripts/vmstate-static-checker.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/scripts/vmstate-static-checker.py b/scripts/vmstate-static-checker.py
index 25aca839a0..2335e25f94 100755
--- a/scripts/vmstate-static-checker.py
+++ b/scripts/vmstate-static-checker.py
@@ -91,6 +91,7 @@ def check_fields_match(name, s_field, d_field):
                       'mem_win_size', 'mig_mem_win_size',
                       'io_win_addr', 'mig_io_win_addr',
                       'io_win_size', 'mig_io_win_size'],
+        'hpet': ['num_timers', 'num_timers_save'],
     }
 
     if not name in changed_names:
-- 
2.48.1

Re: [PATCH] scripts/vmstate-static-checker.py: Add new hpet entry for num_timers

Posted by Thomas Huth 6 months, 2 weeks ago

On 01/05/2025 17.12, Peter Xu wrote:
> The old "num_timers" got a rename.  See commit 1433e38cc8 ("hpet: do not
> overwrite properties on post_load") for more details.  Teach the script to
> accept the new name.
> 
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Signed-off-by: Peter Xu <peterx@redhat.com>
> ---
>   scripts/vmstate-static-checker.py | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/scripts/vmstate-static-checker.py b/scripts/vmstate-static-checker.py
> index 25aca839a0..2335e25f94 100755
> --- a/scripts/vmstate-static-checker.py
> +++ b/scripts/vmstate-static-checker.py
> @@ -91,6 +91,7 @@ def check_fields_match(name, s_field, d_field):
>                         'mem_win_size', 'mig_mem_win_size',
>                         'io_win_addr', 'mig_io_win_addr',
>                         'io_win_size', 'mig_io_win_size'],
> +        'hpet': ['num_timers', 'num_timers_save'],
>       }
>   
>       if not name in changed_names:

Reviewed-by: Thomas Huth <thuth@redhat.com>