On 12/10/16 15:15, David Hildenbrand wrote: >>> + for (cx = 0; ccpus && ccpus[cx]; cx++) { >>> + cpu_single_step(cpu, 0); > > This looks suspicious
why? we set all cpus to single step, since that is the default, and then we clear the single-step property from all CPUs that should be restarted in normal mode, then we restart all CPUs. Those in single-step will indeed only perform one single step, the others will run freely (at least until the first single-step CPU stops again). >>> + } >>> + CPU_FOREACH(cpu) { >>> + cpu_resume(cpu); >>> + } > > Claudio, did you have a look at how s->c_cpu is used later on? I remember > that we > have to take care of some query reply packages. yes, that's set by the H packet and used by the c,s,m,etc packets. vCont ignores it and doesn't change it (see here https://sourceware.org/gdb/onlinedocs/gdb/Packets.html )