On 07.09.2017 07:46, Thomas Huth wrote: >> #include "qemu/cutils.h" >> #include "cpu.h" >> -#include "exec/exec-all.h" > You should maybe mention exec-all.h in the patch description, too. > (I think it can be dropped here because it is included by > translate-all.h already). > > If you tweak the patch description: > Reviewed-by: Thomas Huth <th...@redhat.com>
I'll just drop that hunk although it can also be removed. Yes, this was not included on purpose - these similar names still confuse me :) -- Thanks, David