On 11.08.2023 09:19, Nicola Vetrini wrote:
> The missing header included by this patch provides declarations for
> the functions 'vm_event_{fill_regs,set_registers,monitor_next_interrupt}'
> that are defined in the source file. This also resolves violations
> of MISRA C:2012 Rule 8.4.
>
> Signed-off-by: Nicola Vetrini <[email protected]>
> Fixes: adc75eba8b15 ("x86/vm_event: consolidate hvm_event_fill_regs and
> p2m_vm_event_fill_regs")
> Fixes: 975efd3baa8d ("introduce VM_EVENT_FLAG_SET_REGISTERS")
> Fixes: 9864841914c2 ("x86/vm_event: add support for
> VM_EVENT_REASON_INTERRUPT")
It's hard to see how it can be three commit here. The oldest one is at
fault, I would say.
Also please remember to Cc maintainers.
Jan
> --- a/xen/arch/x86/vm_event.c
> +++ b/xen/arch/x86/vm_event.c
> @@ -20,6 +20,7 @@
>
> #include <xen/sched.h>
> #include <xen/mem_access.h>
> +#include <xen/vm_event.h>
> #include <asm/vm_event.h>
>
> /* Implicitly serialized by the domctl lock. */
