From bff1245572fa8e42666873369caf1ad80e82e630 Mon Sep 17 00:00:00 2001
From: Sean McBride <spmcbride1201@gmail.com>
Date: Tue, 17 Mar 2020 14:58:41 -0400
Subject: [PATCH] chore: order arch files

---
 runtime/include/arch/aarch64/context.h |  9 ++--
 runtime/include/arch/x86_64/context.h  | 67 ++++++++++++++------------
 2 files changed, 38 insertions(+), 38 deletions(-)

diff --git a/runtime/include/arch/aarch64/context.h b/runtime/include/arch/aarch64/context.h
index 9341904..dc05c58 100644
--- a/runtime/include/arch/aarch64/context.h
+++ b/runtime/include/arch/aarch64/context.h
@@ -4,18 +4,13 @@
 #include <unistd.h>
 #include <ucontext.h>
 
-typedef uint64_t reg_t;
 #define ARCH_NREGS 31
 
 /**
  * ARM64 code. Currently Unimplemented
  **/
 
-/*
- * This is the slowpath switch to a preempted sandbox!
- * SIGUSR1 on the current thread and restore mcontext there!
- */
-extern void __attribute__((noreturn)) worker_thread__sandbox_switch_preempt(void);
+typedef uint64_t reg_t;
 
 struct arch_context {
 	reg_t      regs[ARCH_NREGS];
@@ -23,6 +18,8 @@ struct arch_context {
 };
 
 typedef struct arch_context    arch_context_t;
+
+extern void __attribute__((noreturn)) worker_thread__sandbox_switch_preempt(void);
 extern __thread arch_context_t worker_thread__base_context;
 
 static inline void
diff --git a/runtime/include/arch/x86_64/context.h b/runtime/include/arch/x86_64/context.h
index d876c2d..e8cd4b3 100644
--- a/runtime/include/arch/x86_64/context.h
+++ b/runtime/include/arch/x86_64/context.h
@@ -2,8 +2,11 @@
 #define ARCH_X86_64_CONTEXT_H
 
 #include <assert.h>
-#include <unistd.h>
 #include <ucontext.h>
+#include <unistd.h>
+
+#define ARCH_NREGS       (16 /* GP registers */ + 1 /* for IP */)
+#define ARCH_SIG_JMP_OFF 8
 
 /*
  * Register strict ordering>
@@ -27,14 +30,11 @@
  */
 
 typedef uint64_t reg_t;
-#define ARCH_NREGS       (16 /* GP registers */ + 1 /* for IP */)
-#define ARCH_SIG_JMP_OFF 8
 
 /*
  * This is the slowpath switch to a preempted sandbox!
  * SIGUSR1 on the current thread and restore mcontext there!
  */
-extern void __attribute__((noreturn)) worker_thread__sandbox_switch_preempt(void);
 
 struct arch_context {
 	reg_t      regs[ARCH_NREGS];
@@ -42,15 +42,35 @@ struct arch_context {
 };
 
 typedef struct arch_context    arch_context_t;
+
+extern void __attribute__((noreturn)) worker_thread__sandbox_switch_preempt(void);
 extern __thread arch_context_t worker_thread__base_context;
 
-static void
-arch_mcontext_save(arch_context_t *ctx, mcontext_t *mc)
+static void __attribute__((noinline)) arch_context_init(arch_context_t *actx, reg_t ip, reg_t sp)
 {
-	assert(ctx != &worker_thread__base_context);
+	memset(&actx->mctx, 0, sizeof(mcontext_t));
+	memset((void *)actx->regs, 0, sizeof(reg_t) * ARCH_NREGS);
 
-	ctx->regs[5] = 0;
-	memcpy(&ctx->mctx, mc, sizeof(mcontext_t));
+	if (sp) {
+		/*
+		 * context_switch conventions: bp is expected to be on top of the stack
+		 * when co-op context switching..
+		 *
+		 * so push sp on this new stack and use
+		 * that new sp as sp for switching to sandbox!
+		 */
+		asm volatile("movq %%rsp, %%rbx\n\t"
+		             "movq %%rax, %%rsp\n\t"
+		             "pushq %%rax\n\t"
+		             "movq %%rsp, %%rax\n\t"
+		             "movq %%rbx, %%rsp\n\t"
+		             : "=a"(sp)
+		             : "a"(sp)
+		             : "memory", "cc", "rbx");
+	}
+
+	*(actx->regs + 5)  = sp;
+	*(actx->regs + 16) = ip;
 }
 
 static int
@@ -74,33 +94,16 @@ arch_mcontext_restore(mcontext_t *mc, arch_context_t *ctx)
 	return 0;
 }
 
-static void __attribute__((noinline)) arch_context_init(arch_context_t *actx, reg_t ip, reg_t sp)
+static void
+arch_mcontext_save(arch_context_t *ctx, mcontext_t *mc)
 {
-	memset(&actx->mctx, 0, sizeof(mcontext_t));
-	memset((void *)actx->regs, 0, sizeof(reg_t) * ARCH_NREGS);
-
-	if (sp) {
-		/*
-		 * context_switch conventions: bp is expected to be on top of the stack
-		 * when co-op context switching..
-		 *
-		 * so push sp on this new stack and use
-		 * that new sp as sp for switching to sandbox!
-		 */
-		asm volatile("movq %%rsp, %%rbx\n\t"
-		             "movq %%rax, %%rsp\n\t"
-		             "pushq %%rax\n\t"
-		             "movq %%rsp, %%rax\n\t"
-		             "movq %%rbx, %%rsp\n\t"
-		             : "=a"(sp)
-		             : "a"(sp)
-		             : "memory", "cc", "rbx");
-	}
+	assert(ctx != &worker_thread__base_context);
 
-	*(actx->regs + 5)  = sp;
-	*(actx->regs + 16) = ip;
+	ctx->regs[5] = 0;
+	memcpy(&ctx->mctx, mc, sizeof(mcontext_t));
 }
 
+
 static inline int
 arch_context_switch(arch_context_t *ca, arch_context_t *na)
 {