On 5/19/25 16:19, Julian Ganz wrote:
+QEMU_DISABLE_CFI
+static void plugin_vcpu_cb__discon(CPUState *cpu,
+ enum qemu_plugin_discon_type type,
+ uint64_t from)
+{
+ struct qemu_plugin_cb *cb, *next;
+ enum qemu_plugin_event ev;
+ uint64_t to = cpu->cc->get_pc(cpu);
+
+ if (cpu->cpu_index < plugin.num_vcpus) {
+ switch (type) {
+ case QEMU_PLUGIN_DISCON_INTERRUPT:
+ ev = QEMU_PLUGIN_EV_VCPU_INTERRUPT;
+ break;
+ case QEMU_PLUGIN_DISCON_EXCEPTION:
+ ev = QEMU_PLUGIN_EV_VCPU_EXCEPTION;
+ break;
+ case QEMU_PLUGIN_DISCON_HOSTCALL:
+ ev = QEMU_PLUGIN_EV_VCPU_HOSTCALL;
+ break;
No point passing in QEMU_PLUGIN_DISCON_* only to covert it to QEMU_PLUGIN_EV_*.
+void qemu_plugin_vcpu_interrupt_cb(CPUState *cpu, uint64_t from)
+{
+ plugin_vcpu_cb__discon(cpu, QEMU_PLUGIN_DISCON_INTERRUPT, from);
+}
+
+void qemu_plugin_vcpu_exception_cb(CPUState *cpu, uint64_t from)
+{
+ plugin_vcpu_cb__discon(cpu, QEMU_PLUGIN_DISCON_EXCEPTION, from);
+}
+
+void qemu_plugin_vcpu_hostcall_cb(CPUState *cpu, uint64_t from)
+{
+ plugin_vcpu_cb__discon(cpu, QEMU_PLUGIN_DISCON_HOSTCALL, from);
+}
Better to pass in the value we really want here.
r~
