[PATCH 1/4] tests/vmstate-static-checker-data: Remove old dump files

Thomas Huth posted 4 patches 7 months, 2 weeks ago
Maintainers: Peter Xu <peterx@redhat.com>, Fabiano Rosas <farosas@suse.de>, John Snow <jsnow@redhat.com>, Cleber Rosa <crosa@redhat.com>
[PATCH 1/4] tests/vmstate-static-checker-data: Remove old dump files
Posted by Thomas Huth 7 months, 2 weeks ago
From: Thomas Huth <thuth@redhat.com>

These files reference QEMU machine types that have already been
removed and thus are of no use anymore.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS                                  |    1 -
 tests/vmstate-static-checker-data/dump1.json | 1163 ------------------
 tests/vmstate-static-checker-data/dump2.json |  968 ---------------
 3 files changed, 2132 deletions(-)
 delete mode 100644 tests/vmstate-static-checker-data/dump1.json
 delete mode 100644 tests/vmstate-static-checker-data/dump2.json

diff --git a/MAINTAINERS b/MAINTAINERS
index b3f9f2680bc..bb2a66b9866 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3524,7 +3524,6 @@ F: include/qemu/userfaultfd.h
 F: migration/
 F: scripts/vmstate-static-checker.py
 F: tests/functional/test_migration.py
-F: tests/vmstate-static-checker-data/
 F: tests/qtest/migration/
 F: tests/qtest/migration-*
 F: docs/devel/migration/
