[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Minios-devel] [UNIKRAFT PATCH v3 4/8] plat/linuxu: Add linuxu (ARM) interrupts support



Hey Julien,

thanks for your comment. We are welcoming that you guys are also having a look to our patches. This helps us a lot. We are going to change the instruction as you suggested. ;-)

Thanks,

Simon

On 14.06.2018 18:33, Julien Grall wrote:
Hi Simon,

On 14/06/18 15:12, Simon Kuenzer wrote:
From: Costin Lupu <costin.lupu@xxxxxxxxx>

Changes:
- ARM syscall numbers for SIGACTION and SIGPROCMASK
- ARM asm restorer

Signed-off-by: Costin Lupu <costin.lupu@xxxxxxxxx>
---
  plat/linuxu/include/linuxu/syscall-arm_32.h | 4 ++++
  plat/linuxu/irq.c                           | 7 +++++++
  2 files changed, 11 insertions(+)

diff --git a/plat/linuxu/include/linuxu/syscall-arm_32.h b/plat/linuxu/include/linuxu/syscall-arm_32.h
index e28224d..5c1a919 100644
--- a/plat/linuxu/include/linuxu/syscall-arm_32.h
+++ b/plat/linuxu/include/linuxu/syscall-arm_32.h
@@ -46,6 +46,10 @@
  #define __SC_MUNMAP    91
  #define __SC_EXIT       1
  #define __SC_IOCTL     54
+
+#define __SC_RT_SIGACTION 43
+#define __SC_RT_SIGPROCMASK 126
+
  #define __SC_PSELECT6 335
  /* NOTE: from `man syscall`:
diff --git a/plat/linuxu/irq.c b/plat/linuxu/irq.c
index 57e23d3..18fd266 100644
--- a/plat/linuxu/irq.c
+++ b/plat/linuxu/irq.c
@@ -112,7 +112,14 @@ void ukplat_lcpu_irqs_handle_pending(void)
  }
  void __restorer();
+
+#if defined __X86_64__
  asm("__restorer:mov $15,%rax\nsyscall");
+#elif defined __ARM_32__
+asm("__restorer:mov r7, #0x77\nswi 0x0");

"swi" is the pre-UAL name for "svc". The latter is better to use as most likely you are only targeting ARMv7 and onwards platform.

Cheers,


_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.