On 05/04/2016 03:38 AM, Sergey Fedorov wrote:
From: Sergey Fedorov <serge.f...@gmail.com>

cpu_exec() was a huge function also sprinkled with some preprocessor
directives. It's hard to read and see the main loop crowded by all this
code. Restructure cpu_exec() by moving its conceptual parts into
separate static in-line functions. That makes it possible to see the
whole main loop at once, especially its sigsetjmp() handling part.

Signed-off-by: Sergey Fedorov <serge.f...@gmail.com>
Signed-off-by: Sergey Fedorov <sergey.fedo...@linaro.org>

I agree that cpu_exec is a shocking horror. However, this patch is hard enough to read that it might be worth splitting. Is it easy enough to pull out each subroutine in a separate patch?


r~

Reply via email to