diff --git a/tests/vmstate-static-checker-data/dump1.json b/tests/vmstate-static-checker-data/dump1.json
deleted file mode 100644
index 334ec079631..00000000000
--- a/tests/vmstate-static-checker-data/dump1.json
+++ /dev/null
@@ -1,1163 +0,0 @@
-{
-  "vmschkmachine": {
-    "Name": "pc-i440fx-2.1"
-  },
-  "fw_cfg": {
-    "Name": "fw_cfg",
-    "version_id": 2,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "fw_cfg",
-      "version_id": 2,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "cur_entry",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 2
-        },
-        {
-          "field": "cur_offset",
-          "version_id": 0,
-          "field_exists": true,
-          "size": 4
-        },
-        {
-          "field": "cur_offset",
-          "version_id": 2,
-          "field_exists": false,
-          "size": 4
-        }
-      ]
-    }
-  },
-  "fusbh200-ehci-usb": {
-    "Name": "fusbh200-ehci-usb",
-    "version_id": 2,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "ehci-sysbus",
-      "version_id": 2,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "ehci",
-          "version_id": 2,
-          "field_exists": false,
-          "size": 1880,
-          "Description": {
-            "name": "ehci-core",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "usbcmd",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "usbsts",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "usbsts_pending",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "usbsts_frindex",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "usbintr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "frindex",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "ctrldssegment",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "periodiclistbase",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "asynclistaddr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "configflag",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[0]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[1]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[2]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[3]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[4]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[5]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "frame_timer",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 8
-              },
-              {
-                "field": "last_run_ns",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 8
-              },
-              {
-                "field": "async_stepdown",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "astate",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "pstate",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "a_fetch_addr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "p_fetch_addr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              }
-            ]
-          }
-        }
-      ]
-    }
-  },
-  "pci-serial-4x": {
-    "Name": "pci-serial-4x",
-    "version_id": 1,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "pci-serial-multi",
-      "version_id": 1,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "dev",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1944,
-          "Description": {
-            "name": "PCIDevice",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "version_id",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 256
-              },
-              {
-                "field": "irq_state",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 16
-              }
-            ]
-          }
-        },
-        {
-          "field": "state",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 368,
-          "Description": {
-            "name": "serial",
-            "version_id": 3,
-            "minimum_version_id": 2,
-            "Fields": [
-              {
-                "field": "divider",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 2
-              },
-              {
-                "field": "rbr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "ier",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "iir",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "lcr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "mcr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "lsr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "msr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "scr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "fcr_vmstate",
-                "version_id": 3,
-                "field_exists": false,
-                "size": 1
-              }
-            ]
-          }
-        },
-        {
-          "field": "level",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        }
-      ]
-    }
-  },
-  "intel-hda-generic": {
-    "Name": "intel-hda-generic",
-    "version_id": 1,
-    "minimum_version_id": 0,
-    "Description": {
-      "name": "intel-hda",
-      "version_id": 1,
-      "minimum_version_id": 0,
-      "Fields": [
-        {
-          "field": "pci",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1944,
-          "Description": {
-            "name": "PCIDevice",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "version_id",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 256
-              },
-              {
-                "field": "irq_state",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 16
-              }
-            ]
-          }
-        },
-        {
-          "field": "g_ctl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "wake_en",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "state_sts",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "int_ctl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "int_sts",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "wall_clk",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_lbase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_ubase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_rp",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_wp",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_ctl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_sts",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_size",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_lbase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_ubase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_wp",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_cnt",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_ctl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_sts",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_size",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "dp_lbase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "dp_ubase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "icw",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "irr",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "ics",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "st",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 56,
-          "Description": {
-            "name": "intel-hda-stream",
-            "version_id": 1,
-            "minimum_version_id": 0,
-            "Fields": [
-              {
-                "field": "ctl",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "lpib",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "cbl",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "lvi",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "fmt",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "bdlp_lbase",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "bdlp_ubase",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              }
-            ]
-          }
-        },
-        {
-          "field": "rirb_count",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "wall_base_ns",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 8
-        }
-      ]
-    }
-  },
-  "cfi.pflash01": {
-    "Name": "cfi.pflash01",
-    "version_id": 1,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "pflash_cfi01",
-      "version_id": 1,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "wcycle",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        },
-        {
-          "field": "cmd",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        },
-        {
-          "field": "status",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        },
-        {
-          "field": "counter",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 8
-        }
-      ]
-    }
-  },
-  "megasas": {
-    "Name": "megasas",
-    "version_id": 0,
-    "minimum_version_id": 0,
-    "Description": {
-      "name": "megasas",
-      "version_id": 0,
-      "minimum_version_id": 0,
-      "Fields": [
-        {
-          "field": "parent_obj",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1944,
-          "Description": {
-            "name": "PCIDevice",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "version_id",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 256
-              },
-              {
-                "field": "irq_state",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 16
-              }
-            ]
-          }
-        },
-        {
-          "field": "fw_state",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "intr_mask",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "doorbell",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "reply_queue_pa",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 8
-        },
-        {
-          "field": "consumer_pa",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 8
-        },
-        {
-          "field": "producer_pa",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 8
-        }
-      ]
-    }
-  },
-  "PIIX3-xen": {
-    "Name": "PIIX3-xen",
-    "version_id": 3,
-    "minimum_version_id": 2,
-    "Description": {
-      "name": "PIIX3",
-      "version_id": 1,
-      "minimum_version_id": 2,
-      "Fields": [
-        {
-          "field": "dev",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1944,
-          "Description": {
-            "name": "PCIDevice",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "version_id",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 256
-              },
-              {
-                "field": "irq_state",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 16
-              }
-            ]
-          }
-        },
-        {
-          "field": "pci_irq_levels_vmstate",
-          "version_id": 3,
-          "field_exists": false,
-          "size": 4
-        }
-      ],
-      "Subsections": [
-        {
-          "name": "PIIX3/rcr",
-          "version_id": 1,
-          "minimum_version_id": 1,
-          "Fields": [
-            {
-              "field": "rcr",
-              "version_id": 0,
-              "field_exists": false,
-              "size": 1
-            }
-          ]
-        }
-      ]
-    }
-  },
-  "tpci200": {
-    "Name": "tpci200",
-    "version_id": 1,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "tpci200",
-      "version_id": 1,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "dev",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1944,
-          "Description": {
-            "name": "PCIDevice",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "version_id",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 256
-              },
-              {
-                "field": "irq_state",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 16
-              }
-            ]
-          }
-        },
-        {
-          "field": "big_endian",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        },
-        {
-          "field": "ctrl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        },
-        {
-          "field": "status",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 2
-        },
-        {
-          "field": "int_set",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        }
-      ]
-    }
-  },
-  "sun-fdtwo": {
-    "Name": "sun-fdtwo",
-    "version_id": 2,
-    "minimum_version_id": 2,
-    "Description": {
-      "name": "fdc",
-      "version_id": 2,
-      "minimum_version_id": 2,
-      "Fields": [
-        {
-          "field": "state",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 360,
-          "Description": {
-            "name": "fdc",
-            "version_id": 2,
-            "minimum_version_id": 2,
-            "Fields": [
-              {
-                "field": "sra",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "srb",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "dor_vmstate",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "tdr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "dsr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "msr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "status0",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "status1",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "status2",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "fifo",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "data_pos",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "data_len",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "data_state",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "data_dir",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "eot",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "timer0",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "timer1",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "precomp_trk",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "lock",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "pwrd",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "num_floppies",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "drives",
-                "version_id": 1,
-                "field_exists": false,
-                "size": 40,
-                "Description": {
-                  "name": "fdrive",
-                  "version_id": 1,
-                  "minimum_version_id": 1,
-                  "Fields": [
-                    {
-                      "field": "head",
-                      "version_id": 0,
-                      "field_exists": false,
-                      "size": 1
-                    },
-                    {
-                      "field": "track",
-                      "version_id": 0,
-                      "field_exists": false,
-                      "size": 1
-                    },
-                    {
-                      "field": "sect",
-                      "version_id": 0,
-                      "field_exists": false,
-                      "size": 1
-                    }
-                  ],
-                  "Subsections": [
-                    {
-                      "name": "fdrive/media_changed",
-                      "version_id": 1,
-                      "minimum_version_id": 1,
-                      "Fields": [
-                        {
-                          "field": "media_changed",
-                          "version_id": 0,
-                          "field_exists": false,
-                          "size": 1
-                        }
-                      ]
-                    },
-                    {
-                      "name": "fdrive/media_rate",
-                      "version_id": 1,
-                      "minimum_version_id": 1,
-                      "Fields": [
-                        {
-                          "field": "media_rate",
-                          "version_id": 0,
-                          "field_exists": false,
-                          "size": 1
-                        }
-                      ]
-                    }
-                  ]
-                }
-              }
-            ]
-          }
-        }
-      ]
-    }
-  },
-  "usb-kbd": {
-    "Name": "usb-kbd",
-    "version_id": 1,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "usb-kbd",
-      "version_id": 1,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "dev",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4352,
-          "Description": {
-            "name": "USBDevice",
-            "version_id": 1,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "addr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "state",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "remote_wakeup",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "setup_state",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "setup_len",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "setup_index",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "setup_buf",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              }
-            ]
-          }
-        },
-        {
-          "field": "kbd.keycodes",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "head",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "n",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "kbd.modifiers",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 2
-        },
-        {
-          "field": "kbd.leds",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        },
-        {
-          "field": "kbd.key",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        },
-        {
-          "field": "kbd.keys",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "protocol",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "idle",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        }
-      ]
-    }
-  }
-}
diff --git a/tests/vmstate-static-checker-data/dump2.json b/tests/vmstate-static-checker-data/dump2.json
deleted file mode 100644
index 7184e9ccbb5..00000000000
--- a/tests/vmstate-static-checker-data/dump2.json
+++ /dev/null
@@ -1,968 +0,0 @@
-{
-  "vmschkmachine": {
-    "Name": "pc-i440fx-2.2"
-  },
-  "fw_cfg2": {
-    "Name": "fw_cfg",
-    "version_id": 2,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "fw_cfg",
-      "version_id": 2,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "cur_entry",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 2
-        },
-        {
-          "field": "cur_offset",
-          "version_id": 0,
-          "field_exists": true,
-          "size": 4
-        },
-        {
-          "field": "cur_offset",
-          "version_id": 2,
-          "field_exists": false,
-          "size": 4
-        }
-      ]
-    }
-  },
-  "fusbh200-ehci-usb": {
-    "Name": "fusbh200-ehci-usb",
-    "version_id": 1,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "ehci-sysbus",
-      "version_id": 2,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "ehci",
-          "version_id": 2,
-          "field_exists": false,
-          "size": 1880,
-          "Description": {
-            "name": "ehci-core",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "usbcmd",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "usbsts_pending",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "usbsts_frindex",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "usbintr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "frindex",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "ctrldssegment",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "periodiclistbase",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "asynclistaddr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "configflag",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[0]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[1]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[2]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[3]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[4]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "portsc[5]",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "frame_timer",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 8
-              },
-              {
-                "field": "last_run_ns",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 8
-              },
-              {
-                "field": "async_stepdown",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "astate",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "pstate",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "a_fetch_addr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "p_fetch_addr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              }
-            ]
-          }
-        }
-      ]
-    }
-  },
-  "pci-serial-4x": {
-    "Name": "pci-serial-4x",
-    "version_id": 1,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "pci-serial-multi",
-      "version_id": 1,
-      "minimum_version_id": 1
-    }
-  },
-  "intel-hda-generic": {
-    "Name": "intel-hda-generic",
-    "version_id": 1,
-    "minimum_version_id": 0,
-    "Description": {
-      "name": "intel-hda",
-      "version_id": 1,
-      "minimum_version_id": 0,
-      "Fields": [
-        {
-          "field": "pci",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1944
-        },
-        {
-          "field": "g_ctl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "wake_en",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "state_sts",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "int_ctl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "int_sts",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "wall_clk",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_lbase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_ubase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_rp",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_wp",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_ctl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_sts",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "corb_size",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_lbase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_ubase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_wp",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_cnt",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_ctl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_sts",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "rirb_size",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "dp_lbase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "dp_ubase",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "icw",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "irr",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "ics",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "st",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 56,
-          "Description": {
-            "name": "intel-hda-stream",
-            "version_id": 1,
-            "minimum_version_id": 0,
-            "Fields": [
-              {
-                "field": "ctl",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "lpib",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "cbl",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "lvi",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "fmt",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "bdlp_lbase",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "bdlp_ubase",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              }
-            ]
-          }
-        },
-        {
-          "field": "rirb_count",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "wall_base_ns",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 8
-        }
-      ]
-    }
-  },
-  "cfi.pflash01": {
-    "Name": "cfi.pflash01",
-    "version_id": 1,
-    "minimum_version_id": 1
-  },
-  "megasas": {
-    "Name": "megasas",
-    "version_id": 0,
-    "minimum_version_id": 0,
-    "Description": {
-      "name": "megasas",
-      "version_id": 0,
-      "minimum_version_id": 0,
-      "Fields": [
-        {
-          "field": "parent_obj",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1944,
-          "Description": {
-            "name": "PCIDevice",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "version_id",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 256
-              }
-            ]
-          }
-        },
-        {
-          "field": "fw_state",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "intr_mask",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "doorbell",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 4
-        },
-        {
-          "field": "reply_queue_pa",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 8
-        },
-        {
-          "field": "consumer_pa",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 8
-        },
-        {
-          "field": "producer_pa",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 8
-        }
-      ]
-    }
-  },
-  "PIIX3-xen": {
-    "Name": "PIIX3-xen",
-    "version_id": 3,
-    "minimum_version_id": 2,
-    "Description": {
-      "name": "PIIX3",
-      "version_id": 3,
-      "minimum_version_id": 2,
-      "Fields": [
-        {
-          "field": "dev",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1944,
-          "Description": {
-            "name": "PCIDevice",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "version_id",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 256
-              },
-              {
-                "field": "irq_state",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 16
-              }
-            ]
-          }
-        },
-        {
-          "field": "pci_irq_levels_vmstate",
-          "version_id": 3,
-          "field_exists": false,
-          "size": 4
-        }
-      ]
-    }
-  },
-  "tpci200": {
-    "Name": "tpci200",
-    "version_id": 1,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "tpci2002",
-      "version_id": 1,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "dev",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1944,
-          "Description": {
-            "name": "PCIDevice",
-            "version_id": 2,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "version_id",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 256
-              },
-              {
-                "field": "irq_state",
-                "version_id": 2,
-                "field_exists": false,
-                "size": 16
-              }
-            ]
-          }
-        },
-        {
-          "field": "big_endian",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        },
-        {
-          "field": "ctrl",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        },
-        {
-          "field": "status",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 2
-        },
-        {
-          "field": "int_set",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 1
-        }
-      ]
-    }
-  },
-  "sun-fdtwo": {
-    "Name": "sun-fdtwo",
-    "version_id": 2,
-    "minimum_version_id": 2,
-    "Description": {
-      "name": "fdc",
-      "version_id": 1,
-      "minimum_version_id": 2,
-      "Fields": [
-        {
-          "field": "state",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 360,
-          "Description": {
-            "name": "fdc",
-            "version_id": 2,
-            "minimum_version_id": 2,
-            "Fields": [
-              {
-                "field": "sra",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "srb",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "dor_vmstate",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "tdr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "dsr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "msr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "status0",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "status1",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "status2",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "fifo",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "data_pos",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "data_len",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "data_state",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "data_dir",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "eot",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "timer0",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "timer1",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "precomp_trk",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "config",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "lock",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "pwrd",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "num_floppies",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "drives",
-                "version_id": 1,
-                "field_exists": false,
-                "size": 40,
-                "Description": {
-                  "name": "fdrive",
-                  "version_id": 1,
-                  "minimum_version_id": 1,
-                  "Fields": [
-                    {
-                      "field": "head",
-                      "version_id": 0,
-                      "field_exists": false,
-                      "size": 1
-                    },
-                    {
-                      "field": "track",
-                      "version_id": 0,
-                      "field_exists": false,
-                      "size": 1
-                    },
-                    {
-                      "field": "sect",
-                      "version_id": 0,
-                      "field_exists": false,
-                      "size": 1
-                    }
-                  ],
-                  "Subsections": [
-                    {
-                      "name": "fdrive/media_changed",
-                      "version_id": 1,
-                      "minimum_version_id": 1,
-                      "Fields": [
-                        {
-                          "field": "media_changed",
-                          "version_id": 0,
-                          "field_exists": false,
-                          "size": 1
-                        }
-                      ]
-                    }
-                  ]
-                }
-              }
-            ]
-          }
-        }
-      ]
-    }
-  },
-  "usb-kbd": {
-    "Name": "usb-kbd",
-    "version_id": 1,
-    "minimum_version_id": 1,
-    "Description": {
-      "name": "usb-kbd",
-      "version_id": 1,
-      "minimum_version_id": 1,
-      "Fields": [
-        {
-          "field": "dev",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 5832,
-          "Description": {
-            "name": "USBDevice",
-            "version_id": 1,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "addr",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "state",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "remote_wakeup",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "setup_state",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "setup_len",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "setup_index",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "setup_buf",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              }
-            ]
-          }
-        },
-        {
-          "field": "hid",
-          "version_id": 0,
-          "field_exists": false,
-          "size": 312,
-          "Description": {
-            "name": "HIDKeyboardDevice",
-            "version_id": 1,
-            "minimum_version_id": 1,
-            "Fields": [
-              {
-                "field": "kbd.keycodes",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 2
-              },
-              {
-                "field": "head",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "n",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "kbd.modifiers",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 2
-              },
-              {
-                "field": "kbd.leds",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "kbd.key",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              },
-              {
-                "field": "kbd.keys",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "protocol",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 4
-              },
-              {
-                "field": "idle",
-                "version_id": 0,
-                "field_exists": false,
-                "size": 1
-              }
-            ]
-          }
-        }
-      ]
-    }
-  }
-}
-- 
2.49.0
Re: [PATCH 1/4] tests/vmstate-static-checker-data: Remove old dump files
Posted by Peter Xu 7 months, 2 weeks ago
On Tue, Apr 29, 2025 at 05:21:38PM +0200, Thomas Huth wrote:
> From: Thomas Huth <thuth@redhat.com>
> 
> These files reference QEMU machine types that have already been
> removed and thus are of no use anymore.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>

