>>> On 04.08.16 at 13:57, <andrew.coop...@citrix.com> wrote: > debugger_trap_entry() is reachable during early boot where its unconditional > use of current is unsafe. Add a warning to the function to this effect. > > Perform the vector check first, as this allows the compiler to elide the > other > content from most of its callsites. Check guest_mode(regs) before using > current, which makes the path safe on early boot. > > While editing this area, drop DEBUGGER_trap_{entry,fatal}, as hiding a > return > statement in a function-like macro is very antisocial programming; show the > real control flow at each of the callsites. Finally, switch > debugger_trap_{entry,fatal} to having boolean return types, to match their > semantics. > > No behavioural change. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
Reviewed-by: Jan Beulich <jbeul...@suse.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel