On 10.09.2024 12:08, Federico Serafini wrote: > Address violations of MISRA C:2012 Rule 16.3: > "An unconditional `break' statement shall terminate every > switch-clause". > > No functional change. > > Signed-off-by: Federico Serafini <federico.seraf...@bugseng.com> > --- > xen/arch/x86/mm/guest_walk.c | 1 + > xen/arch/x86/mm/hap/hap.c | 2 +- > xen/arch/x86/mm/hap/nested_hap.c | 1 + > xen/arch/x86/mm/paging.c | 2 +- > 4 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/xen/arch/x86/mm/guest_walk.c b/xen/arch/x86/mm/guest_walk.c > index fe7393334f..bc032d697c 100644 > --- a/xen/arch/x86/mm/guest_walk.c > +++ b/xen/arch/x86/mm/guest_walk.c > @@ -497,6 +497,7 @@ guest_walk_tables(const struct vcpu *v, struct p2m_domain > *p2m, > paging_mark_dirty(d, gw->l4mfn); > hvmemul_write_cache(v, l4gpa, &gw->l4e, sizeof(gw->l4e)); > } > + break; > #endif > }
This wants inserting after the #endif, I think. If you agree, I'm happy to adjust while committing. With the adjustment: Acked-by: Jan Beulich <jbeul...@suse.com> Jan