I never noticed these files, but when looking it seems to me these two
dumps are tailored to be used to test the script itself..

See:

$ git ls tests/vmstate-static-checker-data/
e178113ff6 hw: Replace anti-social QOM type names
38ef86b5a6 tests: vmstate static checker: add size mismatch inside substructure
af3713f6b9 tests: vmstate static checker: add substructure for usb-kbd for hid section
c7173a9c18 tests: vmstate static checker: remove Subsections
aa2a12bb82 tests: vmstate static checker: remove a subsection
b5968f0ab3 tests: vmstate static checker: remove Description inside Fields
ff29b8573f tests: vmstate static checker: remove Description
083bac3484 tests: vmstate static checker: remove Fields
1d681c712a tests: vmstate static checker: change description name
fd52ffb9bf tests: vmstate static checker: remove last field in a struct
55e8e0e19c tests: vmstate static checker: remove a field
ab99bdbe33 tests: vmstate static checker: remove a section
7daa3d76df tests: vmstate static checker: minimum_version_id check
4efa6e1d64 tests: vmstate static checker: version mismatch inside a Description
a81d3fad87 tests: vmstate static checker: add version error in main section
bc178dc563 tests: vmstate static checker: incompat machine types
a10413e4fc tests: vmstate static checker: add dump1 and dump2 files

