On Thu, Feb 15, 2018 at 10:03:03PM +0100, Christoffer Dall wrote: > Instead of having multiple calls from the world switch path to the debug > logic, each figuring out if the dirty bit is set and if we should > save/restore the debug registers, let's just provide two hooks to the > debug save/restore functionality, one for switching to the guest > context, and one for switching to the host context, and we get the > benefit of only having to evaluate the dirty flag once on each path, > plus we give the compiler some more room to inline some of this > functionality. > > Reviewed-by: Marc Zyngier <marc.zyng...@arm.com> > Signed-off-by: Christoffer Dall <christoffer.d...@linaro.org> > --- > > Notes: > Changes since v1: > - Remove leading underscores from local variables > > arch/arm64/include/asm/kvm_hyp.h | 10 ++----- > arch/arm64/kvm/hyp/debug-sr.c | 56 > +++++++++++++++++++++++++++------------- > arch/arm64/kvm/hyp/switch.c | 6 ++--- > 3 files changed, 42 insertions(+), 30 deletions(-) >
Reviewed-by: Andrew Jones <drjo...@redhat.com> _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
