On 13.11.2024 09:17, Federico Serafini wrote: > --- a/tools/tests/x86_emulator/x86-emulate.h > +++ b/tools/tests/x86_emulator/x86-emulate.h > @@ -70,6 +70,16 @@ > extern uint32_t mxcsr_mask; > extern struct cpu_policy cpu_policy; > > +/* > + * Pseudo keyword 'fallthrough' to make explicit the fallthrough intention at > + * the end of a case statement block. > + */ > +#if !defined(__clang__) && (__GNUC__ >= 7) > +# define fallthrough __attribute__((__fallthrough__)) > +#else > +# define fallthrough do {} while (0) /* fallthrough */ > +#endif > + > #define MMAP_SZ 16384 > bool emul_test_init(void); >
I'm sure you saw my reply to Stefano where I said "Yes, just with the addition not at the bottom of the file, but where the other compatibility definitions are." Yes, you fulfilled the first half, but then you still put it in the middle of testing-specific definitions / declarations. In the interest of saving yet another round trip and to avoid yet further misunderstanding, I'll take care of the further movement while committing. Then Acked-by: Jan Beulich <jbeul...@suse.com> Jan