So dump2.json was deliberately modified to trigger all kinds of error that
the python script can detect.

Maybe we can still keep it there, but at least make it clearer that it's
testing the script?  For example, we could create a bash under the same dir
running the script over the two dumps and comparing the results to be the
expected one?

Currently the output:

$ scripts/vmstate-static-checker.py -s ./tests/vmstate-static-checker-data/dump1.json -d ./tests/vmstate-static-checker-data/dump2.json 
Warning: checking incompatible machine types: "pc-i440fx-2.1", "pc-i440fx-2.2"
Section "fw_cfg" does not exist in dest
Section "fusbh200-ehci-usb" version error: 2 > 1
Section "fusbh200-ehci-usb", Description "ehci-core": expected field "usbsts", got "usbsts_pending"; skipping rest
Section "pci-serial-4x" Description "pci-serial-multi": Entry "Fields" missing
Section "intel-hda-generic", Description "intel-hda", Field "pci": missing description
Section "cfi.pflash01": Entry "Description" missing
Section "megasas", Description "PCIDevice": expected field "irq_state", while dest has no further fields
Section "PIIX3-xen" Description "PIIX3": minimum version error: 1 < 2
Section "PIIX3-xen" Description "PIIX3": Entry "Subsections" missing
Section "tpci200": Description "tpci200" missing, got "tpci2002" instead; skipping
Section "sun-fdtwo" Description "fdc": version error: 2 > 1
Section "sun-fdtwo", Description "fdrive": Subsection "fdrive/media_rate" not found
Section "usb-kbd" Description "usb-kbd" Field "kbd.keycodes" size mismatch: 4 , 2

