Merge pull request #5 from retrage/parameterized-build

build: Introduce Parameterize Build

Author: retrage

.github/workflows/build.yaml

@@ -19,9 +19,25 @@ jobs:
           DEBIAN_FRONTEND=noninteractive sudo apt update -y
           DEBIAN_FRONTEND=noninteractive sudo apt install -y build-essential gcc-aarch64-linux-gnu clang-format
       - name: Build
+        env:
+          CC: aarch64-linux-gnu-gcc
+          FULL_CONTEXT: 0
+          SYSCALL_RECORD: 0
         run: |
-          CC=aarch64-linux-gnu-gcc make
-          CC=aarch64-linux-gnu-gcc make -C apps/basic
+          make clean
+          make -C apps/basic clean
+          make
+          make -C apps/basic
+      - name: Build
+        env:
+          CC: aarch64-linux-gnu-gcc
+          FULL_CONTEXT: 1
+          SYSCALL_RECORD: 1
+        run: |
+          make clean
+          make -C apps/basic clean
+          make
+          make -C apps/basic
       - name: Format
         run: |
           make fmt

Makefile

@@ -1,21 +1,33 @@
+#!/usr/bin/env make
+# SPDX-License-Identifier: Apache-2.0
+# Copyright (C) 2025 Akira Moroo
+
 PROGS = libsvchook.so
 
-CC ?= gcc
 CLANG_FORMAT ?= clang-format
 
 CLEANFILES = $(PROGS) *.o *.d
 
-SRCDIR ?= ./
-
-NO_MAN=
-CFLAGS = -O3 -pipe
+CFLAGS = -O3
+CFLAGS += -pipe
 CFLAGS += -g
-CFLAGS += -DMINIMAL_CONTEXT
-CFLAGS += -Werror -Wall -Wunused-function
+CFLAGS += -Werror
+CFLAGS += -Wall
+CFLAGS += -Wunused-function
 CFLAGS += -Wextra
 CFLAGS += -fPIC
 
-LDFLAGS += -shared -rdynamic -ldl
+ifeq ($(FULL_CONTEXT), 1)
+CFLAGS += -DFULL_CONTEXT
+endif
+
+ifeq ($(SYSCALL_RECORD), 1)
+CFLAGS += -DSUPPLEMENTAL__SYSCALL_RECORD
+endif
+
+LDFLAGS += -shared
+LDFLAGS += -rdynamic
+LDFLAGS += -ldl
 
 C_SRCS = main.c
 OBJS = $(C_SRCS:.c=.o)

apps/basic/Makefile

@@ -1,15 +1,18 @@
+#!/usr/bin/env make
+# SPDX-License-Identifier: Apache-2.0
+# Copyright (C) 2025 Akira Moroo
+
 PROGS = libsvchook_basic.so
 
-CC ?= gcc
 CLANG_FORMAT ?= clang-format
 
 CLEANFILES = $(PROGS) *.o *.d
 
-SRCDIR ?= ./
-
-NO_MAN=
-CFLAGS = -O3 -pipe
-CFLAGS += -Werror -Wall -Wunused-function
+CFLAGS = -O3
+CFLAGS += -pipe
+CFLAGS += -Werror
+CFLAGS += -Wall
+CFLAGS += -Wunused-function
 CFLAGS += -Wextra
 CFLAGS += -fPIC
 
@@ -18,7 +21,6 @@ LDFLAGS += -shared
 C_SRCS = main.c
 OBJS = $(C_SRCS:.c=.o)
 
-.PHONY: all
 all: $(PROGS)
 
 $(PROGS): $(OBJS)
@@ -29,3 +31,5 @@ clean:
 
 fmt:
 	$(CLANG_FORMAT) -i $(C_SRCS)
+
+.PHONY: all clean fmt

apps/basic/main.c

@@ -1,3 +1,6 @@
+// SPDX-License-Identifier: Apache-2.0
+// Copyright (C) 2025 Akira Moroo
+
 #include <stdio.h>
 
 typedef long (*syscall_fn_t)(long, long, long, long, long, long, long, long);

main.c

@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: Apache-2.0
-// Copyright (C) 2024 Akira Moroo
+// Copyright (C) 2024-2025 Akira Moroo
 
 #ifndef _GNU_SOURCE
 #define _GNU_SOURCE
@@ -52,8 +52,7 @@ extern long enter_syscall(int64_t, int64_t, int64_t, int64_t, int64_t, int64_t,
                           int64_t, int64_t);
 extern void asm_syscall_hook(void);
 
-#ifdef MINIMAL_CONTEXT
-/* MINIMAL_CONTEXT will improve performance */
+#ifndef FULL_CONTEXT
 #define CONTEXT_SIZE 64
 // clang-format off
 #define __OP_CONTEXT(op, reg) \
@@ -64,6 +63,7 @@ extern void asm_syscall_hook(void);
 // clang-format on
 #else
 #define CONTEXT_SIZE 256
+/* FULL_CONTEXT saves all registers, but may decrease the performance. */
 // clang-format off
 #define __OP_CONTEXT(op, reg) \
   #op " xzr, x1, [" #reg ",#0] \n\t" \
@@ -83,7 +83,7 @@ extern void asm_syscall_hook(void);
   #op " x28, x29, [" #reg ",#224] \n\t" \
   #op " x30, xzr, [" #reg ",#240] \n\t"
 // clang-format on
-#endif /* !MINIMAL_CONTEXT */
+#endif /* FULL_CONTEXT */
 
 #define SAVE_CONTEXT(reg) __OP_CONTEXT(stp, reg)
 #define RESTORE_CONTEXT(reg) __OP_CONTEXT(ldp, reg)