On 9/30/2026 1:23 PM, Marco Liebel wrote:
On Sat, Sep 26, 2026 at 2:23 PM Brian Cain <[email protected]> wrote:
Signed-off-by: Brian Cain <[email protected]>
---
  target/hexagon/mmvec/mmvec.h     | 55 ++++++++++++++++++++------------
  hw/hexagon/hexagon_hvx_context.c |  5 +++
  target/hexagon/hex_common.py     | 29 +++++++++++++++++
  3 files changed, 68 insertions(+), 21 deletions(-)

  /* gvec needs VRegs/QRegs 16-aligned within the struct. */
@@ -26,6 +30,7 @@ static const VMStateDescription vmstate_mmvector = {
      .minimum_version_id = 1,
      .fields = (const VMStateField[]){
          VMSTATE_UINT64_ARRAY(ud, MMVector, MAX_VEC_SIZE_BYTES / 8),
+        VMSTATE_UINT8_ARRAY(ext, MMVector, MAX_VEC_SIZE_BYTES / 4),
          VMSTATE_END_OF_LIST()
      }
  };
Does the version id need to be updated?


yes -ish...

I deliberately omitted it because it seemed like a lot of churn to keep updating the version IDs among all of the different features impacted with the upstreaming work going on.  I'd hoped/expected I could make a single update to all of the vmstate-impacted structures for 11.2 rather than incremental ones each time they changed.  Not sure if there's a best practice here.


Reply via email to