semantic: 0.775
mistranslation: 0.640
ppc: 0.544
graphic: 0.498
assembly: 0.449
PID: 0.405
architecture: 0.360
device: 0.352
vnc: 0.337
socket: 0.304
register: 0.268
peripherals: 0.265
files: 0.243
arm: 0.232
performance: 0.215
risc-v: 0.207
kernel: 0.207
network: 0.201
x86: 0.189
user-level: 0.185
virtual: 0.156
debug: 0.154
boot: 0.153
hypervisor: 0.149
TCG: 0.142
permissions: 0.127
i386: 0.112
VMM: 0.070
KVM: 0.050

Replace comma with semicolon in trace/simple.c

In the master branch in trace/simple.c in writeout_thread (https://github.com/qemu/qemu/blob/master/trace/simple.c#L174) we currently have:
  dropped.rec.length = sizeof(TraceRecord) + sizeof(uint64_t),
  dropped.rec.pid = trace_pid;

It seems to me like a typo that the first line ends with a comma.
Currently this causes no harm, but I think this should be fixed.

It's perfect valid C to terminate a statement with "," instead of ";" - it just has a different meaning. Consider this:

#include <stdio.h>

int main()
{
    if (0)
        printf("Hello!\n"),
        
    printf("Good bye!\n");

    return 0;
}

At a first glance, you'd expect this program to print "Good bye!" - but it does not. Actually, the "," is used here to put the two printf statements into the same block, so this program is the same as:

    if (0) {
        printf("Hello!\n");
        printf("Good bye!\n");
    }

Thus, there is no real bug in simple.c here, but of course it would be better style to clean this up and use ";" instead.

By the way, two lines earlier there is another line ending in ",":

    dropped.rec.event = DROPPED_EVENT_ID,

Fixed in commit 7ff5920717d413d8b7c3ba13d9, which will be in the upcoming 4.0 release.