On 5/27/21 1:46 AM, Richard Henderson wrote: > From: Philippe Mathieu-Daudé <f4...@amsat.org> > > Only 2 headers require "exec/tb-context.h". Instead of having > all files including "exec/exec-all.h" also including it, directly > include it where it is required: > - accel/tcg/cpu-exec.c > - accel/tcg/translate-all.c > > For plugins/plugin.h, we were implicitly relying on > exec/exec-all.h -> exec/tb-context.h -> qemu/qht.h > which is now included directly. > > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> > Message-Id: <20210524170453.3791436-2-f4...@amsat.org> > [rth: Fix plugins/plugin.h compilation]
Ah, I see you squashed: https://www.mail-archive.com/qemu-devel@nongnu.org/msg810344.html I was hoping it goes via Alex's testing-next tree or trivial first, so this wouldn't be a problem when you apply this patch. Thanks anyway :) Phil. > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> > --- > include/exec/exec-all.h | 1 - > include/tcg/tcg.h | 1 - > plugins/plugin.h | 1 + > accel/tcg/cpu-exec.c | 1 + > accel/tcg/translate-all.c | 1 + > 5 files changed, 3 insertions(+), 2 deletions(-) > diff --git a/plugins/plugin.h b/plugins/plugin.h > index 1aa29dcadd..55017e3581 100644 > --- a/plugins/plugin.h > +++ b/plugins/plugin.h > @@ -13,6 +13,7 @@ > #define _PLUGIN_INTERNAL_H_ > > #include <gmodule.h> > +#include "qemu/qht.h" > > #define QEMU_PLUGIN_MIN_VERSION 0