So the bash can make sure the result is exactly that.  If we want, we could
even route that to CI, but I'd say optional.

-- 
Peter Xu
Re: [PATCH 1/4] tests/vmstate-static-checker-data: Remove old dump files
Posted by Thomas Huth 7 months, 2 weeks ago
On 30/04/2025 00.13, Peter Xu wrote:
> On Tue, Apr 29, 2025 at 05:21:38PM +0200, Thomas Huth wrote:
>> From: Thomas Huth <thuth@redhat.com>
>>
>> These files reference QEMU machine types that have already been
>> removed and thus are of no use anymore.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
> 
> I never noticed these files, but when looking it seems to me these two
> dumps are tailored to be used to test the script itself..
> 
> See:
> 
> $ git ls tests/vmstate-static-checker-data/
> e178113ff6 hw: Replace anti-social QOM type names
> 38ef86b5a6 tests: vmstate static checker: add size mismatch inside substructure
> af3713f6b9 tests: vmstate static checker: add substructure for usb-kbd for hid section
> c7173a9c18 tests: vmstate static checker: remove Subsections
> aa2a12bb82 tests: vmstate static checker: remove a subsection
> b5968f0ab3 tests: vmstate static checker: remove Description inside Fields
> ff29b8573f tests: vmstate static checker: remove Description
> 083bac3484 tests: vmstate static checker: remove Fields
> 1d681c712a tests: vmstate static checker: change description name
> fd52ffb9bf tests: vmstate static checker: remove last field in a struct
> 55e8e0e19c tests: vmstate static checker: remove a field
> ab99bdbe33 tests: vmstate static checker: remove a section
> 7daa3d76df tests: vmstate static checker: minimum_version_id check
> 4efa6e1d64 tests: vmstate static checker: version mismatch inside a Description
> a81d3fad87 tests: vmstate static checker: add version error in main section
> bc178dc563 tests: vmstate static checker: incompat machine types
> a10413e4fc tests: vmstate static checker: add dump1 and dump2 files
> 
> So dump2.json was deliberately modified to trigger all kinds of error that
> the python script can detect.

