cpu: Make the protobuf inst tracer accept variable sized instructions.
authorGabe Black <gabeblack@google.com>
Thu, 25 Jan 2018 08:03:26 +0000 (00:03 -0800)
committerGabe Black <gabeblack@google.com>
Thu, 22 Mar 2018 18:29:37 +0000 (18:29 +0000)
commitd28bc0f946fcb7d6ad90b74f917db9fa9d311191
tree28d0fbafd391b77f50ee48ae27a1d9199c5e6ae1
parent8f1a604ec3ba2c54bbceef8d3ac188aa59d3a78e
cpu: Make the protobuf inst tracer accept variable sized instructions.

This change adds an inst_bytes field which is of type bytes, and puts
it in a oneof with the previously required inst field. If an
instruction's encoding happens to be 4 bytes long, the original inst
field will be used. Otherwise, the new variably sized inst_bytes field
will be used.

Because this tracer doesn't have visibility into how the data in
inst_bytes is structured, it can't do any endian conversion itself.
To maintain compatibility between producers and consumers who may have
different endiannesses, all data should be manually converted to
little endian before being stored in this field.

inst will be converted into little endian by protobuf, and so
compatibility doesn't have to be handled manually.

Change-Id: I290713f70e7124d8aa9550c022c71334939d84a6
Reviewed-on: https://gem5-review.googlesource.com/7561
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Maintainer: Gabe Black <gabeblack@google.com>
src/proto/inst.proto