Hi Julien,

On 09.08.17 12:53, Julien Grall wrote:
Hi Volodymyr,

On 08/08/17 21:08, Volodymyr Babchuk wrote:
From: Volodymyr Babchuk <vlad.babc...@gmail.com>

The following list of functions:

 - advance_pc()
 - check_conditional_instr()
 - inject_undef_exception()
 - inject_iabt_exception()
 - inject_dabt_exception()
 - inject_vabt_exception()

are now globaly visible. This change is needed becase we plan to handle SMCs
in different file and handler code needs to alter state of a guest.

Signed-off-by: Volodymyr Babchuk <vlad.babc...@gmail.com>
 xen/arch/arm/traps.c            | 16 ++++++----------
 xen/include/asm-arm/processor.h | 11 +++++++++++

I would much prefer if you introduce a new header traps.h rather than piggy-back on the now growing processor.h.
Probably, better idea is to move this functions to processor.c or to newlycreated file like processor_state.c, because actually this functions can
have broader use, than traps handling. What do you think?

I have actually a series that will do that, but I am happy if you introduce it before me.


Xen-devel mailing list

Reply via email to