On Tue, May 26, 2015 at 10:18 AM, Paolo Bonzini <pbonz...@redhat.com> wrote: > > > On 26/05/2015 19:05, Peter Crosthwaite wrote: >> OK, but it is an odd one out, would you be willing to accept >> the double include that seems to be widespread? >> >> +#include "qemu_common.h" >> #include "cpu.h" >> -#include "exec/cpu-all.h" > > This is exactly what I meant. What is odd about it? >
Nothing at all. It's just the other of the two valid resolutions to my problem. I lost the coinflip when I wrote the patch :) Will respin tonight. Regards, Peter > In fact, including exec/cpu-all.h outside cpu.h is definitely odd. > > Paolo >