Oh, good point, I should have looked at the history, too! ... the files 
clearly lacked an explanation / script that is using them!

> Maybe we can still keep it there, but at least make it clearer that it's
> testing the script?  For example, we could create a bash under the same dir
> running the script over the two dumps and comparing the results to be the
> expected one?
> 
> Currently the output:
> 
> $ scripts/vmstate-static-checker.py -s ./tests/vmstate-static-checker-data/dump1.json -d ./tests/vmstate-static-checker-data/dump2.json
> Warning: checking incompatible machine types: "pc-i440fx-2.1", "pc-i440fx-2.2"
> Section "fw_cfg" does not exist in dest
> Section "fusbh200-ehci-usb" version error: 2 > 1
> Section "fusbh200-ehci-usb", Description "ehci-core": expected field "usbsts", got "usbsts_pending"; skipping rest
> Section "pci-serial-4x" Description "pci-serial-multi": Entry "Fields" missing
> Section "intel-hda-generic", Description "intel-hda", Field "pci": missing description
> Section "cfi.pflash01": Entry "Description" missing
> Section "megasas", Description "PCIDevice": expected field "irq_state", while dest has no further fields
> Section "PIIX3-xen" Description "PIIX3": minimum version error: 1 < 2
> Section "PIIX3-xen" Description "PIIX3": Entry "Subsections" missing
> Section "tpci200": Description "tpci200" missing, got "tpci2002" instead; skipping
> Section "sun-fdtwo" Description "fdc": version error: 2 > 1
> Section "sun-fdtwo", Description "fdrive": Subsection "fdrive/media_rate" not found
> Section "usb-kbd" Description "usb-kbd" Field "kbd.keycodes" size mismatch: 4 , 2
> 
> So the bash can make sure the result is exactly that.  If we want, we could
> even route that to CI, but I'd say optional.

I can include such a test in the functional python test, no need for a 
separate bash script here, I think. I'll try to include that in v2.

  Thomas
Re: [PATCH 1/4] tests/vmstate-static-checker-data: Remove old dump files
Posted by Philippe Mathieu-Daudé 7 months, 2 weeks ago
On 29/4/25 17:21, Thomas Huth wrote:
> From: Thomas Huth <thuth@redhat.com>
> 
> These files reference QEMU machine types that have already been
> removed and thus are of no use anymore.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   MAINTAINERS                                  |    1 -
>   tests/vmstate-static-checker-data/dump1.json | 1163 ------------------
>   tests/vmstate-static-checker-data/dump2.json |  968 ---------------
>   3 files changed, 2132 deletions(-)
>   delete mode 100644 tests/vmstate-static-checker-data/dump1.json
>   delete mode 100644 tests/vmstate-static-checker-data/dump2.json

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>