On 2018-08-14 12:50, Ralf Ramsauer wrote:
Prepares consolidation of arch_cpu_resume/suspend. With this, we have
the same path on all architectures for suspending CPUs. This allows us
to consolidate code in the next step.

Signed-off-by: Ralf Ramsauer <ralf.ramsa...@oth-regensburg.de>
---
  hypervisor/arch/arm-common/control.c             | 2 +-
  hypervisor/arch/arm-common/include/asm/control.h | 5 +++++
  hypervisor/arch/x86/control.c                    | 2 +-
  hypervisor/arch/x86/include/asm/control.h        | 6 ++++++
  4 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/hypervisor/arch/arm-common/control.c 
b/hypervisor/arch/arm-common/control.c
index 9aab822b..22101e83 100644
--- a/hypervisor/arch/arm-common/control.c
+++ b/hypervisor/arch/arm-common/control.c
@@ -65,7 +65,7 @@ void arch_suspend_cpu(unsigned int cpu_id)
                 * The target CPU, in turn, will leave the guest and handle the
                 * request in the event loop.
                 */
-               arm_cpu_kick(cpu_id);
+               arch_send_event(target_data);
while (!target_data->cpu_suspended)
                        cpu_relax();
diff --git a/hypervisor/arch/arm-common/include/asm/control.h 
b/hypervisor/arch/arm-common/include/asm/control.h
index bafef2c8..a1771302 100644
--- a/hypervisor/arch/arm-common/include/asm/control.h
+++ b/hypervisor/arch/arm-common/include/asm/control.h
@@ -33,6 +33,11 @@ void arm_cpu_reset(unsigned long pc);
  void arm_cpu_park(void);
  void arm_cpu_kick(unsigned int cpu_id);
+static inline void arch_send_event(struct public_per_cpu *target_data)
+{
+       arm_cpu_kick(target_data->cpu_id);
+}
+
  #endif /* !__ASSEMBLY__ */
#endif /* !_JAILHOUSE_ASM_CONTROL_H */
diff --git a/hypervisor/arch/x86/control.c b/hypervisor/arch/x86/control.c
index 7f171838..6b39eb9d 100644
--- a/hypervisor/arch/x86/control.c
+++ b/hypervisor/arch/x86/control.c
@@ -138,7 +138,7 @@ void arch_suspend_cpu(unsigned int cpu_id)
                 * The target CPU, in turn, will leave the guest and handle the
                 * request in the event loop.
                 */
-               apic_send_nmi_ipi(target_data);
+               arch_send_event(target_data);
while (!target_data->cpu_suspended)
                        cpu_relax();
diff --git a/hypervisor/arch/x86/include/asm/control.h 
b/hypervisor/arch/x86/include/asm/control.h
index 2566e115..0fcdd812 100644
--- a/hypervisor/arch/x86/include/asm/control.h
+++ b/hypervisor/arch/x86/include/asm/control.h
@@ -11,6 +11,7 @@
   */
#include <jailhouse/percpu.h>
+#include <asm/apic.h>
struct exception_frame; @@ -23,3 +24,8 @@ void x86_check_events(void); void __attribute__((noreturn))
  x86_exception_handler(struct exception_frame *frame);
+
+static inline void arch_send_event(struct public_per_cpu *target_data)
+{
+       apic_send_nmi_ipi(target_data);
+}


This case could be handled by an alias instead (because the signature of both functions is identical). See also arch_panic_park vs. arm_cpu_park.

Jan

--
You received this message because you are subscribed to the Google Groups 
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jailhouse-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to