[PATCH 0/4] migration: Remove extra type-checking from vmstate macros
Fabiano Rosas <[email protected]>
| Newsgroups | gmane.comp.emulators.qemu |
|---|---|
| Message-ID | <[email protected]> |
Hi, this is basically what I ranted about in: https://lore.kernel.org/r/[email protected] I'm replacing the per-integer-size type checks with a single "int that fits in 32bit" check. This allows several lines of duplicated code to be removed. I haven't changed the macro names in the device code yet. If this series gets positive feedback then I'll send per-subsystem patches doing that. CI run: https://gitlab.com/farosas/qemu/-/pipelines/2716814081 Also tested: - migration-test --full --thorough - x86_64 compat run forwards and backwards for previous 3 QEMU releases - s390x compat run forwards and backwards for previous 2 QEMU releases - ppc64 compat run forwards and backwards for previous QEMU release - migration-test smoke ASAN/UBSAN run Fabiano Rosas (4): migration: Remove VMSTATE_ARRAY_INT32_UNSAFE migration: Introduce VMStateOffset migration: Remove redundant flags migration: Remove duplicate vmstate macros include/migration/vmstate.h | 255 ++++++++++------------------- migration/savevm.c | 10 +- migration/vmstate.c | 50 ++++-- rust/bindings/migration-sys/lib.rs | 15 +- rust/migration/src/vmstate.rs | 30 +++- rust/tests/tests/vmstate_tests.rs | 30 ++-- 6 files changed, 177 insertions(+), 213 deletions(-) -- 2.53.0