Am 26.05.2015 um 10:05 schrieb Paolo Bonzini: > On 26/05/2015 08:10, Andreas Färber wrote: >> Am 25.05.2015 um 15:08 schrieb Paolo Bonzini: >>> On 25/05/2015 08:22, Peter Crosthwaite wrote: >>>> Hi Andreas, Richard and all, >>>> >>>> I'm moving towards the goal of having no core code usages of ENV_GET_CPU. >>>> This has two advantages: >>>> >>>> 1: It means we are closer to common-obj'ing core code like exec.c, cpus.c >>>> and friends. >>>> 2: Multi arch is easier if ENV_GET_CPU stays arch specific. It means I >>>> don't need those patches where I reorder the env within the arch specific >>>> CPUState. This allows continuing placement of arch specifics before the >>>> env in the CPU container (which has TCG perf advantages). >>>> >>>> There's a couple more after this pack to get the multi-arch thing going, >>>> but due to point 1, I'm sending this ahead as I think it has standalone >>>> value. >>>> >>>> Regards, >>>> Peter >>>> >>>> Peter Crosthwaite (4): >>>> translate-all: Change tb_flush env argument to cpu >>>> gdbserver: _fork: Change fn to accept cpu instead of env >>>> cpus: Change tcg_cpu_exec arg to cpu, not env >>>> cpus: Change exec_init arg to cpu, not env >> [...] >>> >>> Thanks, queued for 2.4. >> >> Apparently after qom-next you also want to take over qom-cpu, once again >> without pinging me first. > > Uhm... > > Main loop > M: Paolo Bonzini <pbonz...@redhat.com> > S: Maintained > F: cpus.c > F: main-loop.c > F: qemu-timer.c > F: vl.c > > translate-all.c is "Odd fixes" with no specific maintainer, and > gdbserver.c is not in MAINTAINERS altogether.
ENV_GET_CPU() is my QOM CPU macro. You picked the patchset up just hours after it arrived on the list, on a holiday, without giving me a chance to review. It's not about which tree it goes through, it's about you not asking first - which I reminded you of just days ago, so this appears deliberate. And as you can see above, the cover letter was addressed to Richard and me, so we should get a chance to respond a) with change requests and b) with Reviewed-bys or Acked-bys that actually get put on the patch and not repeatedly ignored like on your last pull. Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Dilip Upmanyu, Graham Norton; HRB 21284 (AG Nürnberg)