From 25a4f188109cbdbc98b9e6bba97650ae315979a1 Mon Sep 17 00:00:00 2001
From: Georgiy Lebedev <lebedev.gk@phystech.edu>
Date: Fri, 8 Sep 2023 13:45:34 +0300
Subject: [PATCH] Temporarily disable UPF feature and tests

UPF feature for vanilla firecracker requires a separate integration effort
(#807), so this feature and all tests for it are temporarily disabled.

Signed-off-by: Georgiy Lebedev <lebedev.gk@phystech.edu>
---
 .github/workflows/nightly_tests.yml |  4 +++-
 ctriface/Makefile                   |  4 +++-
 ctriface/iface_test.go              | 13 ++++++++++++-
 vhive.go                            |  5 +++++
 vhive_test.go                       |  5 +++++
 5 files changed, 28 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/nightly_tests.yml b/.github/workflows/nightly_tests.yml
index b47671ae3..06df79bc2 100644
--- a/.github/workflows/nightly_tests.yml
+++ b/.github/workflows/nightly_tests.yml
@@ -58,7 +58,9 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        vhive_args: ["-dbg", "-dbg -snapshots", "-dbg -snapshots -upf"]
+        # User-level page faults are temporarily disabled (gh-807)
+        # vhive_args: ["-dbg", "-dbg -snapshots", "-dbg -snapshots -upf"]
+        vhive_args: [ "-dbg", "-dbg -snapshots" ]
     env:
         GITHUB_RUN_ID: ${{ github.run_id }}
         GITHUB_JOB: ${{ github.job }}
diff --git a/ctriface/Makefile b/ctriface/Makefile
index d88495d57..6b16fa7dd 100644
--- a/ctriface/Makefile
+++ b/ctriface/Makefile
@@ -23,7 +23,9 @@
 EXTRAGOARGS:=-v -race -cover
 EXTRATESTFILES:=iface_test.go iface.go orch_options.go orch.go
 BENCHFILES:=bench_test.go iface.go orch_options.go orch.go
-WITHUPF:=-upf
+# User-level page faults are temporarily disabled (gh-807)
+# WITHUPF:=-upf
+WITHUPF:=
 WITHLAZY:=-lazy
 GOBENCH:=-v -timeout 1500s
 CTRDLOGDIR:=/tmp/ctrd-logs
diff --git a/ctriface/iface_test.go b/ctriface/iface_test.go
index cda2e97a0..fcf4e3cc9 100644
--- a/ctriface/iface_test.go
+++ b/ctriface/iface_test.go
@@ -41,9 +41,20 @@ var (
 	isUPFEnabled = flag.Bool("upf", false, "Set UPF enabled")
 	isLazyMode   = flag.Bool("lazy", false, "Set lazy serving on or off")
 	//nolint:deadcode,unused,varcheck
-	isWithCache  = flag.Bool("withCache", false, "Do not drop the cache before measurements")
+	isWithCache = flag.Bool("withCache", false, "Do not drop the cache before measurements")
 )
 
+func TestMain(m *testing.M) {
+	flag.Parse()
+
+	if *isUPFEnabled {
+		log.Error("User-level page faults are temporarily disabled (gh-807)")
+		os.Exit(-1)
+	}
+
+	os.Exit(m.Run())
+}
+
 func TestPauseSnapResume(t *testing.T) {
 	log.SetFormatter(&log.TextFormatter{
 		TimestampFormat: ctrdlog.RFC3339NanoFixed,
diff --git a/vhive.go b/vhive.go
index 39b09630d..b9b2542a8 100644
--- a/vhive.go
+++ b/vhive.go
@@ -89,6 +89,11 @@ func main() {
 		return
 	}
 
+	if *isUPFEnabled {
+		log.Error("User-level page faults are temporarily disabled (gh-807)")
+		return
+	}
+
 	if *isUPFEnabled && !*isSnapshotsEnabled {
 		log.Error("User-level page faults are not supported without snapshots")
 		return
diff --git a/vhive_test.go b/vhive_test.go
index bd57750d4..07e6505e0 100644
--- a/vhive_test.go
+++ b/vhive_test.go
@@ -65,6 +65,11 @@ func TestMain(m *testing.M) {
 
 	flag.Parse()
 
+	if *isUPFEnabled {
+		log.Error("User-level page faults are temporarily disabled (gh-807)")
+		os.Exit(-1)
+	}
+
 	log.Infof("Orchestrator snapshots enabled: %t", *isSnapshotsEnabledTest)
 	log.Infof("Orchestrator UPF enabled: %t", *isUPFEnabledTest)
 	log.Infof("Orchestrator lazy serving mode enabled: %t", *isLazyModeTest)