On 6/17/2016 12:28 AM, Julien Grall wrote:
On 16/06/2016 20:24, Corneliu ZUZU wrote:
On 6/16/2016 5:26 PM, Julien Grall wrote:
Hi Julien. Yes, I agree that it's complex, I would have preferred to
split it up too and I actually tried, but the changes are tightly
coupled and they don't seem to be 'split-able'.
After I reviewed this patch I think it could be simplified a lot.
Most of the registers not trapped by vm-event could be emulated with
one-line WRITE_SYSREGxx and does not require specific case depending
on the architecture.
You can give a look how we handle the domain context switch in C
(arch/arm/domain.c).
Only few of the registers will require more than one-line (such as
MAIR*, *FAR) which could be over a macro.
Regards,
Ok, I'll take a look again but I'm not that optimistic, I remember
trying pretty hard to simplify this (and kind of failing), I got the
best I could.
I think it had to do with the fact the SCTLR macros & such got expanded
too early (where I wanted to use them as prefixes to construct other
macros).
Having said this, I'll look into your suggestions and reinspect the code
to recall why those macros got to be the way the are and get back with
conclusions.
Regards,
Corneliu.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
