On 08.02.2025 01:02, Andrew Cooper wrote: > Three architectures have an identical dump_execution_state(), and PPC has a > stub for show_execution_state() that just isn't wired up yet. > > show_execution_state() is declared in a common header, meaning that > dump_execution_state() really ought to be too. Move them both into xen/bug.h > as they're tightly tied to run_in_exception_handler().
Hmm, show_execution_state() certainly has wider use than just with run_in_exception_handler(). I don't think kernel.h was a great home for its decl, but I'm thinking the same of bug.h. Nevertheless (not the least short of having any better suggestion) ... > Drop the include of > xen/kernel.h from ubsan.c which was required reviously for RISC-V to compile. > > No functional change. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> Reviewed-by: Jan Beulich <jbeul...@suse.com> Jan
