Move defs into header file and add Frame struct.

Author: Rot127

contrib/plugins/bap-tracing/frame.c

@@ -0,0 +1,24 @@
+#include "tracing.h"
+
+Frame *frame_new_std(uint64_t addr, int vcpu_id) {
+  Frame *frame = g_new(Frame, 1);
+  frame__init(frame);
+
+  StdFrame *sframe = g_new(StdFrame, 1);
+  std_frame__init(sframe);
+  frame->std_frame = sframe;
+
+  sframe->address = addr;
+  sframe->thread_id = vcpu_id;
+
+  OperandValueList *ol_in = g_new(OperandValueList, 1);
+  operand_value_list__init(ol_in);
+  ol_in->n_elem = 0;
+  sframe->operand_pre_list = ol_in;
+
+  OperandValueList *ol_out = g_new(OperandValueList, 1);
+  operand_value_list__init(ol_out);
+  ol_out->n_elem = 0;
+  sframe->operand_post_list = ol_out;
+  return frame;
+}

contrib/plugins/bap-tracing/tracing.c

@@ -1,21 +1,9 @@
 // SPDX-FileCopyrightText: 2025 Rot127 <[email protected]>
 // SPDX-License-Identifier: GPL-2.0-only
 
-#include <qemu-plugin.h>
+#include <glib.h>
 
-QEMU_PLUGIN_EXPORT int qemu_plugin_version = QEMU_PLUGIN_VERSION;
-
-typedef struct {
-  // Current instruction related things.
-} VCPU;
-
-typedef struct {
-  GRWLock vcpus_array_lock;
-  GArray *vcpus;
-
-  GRWLock frame_buffer_lock;
-  GPtrArray *frame_buffer;
-} TraceState;
+#include "tracing.h"
 
 static TraceState state;
 

contrib/plugins/bap-tracing/tracing.h

@@ -0,0 +1,58 @@
+#ifndef BAP_TRACING_H
+#define BAP_TRACING_H
+
+#include <qemu-plugin.h>
+#include <stdio.h>
+
+#include "tracewrap.h"
+
+QEMU_PLUGIN_EXPORT int qemu_plugin_version = QEMU_PLUGIN_VERSION;
+
+#define FRAME_BUFFER_SIZE 1024
+
+typedef struct {
+  Frame **fbuf;
+  size_t len;
+} FrameBuffer;
+
+typedef struct {
+  // Current instruction related things.
+} VCPU;
+
+typedef struct {
+  GRWLock vcpus_array_lock;
+  GArray *vcpus;
+
+  GRWLock frame_buffer_lock;
+  FrameBuffer *frame_buffer;
+
+  GRWLock file_lock;
+  FILE *file;
+} TraceState;
+
+VCPU *vcpu_new(void);
+
+/**
+ * \brief Initializes a frame buffer with space for \p size frames.
+ * Returns the buffer or NULL in case of failure.
+ */
+FrameBuffer *frame_buffer_init(size_t size);
+
+/**
+ * \brief Push a frame into the buffer.
+ * Returns true on success. False otherwise.
+ */
+bool frame_buffer_push(FrameBuffer *buf, Frame *frame);
+
+/**
+ * \brief Flusehs the buffer and returns it's content.
+ * The size of the returned buffer is written to \p fbuf_size.
+ */
+Frame **frame_buffer_flush(FrameBuffer *buf, size_t *fbuf_size);
+
+/**
+ * \brief Create new std frame
+ */
+Frame *frame_new_std(uint64_t addr, int vcpu_id);
+
+#endif